Don't test order of delayed events in GET response

as the MSC does not (yet) specify an order for them

Author: AndrewFerr

tests/msc4140/delayed_event_test.go

@@ -87,24 +87,11 @@ func TestDelayedEvents(t *testing.T) {
 			})
 		}
 
-		checkContent := func(val gjson.Result) error {
-			content := val.Get("content").Map()
-			if l := len(content); l != 1 {
-				return fmt.Errorf("wrong number of content fields: expected 1, got %d", l)
-			}
-			countExpected++
-			if countActual := content[countKey].Uint(); countActual != countExpected {
-				return fmt.Errorf("wrong count in delayed event content: expected %v, got %v", countExpected, countActual)
-			}
-			return nil
-		}
-
 		res = getDelayedEvents(t, user)
 		countExpected = 0
 		must.MatchResponse(t, res, match.HTTPResponse{
 			JSON: []match.JSON{
 				match.JSONKeyArrayOfSize("delayed_events", numEvents),
-				match.JSONArrayEach("delayed_events", checkContent),
 			},
 		})
 
@@ -131,7 +118,17 @@ func TestDelayedEvents(t *testing.T) {
 		countExpected = 0
 		must.MatchResponse(t, res, match.HTTPResponse{
 			JSON: []match.JSON{
-				match.JSONArrayEach("chunk", checkContent),
+				match.JSONArrayEach("chunk", func(val gjson.Result) error {
+					content := val.Get("content").Map()
+					if l := len(content); l != 1 {
+						return fmt.Errorf("wrong number of content fields: expected 1, got %d", l)
+					}
+					countExpected++
+					if countActual := content[countKey].Uint(); countActual != countExpected {
+						return fmt.Errorf("wrong count in delayed event content: expected %v, got %v", countExpected, countActual)
+					}
+					return nil
+				}),
 			},
 		})
 	})