Skip to content

Fix test_celery_integration: unreachable code and stale attributes#65863

Open
wjddn279 wants to merge 1 commit intoapache:mainfrom
wjddn279:fix-celery-integration-test
Open

Fix test_celery_integration: unreachable code and stale attributes#65863
wjddn279 wants to merge 1 commit intoapache:mainfrom
wjddn279:fix-celery-integration-test

Conversation

@wjddn279
Copy link
Copy Markdown
Contributor

After #63888, broken conventions in this integration test file makes test silently skipped

https://github.com/apache/airflow/actions/runs/24937419589/job/73026251221?pr=65847

image

Out of 10 tests in the file, only 4 were actually executing. This PR restores the remaining 6.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    claude code (opus 4.7)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@wjddn279
Copy link
Copy Markdown
Contributor Author

@SameerMesiah97 Could you double-check whether this is the intended change?

@eladkal eladkal requested a review from jscheffl April 26, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants