replace ICQ Interests and Affiliations slices (#111)

* replace ICQ Interests and Affiliations slices

* remove affil/count checking & associated tests

foodgroup/icq.go

@@ -461,9 +461,6 @@ func (s ICQService) OfflineMsgReq(ctx context.Context, sess *state.Session, seq
 }
 
 func (s ICQService) SetAffiliations(ctx context.Context, sess *state.Session, req wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations, seq uint16) error {
-	if len(req.PastAffiliations) != 3 || len(req.Affiliations) != 3 {
-		return fmt.Errorf("%w: expected 3 past affiliations and 3 affiliations", errICQBadRequest)
-	}
 	u := state.ICQAffiliations{
 		PastCode1:       req.PastAffiliations[0].Code,
 		PastKeyword1:    req.PastAffiliations[0].Keyword,
@@ -519,9 +516,6 @@ func (s ICQService) SetEmails(ctx context.Context, sess *state.Session, req wire
 }
 
 func (s ICQService) SetInterests(ctx context.Context, sess *state.Session, req wire.ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests, seq uint16) error {
-	if len(req.Interests) != 4 {
-		return fmt.Errorf("%w: expected 4 interests", errICQBadRequest)
-	}
 	u := state.ICQInterests{
 		Code1:    req.Interests[0].Code,
 		Keyword1: req.Interests[0].Keyword,
@@ -656,7 +650,7 @@ func (s ICQService) affiliations(ctx context.Context, sess *state.Session, user
 			ReqSubType: wire.ICQDBQueryMetaReplyAffiliations,
 			Success:    wire.ICQStatusCodeOK,
 			ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-				PastAffiliations: []struct {
+				PastAffiliations: [3]struct {
 					Code    uint16
 					Keyword string `oscar:"len_prefix=uint16,nullterm"`
 				}{
@@ -673,7 +667,7 @@ func (s ICQService) affiliations(ctx context.Context, sess *state.Session, user
 						Keyword: user.ICQAffiliations.PastKeyword3,
 					},
 				},
-				Affiliations: []struct {
+				Affiliations: [3]struct {
 					Code    uint16
 					Keyword string `oscar:"len_prefix=uint16,nullterm"`
 				}{

foodgroup/icq_test.go

@@ -1706,7 +1706,7 @@ func TestICQService_FullUserInfo(t *testing.T) {
 													Success:    wire.ICQStatusCodeOK,
 													ReqSubType: wire.ICQDBQueryMetaReplyAffiliations,
 													ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-														PastAffiliations: []struct {
+														PastAffiliations: [3]struct {
 															Code    uint16
 															Keyword string `oscar:"len_prefix=uint16,nullterm"`
 														}{
@@ -1723,7 +1723,7 @@ func TestICQService_FullUserInfo(t *testing.T) {
 																Keyword: "Previous Job",
 															},
 														},
-														Affiliations: []struct {
+														Affiliations: [3]struct {
 															Code    uint16
 															Keyword string `oscar:"len_prefix=uint16,nullterm"`
 														}{
@@ -1978,7 +1978,7 @@ func TestICQService_SetAffiliations(t *testing.T) {
 			seq:  1,
 			sess: newTestSession("100003", sessOptUIN(100003)),
 			req: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-				PastAffiliations: []struct {
+				PastAffiliations: [3]struct {
 					Code    uint16
 					Keyword string `oscar:"len_prefix=uint16,nullterm"`
 				}{
@@ -1995,7 +1995,7 @@ func TestICQService_SetAffiliations(t *testing.T) {
 						Keyword: "kw3",
 					},
 				},
-				Affiliations: []struct {
+				Affiliations: [3]struct {
 					Code    uint16
 					Keyword string `oscar:"len_prefix=uint16,nullterm"`
 				}{
@@ -2067,32 +2067,6 @@ func TestICQService_SetAffiliations(t *testing.T) {
 				},
 			},
 		},
-		{
-			name: "err: unexpected affiliations count",
-			seq:  1,
-			sess: newTestSession("100003", sessOptUIN(100003)),
-			req: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-				PastAffiliations: []struct {
-					Code    uint16
-					Keyword string `oscar:"len_prefix=uint16,nullterm"`
-				}{
-					{
-						Code:    1,
-						Keyword: "kw1",
-					},
-				},
-				Affiliations: []struct {
-					Code    uint16
-					Keyword string `oscar:"len_prefix=uint16,nullterm"`
-				}{
-					{
-						Code:    4,
-						Keyword: "kw4",
-					},
-				},
-			},
-			wantErr: errICQBadRequest,
-		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -2317,7 +2291,7 @@ func TestICQService_SetInterests(t *testing.T) {
 			seq:  1,
 			sess: newTestSession("100003", sessOptUIN(100003)),
 			req: wire.ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests{
-				Interests: []struct {
+				Interests: [4]struct {
 					Code    uint16
 					Keyword string `oscar:"len_prefix=uint16,nullterm"`
 				}{
@@ -2389,27 +2363,6 @@ func TestICQService_SetInterests(t *testing.T) {
 				},
 			},
 		},
-		{
-			name: "err: unexpected interest count",
-			seq:  1,
-			sess: newTestSession("100003", sessOptUIN(100003)),
-			req: wire.ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests{
-				Interests: []struct {
-					Code    uint16
-					Keyword string `oscar:"len_prefix=uint16,nullterm"`
-				}{
-					{
-						Code:    1,
-						Keyword: "kw1",
-					},
-					{
-						Code:    2,
-						Keyword: "kw2",
-					},
-				},
-			},
-			wantErr: errICQBadRequest,
-		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

server/oscar/handler/icq_test.go

@@ -657,7 +657,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 									},
 									ReqSubType: wire.ICQDBQueryMetaReqSetInterests,
 									MetaRequest: wire.ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests{
-										Interests: []struct {
+										Interests: [4]struct {
 											Code    uint16
 											Keyword string `oscar:"len_prefix=uint16,nullterm"`
 										}{
@@ -676,7 +676,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 			allMockParams: allMockParams{
 				setInterests: &mockParam{
 					req: wire.ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests{
-						Interests: []struct {
+						Interests: [4]struct {
 							Code    uint16
 							Keyword string `oscar:"len_prefix=uint16,nullterm"`
 						}{
@@ -703,7 +703,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 									},
 									ReqSubType: wire.ICQDBQueryMetaReqSetAffiliations,
 									MetaRequest: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-										PastAffiliations: []struct {
+										PastAffiliations: [3]struct {
 											Code    uint16
 											Keyword string `oscar:"len_prefix=uint16,nullterm"`
 										}{
@@ -711,7 +711,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 												Keyword: "a_past_affiliation",
 											},
 										},
-										Affiliations: []struct {
+										Affiliations: [3]struct {
 											Code    uint16
 											Keyword string `oscar:"len_prefix=uint16,nullterm"`
 										}{
@@ -730,7 +730,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 			allMockParams: allMockParams{
 				setAffiliations: &mockParam{
 					req: wire.ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations{
-						PastAffiliations: []struct {
+						PastAffiliations: [3]struct {
 							Code    uint16
 							Keyword string `oscar:"len_prefix=uint16,nullterm"`
 						}{
@@ -738,7 +738,7 @@ func TestICQHandler_DBQuery(t *testing.T) {
 								Keyword: "a_past_affiliation",
 							},
 						},
-						Affiliations: []struct {
+						Affiliations: [3]struct {
 							Code    uint16
 							Keyword string `oscar:"len_prefix=uint16,nullterm"`
 						}{

wire/snacs.go

@@ -1802,18 +1802,18 @@ type ICQ_0x07D0_0x0406_DBQueryMetaReqSetNotes struct {
 }
 
 type ICQ_0x07D0_0x0410_DBQueryMetaReqSetInterests struct {
-	Interests []struct {
+	Interests [4]struct {
 		Code    uint16
 		Keyword string `oscar:"len_prefix=uint16,nullterm"`
 	} `oscar:"count_prefix=uint8"`
 }
 
 type ICQ_0x07D0_0x041A_DBQueryMetaReqSetAffiliations struct {
-	PastAffiliations []struct {
+	PastAffiliations [3]struct {
 		Code    uint16
 		Keyword string `oscar:"len_prefix=uint16,nullterm"`
 	} `oscar:"count_prefix=uint8"`
-	Affiliations []struct {
+	Affiliations [3]struct {
 		Code    uint16
 		Keyword string `oscar:"len_prefix=uint16,nullterm"`
 	} `oscar:"count_prefix=uint8"`