nuclearpinguin edited a comment on issue #5931: [AIRFLOW-5322] Fix flaky gcp transfer hook test URL: https://github.com/apache/airflow/pull/5931#issuecomment-525473786 That was one of the errors: ``` ====================================================================== 21) FAIL: test_wait_for_transfer_job (tests.gcp.hooks.test_cloud_storage_transfer_service.TestGCPTransferServiceHookWithPassedProjectId) ---------------------------------------------------------------------- Traceback (most recent call last): /usr/local/lib/python3.5/unittest/mock.py line 1159 in patched return func(*args, **keywargs) tests/gcp/hooks/test_cloud_storage_transfer_service.py line 242 in test_wait_for_transfer_job list_method.assert_has_calls(calls) /usr/local/lib/python3.5/unittest/mock.py line 826 in assert_has_calls ) from cause AssertionError: Calls not found. est-job"], "project_id": "project-id"}', name='transferOperations'), call().execute(num_retries=5)] Expected: [call(filter='{"job_names": ["transferJobs/test-job"], "project_id": "project-id"}', name='transferOperations'), call().execute(num_retries=5), call(filter='{"job_names": ["transferJobs/t Actual: [call(filter='{"project_id": "project-id", "job_names": ["transferJobs/test-job"]}', name='transferOperations'), call().execute(num_retries=5), call(filter='{"project_id": "project-id", "job_names": ["transferJobs/test-job"]}', name='transferOperations'), call().execute(num_retries=5)] ``` You can see that the `filter` parameter differs in key order. But it could be also issue of `has_calls` so I will add `any_order=True`.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
