Taragolis commented on code in PR #37916:
URL: https://github.com/apache/airflow/pull/37916#discussion_r1513532765
##########
tests/providers/apache/beam/operators/test_beam.py:
##########
@@ -286,7 +287,7 @@ def test_exec_dataflow_runner(self, gcs_hook,
dataflow_hook_mock, beam_hook_mock
"jobName": job_name,
"stagingLocation": "gs://test/staging",
"region": "us-central1",
- "labels": {"foo": "bar", "airflow-version": TEST_VERSION},
+ "labels": {"foo": "bar"},
Review Comment:
Yeah tests cases broken completely. It is expects on the specific order, and
have a side effect one after each other.
Single test in main failed
```console
pytest
tests/providers/apache/beam/operators/test_beam.py::TestBeamRunJavaPipelineOperator::test_exec_dataflow_runne
...
============================================================================= 1
failed, 4 warnings in 2.34s
==============================================================================
```
Entire pass
```console
pytest tests/providers/apache/beam/operators/test_beam.py
...
=============================================================================
37 passed, 4 warnings in 2.20s
=============================================================================
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
