RNHTTR commented on a change in pull request #14883:
URL: https://github.com/apache/airflow/pull/14883#discussion_r602787469
##########
File path: tests/executors/test_celery_executor.py
##########
@@ -469,3 +470,23 @@ def test_should_support_base_backend(self):
)
assert result == {'123': ('SUCCESS', None), '456': ("PENDING", None)}
+
+ @pytest.mark.integration("redis")
+ @pytest.mark.integration("rabbitmq")
+ @pytest.mark.backend("mysql", "postgres")
+ def test_should_support_base_backend_from_try_adopt_task_instances(self):
Review comment:
Good call.
I refactored the tests for the possible iterations of `get_many` to set the
log mode to `DEBUG` and updated the debug statement to account for being a map
object vs being a list object. If it's a map object and the backend is not a
base backend, the length of `async_results` will always be 0 because it comes
in as an iterator and `_tasks_list_to_task_ids` converts it to a set.
--
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]
