fixup! Add managed-access integration test for verbatim manifests (#6098

)
DataBiosphere · May 1, 2024 · 24209d0 · 24209d0
1 parent 6073a24
commit 24209d0
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/integration_test.py b/test/integration_test.py
@@ -1586,9 +1586,9 @@ def dump_verbatim_manifest(bundles: Iterable[str]) -> set[str]:
             self.assertEqual(len(manifest_lines), len(manifest_content))
             return manifest_content
 
-        for manifest_format in ManifestFormat.verbatim_jsonl, ManifestFormat.verbatim_pfb:
-            if manifest_format in metadata_plugin.manifest_formats:
-                with self.subTest(format=manifest_format):
+        for format_ in ManifestFormat.verbatim_jsonl, ManifestFormat.verbatim_pfb:
+            if format_ in metadata_plugin.manifest_formats:
+                with self.subTest(format=format_):
                     unauthorized = dump_verbatim_manifest(all_bundles)
                     with self._service_account_credentials:
                         authorized = dump_verbatim_manifest(all_bundles)