dheerajturaga commented on PR #72857: URL: https://github.com/apache/airflow/pull/72857#issuecomment-5687266873
Thanks for adding coverage for this module. Could you please: - Rebase onto current `main` to resolve the conflict in `test_project_structure.py`. - Avoid duplicating coverage already present in `test_celery_executor.py`, including visibility timeouts, result-backend parsing/fallbacks, extra configuration, and SSL handling. Please move the relevant existing tests into `test_default_celery.py` and retain the genuinely new cases. This keeps each behavior covered in one place and avoids maintaining parallel test suites. --- Drafted-by: Codex (GPT-5); reviewed by @dheerajturaga before posting -- 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]
