[1/2] Manifest request fails if filter params are rearranged (#6417)

Add test coverage
DataBiosphere · Jul 24, 2024 · c53779f · c53779f
1 parent d709b1a
commit c53779f
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/test/service/test_manifest_async.py b/test/service/test_manifest_async.py
@@ -314,7 +314,16 @@ def test(self,
                                                          operation_name='StartExecution',
                                                          error_code='ExecutionAlreadyExists')
                     _sfn.start_execution.side_effect = exception
-                    response = requests.put(url=str(initial_url), allow_redirects=False)
+
+                    # Introduce an insignificant difference in the SFN input by
+                    # reordering the `filters` dictionary entries. The repeated
+                    # request should be considered valid and matching the completed
+                    # step function execution.
+                    url = initial_url.copy()
+                    filters = json.loads(url.args['filters'])
+                    url.args['filters'] = json.dumps(dict(reversed(filters.items())))
+
+                    response = requests.put(url=str(url), allow_redirects=False)
                     _sfn.reset_mock(side_effect=True)
                     if fetch:
                         self.assertEqual(200, response.status_code)