Skip to content

Conversation

@giannistbs
Copy link
Contributor

Summary of changes

  • Adds the missing COLOR_GREEN environment variable to scripts/ci/testing/run_unit_tests.sh for consistent color output.
  • Fixes a typo in the providers tests success message, replacing the undefined ${COLOR_GREEB} with ${COLOR_GREEN}.
  • This prevents an "unbound variable" error when running the script with bash -u and ensures successful test completion messages are displayed in green.

Additional context

No functional changes to test logic; this is a minor fix to improve script robustness and output clarity.

…r\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u.
@giannistbs giannistbs changed the title fix: add missing COLOR_GREEN variable and correct providers tests success message color Add missing COLOR_GREEN variable and correct providers tests success message color Jun 4, 2025
@pierrejeambrun pierrejeambrun merged commit f04a282 into apache:main Jun 5, 2025
44 checks passed
@boring-cyborg
Copy link

boring-cyborg bot commented Jun 5, 2025

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

@pierrejeambrun
Copy link
Member

cc: @potiuk

@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Jun 5, 2025
github-actions bot pushed a commit that referenced this pull request Jun 5, 2025
…message color\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u. (#51399)

(cherry picked from commit f04a282)

Co-authored-by: Giannis Tampakis <giannistampakis04@gmail.com>
@github-actions
Copy link

github-actions bot commented Jun 5, 2025

Backport successfully created: v3-0-test

Status Branch Result
v3-0-test PR Link

@pierrejeambrun pierrejeambrun changed the title Add missing COLOR_GREEN variable and correct providers tests success message color fix: run_unit_tests.sh Jun 5, 2025
github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jun 5, 2025
…message color\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u. (apache#51399)

(cherry picked from commit f04a282)

Co-authored-by: Giannis Tampakis <giannistampakis04@gmail.com>
@giannistbs giannistbs deleted the fix/providers-success-color branch June 5, 2025 17:25
pierrejeambrun pushed a commit that referenced this pull request Jun 6, 2025
…message color\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u. (#51399) (#51458)

(cherry picked from commit f04a282)

Co-authored-by: Giannis Tampakis <giannistampakis04@gmail.com>
kaxil pushed a commit that referenced this pull request Jun 6, 2025
…message color\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u. (#51399) (#51458)

(cherry picked from commit f04a282)

Co-authored-by: Giannis Tampakis <giannistampakis04@gmail.com>
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
…r\n\nAdds missing COLOR_GREEN variable and fixes typo in providers tests success message to use the correct color. Prevents unbound variable error with bash -u. (apache#51399)
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants