Fix external deps test

Summary: Broken by D66377231 Reviewed By: IanChilds Differential Revision: D66498117 fbshipit-source-id: c2de92d6ebd2753626f9b433de43794eeabddd83
facebook · Nov 26, 2024 · 2caeefc · 2caeefc
1 parent 2374a1e
commit 2caeefc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/tests/core/build/test_executor_with_dependencies.py b/tests/core/build/test_executor_with_dependencies.py
@@ -53,7 +53,8 @@ async def test_good_target_with_dependencies(buck: Buck) -> None:
         assert len(deps) == 1
         assert deps[0]["smc_tier"] == "noop"
         assert deps[0]["id"] == "foo"
-        assert deps[0]["reservation_id"] == "noop"
+        # reservation_id is a random string which is 20 characters long
+        assert len(deps[0]["reservation_id"]) == 20
 
     # Make sure it actually did run on RE.
     out = await read_what_ran(buck)

diff --git a/tests/e2e/build/test_build_inplace.py b/tests/e2e/build/test_build_inplace.py
@@ -802,7 +802,8 @@ async def test_genrule_with_remote_execution_dependencies(buck: Buck) -> None:
         assert len(deps) == 1
         assert deps[0]["smc_tier"] == "noop"
         assert deps[0]["id"] == "foo"
-        assert deps[0]["reservation_id"] == "noop"
+        # reservation_id is a random string which is 20 characters long
+        assert len(deps[0]["reservation_id"]) == 20
 
 
 async def read_io_provider_for_last_build(buck: Buck) -> None: