| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440 |
- 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{
- relayToSelfParams: relayToSelfParams{
- {
- screenName: state.NewIdentScreenName("Chatting Chuck"),
- message: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.OService,
- SubGroup: wire.OServiceUserInfoUpdate,
- RequestID: wire.ReqIDFromServer,
- },
- },
- },
- },
- },
- },
- 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{
- relayToSelfParams: relayToSelfParams{
- {
- screenName: state.NewIdentScreenName("Chatting Chuck"),
- message: wire.SNACMessage{
- Frame: wire.SNACFrame{
- FoodGroup: wire.OService,
- SubGroup: wire.OServiceUserInfoUpdate,
- RequestID: wire.ReqIDFromServer,
- },
- },
- },
- },
- },
- },
- 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)
- }
- for _, params := range tc.mockParams.relayToSelfParams {
- p := params
- messageRelayer.EXPECT().
- RelayToSelf(mock.Anything, mock.Anything, mock.MatchedBy(func(msg wire.SNACMessage) bool {
- if msg.Frame != p.message.Frame {
- return false
- }
- body, ok := msg.Body.(wire.SNAC_0x01_0x0F_OServiceUserInfoUpdate)
- return ok && len(body.UserInfo) == 1 &&
- state.NewIdentScreenName(body.UserInfo[0].ScreenName) == p.screenName
- })).
- Once()
- }
- 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)
- })
- }
- }
|