Skip to content

feat: More cross-track itest assertions#314

Draft
MichaelThamm wants to merge 6 commits intomainfrom
fix/itests
Draft

feat: More cross-track itest assertions#314
MichaelThamm wants to merge 6 commits intomainfrom
fix/itests

Conversation

@MichaelThamm
Copy link
Copy Markdown
Contributor

@MichaelThamm MichaelThamm commented May 1, 2026

Issue

  • Consider moving everything to a quality_gate method to keep the tests DRY

Solution

Checklist

  • I have added or updated relevant documentation.
  • PR title makes an appropriate release note and follows conventional commits syntax.
  • Merge target is the correct branch, and relevant tandem backport PRs opened.
  • Check if replace juju_application is the only (if even) case which causes data loss

Context

Testing Instructions

Upgrade Notes

Comment thread tests/integration/helpers.py Outdated
Comment thread tests/integration/helpers.py
Comment thread tests/integration/cos_lite/tls_full/test_upgrade_cos_lite_tls_full.py Outdated
wait_for_active_idle_without_error([ca_model, cos_model])
tls_ctx = get_tls_context(tmp_path, ca_model, "self-signed-certificates")
catalogue_apps_are_reachable(cos_model, tls_ctx)
assert not tf_manager.plan_has_changes(**jubilant_args)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We probably don't want this assertion and can say in docs to "run terraform apply until no more changes"

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant