| 1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426 |
- package foodgroup
- import (
- "context"
- "io"
- "log/slog"
- "net/mail"
- "testing"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/mock"
- "github.com/mk6i/open-oscar-server/config"
- "github.com/mk6i/open-oscar-server/state"
- "github.com/mk6i/open-oscar-server/wire"
- )
- func TestAdminService_ConfirmRequest(t *testing.T) {
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "unconfirmed account sends confirmation request",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmRequest,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x06_AdminConfirmRequest{},
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmReply,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x07_AdminConfirmReply{
- Status: wire.AdminAcctConfirmStatusEmailSent,
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerEmailAddressParams: accountManagerEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: &mail.Address{
- Address: "chuck@aol.com",
- },
- err: nil,
- },
- },
- accountManagerConfirmStatusParams: accountManagerConfirmStatusParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- confirmStatus: false,
- err: nil,
- },
- },
- accountManagerUpdateConfirmStatusParams: accountManagerUpdateConfirmStatusParams{
- {
- confirmStatus: true,
- screenName: state.NewIdentScreenName("chattingchuck"),
- err: nil,
- },
- },
- },
- buddyBroadcasterParams: buddyBroadcasterParams{
- broadcastBuddyArrivedParams: broadcastBuddyArrivedParams{
- {
- screenName: state.DisplayScreenName("Chatting Chuck"),
- },
- },
- },
- },
- },
- {
- name: "already confirmed account sends confirmation request",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmRequest,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x06_AdminConfirmRequest{},
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmReply,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x07_AdminConfirmReply{
- Status: wire.AdminAcctConfirmStatusAlreadyConfirmed,
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerEmailAddressParams: accountManagerEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: &mail.Address{
- Address: "chuck@aol.com",
- },
- err: nil,
- },
- },
- accountManagerConfirmStatusParams: accountManagerConfirmStatusParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- confirmStatus: true,
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "acccount with no email address sends confirmation request",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmRequest,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x06_AdminConfirmRequest{},
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminAcctConfirmReply,
- RequestID: 1234,
- },
- Body: wire.SNAC_0x07_0x07_AdminConfirmReply{
- Status: wire.AdminAcctConfirmStatusServerError,
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerEmailAddressParams: accountManagerEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: nil,
- err: state.ErrNoEmailAddress,
- },
- },
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- messageRelayer := newMockMessageRelayer(t)
- accountManager := newMockAccountManager(t)
- buddyBroadcaster := newMockbuddyBroadcaster(t)
- for _, params := range tc.mockParams.accountManagerEmailAddressParams {
- accountManager.EXPECT().
- EmailAddress(matchContext(), params.screenName).
- Return(params.emailAddress, params.err)
- }
- for _, params := range tc.mockParams.accountManagerConfirmStatusParams {
- accountManager.EXPECT().
- ConfirmStatus(matchContext(), params.screenName).
- Return(params.confirmStatus, params.err)
- }
- for _, params := range tc.mockParams.accountManagerUpdateConfirmStatusParams {
- accountManager.EXPECT().
- UpdateConfirmStatus(matchContext(), params.screenName, params.confirmStatus).
- Return(params.err)
- }
- for _, params := range tc.mockParams.broadcastBuddyArrivedParams {
- buddyBroadcaster.EXPECT().
- BroadcastBuddyArrived(mock.Anything, tc.instance.IdentScreenName(), tc.instance.Session().TLVUserInfo()).
- Return(params.err)
- }
- svc := AdminService{
- messageRelayer: messageRelayer,
- accountManager: accountManager,
- buddyBroadcaster: buddyBroadcaster,
- }
- outputSNAC, err := svc.ConfirmRequest(context.Background(), tc.instance, tc.inputSNAC.Frame)
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
- func TestAdminService_InfoQuery(t *testing.T) {
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "user requests account registration status",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoQuery,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x02_AdminInfoQuery{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVRegistrationStatus, uint16(0x00))},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x03_AdminInfoReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVRegistrationStatus, wire.AdminInfoRegStatusLimitDisclosure),
- },
- },
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerRegStatusParams: accountManagerRegStatusParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- regStatus: wire.AdminInfoRegStatusLimitDisclosure,
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "user requests account email address",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoQuery,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x02_AdminInfoQuery{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, uint16(0x00))},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x03_AdminInfoReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, "chattingchuck@aol.com"),
- },
- },
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerEmailAddressParams: accountManagerEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: &mail.Address{
- Address: "chattingchuck@aol.com",
- },
- err: nil,
- },
- },
- },
- },
- },
- {
- name: "user requests account email address but not set",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoQuery,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x02_AdminInfoQuery{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, uint16(0x00))},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x03_AdminInfoReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, ""),
- },
- },
- },
- },
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerEmailAddressParams: accountManagerEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: nil,
- err: state.ErrNoEmailAddress,
- },
- },
- },
- },
- },
- {
- name: "user requests formatted screenname",
- instance: newTestInstance("ChattingChuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoQuery,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x02_AdminInfoQuery{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, uint16(0x00))},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x03_AdminInfoReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "ChattingChuck"),
- },
- },
- },
- },
- },
- {
- name: "user requests invalid TLV",
- instance: newTestInstance("ChattingChuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoQuery,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x02_AdminInfoQuery{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(uint16(0x99), uint16(0x00))},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminErr,
- RequestID: 1337,
- },
- Body: wire.SNACError{
- Code: wire.ErrorCodeNotSupportedByHost,
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- messageRelayer := newMockMessageRelayer(t)
- accountManager := newMockAccountManager(t)
- buddyBroadcaster := newMockbuddyBroadcaster(t)
- for _, params := range tc.mockParams.accountManagerRegStatusParams {
- accountManager.EXPECT().
- RegStatus(matchContext(), params.screenName).
- Return(params.regStatus, params.err)
- }
- for _, params := range tc.mockParams.accountManagerEmailAddressParams {
- accountManager.EXPECT().
- EmailAddress(matchContext(), params.screenName).
- Return(params.emailAddress, params.err)
- }
- svc := AdminService{
- messageRelayer: messageRelayer,
- accountManager: accountManager,
- buddyBroadcaster: buddyBroadcaster,
- }
- outputSNAC, err := svc.InfoQuery(context.Background(), tc.instance, tc.inputSNAC.Frame, tc.inputSNAC.Body.(wire.SNAC_0x07_0x02_AdminInfoQuery))
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
- func TestAdminService_InfoChangeRequest_ScreenName(t *testing.T) {
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "user changes screen name format successfully aim < 6",
- instance: newTestInstance("chattingchuck"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUpdateDisplayScreenNameParams: accountManagerUpdateDisplayScreenNameParams{
- {
- displayScreenName: state.DisplayScreenName("Chatting Chuck"),
- },
- },
- },
- buddyBroadcasterParams: buddyBroadcasterParams{
- broadcastBuddyArrivedParams: broadcastBuddyArrivedParams{
- {
- screenName: state.DisplayScreenName("Chatting Chuck"),
- },
- },
- },
- messageRelayerParams: messageRelayerParams{
- relayToScreenNameParams: relayToScreenNameParams{
- {
- screenName: state.NewIdentScreenName("Chatting Chuck"),
- message: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.OService,
- SubGroup: wire.OServiceUserInfoUpdate,
- },
- Body: newOServiceUserInfoUpdate(newTestInstance("Chatting Chuck")),
- },
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "Chatting Chuck"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "Chatting Chuck"),
- },
- },
- },
- },
- },
- {
- name: "user changes screen name format successfully aim >= 6",
- instance: newTestInstance("chattingchuck", sessOptSetFoodGroupVersion(wire.OService, 4)),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUpdateDisplayScreenNameParams: accountManagerUpdateDisplayScreenNameParams{
- {
- displayScreenName: state.DisplayScreenName("Chatting Chuck"),
- },
- },
- },
- buddyBroadcasterParams: buddyBroadcasterParams{
- broadcastBuddyArrivedParams: broadcastBuddyArrivedParams{
- {
- screenName: state.DisplayScreenName("Chatting Chuck"),
- },
- },
- },
- messageRelayerParams: messageRelayerParams{
- relayToScreenNameParams: relayToScreenNameParams{
- {
- screenName: state.NewIdentScreenName("Chatting Chuck"),
- message: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.OService,
- SubGroup: wire.OServiceUserInfoUpdate,
- },
- Body: newOServiceUserInfoUpdate(newTestInstance("Chatting Chuck", sessOptSetFoodGroupVersion(wire.OService, 4))),
- },
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "Chatting Chuck"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "Chatting Chuck"),
- },
- },
- },
- },
- },
- {
- name: "proposed screen name is too long",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "c h a t t i n g c h u c k")},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidNickNameLength),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- {
- name: "proposed screen name does not match session screen name",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "QuietQuinton")},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorValidateNickName),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- {
- name: "proposed screen name ends in a space",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVScreenNameFormatted, "ChattingChuck ")},
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidNickName),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- messageRelayer := newMockMessageRelayer(t)
- accountManager := newMockAccountManager(t)
- mockBuddyBroadcaster := newMockbuddyBroadcaster(t)
- for _, params := range tc.mockParams.accountManagerUpdateDisplayScreenNameParams {
- accountManager.EXPECT().
- UpdateDisplayScreenName(matchContext(), params.displayScreenName).
- Return(params.err)
- }
- for _, params := range tc.mockParams.broadcastBuddyArrivedParams {
- mockBuddyBroadcaster.EXPECT().
- BroadcastBuddyArrived(mock.Anything, state.NewIdentScreenName(params.screenName.String()), mock.MatchedBy(func(userInfo wire.TLVUserInfo) bool {
- return userInfo.ScreenName == params.screenName.String()
- })).
- Return(params.err)
- }
- for _, params := range tc.mockParams.relayToScreenNameParams {
- p := params
- messageRelayer.EXPECT().
- RelayToScreenName(mock.Anything, p.screenName, p.message)
- }
- svc := AdminService{
- accountManager: accountManager,
- buddyBroadcaster: mockBuddyBroadcaster,
- messageRelayer: messageRelayer,
- }
- outputSNAC, err := svc.InfoChangeRequest(context.Background(), tc.instance, tc.inputSNAC.Frame, tc.inputSNAC.Body.(wire.SNAC_0x07_0x04_AdminInfoChangeRequest))
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
- func TestAdminService_InfoChangeRequest_EmailAddress(t *testing.T) {
- // One case needs a 320 character long email address
- longEmailAddress := "longemailaddress@"
- for i := 0; i < 50; i++ {
- longEmailAddress += "domain"
- }
- longEmailAddress += ".com"
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "user changes email address successfully",
- instance: newTestInstance("chattingchuck"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUpdateEmailAddressParams: accountManagerUpdateEmailAddressParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- emailAddress: &mail.Address{
- Address: "chattingchuck@aol.com",
- },
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, "chattingchuck@aol.com"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, "chattingchuck@aol.com"),
- },
- },
- },
- },
- },
- {
- name: "proposed email address invalid rfc 5322 format",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, "chattingchuck@@@@@@@aol.com"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidEmail),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- {
- name: "proposed email address too long",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVEmailAddress, longEmailAddress),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidEmailLength),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- messageRelayer := newMockMessageRelayer(t)
- accountManager := newMockAccountManager(t)
- buddyBroadcaster := newMockbuddyBroadcaster(t)
- for _, params := range tc.mockParams.accountManagerUpdateEmailAddressParams {
- accountManager.EXPECT().
- UpdateEmailAddress(matchContext(), params.screenName, params.emailAddress).
- Return(params.err)
- }
- svc := AdminService{
- accountManager: accountManager,
- buddyBroadcaster: buddyBroadcaster,
- messageRelayer: messageRelayer,
- }
- outputSNAC, err := svc.InfoChangeRequest(context.Background(), tc.instance, tc.inputSNAC.Frame, tc.inputSNAC.Body.(wire.SNAC_0x07_0x04_AdminInfoChangeRequest))
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
- func TestAdminService_InfoChangeRequest_RegStatus(t *testing.T) {
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "user changes reg preference successfully",
- instance: newTestInstance("chattingchuck"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUpdateRegStatusParams: accountManagerUpdateRegStatusParams{
- {
- screenName: state.NewIdentScreenName("chattingchuck"),
- regStatus: wire.AdminInfoRegStatusNoDisclosure,
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVRegistrationStatus, wire.AdminInfoRegStatusNoDisclosure),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVRegistrationStatus, wire.AdminInfoRegStatusNoDisclosure),
- },
- },
- },
- },
- },
- {
- name: "proposed reg preference invalid",
- instance: newTestInstance("chattingchuck"),
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVRegistrationStatus, uint16(0x1337)),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidRegistrationPreference),
- wire.NewTLVBE(wire.AdminTLVUrl, ""),
- },
- },
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- messageRelayer := newMockMessageRelayer(t)
- accountManager := newMockAccountManager(t)
- buddyBroadcaster := newMockbuddyBroadcaster(t)
- for _, params := range tc.mockParams.accountManagerUpdateRegStatusParams {
- accountManager.EXPECT().
- UpdateRegStatus(context.Background(), params.screenName, params.regStatus).
- Return(params.err)
- }
- svc := AdminService{
- accountManager: accountManager,
- buddyBroadcaster: buddyBroadcaster,
- messageRelayer: messageRelayer,
- }
- outputSNAC, err := svc.InfoChangeRequest(context.Background(), tc.instance, tc.inputSNAC.Frame, tc.inputSNAC.Body.(wire.SNAC_0x07_0x04_AdminInfoChangeRequest))
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
- func TestAdminService_InfoChangeRequest_Password(t *testing.T) {
- cases := []struct {
- // name is the unit test name
- name string
- // cfg is the app configuration
- cfg config.Config
- // inputSNAC is the SNAC sent from the client to the server
- inputSNAC wire.SNACMessage
- // mockParams is the list of params sent to mocks that satisfy this
- // method's dependencies
- mockParams mockParams
- // instance is the session of the user
- instance *state.SessionInstance
- // expectOutput is the SNAC sent from the server to client
- expectOutput wire.SNACMessage
- // expectErr is the expected error returned
- expectErr error
- }{
- {
- name: "user changes password successfully",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerSetUserPasswordParams: accountManagerSetUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("me"),
- password: "newpass",
- },
- },
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- result: func() *state.User {
- user := &state.User{
- AuthKey: "auth_key",
- }
- assert.NoError(t, user.HashPassword("oldpass"))
- return user
- }(),
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpass"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- },
- },
- },
- },
- },
- {
- name: "user changes password, invalid new password length",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerSetUserPasswordParams: accountManagerSetUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("me"),
- password: "newpass",
- err: state.ErrPasswordInvalid,
- },
- },
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- result: func() *state.User {
- user := &state.User{
- AuthKey: "auth_key",
- }
- assert.NoError(t, user.HashPassword("oldpass"))
- return user
- }(),
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpass"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorInvalidPasswordLength),
- },
- },
- },
- },
- },
- {
- name: "user changes password, set user password runtime error",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerSetUserPasswordParams: accountManagerSetUserPasswordParams{
- {
- screenName: state.NewIdentScreenName("me"),
- password: "newpass",
- err: io.EOF,
- },
- },
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- result: func() *state.User {
- user := &state.User{
- AuthKey: "auth_key",
- }
- assert.NoError(t, user.HashPassword("oldpass"))
- return user
- }(),
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpass"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorAllOtherErrors),
- },
- },
- },
- },
- },
- {
- name: "user changes password, old password is invalid",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- result: func() *state.User {
- user := &state.User{
- AuthKey: "auth_key",
- }
- assert.NoError(t, user.HashPassword("oldpass"))
- return user
- }(),
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpassbad"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorValidatePassword),
- },
- },
- },
- },
- },
- {
- name: "user changes password, user not found",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- result: nil,
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpass"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorAllOtherErrors),
- },
- },
- },
- },
- },
- {
- name: "user changes password, user lookup runtime error",
- instance: newTestInstance("me"),
- mockParams: mockParams{
- accountManagerParams: accountManagerParams{
- accountManagerUserParams: accountManagerUserParams{
- {
- screenName: state.NewIdentScreenName("me"),
- err: io.EOF,
- },
- },
- },
- },
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVOldPassword, "oldpass"),
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorAllOtherErrors),
- },
- },
- },
- },
- },
- {
- name: "user changes password, missing old password",
- instance: newTestInstance("me"),
- mockParams: mockParams{},
- inputSNAC: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeRequest,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x04_AdminInfoChangeRequest{
- TLVRestBlock: wire.TLVRestBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, "newpass"),
- },
- },
- },
- },
- expectOutput: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.Admin,
- SubGroup: wire.AdminInfoChangeReply,
- RequestID: 1337,
- },
- Body: wire.SNAC_0x07_0x05_AdminChangeReply{
- Permissions: wire.AdminInfoPermissionsReadWrite,
- TLVBlock: wire.TLVBlock{
- TLVList: wire.TLVList{
- wire.NewTLVBE(wire.AdminTLVNewPassword, []byte{}),
- wire.NewTLVBE(wire.AdminTLVErrorCode, wire.AdminInfoErrorNeedOldPassword),
- },
- },
- },
- },
- },
- }
- for _, tc := range cases {
- t.Run(tc.name, func(t *testing.T) {
- accountManager := newMockAccountManager(t)
- for _, params := range tc.mockParams.accountManagerUserParams {
- accountManager.EXPECT().
- User(matchContext(), params.screenName).
- Return(params.result, params.err)
- }
- for _, params := range tc.mockParams.accountManagerSetUserPasswordParams {
- accountManager.EXPECT().
- SetUserPassword(matchContext(), params.screenName, params.password).
- Return(params.err)
- }
- svc := AdminService{
- accountManager: accountManager,
- logger: slog.Default(),
- }
- outputSNAC, err := svc.InfoChangeRequest(context.Background(), tc.instance, tc.inputSNAC.Frame, tc.inputSNAC.Body.(wire.SNAC_0x07_0x04_AdminInfoChangeRequest))
- assert.ErrorIs(t, err, tc.expectErr)
- if tc.expectErr != nil {
- return
- }
- assert.Equal(t, tc.expectOutput, outputSNAC)
- })
- }
- }
|