Fix integration tests to be inline with AF3 by neel-astro · Pull Request #1860 · astronomer/astro-cli

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@neel-astro

Copy link

Contributor

Description

Changes:

  • Updated the expected containers in CLI ITs to be inline with AF3 components
  • Updated the integration to reduce flakyness due to sleep and teardown component containers even during failures
Copy link

Contributor

@pritt20 pritt20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @neel-astro !!

@neel-astro neel-astro merged commit 1c2150e into main

May 26, 2025

6 checks passed

@neel-astro neel-astro deleted the fix/its branch

May 26, 2025 15:55

neel-astro added a commit that referenced this pull request

May 26, 2025
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pritt20 pritt20 pritt20 approved these changes

@jeremybeard jeremybeard Awaiting requested review from jeremybeard

@kushalmalani kushalmalani Awaiting requested review from kushalmalani

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@neel-astro @pritt20