| 12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250 |
- package http
- import (
- "io"
- "log/slog"
- "net/http"
- "net/http/httptest"
- "net/mail"
- "strings"
- "testing"
- "time"
- "github.com/google/uuid"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/mock"
- "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)
- 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},{"id":"userb","screen_name":"userB","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":false},{"id":"100003","screen_name":"100003","online_seconds":0,"away_message":"","idle_seconds":0,"is_icq":true}]}`,
- 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)
- 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}]}`,
- 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().
- RetrieveByScreenName(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 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}`,
- statusCode: http.StatusOK,
- mockParams: mockParams{
- userManagerParams: userManagerParams{
- getUserParams: getUserParams{
- {
- screenName: state.NewIdentScreenName("userA"),
- result: &state.User{
- DisplayScreenName: "userA",
- IdentScreenName: state.NewIdentScreenName("userA"),
- },
- },
- },
- },
- accountRetrieverParams: accountRetrieverParams{
- 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",
- },
- },
- },
- },
- },
- }
- 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)
- }
- accountRetriever := newMockAccountRetriever(t)
- for _, params := range tc.mockParams.accountRetrieverParams.emailAddressByNameParams {
- accountRetriever.EXPECT().
- EmailAddressByName(params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountRetrieverParams.regStatusByNameParams {
- accountRetriever.EXPECT().
- RegStatusByName(params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountRetrieverParams.confirmStatusByNameParams {
- accountRetriever.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, accountRetriever, 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 TestUserBuddyIconHandler_GET(t *testing.T) {
- 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 buddy icon",
- requestScreenName: state.NewIdentScreenName("userA"),
- want: string([]byte{'i', 'c', 'o', 'n'}),
- 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: []byte{'i', 'c', 'o', 'n'},
- },
- },
- },
- },
- },
- {
- 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},{"id":"userb","screen_name":"userB","is_icq":false},{"id":"100003","screen_name":"100003","is_icq":true}]`,
- 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\u0026exchange=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\u0026exchange=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\u0026exchange=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\u0026exchange=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\u0026exchange=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\u0026exchange=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.Slice(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)
- }
- })
- }
- }
|