Update getPipelines to return a set of behaviors

MatthiWare · May 12, 2024 · def0084 · def0084
1 parent af80949
commit def0084
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/lib/src/request/pipeline/pipeline_behavior_store.dart b/lib/src/request/pipeline/pipeline_behavior_store.dart
@@ -61,16 +61,16 @@ class PipelineBehaviorStore implements PipelineConfigurator {
   }
 
   /// Returns all [PipelineBehavior]'s that match.
-  List<PipelineBehavior> getPipelines(
+  Set<PipelineBehavior> getPipelines(
     Request request,
   ) {
     final requestType = request.runtimeType;
 
     final behaviors = _typedBehaviors[requestType];
 
-    return [
+    return {
       if (behaviors != null) ...behaviors,
       ..._genericBehaviors,
-    ];
+    };
   }
 }
diff --git a/test/unit/request/pipeline_behavior/pipeline_behavior_store_test.dart b/test/unit/request/pipeline_behavior/pipeline_behavior_store_test.dart
@@ -252,14 +252,14 @@ void main() {
 
         expect(
           pipelineBehaviorStore.getPipelines(mockRequest),
-          [
+          {
             correctBehavior,
             PipelineBehavior.function(correctBehavior.handle),
             PipelineBehavior.factory(correctFactory),
             logBehavior,
             PipelineBehavior.function(logBehavior.handle),
             PipelineBehavior.factory(logBehaviorFactory),
-          ],
+          },
         );
       });
     });

diff --git a/test/unit/request/requests_manager_test.dart b/test/unit/request/requests_manager_test.dart
@@ -99,7 +99,7 @@ void main() {
             .thenReturn(mockRequestHandler);
 
         when(() => mockPipelineBehaviorStore.getPipelines(mockRequest))
-            .thenReturn([]);
+            .thenReturn({});
 
         final result = await requestsManager.send(mockRequest);
 
@@ -134,7 +134,7 @@ void main() {
         });
 
         when(() => mockPipelineBehaviorStore.getPipelines(mockRequest))
-            .thenReturn([mockBehavior]);
+            .thenReturn({mockBehavior});
 
         final result = await requestsManager.send(mockRequest);
 
@@ -166,7 +166,7 @@ void main() {
         });
 
         when(() => mockPipelineBehaviorStore.getPipelines(mockRequest))
-            .thenReturn([mockWrongBehavior]);
+            .thenReturn({mockWrongBehavior});
 
         await expectLater(
           () => requestsManager.send(mockRequest),