| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546254725482549255025512552255325542555255625572558255925602561256225632564256525662567256825692570257125722573257425752576257725782579258025812582258325842585258625872588258925902591259225932594259525962597259825992600260126022603260426052606260726082609261026112612261326142615261626172618261926202621262226232624262526262627262826292630263126322633263426352636263726382639264026412642264326442645264626472648264926502651265226532654265526562657265826592660266126622663266426652666266726682669267026712672267326742675267626772678267926802681268226832684268526862687268826892690269126922693269426952696269726982699270027012702270327042705270627072708270927102711271227132714271527162717271827192720272127222723272427252726272727282729273027312732273327342735273627372738273927402741274227432744274527462747274827492750275127522753275427552756275727582759276027612762276327642765276627672768276927702771277227732774277527762777277827792780278127822783278427852786278727882789279027912792279327942795279627972798279928002801280228032804280528062807280828092810281128122813281428152816281728182819282028212822282328242825282628272828282928302831283228332834283528362837283828392840284128422843284428452846284728482849285028512852285328542855285628572858285928602861286228632864286528662867286828692870287128722873287428752876287728782879288028812882288328842885288628872888288928902891289228932894289528962897289828992900290129022903290429052906290729082909291029112912291329142915291629172918291929202921292229232924292529262927292829292930293129322933293429352936293729382939294029412942294329442945294629472948294929502951295229532954295529562957295829592960296129622963296429652966296729682969297029712972297329742975297629772978297929802981298229832984298529862987 |
- 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":"","is_bot":false}`,
- 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{
- EmailAddressParams: EmailAddressParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &mail.Address{
- Address: "userA@aol.com",
- },
- },
- },
- RegStatusParams: RegStatusParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: uint16(0x02),
- },
- },
- ConfirmStatusParams: ConfirmStatusParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: true,
- },
- },
- },
- profileRetrieverParams: profileRetrieverParams{
- retrieveProfileParams: retrieveProfileParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: "My Profile Text",
- },
- },
- },
- },
- },
- {
- name: "valid aim bot 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":"","is_bot":true}`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- IsBot: true,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- EmailAddressParams: EmailAddressParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &mail.Address{
- Address: "userA@aol.com",
- },
- },
- },
- RegStatusParams: RegStatusParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: uint16(0x02),
- },
- },
- ConfirmStatusParams: ConfirmStatusParams{
- {
- 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","is_bot":false}`,
- 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{
- EmailAddressParams: EmailAddressParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: &mail.Address{
- Address: "userB@aol.com",
- },
- },
- },
- RegStatusParams: RegStatusParams{
- {
- screenName: state.NewIdentScreenName("userB"),
- result: uint16(0x02),
- },
- },
- ConfirmStatusParams: ConfirmStatusParams{
- {
- 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(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- accountManager := newMockAccountManager(t)
- for _, params := range tc.mockParams.accountManagerParams.EmailAddressParams {
- accountManager.EXPECT().
- EmailAddress(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountManagerParams.RegStatusParams {
- accountManager.EXPECT().
- RegStatus(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountManagerParams.ConfirmStatusParams {
- accountManager.EXPECT().
- ConfirmStatus(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- profileRetriever := newMockProfileRetriever(t)
- for _, params := range tc.mockParams.profileRetrieverParams.retrieveProfileParams {
- profileRetriever.EXPECT().
- Profile(matchContext(), 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,
- },
- },
- },
- },
- },
- },
- {
- name: "setting bot flag (before: false, after: true)",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNoContent,
- body: `{"is_bot":true}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- IsBot: false,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- setBotStatusParams: setBotStatusParams{
- {
- isBot: true,
- screenName: state.NewIdentScreenName("userA"),
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "setting bot flag (before: true, after: false)",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNoContent,
- body: `{"is_bot":false}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- IsBot: true,
- },
- },
- },
- },
- accountManagerParams: accountManagerParams{
- setBotStatusParams: setBotStatusParams{
- {
- isBot: false,
- screenName: state.NewIdentScreenName("userA"),
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "setting bot flag (before: true, after: true)",
- requestScreenName: state.NewIdentScreenName("userA"),
- statusCode: http.StatusNotModified,
- body: `{"is_bot":true}`,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- SuspendedStatus: 0x0,
- IsBot: true,
- },
- },
- },
- },
- },
- },
- }
- 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(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- accountManager := newMockAccountManager(t)
- for _, params := range tc.mockParams.accountManagerParams.updateSuspendedStatusParams {
- accountManager.EXPECT().
- UpdateSuspendedStatus(matchContext(), params.suspendedStatus, params.screenName).
- Return(params.err)
- }
- for _, params := range tc.mockParams.accountManagerParams.setBotStatusParams {
- accountManager.EXPECT().
- SetBotStatus(matchContext(), params.isBot, 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{
- buddyIconMetadataParams: buddyIconMetadataParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []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{
- buddyIconMetadataParams: buddyIconMetadataParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []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{
- buddyIconMetadataParams: buddyIconMetadataParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &wire.BARTID{
- Type: wire.BARTTypesBuddyIcon,
- BARTInfo: wire.BARTInfo{
- Flags: 0x00,
- Hash: []byte{'t', 'h', 'e', 'h', 'a', 's', 'h'},
- },
- },
- },
- },
- },
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []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{
- buddyIconMetadataParams: buddyIconMetadataParams{
- {
- 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{
- buddyIconMetadataParams: buddyIconMetadataParams{
- {
- 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(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- feedbagRetriever := newMockFeedBagRetriever(t)
- for _, params := range tc.mockParams.feedBagRetrieverParams.buddyIconMetadataParams {
- feedbagRetriever.EXPECT().
- BuddyIconMetadata(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- bartRetriever := newMockBARTAssetManager(t)
- for _, params := range tc.mockParams.bartAssetManagerParams.bartItemParams {
- bartRetriever.EXPECT().
- BARTItem(matchContext(), params.hash).
- 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":"","is_bot":false},{"id":"userb","screen_name":"userB","is_icq":false,"suspended_status":"","is_bot":true},{"id":"100003","screen_name":"100003","is_icq":true,"suspended_status":"","is_bot":false}]`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- allUsersParams: allUsersParams{
- {
- result: []state.User{
- {
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- {
- DisplayScreenName: "userB",
- IdentScreenName: state.NewIdentScreenName("userB"),
- IsBot: true,
- },
- {
- 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) {
- request := httptest.NewRequest(http.MethodGet, "/user", nil)
- responseRecorder := httptest.NewRecorder()
- userManager := newMockUserManager(t)
- for _, params := range tc.mockParams.userManagerParams.allUsersParams {
- userManager.EXPECT().
- AllUsers(matchContext()).
- Return(params.result, params.err)
- }
- getUserHandler(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 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(matchContext(), 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(matchContext(), 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(matchContext(), 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(matchContext(), 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 TestDeletePublicChatHandler(t *testing.T) {
- tt := []struct {
- name string
- body string
- want string
- statusCode int
- mockParams mockParams
- }{
- {
- name: "successful deletion of single chat room",
- body: `{"names":["TestRoom"]}`,
- want: `Chat rooms deleted successfully.`,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- chatRoomDeleterParams: chatRoomDeleterParams{
- deleteChatRoomsParams: deleteChatRoomsParams{
- {
- exchange: state.PublicExchange,
- names: []string{"TestRoom"},
- },
- },
- },
- },
- },
- {
- name: "successful deletion of multiple chat rooms",
- body: `{"names":["Room1", "Room2", "Room3"]}`,
- want: `Chat rooms deleted successfully.`,
- statusCode: http.StatusNoContent,
- mockParams: mockParams{
- chatRoomDeleterParams: chatRoomDeleterParams{
- deleteChatRoomsParams: deleteChatRoomsParams{
- {
- exchange: state.PublicExchange,
- names: []string{"Room1", "Room2", "Room3"},
- },
- },
- },
- },
- },
- {
- name: "empty names array",
- body: `{"names":[]}`,
- want: `no chat room names provided`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "malformed JSON",
- body: `{"names":["Room1"`, // missing closing brackets
- want: `malformed input`,
- statusCode: http.StatusBadRequest,
- },
- {
- name: "deletion error",
- body: `{"names":["TestRoom"]}`,
- want: `internal server error`,
- statusCode: http.StatusInternalServerError,
- mockParams: mockParams{
- chatRoomDeleterParams: chatRoomDeleterParams{
- deleteChatRoomsParams: deleteChatRoomsParams{
- {
- exchange: state.PublicExchange,
- names: []string{"TestRoom"},
- err: errors.New("database error"),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, "/chat/room/public", strings.NewReader(tc.body))
- responseRecorder := httptest.NewRecorder()
- chatRoomDeleter := newMockChatRoomDeleter(t)
- for _, params := range tc.mockParams.chatRoomDeleterParams.deleteChatRoomsParams {
- chatRoomDeleter.EXPECT().
- DeleteChatRooms(matchContext(), params.exchange, params.names).
- Return(params.err)
- }
- deletePublicChatHandler(responseRecorder, request, chatRoomDeleter, 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(matchContext(), 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()
- request := httptest.NewRequest(http.MethodGet, "/directory/category", nil)
- directoryManager := newMockDirectoryManager(t)
- for _, params := range tc.mockParams.categoriesParams {
- directoryManager.EXPECT().
- Categories(matchContext()).
- Return(params.result, params.err)
- }
- getDirectoryCategoryHandler(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 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(matchContext(), 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(matchContext(), 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(matchContext(), 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(matchContext(), 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(matchContext(), 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)
- }
- })
- }
- }
- func TestBARTByTypeHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- queryParams string
- wantStatusCode int
- wantResponse string
- mockParams mockParams
- }{
- {
- name: "success with items",
- queryParams: "?type=1",
- wantStatusCode: http.StatusOK,
- wantResponse: `[{"hash":"2B000001E4","type":1},{"hash":"2B000001B7","type":1}]`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- listBARTItemsParams: listBARTItemsParams{
- {
- itemType: 1,
- result: []state.BARTItem{
- {Hash: "2B000001E4", Type: 1},
- {Hash: "2B000001B7", Type: 1},
- },
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "success with empty list",
- queryParams: "?type=2",
- wantStatusCode: http.StatusOK,
- wantResponse: `[]`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- listBARTItemsParams: listBARTItemsParams{
- {
- itemType: 2,
- result: []state.BARTItem{},
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "missing type parameter",
- queryParams: "",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"type query parameter is required"}`,
- },
- {
- name: "invalid type parameter",
- queryParams: "?type=invalid",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"invalid type ID"}`,
- },
- {
- name: "internal server error",
- queryParams: "?type=1",
- wantStatusCode: http.StatusInternalServerError,
- wantResponse: `{"message":"internal server error"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- listBARTItemsParams: listBARTItemsParams{
- {
- itemType: 1,
- result: nil,
- err: errors.New("database error"),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/bart"+tc.queryParams, nil)
- responseRecorder := httptest.NewRecorder()
- mockBARTManager := newMockBARTAssetManager(t)
- for _, params := range tc.mockParams.bartAssetManagerParams.listBARTItemsParams {
- mockBARTManager.EXPECT().
- ListBARTItems(matchContext(), params.itemType).
- Return(params.result, params.err)
- }
- getBARTByTypeHandler(responseRecorder, request, mockBARTManager, slog.Default())
- assert.Equal(t, tc.wantStatusCode, responseRecorder.Code)
- assert.JSONEq(t, tc.wantResponse, responseRecorder.Body.String())
- })
- }
- }
- func TestBARTHandler_GET(t *testing.T) {
- tt := []struct {
- name string
- hash string
- wantStatusCode int
- wantResponse string
- wantHeaders map[string]string
- mockParams mockParams
- }{
- {
- name: "success with valid hash",
- hash: "2B000001E4",
- wantStatusCode: http.StatusOK,
- wantResponse: "binary data",
- wantHeaders: map[string]string{"Content-Type": "application/octet-stream"},
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- result: []byte("binary data"),
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "missing hash parameter",
- hash: "",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"hash is required"}`,
- },
- {
- name: "invalid hash format",
- hash: "invalid-hex",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"invalid hash format"}`,
- },
- {
- name: "asset not found",
- hash: "2B000001E4",
- wantStatusCode: http.StatusNotFound,
- wantResponse: `{"message":"BART asset not found"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- result: []byte{},
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "internal server error",
- hash: "2B000001E4",
- wantStatusCode: http.StatusInternalServerError,
- wantResponse: `{"message":"internal server error"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- bartItemParams: bartItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- result: nil,
- err: errors.New("database error"),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodGet, "/bart/"+tc.hash, nil)
- // Set the path value manually for testing
- if tc.hash != "" {
- request.SetPathValue("hash", tc.hash)
- }
- responseRecorder := httptest.NewRecorder()
- mockBARTManager := newMockBARTAssetManager(t)
- for _, params := range tc.mockParams.bartAssetManagerParams.bartItemParams {
- mockBARTManager.EXPECT().
- BARTItem(matchContext(), params.hash).
- Return(params.result, params.err)
- }
- getBARTHandler(responseRecorder, request, mockBARTManager, slog.Default())
- assert.Equal(t, tc.wantStatusCode, responseRecorder.Code)
- if tc.wantHeaders != nil {
- for key, value := range tc.wantHeaders {
- assert.Equal(t, value, responseRecorder.Header().Get(key))
- }
- }
- if tc.wantStatusCode == http.StatusOK {
- assert.Equal(t, tc.wantResponse, responseRecorder.Body.String())
- } else {
- assert.JSONEq(t, tc.wantResponse, responseRecorder.Body.String())
- }
- })
- }
- }
- func TestBARTHandler_POST(t *testing.T) {
- tt := []struct {
- name string
- hash string
- queryParams string
- requestBody string
- wantStatusCode int
- wantResponse string
- mockParams mockParams
- }{
- {
- name: "success with valid data",
- hash: "2B000001E4",
- queryParams: "?type=1",
- requestBody: "binary data",
- wantStatusCode: http.StatusCreated,
- wantResponse: `{"hash":"2b000001e4","type":1}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- insertBARTItemParams: insertBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- blob: []byte("binary data"),
- itemType: 1,
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "missing hash parameter",
- hash: "",
- queryParams: "?type=1",
- requestBody: "binary data",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"hash path parameter is required"}`,
- },
- {
- name: "invalid hash format",
- hash: "invalid-hex",
- queryParams: "?type=1",
- requestBody: "binary data",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"invalid hash format"}`,
- },
- {
- name: "missing type parameter",
- hash: "2B000001E4",
- queryParams: "",
- requestBody: "binary data",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"type query parameter is required"}`,
- },
- {
- name: "invalid type parameter",
- hash: "2B000001E4",
- queryParams: "?type=invalid",
- requestBody: "binary data",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"invalid type ID"}`,
- },
- {
- name: "failed to read request body",
- hash: "2B000001E4",
- queryParams: "?type=1",
- requestBody: "", // This will cause an error when reading
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"failed to read request body"}`,
- },
- {
- name: "asset already exists",
- hash: "2B000001E4",
- queryParams: "?type=1",
- requestBody: "binary data",
- wantStatusCode: http.StatusConflict,
- wantResponse: `{"message":"BART asset already exists"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- insertBARTItemParams: insertBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- blob: []byte("binary data"),
- itemType: 1,
- err: state.ErrBARTItemExists,
- },
- },
- },
- },
- },
- {
- name: "internal server error",
- hash: "2B000001E4",
- queryParams: "?type=1",
- requestBody: "binary data",
- wantStatusCode: http.StatusInternalServerError,
- wantResponse: `{"message":"internal server error"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- insertBARTItemParams: insertBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- blob: []byte("binary data"),
- itemType: 1,
- err: errors.New("database error"),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- var requestBody io.Reader
- if tc.requestBody != "" {
- requestBody = strings.NewReader(tc.requestBody)
- } else {
- requestBody = &errorReader{}
- }
- request := httptest.NewRequest(http.MethodPost, "/bart/"+tc.hash+tc.queryParams, requestBody)
- // Set the path value manually for testing
- if tc.hash != "" {
- request.SetPathValue("hash", tc.hash)
- }
- responseRecorder := httptest.NewRecorder()
- mockBARTManager := newMockBARTAssetManager(t)
- for _, params := range tc.mockParams.bartAssetManagerParams.insertBARTItemParams {
- mockBARTManager.EXPECT().
- InsertBARTItem(matchContext(), params.hash, params.blob, params.itemType).
- Return(params.err)
- }
- postBARTHandler(responseRecorder, request, mockBARTManager, slog.Default())
- assert.Equal(t, tc.wantStatusCode, responseRecorder.Code)
- assert.JSONEq(t, tc.wantResponse, responseRecorder.Body.String())
- })
- }
- }
- func TestBARTHandler_DELETE(t *testing.T) {
- tt := []struct {
- name string
- hash string
- wantStatusCode int
- wantResponse string
- mockParams mockParams
- }{
- {
- name: "success with valid hash",
- hash: "2B000001E4",
- wantStatusCode: http.StatusOK,
- wantResponse: `{"message":"BART asset deleted successfully."}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- deleteBARTItemParams: deleteBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "missing hash parameter",
- hash: "",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"hash path parameter is required"}`,
- },
- {
- name: "invalid hash format",
- hash: "invalid-hex",
- wantStatusCode: http.StatusBadRequest,
- wantResponse: `{"message":"invalid hash format"}`,
- },
- {
- name: "asset not found",
- hash: "2B000001E4",
- wantStatusCode: http.StatusNotFound,
- wantResponse: `{"message":"BART asset not found"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- deleteBARTItemParams: deleteBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- err: state.ErrBARTItemNotFound,
- },
- },
- },
- },
- },
- {
- name: "internal server error",
- hash: "2B000001E4",
- wantStatusCode: http.StatusInternalServerError,
- wantResponse: `{"message":"internal server error"}`,
- mockParams: mockParams{
- bartAssetManagerParams: bartAssetManagerParams{
- deleteBARTItemParams: deleteBARTItemParams{
- {
- hash: []byte{0x2B, 0x00, 0x00, 0x01, 0xE4},
- err: errors.New("database error"),
- },
- },
- },
- },
- },
- }
- for _, tc := range tt {
- t.Run(tc.name, func(t *testing.T) {
- request := httptest.NewRequest(http.MethodDelete, "/bart/"+tc.hash, nil)
- // Set the path value manually for testing
- if tc.hash != "" {
- request.SetPathValue("hash", tc.hash)
- }
- responseRecorder := httptest.NewRecorder()
- mockBARTManager := newMockBARTAssetManager(t)
- for _, params := range tc.mockParams.bartAssetManagerParams.deleteBARTItemParams {
- mockBARTManager.EXPECT().
- DeleteBARTItem(matchContext(), params.hash).
- Return(params.err)
- }
- deleteBARTHandler(responseRecorder, request, mockBARTManager, slog.Default())
- assert.Equal(t, tc.wantStatusCode, responseRecorder.Code)
- assert.JSONEq(t, tc.wantResponse, responseRecorder.Body.String())
- })
- }
- }
- // errorReader is a helper type that always returns an error when reading
- type errorReader struct{}
- func (er *errorReader) Read(p []byte) (n int, err error) {
- return 0, errors.New("read error")
- }
|