| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617161816191620162116221623162416251626162716281629163016311632163316341635163616371638163916401641164216431644164516461647164816491650165116521653165416551656165716581659166016611662166316641665166616671668166916701671167216731674167516761677167816791680168116821683168416851686168716881689169016911692169316941695169616971698169917001701170217031704170517061707170817091710171117121713171417151716171717181719172017211722172317241725172617271728172917301731173217331734173517361737173817391740174117421743174417451746174717481749175017511752175317541755175617571758175917601761176217631764176517661767176817691770177117721773177417751776177717781779178017811782178317841785178617871788178917901791179217931794179517961797179817991800180118021803180418051806180718081809181018111812181318141815181618171818181918201821182218231824182518261827182818291830183118321833183418351836183718381839184018411842184318441845184618471848184918501851185218531854185518561857185818591860186118621863186418651866186718681869187018711872187318741875187618771878187918801881188218831884188518861887188818891890189118921893189418951896189718981899190019011902190319041905190619071908190919101911191219131914191519161917191819191920192119221923192419251926192719281929193019311932193319341935193619371938193919401941194219431944194519461947194819491950195119521953195419551956195719581959196019611962196319641965196619671968196919701971197219731974197519761977197819791980198119821983198419851986198719881989199019911992199319941995199619971998199920002001200220032004200520062007200820092010201120122013201420152016201720182019202020212022202320242025202620272028202920302031203220332034203520362037203820392040204120422043204420452046204720482049205020512052205320542055205620572058205920602061206220632064206520662067206820692070207120722073207420752076207720782079208020812082208320842085208620872088208920902091209220932094209520962097209820992100210121022103210421052106210721082109211021112112211321142115211621172118211921202121212221232124212521262127212821292130213121322133213421352136213721382139214021412142214321442145214621472148214921502151215221532154215521562157215821592160216121622163216421652166216721682169217021712172217321742175217621772178217921802181218221832184218521862187218821892190219121922193219421952196219721982199220022012202220322042205220622072208220922102211221222132214221522162217221822192220222122222223222422252226222722282229223022312232223322342235223622372238223922402241224222432244224522462247224822492250225122522253225422552256225722582259226022612262226322642265226622672268226922702271227222732274227522762277227822792280228122822283228422852286228722882289229022912292229322942295229622972298229923002301 |
- package http
- import (
- "errors"
- "fmt"
- "io"
- "log/slog"
- "net/http"
- "net/http/httptest"
- "net/mail"
- "net/netip"
- "strings"
- "testing"
- "time"
- "github.com/google/uuid"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/mock"
- "github.com/mk6i/retro-aim-server/config"
- "github.com/mk6i/retro-aim-server/state"
- "github.com/mk6i/retro-aim-server/wire"
- )
- func TestSessionHandler_GET(t *testing.T) {
- fnNewSess := func(screenName string, uin uint32) *state.Session {
- sess := state.NewSession()
- sess.SetIdentScreenName(state.NewIdentScreenName(screenName))
- sess.SetDisplayScreenName(state.DisplayScreenName(screenName))
- sess.SetUIN(uin)
- ip, _ := netip.ParseAddrPort("1.2.3.4:1234")
- sess.SetRemoteAddr(&ip)
- return sess
- }
- tt := []struct {
- name string
- want string
- statusCode int
- timeSinceFunc func(t time.Time) time.Duration
- mockParams mockParams
- }{
- {
- name: "without sessions",
- want: `{"count":0,"sessions":[]}`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- sessionRetrieverAllSessionsParams: sessionRetrieverAllSessionsParams{
- {
- result: []*state.Session{},
- },
- },
- },
- },
- },
- {
- name: "with sessions",
- want: `{"count":3,"sessions":[{"id":"usera","screen_name":"userA","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":false,"remote_addr":"1.2.3.4","remote_port":1234},{"id":"userb","screen_name":"userB","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":false,"remote_addr":"1.2.3.4","remote_port":1234},{"id":"100003","screen_name":"100003","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":true,"remote_addr":"1.2.3.4","remote_port":1234}]}`,
- statusCode: http.StatusOK,
- timeSinceFunc: func(t time.Time) time.Duration { t0 := time.Now(); return t0.Sub(t0) },
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- sessionRetrieverAllSessionsParams: sessionRetrieverAllSessionsParams{
- {
- result: []*state.Session{
- fnNewSess("userA", 0),
- fnNewSess("userB", 0),
- fnNewSess("100003", 100003),
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/session", nil)
- responseRecorder := httptest.NewRecorder()
- sessionRetriever := newMockSessionRetriever(t)
- for _, params := range tc.mockParams.sessionRetrieverParams.sessionRetrieverAllSessionsParams {
- sessionRetriever.EXPECT().
- AllSessions().
- Return(params.result)
- }
- getSessionHandler(responseRecorder, request, sessionRetriever, tc.timeSinceFunc)
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestSessionHandlerScreenname_GET(t *testing.T) {
- fnNewSess := func(screenName string, uin uint32) *state.Session {
- sess := state.NewSession()
- sess.SetIdentScreenName(state.NewIdentScreenName(screenName))
- sess.SetDisplayScreenName(state.DisplayScreenName(screenName))
- sess.SetUIN(uin)
- ip, _ := netip.ParseAddrPort("1.2.3.4:1234")
- sess.SetRemoteAddr(&ip)
- return sess
- }
- tt := []struct {
- name string
- sessions []*state.Session
- requestScreenName state.IdentScreenName
- want string
- statusCode int
- timeSinceFunc func(t time.Time) time.Duration
- mockParams mockParams
- }{
- {
- name: "no session for screenname",
- sessions: []*state.Session{},
- requestScreenName: state.NewIdentScreenName("userA"),
- want: `session not found`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- retrieveSessionByNameParams: retrieveSessionByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "active session found for screenname",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: `{"count":1,"sessions":[{"id":"usera","screen_name":"userA","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":false,"remote_addr":"1.2.3.4","remote_port":1234}]}`,
- statusCode: http.StatusOK,
- timeSinceFunc: func(t time.Time) time.Duration { t0 := time.Now(); return t0.Sub(t0) },
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- retrieveSessionByNameParams: retrieveSessionByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: fnNewSess("userA", 0),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/session/"+tc.requestScreenName.String(), nil)
- request.SetPathValue("screenname", tc.requestScreenName.String())
- responseRecorder := httptest.NewRecorder()
- sessionRetriever := newMockSessionRetriever(t)
- for _, params := range tc.mockParams.sessionRetrieverParams.retrieveSessionByNameParams {
- sessionRetriever.EXPECT().
- RetrieveSession(params.screenName).
- Return(params.result)
- }
- getSessionHandler(responseRecorder, request, sessionRetriever, tc.timeSinceFunc)
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestSessionHandlerScreenname_DELETE(t *testing.T) {
- fnNewSess := func(screenName string) *state.Session {
- sess := state.NewSession()
- sess.SetIdentScreenName(state.NewIdentScreenName(screenName))
- sess.SetDisplayScreenName(state.DisplayScreenName(screenName))
- ip, _ := netip.ParseAddrPort("1.2.3.4:1234")
- sess.SetRemoteAddr(&ip)
- return sess
- }
- tt := []struct {
- name string
- session *state.Session
- requestScreenName state.IdentScreenName
- statusCode int
- mockParams mockParams
- }{
- {
- name: "delete an active session",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- retrieveSessionByNameParams: retrieveSessionByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: fnNewSess("userA"),
- },
- },
- },
- },
- },
- {
- name: "delete a non-existent session",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- sessionRetrieverParams: sessionRetrieverParams{
- retrieveSessionByNameParams: retrieveSessionByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, "/session/"+tc.requestScreenName.String(), nil)
- request.SetPathValue("screenname", tc.requestScreenName.String())
- responseRecorder := httptest.NewRecorder()
- sessionRetriever := newMockSessionRetriever(t)
- for _, params := range tc.mockParams.sessionRetrieverParams.retrieveSessionByNameParams {
- sessionRetriever.EXPECT().
- RetrieveSession(params.screenName).
- Return(params.result)
- }
- deleteSessionHandler(responseRecorder, request, sessionRetriever)
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- })
- }
- }
- func TestUserAccountHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- requestScreenName state.IdentScreenName
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "invalid account",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: `user not found`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "valid aim account",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: `{"id":"usera","screen_name":"userA","profile":"My Profile Text","email_address":"\u003cuserA@aol.com\u003e","reg_status":2,"confirmed":true,"is_icq":false,"suspended_status":""}`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- emailAddressByNameParams: emailAddressByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &mail.Address{
- Address: "userA@aol.com",
- },
- },
- },
- regStatusByNameParams: regStatusByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: uint16(0x02),
- },
- },
- confirmStatusByNameParams: confirmStatusByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: true,
- },
- },
- },
- profileRetrieverParams: profileRetrieverParams{
- retrieveProfileParams: retrieveProfileParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: "My Profile Text",
- },
- },
- },
- },
- },
- {
- name: "suspended aim account",
- requestScreenName: state.NewIdentScreenName("userB"),
- want: `{"id":"userb","screen_name":"userB","profile":"My Profile Text","email_address":"\u003cuserB@aol.com\u003e","reg_status":2,"confirmed":true,"is_icq":false,"suspended_status":"suspended"}`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: &state.User{
- DisplayScreenName: "userB",
- IdentScreenName: state.NewIdentScreenName("userB"),
- SuspendedStatus: wire.LoginErrSuspendedAccount,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- emailAddressByNameParams: emailAddressByNameParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: &mail.Address{
- Address: "userB@aol.com",
- },
- },
- },
- regStatusByNameParams: regStatusByNameParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: uint16(0x02),
- },
- },
- confirmStatusByNameParams: confirmStatusByNameParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: true,
- },
- },
- },
- profileRetrieverParams: profileRetrieverParams{
- retrieveProfileParams: retrieveProfileParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: "My Profile Text",
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/user/"+tc.requestScreenName.String()+"/account", nil)
- request.SetPathValue("screenname", tc.requestScreenName.String())
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.getUserParams {
- userManager.EXPECT().
- User(params.screenName).
- Return(params.result, params.err)
- }
- accountManager := newMockAccountManager(t)
- for _, params := range tc.mockParams.accountManagerParams.emailAddressByNameParams {
- accountManager.EXPECT().
- EmailAddressByName(params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountManagerParams.regStatusByNameParams {
- accountManager.EXPECT().
- RegStatusByName(params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountManagerParams.confirmStatusByNameParams {
- accountManager.EXPECT().
- ConfirmStatusByName(params.screenName).
- Return(params.result, params.err)
- }
- profileRetriever := newMockProfileRetriever(t)
- for _, params := range tc.mockParams.profileRetrieverParams.retrieveProfileParams {
- profileRetriever.EXPECT().
- Profile(params.screenName).
- Return(params.result, params.err)
- }
- getUserAccountHandler(responseRecorder, request, userManager, accountManager, profileRetriever, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserAccountHandler_PATCH(t *testing.T) {
- tt := []struct {
- name string
- requestScreenName state.IdentScreenName
- want string
- body string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "suspending a non-existent account",
- requestScreenName: state.NewIdentScreenName("userA"),
- body: `{"suspended_status":"suspended"}`,
- want: `user not found`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "patching with invalid suspended_status value",
- requestScreenName: state.NewIdentScreenName("userA"),
- body: `{"suspended_status":"thisisinvalid"}`,
- want: `{"message":"suspended_status must be empty str or one of deleted,expired,suspended,suspended_age"}`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{},
- },
- },
- },
- },
- },
- {
- name: "suspending an active aim account",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNoContent,
- body: `{"suspended_status":"suspended"}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- updateSuspendedStatusParams: updateSuspendedStatusParams{
- {
- suspendedStatus: wire.LoginErrSuspendedAccount,
- screenName: state.NewIdentScreenName("userA"),
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "unsuspending a suspended aim account",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNoContent,
- body: `{"suspended_status":""}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: wire.LoginErrSuspendedAccount,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- updateSuspendedStatusParams: updateSuspendedStatusParams{
- {
- suspendedStatus: 0x0,
- screenName: state.NewIdentScreenName("userA"),
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "suspending an already suspended aim account",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNotModified,
- body: `{"suspended_status":"suspended"}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: wire.LoginErrSuspendedAccount,
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPatch, "/user/"+tc.requestScreenName.String()+"/account", strings.NewReader(tc.body))
- request.SetPathValue("screenname", tc.requestScreenName.String())
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.getUserParams {
- userManager.EXPECT().
- User(params.screenName).
- Return(params.result, params.err)
- }
- accountManager := newMockAccountManager(t)
- for _, params := range tc.mockParams.accountManagerParams.updateSuspendedStatusParams {
- accountManager.EXPECT().
- UpdateSuspendedStatus(params.suspendedStatus, params.screenName).
- Return(params.err)
- }
- patchUserAccountHandler(responseRecorder, request, userManager, accountManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserBuddyIconHandler_GET(t *testing.T) {
- sampleGIF := []byte{
- 0x47, 0x49, 0x46, 0x38, 0x39, 0x61, 0x32, 0x00, 0x32, 0x00, 0x80, 0x00, 0x00, 0x00, 0x00, 0x00,
- 0x00, 0x00, 0x00, 0x21, 0xf9, 0x04, 0x01, 0x00, 0x00, 0x00, 0x00, 0x2c, 0x00, 0x00, 0x00, 0x00,
- 0x32, 0x00, 0x32, 0x00, 0x00, 0x02, 0x02, 0x44, 0x01, 0x00, 0x3b}
- sampleJPG := []byte{0xFF, 0xD8, 0xFF, 0x43, 0x13, 0x37}
- tt := []struct {
- name string
- requestScreenName state.IdentScreenName
- want string
- statusCode int
- contentType string
- mockParams mockParams
- }{
- {
- name: "invalid account",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: `user not found`,
- statusCode: http.StatusNotFound,
- contentType: "text/plain; charset=utf-8",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "account with gif buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: string(sampleGIF),
- statusCode: http.StatusOK,
- contentType: "image/gif",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- feedBagRetrieverParams: feedBagRetrieverParams{
- buddyIconRefByNameParams: buddyIconRefByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartRetrieverParams: bartRetrieverParams{
- bartRetrieveParams: bartRetrieveParams{
- {
- itemHash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- result: sampleGIF,
- },
- },
- },
- },
- },
- {
- name: "account with jpg buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: string(sampleJPG),
- statusCode: http.StatusOK,
- contentType: "image/jpeg",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- feedBagRetrieverParams: feedBagRetrieverParams{
- buddyIconRefByNameParams: buddyIconRefByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartRetrieverParams: bartRetrieverParams{
- bartRetrieveParams: bartRetrieveParams{
- {
- itemHash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- result: sampleJPG,
- },
- },
- },
- },
- },
- {
- name: "account with unknown format buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: string([]byte{0x13, 0x37, 0x13, 0x37, 0x13, 0x37}),
- statusCode: http.StatusOK,
- contentType: "application/octet-stream",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- feedBagRetrieverParams: feedBagRetrieverParams{
- buddyIconRefByNameParams: buddyIconRefByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartRetrieverParams: bartRetrieverParams{
- bartRetrieveParams: bartRetrieveParams{
- {
- itemHash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- result: []byte{0x13, 0x37, 0x13, 0x37, 0x13, 0x37},
- },
- },
- },
- },
- },
- {
- name: "account with cleared buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: "icon not found",
- statusCode: http.StatusNotFound,
- contentType: "text/plain; charset=utf-8",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- feedBagRetrieverParams: feedBagRetrieverParams{
- buddyIconRefByNameParams: buddyIconRefByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: wire.GetClearIconHash(),
- },
- },
- },
- },
- },
- },
- },
- {
- name: "account with no buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: "icon not found",
- statusCode: http.StatusNotFound,
- contentType: "text/plain; charset=utf-8",
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- feedBagRetrieverParams: feedBagRetrieverParams{
- buddyIconRefByNameParams: buddyIconRefByNameParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: nil,
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/user/"+tc.requestScreenName.String()+"/icon", nil)
- request.SetPathValue("screenname", tc.requestScreenName.String())
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.getUserParams {
- userManager.EXPECT().
- User(params.screenName).
- Return(params.result, params.err)
- }
- feedbagRetriever := newMockFeedBagRetriever(t)
- for _, params := range tc.mockParams.feedBagRetrieverParams.buddyIconRefByNameParams {
- feedbagRetriever.EXPECT().
- BuddyIconRefByName(params.screenName).
- Return(params.result, params.err)
- }
- bartRetriever := newMockBARTRetriever(t)
- for _, params := range tc.mockParams.bartRetrieverParams.bartRetrieveParams {
- bartRetriever.EXPECT().
- BARTRetrieve(params.itemHash).
- Return(params.result, params.err)
- }
- getUserBuddyIconHandler(responseRecorder, request, userManager, feedbagRetriever, bartRetriever, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- contentType := responseRecorder.Header().Get("Content-Type")
- if contentType != tc.contentType {
- t.Errorf("Want content type '%s', got '%s'", tc.contentType, contentType)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "empty user store",
- want: `[]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- allUsersParams: allUsersParams{
- {
- result: []state.User{},
- },
- },
- },
- },
- },
- {
- name: "user store containing 3 users",
- want: `[{"id":"usera","screen_name":"userA","is_icq":false,"suspended_status":""},{"id":"userb","screen_name":"userB","is_icq":false,"suspended_status":""},{"id":"100003","screen_name":"100003","is_icq":true,"suspended_status":""}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- allUsersParams: allUsersParams{
- {
- result: []state.User{
- {
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- {
- DisplayScreenName: "userB",
- IdentScreenName: state.NewIdentScreenName("userB"),
- },
- {
- DisplayScreenName: "100003",
- IdentScreenName: state.NewIdentScreenName("100003"),
- IsICQ: true,
- },
- },
- },
- },
- },
- },
- },
- {
- name: "user handler error",
- want: `internal server error`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- allUsersParams: allUsersParams{
- {
- result: []state.User{},
- err: io.EOF,
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.allUsersParams {
- userManager.EXPECT().
- AllUsers().
- Return(params.result, params.err)
- }
- getUserHandler(responseRecorder, userManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserHandler_POST(t *testing.T) {
- tt := []struct {
- name string
- body string
- UUID uuid.UUID
- want string
- password string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "with valid AIM user",
- body: `{"screen_name":"userA", "password":"thepassword"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `User account created successfully.`,
- password: "thepassword",
- statusCode: http.StatusCreated,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- insertUserParams: insertUserParams{
- {
- u: state.User{
- AuthKey: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b").String(),
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "with valid ICQ user",
- body: `{"screen_name":"100003", "password":"thepass"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `User account created successfully.`,
- password: "thepass",
- statusCode: http.StatusCreated,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- insertUserParams: insertUserParams{
- {
- u: state.User{
- AuthKey: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b").String(),
- DisplayScreenName: "100003",
- IdentScreenName: state.NewIdentScreenName("100003"),
- IsICQ: true,
- },
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "with malformed body",
- body: `{"screen_name":"userA", "password":"thepassword"`, // missing closing }
- want: `malformed input`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "user handler error",
- body: `{"screen_name":"userA", "password":"thepassword"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `internal server error`,
- password: "thepassword",
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- insertUserParams: insertUserParams{
- {
- u: state.User{
- AuthKey: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b").String(),
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- err: io.EOF,
- },
- },
- },
- },
- },
- {
- name: "duplicate user",
- body: `{"screen_name":"userA", "password":"thepassword"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `user already exists`,
- password: "thepassword",
- statusCode: http.StatusConflict,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- insertUserParams: insertUserParams{
- {
- u: state.User{
- AuthKey: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b").String(),
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- err: state.ErrDupUser,
- },
- },
- },
- },
- },
- {
- name: "invalid AIM screen name",
- body: `{"screen_name":"a", "password":"thepassword"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `invalid screen name: screen name must be between 3 and 16 characters`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "invalid AIM password",
- body: `{"screen_name":"userA", "password":"1"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `invalid password: invalid password length: password length must be between 4-16 characters`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "invalid ICQ UIN",
- body: `{"screen_name":"1000", "password":"thepass"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `invalid uin: uin must be a number in the range 10000-2147483646`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "invalid ICQ password",
- body: `{"screen_name":"100003", "password":"thelongpassword"}`,
- UUID: uuid.MustParse("07c70701-ba68-49a9-9f9b-67a53816e37b"),
- want: `invalid password: invalid password length: password must be between 6-8 characters`,
- statusCode: http.StatusBadRequest,
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPost, "/user", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.insertUserParams {
- assert.NoError(t, params.u.HashPassword(tc.password))
- userManager.EXPECT().
- InsertUser(params.u).
- Return(params.err)
- }
- newUUID := func() uuid.UUID { return tc.UUID }
- postUserHandler(responseRecorder, request, userManager, newUUID, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserHandler_DELETE(t *testing.T) {
- tt := []struct {
- name string
- body string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "with valid user",
- body: `{"screen_name":"userA"}`,
- want: `User account successfully deleted.`,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- deleteUserParams: deleteUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- },
- {
- name: "with non-existent user",
- body: `{"screen_name":"userA"}`,
- want: `user does not exist`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- deleteUserParams: deleteUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- err: state.ErrNoUser,
- },
- },
- },
- },
- },
- {
- name: "with malformed body",
- body: `{"screen_name":"userA"`, // missing closing }
- want: `malformed input`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "user handler error",
- body: `{"screen_name":"userA"}`,
- want: `internal server error`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- deleteUserParams: deleteUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- err: io.EOF,
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, "/user", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.deleteUserParams {
- userManager.EXPECT().
- DeleteUser(params.screenName).
- Return(params.err)
- }
- deleteUserHandler(responseRecorder, request, userManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestUserPasswordHandler_PUT(t *testing.T) {
- tt := []struct {
- name string
- body string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "user with valid password",
- body: `{"screen_name":"userA", "password":"thenewpassword"}`,
- want: `Password successfully reset.`,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- setUserPasswordParams: setUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- newPassword: "thenewpassword",
- },
- },
- },
- },
- },
- {
- name: "user with invalid password",
- body: `{"screen_name":"userA", "password":"a"}`,
- want: `invalid password length`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- setUserPasswordParams: setUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- newPassword: "a",
- err: state.ErrPasswordInvalid,
- },
- },
- },
- },
- },
- {
- name: "with malformed body",
- body: `{"screen_name":"userA", "password":"thepassword"`, // missing closing }
- want: `malformed input`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "password updater returns runtime error",
- body: `{"screen_name":"userA", "password":"thepassword"}`,
- want: `internal server error`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- setUserPasswordParams: setUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- newPassword: "thepassword",
- err: io.EOF,
- },
- },
- },
- },
- },
- {
- name: "user doesn't exist",
- body: `{"screen_name":"userA", "password":"thepassword"}`,
- want: `user does not exist`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- setUserPasswordParams: setUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- newPassword: "thepassword",
- err: state.ErrNoUser,
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPut, "/user", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.setUserPasswordParams {
- userManager.EXPECT().
- SetUserPassword(params.screenName, params.newPassword).
- Return(params.err)
- }
- putUserPasswordHandler(responseRecorder, request, userManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestPublicChatHandler_GET(t *testing.T) {
- fnNewSess := func(screenName string) *state.Session {
- sess := state.NewSession()
- sess.SetIdentScreenName(state.NewIdentScreenName(screenName))
- sess.SetDisplayScreenName(state.DisplayScreenName(screenName))
- return sess
- }
- chatRoom1 := state.NewChatRoom("chat-room-1-name", state.NewIdentScreenName("chat-room-1-creator"), state.PublicExchange)
- chatRoom2 := state.NewChatRoom("chat-room-2-name", state.NewIdentScreenName("chat-room-1-creator"), state.PublicExchange)
- tt := []struct {
- name string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "multiple chat rooms with participants",
- want: `[{"name":"chat-room-1-name","create_time":"0001-01-01T00:00:00Z","url":"aim:gochat?roomname=chat-room-1-name&exchange=5","participants":[{"id":"usera","screen_name":"userA"},{"id":"userb","screen_name":"userB"}]},{"name":"chat-room-2-name","create_time":"0001-01-01T00:00:00Z","url":"aim:gochat?roomname=chat-room-2-name&exchange=5","participants":[{"id":"userc","screen_name":"userC"},{"id":"userd","screen_name":"userD"}]}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PublicExchange,
- result: []state.ChatRoom{
- chatRoom1,
- chatRoom2,
- },
- },
- },
- },
- chatSessionRetrieverParams: chatSessionRetrieverParams{
- chatSessionRetrieverAllSessionsParams: chatSessionRetrieverAllSessionsParams{
- {
- cookie: chatRoom1.Cookie(),
- result: []*state.Session{
- fnNewSess("userA"),
- fnNewSess("userB"),
- },
- },
- {
- cookie: chatRoom2.Cookie(),
- result: []*state.Session{
- fnNewSess("userC"),
- fnNewSess("userD"),
- },
- },
- },
- },
- },
- },
- {
- name: "chat room without participants",
- want: `[{"name":"chat-room-1-name","create_time":"0001-01-01T00:00:00Z","url":"aim:gochat?roomname=chat-room-1-name&exchange=5","participants":[]}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PublicExchange,
- result: []state.ChatRoom{
- chatRoom1,
- },
- },
- },
- },
- chatSessionRetrieverParams: chatSessionRetrieverParams{
- chatSessionRetrieverAllSessionsParams: chatSessionRetrieverAllSessionsParams{
- {
- cookie: chatRoom1.Cookie(),
- result: []*state.Session{},
- },
- },
- },
- },
- },
- {
- name: "no chat rooms",
- want: `[]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PublicExchange,
- result: []state.ChatRoom{},
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/chat/room/public", nil)
- responseRecorder := httptest.NewRecorder()
- chatRoomRetriever := newMockChatRoomRetriever(t)
- for _, params := range tc.mockParams.chatRoomRetrieverParams.allChatRoomsParams {
- chatRoomRetriever.EXPECT().
- AllChatRooms(params.exchange).
- Return(params.result, params.err)
- }
- chatSessionRetriever := newMockChatSessionRetriever(t)
- for _, params := range tc.mockParams.chatSessionRetrieverParams.chatSessionRetrieverAllSessionsParams {
- chatSessionRetriever.EXPECT().
- AllSessions(params.cookie).
- Return(params.result)
- }
- getPublicChatHandler(responseRecorder, request, chatRoomRetriever, chatSessionRetriever, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestPrivateChatHandler_GET(t *testing.T) {
- fnNewSess := func(screenName string) *state.Session {
- sess := state.NewSession()
- sess.SetIdentScreenName(state.NewIdentScreenName(screenName))
- sess.SetDisplayScreenName(state.DisplayScreenName(screenName))
- return sess
- }
- chatRoom1 := state.NewChatRoom("chat-room-1-name", state.NewIdentScreenName("chat-room-1-creator"), state.PrivateExchange)
- chatRoom2 := state.NewChatRoom("chat-room-2-name", state.NewIdentScreenName("chat-room-2-creator"), state.PrivateExchange)
- tt := []struct {
- name string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "multiple chat rooms with participants",
- want: `[{"name":"chat-room-1-name","create_time":"0001-01-01T00:00:00Z","creator_id":"chat-room-1-creator","url":"aim:gochat?roomname=chat-room-1-name&exchange=4","participants":[{"id":"usera","screen_name":"userA"},{"id":"userb","screen_name":"userB"}]},{"name":"chat-room-2-name","create_time":"0001-01-01T00:00:00Z","creator_id":"chat-room-2-creator","url":"aim:gochat?roomname=chat-room-2-name&exchange=4","participants":[{"id":"userc","screen_name":"userC"},{"id":"userd","screen_name":"userD"}]}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PrivateExchange,
- result: []state.ChatRoom{
- chatRoom1,
- chatRoom2,
- },
- },
- },
- },
- chatSessionRetrieverParams: chatSessionRetrieverParams{
- chatSessionRetrieverAllSessionsParams: chatSessionRetrieverAllSessionsParams{
- {
- cookie: chatRoom1.Cookie(),
- result: []*state.Session{
- fnNewSess("userA"),
- fnNewSess("userB"),
- },
- },
- {
- cookie: chatRoom2.Cookie(),
- result: []*state.Session{
- fnNewSess("userC"),
- fnNewSess("userD"),
- },
- },
- },
- },
- },
- },
- {
- name: "chat room without participants",
- want: `[{"name":"chat-room-1-name","create_time":"0001-01-01T00:00:00Z","creator_id":"chat-room-1-creator","url":"aim:gochat?roomname=chat-room-1-name&exchange=4","participants":[]}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PrivateExchange,
- result: []state.ChatRoom{
- chatRoom1,
- },
- },
- },
- },
- chatSessionRetrieverParams: chatSessionRetrieverParams{
- chatSessionRetrieverAllSessionsParams: chatSessionRetrieverAllSessionsParams{
- {
- cookie: chatRoom1.Cookie(),
- result: []*state.Session{},
- },
- },
- },
- },
- },
- {
- name: "no chat rooms",
- want: `[]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- chatRoomRetrieverParams: chatRoomRetrieverParams{
- allChatRoomsParams: allChatRoomsParams{
- {
- exchange: state.PrivateExchange,
- result: []state.ChatRoom{},
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/chat/room/private", nil)
- responseRecorder := httptest.NewRecorder()
- chatRoomRetriever := newMockChatRoomRetriever(t)
- for _, params := range tc.mockParams.chatRoomRetrieverParams.allChatRoomsParams {
- chatRoomRetriever.EXPECT().
- AllChatRooms(params.exchange).
- Return(params.result, params.err)
- }
- chatSessionRetriever := newMockChatSessionRetriever(t)
- for _, params := range tc.mockParams.chatSessionRetrieverParams.chatSessionRetrieverAllSessionsParams {
- chatSessionRetriever.EXPECT().
- AllSessions(params.cookie).
- Return(params.result)
- }
- getPrivateChatHandler(responseRecorder, request, chatRoomRetriever, chatSessionRetriever, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestInstantMessageHandler_POST(t *testing.T) {
- type relayToScreenNameInputs struct {
- sender state.IdentScreenName
- recipient state.IdentScreenName
- msg string
- }
- tt := []struct {
- name string
- relayToScreenNameInputs []relayToScreenNameInputs
- body string
- want string
- statusCode int
- }{
- {
- name: "send an instant message",
- relayToScreenNameInputs: []relayToScreenNameInputs{
- {
- sender: state.NewIdentScreenName("sender_sn"),
- recipient: state.NewIdentScreenName("recip_sn"),
- msg: "hello world!",
- },
- },
- body: `{"from":"sender_sn","to":"recip_sn","text":"hello world!"}`,
- want: `Message sent successfully.`,
- statusCode: http.StatusOK,
- },
- {
- name: "with malformed body",
- body: `{"screen_name":"userA", "password":"thepassword"`,
- want: `malformed input`,
- statusCode: http.StatusBadRequest,
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPost, "/user", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- messageRelayer := newMockMessageRelayer(t)
- for _, params := range tc.relayToScreenNameInputs {
- validateSNAC := func(msg wire.SNACMessage) bool {
- body := msg.Body.(wire.SNAC_0x04_0x07_ICBMChannelMsgToClient)
- assert.Equal(t, params.sender.String(), body.TLVUserInfo.ScreenName)
- b, ok := body.Bytes(wire.ICBMTLVAOLIMData)
- assert.True(t, ok)
- txt, err := wire.UnmarshalICBMMessageText(b)
- assert.NoError(t, err)
- assert.Equal(t, params.msg, txt)
- return true
- }
- messageRelayer.EXPECT().
- RelayToScreenName(mock.Anything, params.recipient, mock.MatchedBy(validateSNAC))
- }
- postInstantMessageHandler(responseRecorder, request, messageRelayer, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestVersionHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- want string
- statusCode int
- buildInfo config.Build
- }{
- {
- name: "get ras version",
- want: `{"version":"13.3.7","commit":"asdfASDF12345678","date":"2024-03-01"}`,
- statusCode: http.StatusOK,
- buildInfo: config.Build{
- Version: "13.3.7",
- Commit: "asdfASDF12345678",
- Date: "2024-03-01",
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- responseRecorder := httptest.NewRecorder()
- getVersionHandler(responseRecorder, tc.buildInfo)
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryCategoryHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "no categories",
- want: `[]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- categoriesParams: categoriesParams{
- {
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "error fetching categories",
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- categoriesParams: categoriesParams{
- {
- result: nil,
- err: errors.New("error fetching categories"),
- },
- },
- },
- },
- },
- {
- name: "fetch some categories",
- want: `[{"id":1,"name":"category-1"},{"id":2,"name":"category-2"}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- categoriesParams: categoriesParams{
- {
- result: []state.Category{
- {
- ID: 1,
- Name: "category-1",
- },
- {
- ID: 2,
- Name: "category-2",
- },
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.categoriesParams {
- directoryManager.EXPECT().
- Categories().
- Return(params.result, params.err)
- }
- getDirectoryCategoryHandler(responseRecorder, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryCategoryKeywordHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- categoryID int
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "category not found",
- categoryID: 1,
- want: `{"message":"category not found"}`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- keywordsByCategoryParams: keywordsByCategoryParams{
- {
- categoryID: 1,
- result: nil,
- err: state.ErrKeywordCategoryNotFound,
- },
- },
- },
- },
- },
- {
- name: "error fetching keywords by category",
- categoryID: 1,
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- keywordsByCategoryParams: keywordsByCategoryParams{
- {
- categoryID: 1,
- result: nil,
- err: errors.New("error fetching keywords by category"),
- },
- },
- },
- },
- },
- {
- name: "invalid category ID",
- categoryID: -1,
- want: `{"message":"invalid category ID"}`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- keywordsByCategoryParams: keywordsByCategoryParams{},
- },
- },
- },
- {
- name: "no keywords",
- categoryID: 1,
- want: `[]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- keywordsByCategoryParams: keywordsByCategoryParams{
- {
- categoryID: 1,
- result: nil,
- },
- },
- },
- },
- },
- {
- name: "fetch some keywords by category",
- categoryID: 1,
- want: `[{"id":1,"name":"keyword-1"},{"id":2,"name":"keyword-2"}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- keywordsByCategoryParams: keywordsByCategoryParams{
- {
- categoryID: 1,
- result: []state.Keyword{
- {
- ID: 1,
- Name: "keyword-1",
- },
- {
- ID: 2,
- Name: "keyword-2",
- },
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, fmt.Sprintf("/directory/category/%d/keyword", tc.categoryID), nil)
- request.SetPathValue("id", fmt.Sprintf("%d", tc.categoryID))
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.keywordsByCategoryParams {
- directoryManager.EXPECT().
- KeywordsByCategory(params.categoryID).
- Return(params.result, params.err)
- }
- getDirectoryCategoryKeywordHandler(responseRecorder, request, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryCategoryHandler_DELETE(t *testing.T) {
- tt := []struct {
- name string
- categoryID int
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "category not found",
- categoryID: 1,
- want: `{"message":"category not found"}`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteCategoryParams: deleteCategoryParams{
- {
- categoryID: 1,
- err: state.ErrKeywordCategoryNotFound,
- },
- },
- },
- },
- },
- {
- name: "keyword in use by user",
- categoryID: 1,
- want: `{"message":"can't delete because category in use by a user"}`,
- statusCode: http.StatusConflict,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteCategoryParams: deleteCategoryParams{
- {
- categoryID: 1,
- err: state.ErrKeywordInUse,
- },
- },
- },
- },
- },
- {
- name: "runtime error",
- categoryID: 1,
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteCategoryParams: deleteCategoryParams{
- {
- categoryID: 1,
- err: errors.New("error deleting keyword"),
- },
- },
- },
- },
- },
- {
- name: "successful deletion",
- categoryID: 1,
- want: ``,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteCategoryParams: deleteCategoryParams{
- {
- categoryID: 1,
- },
- },
- },
- },
- },
- {
- name: "invalid category ID",
- categoryID: -1,
- want: `invalid category ID`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteCategoryParams: deleteCategoryParams{},
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, fmt.Sprintf("/directory/category/%d/keyword", tc.categoryID), nil)
- request.SetPathValue("id", fmt.Sprintf("%d", tc.categoryID))
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.deleteCategoryParams {
- directoryManager.EXPECT().
- DeleteCategory(params.categoryID).
- Return(params.err)
- }
- deleteDirectoryCategoryHandler(responseRecorder, request, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryCategoryHandler_POST(t *testing.T) {
- tt := []struct {
- name string
- body string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "category already exists",
- body: `{"name":"the_category"}`,
- want: `{"message":"category already exists"}`,
- statusCode: http.StatusConflict,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createCategoryParams: createCategoryParams{
- {
- name: "the_category",
- err: state.ErrKeywordCategoryExists,
- },
- },
- },
- },
- },
- {
- name: "runtime error",
- body: `{"name":"the_category"}`,
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createCategoryParams: createCategoryParams{
- {
- name: "the_category",
- err: errors.New("error creating category"),
- },
- },
- },
- },
- },
- {
- name: "bad input",
- body: `{"name":"the_category"`,
- want: `{"message":"malformed input"}`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createCategoryParams: createCategoryParams{},
- },
- },
- },
- {
- name: "successful creation",
- body: `{"name":"the_category"}`,
- want: `{"id":1,"name":"the_category"}`,
- statusCode: http.StatusCreated,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createCategoryParams: createCategoryParams{
- {
- name: "the_category",
- result: state.Category{
- ID: 1,
- Name: "the_category",
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPost, "/directory/category", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.createCategoryParams {
- directoryManager.EXPECT().
- CreateCategory(params.name).
- Return(params.result, params.err)
- }
- postDirectoryCategoryHandler(responseRecorder, request, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryKeywordHandler_POST(t *testing.T) {
- tt := []struct {
- name string
- body string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "keyword already exists",
- body: `{"category_id":1,"name":"the_keyword"}`,
- want: `{"message":"keyword already exists"}`,
- statusCode: http.StatusConflict,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createKeywordParams: createKeywordParams{
- {
- name: "the_keyword",
- categoryID: 1,
- err: state.ErrKeywordExists,
- },
- },
- },
- },
- },
- {
- name: "category not found",
- body: `{"category_id":1,"name":"the_keyword"}`,
- want: `{"message":"category not found"}`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createKeywordParams: createKeywordParams{
- {
- name: "the_keyword",
- categoryID: 1,
- err: state.ErrKeywordCategoryNotFound,
- },
- },
- },
- },
- },
- {
- name: "runtime error",
- body: `{"category_id":1,"name":"the_keyword"}`,
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createKeywordParams: createKeywordParams{
- {
- name: "the_keyword",
- categoryID: 1,
- err: errors.New("error creating keyword"),
- },
- },
- },
- },
- },
- {
- name: "bad input",
- body: `{"category_id":1,"name":"the_keyword"`,
- want: `{"message":"malformed input"}`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createKeywordParams: createKeywordParams{},
- },
- },
- },
- {
- name: "successful creation",
- body: `{"category_id":1,"name":"the_keyword"}`,
- want: `{"id":1,"name":"the_keyword"}`,
- statusCode: http.StatusCreated,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- createKeywordParams: createKeywordParams{
- {
- name: "the_keyword",
- categoryID: 1,
- result: state.Keyword{
- ID: 1,
- Name: "the_keyword",
- },
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodPost, "/directory/keyword", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.createKeywordParams {
- directoryManager.EXPECT().
- CreateKeyword(params.name, params.categoryID).
- Return(params.result, params.err)
- }
- postDirectoryKeywordHandler(responseRecorder, request, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
- func TestDirectoryKeywordHandler_DELETE(t *testing.T) {
- tt := []struct {
- name string
- categoryID int
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "keyword not found",
- categoryID: 1,
- want: `{"message":"keyword not found"}`,
- statusCode: http.StatusNotFound,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteKeywordParams: deleteKeywordParams{
- {
- id: 1,
- err: state.ErrKeywordNotFound,
- },
- },
- },
- },
- },
- {
- name: "keyword in use by user",
- categoryID: 1,
- want: `{"message":"can't delete because category in use by a user"}`,
- statusCode: http.StatusConflict,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteKeywordParams: deleteKeywordParams{
- {
- id: 1,
- err: state.ErrKeywordInUse,
- },
- },
- },
- },
- },
- {
- name: "runtime error",
- categoryID: 1,
- want: `{"message":"internal server error"}`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteKeywordParams: deleteKeywordParams{
- {
- id: 1,
- err: errors.New("error deleting keyword"),
- },
- },
- },
- },
- },
- {
- name: "successful deletion",
- categoryID: 1,
- want: ``,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteKeywordParams: deleteKeywordParams{
- {
- id: 1,
- },
- },
- },
- },
- },
- {
- name: "invalid keyword ID",
- categoryID: -1,
- want: `{"message":"invalid keyword ID"}`,
- statusCode: http.StatusBadRequest,
- mockParams: mockParams{
- directoryManagerParams: directoryManagerParams{
- deleteKeywordParams: deleteKeywordParams{},
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, fmt.Sprintf("/directory/keyword/%d", tc.categoryID), nil)
- request.SetPathValue("id", fmt.Sprintf("%d", tc.categoryID))
- responseRecorder := httptest.NewRecorder()
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.deleteKeywordParams {
- directoryManager.EXPECT().
- DeleteKeyword(params.id).
- Return(params.err)
- }
- deleteDirectoryKeywordHandler(responseRecorder, request, directoryManager, slog.Default())
- if responseRecorder.Code != tc.statusCode {
- t.Errorf("Want status '%d', got '%d'", tc.statusCode, responseRecorder.Code)
- }
- if strings.TrimSpace(responseRecorder.Body.String()) != tc.want {
- t.Errorf("Want '%s', got '%s'", tc.want, responseRecorder.Body)
- }
- })
- }
- }
|