cruseakshay commented on PR #60650: URL: https://github.com/apache/airflow/pull/60650#issuecomment-3788400680
> @cruseakshay Also, the changes made in the data_factory trigger, like the added _build_trigger_event method like I asked, would be nice to see a test for it as well. Is the trigger data_factory tests also testing the context manger? hook-level test that covers the context manager behavior: `AzureDataFactoryAsyncHook` already has `test_context_manager_calls_close` in the hook tests that explicitly verifies async with hook: calls close(). Haven't seen any trigger tests explicitly assert `__aenter__/__aexit__` calls. -- 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]
