Skip to content

Commit df98d10

Browse files
generatedunixname1608173377072046meta-codesync[bot]
authored andcommitted
Fix test_main_calls_align_with_args for removed --covariance args
Summary: WARNING: Generated by Autopilot (alpha) — review carefully, verify the underlying claim before accepting. Agent: General Agent | Trajectory: https://www.internalfb.com/intern/devai/devmate/inspector/741347ae-05b8-47d0-a205-b5295f06cf93/ | SC job: https://www.internalfb.com/intern/sandcastle/instance/45035997033129948/ --- ## The issue - D125282253 removed the `--covariance` and `--std-threshold` flags from `run_ba.py`, so `align` now takes `(path, rec_a, rec_b, rigid, px_threshold)`. - `test_main_calls_align_with_args` (added in D125284298) still asserted the old 7-argument call `("/data", 2, 3, True, False, 0.016, 0.3)`. ## The fix - Update the assertion to the current signature: `("/data", 2, 3, True, 0.016)`. Reviewed By: YaroslavHavrylovych Differential Revision: D125670887 fbshipit-source-id: 4bef6d721f488ff2b5198e1fa3e729dbe82251a4
1 parent 2baa8ee commit df98d10

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎annotation_gui_gcp/tests/test_run_ba.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -500,4 +500,4 @@ def test_main_calls_align_with_args(self) -> None:
500500
):
501501
with self.assertRaises(SystemExit):
502502
run_ba.main()
503-
align.assert_called_once_with("/data", 2, 3, True, False, 0.016, 0.3)
503+
align.assert_called_once_with("/data", 2, 3, True, 0.016)

0 commit comments

Comments
 (0)