Skip to content

Commit b8360cb

Browse files
committed
Force a query with a HACK within the tests
1 parent a4a5dba commit b8360cb

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

tests/integration/test_metadata_model.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ async def test_submit_filebased_manifest_file_and_entities_valid_manifest_submit
139139
dir=create_temp_folder(path=tempfile.gettempdir()),
140140
) as tmp_file:
141141
df.to_csv(tmp_file.name, index=False)
142-
143142
# WHEN the manifest is submitted (Assertions are handled in the helper method)
144143
self._submit_and_verify_manifest(
145144
helpers=helpers,
@@ -229,7 +228,6 @@ async def test_submit_filebased_manifest_file_and_entities_mock_filename(
229228
dir=create_temp_folder(path=tempfile.gettempdir()),
230229
) as tmp_file:
231230
df.to_csv(tmp_file.name, index=False)
232-
233231
# WHEN the manifest is submitted (Assertions are handled in the helper method)
234232
self._submit_and_verify_manifest(
235233
helpers=helpers,
@@ -450,6 +448,11 @@ def _submit_and_verify_manifest(
450448
raise ValueError(
451449
"expected_manifest_id or expected_manifest_name must be provided"
452450
)
451+
# HACK: must requery the fileview to get new files, since SynapseStorage will query the last state
452+
# of the fileview which may not contain any new folders in the fileview.
453+
# This is a workaround to fileviews not always containing the latest information
454+
# Since the tests don't always follow a similar process as testing resources are created and destroyed
455+
synapse_store.query_fileview(force_requery=True)
453456

454457
# Spies
455458
if already_spied:

0 commit comments

Comments
 (0)