Skip to content

Fix Breeze unit tests for milestone tagging#63031

Open
Ironankit525 wants to merge 1 commit intoapache:mainfrom
Ironankit525:fix-breeze-test-github-repo-env
Open

Fix Breeze unit tests for milestone tagging#63031
Ironankit525 wants to merge 1 commit intoapache:mainfrom
Ironankit525:fix-breeze-test-github-repo-env

Conversation

@Ironankit525
Copy link

The .github/workflows/ci.yml run for Breeze unit tests is failing in forks because the GITHUB_REPOSITORY environment variable overrules option_github_repository default of apache/airflow.

This causes two test assertions to fail when checking the comment URL generated. Explicitly passing --github-repository apache/airflow in these tests stabilizes them for forks.

The GITHUB_REPOSITORY environment variable is set in the CI pipeline
to the repository fork (e.g. Ironankit525/airflow).
This overrides the default value of 'apache/airflow' in the
option_github_repository of the Breeze CLI.

This causes the test assertions to fail when they expect the
comment URL to use 'apache/airflow' but it actually uses the
fork repository.

This commit explicitly passes '--github-repository apache/airflow'
in the test arguments to ensure they are unaffected by this environment
variable.
@Ironankit525 Ironankit525 force-pushed the fix-breeze-test-github-repo-env branch from efaa515 to 39a5313 Compare March 7, 2026 10:58
@eladkal eladkal added changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) and removed area:providers provider:openfaas provider:apache-hive labels Mar 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants