Skip to content

Fail tests if plugin/package deployment fails init#530

Merged
eob merged 1 commit intomainfrom
deploy_plugin_test_fix
Aug 24, 2023
Merged

Fail tests if plugin/package deployment fails init#530
eob merged 1 commit intomainfrom
deploy_plugin_test_fix

Conversation

@eob
Copy link
Copy Markdown
Contributor

@eob eob commented Aug 24, 2023

wait_for_init only checks to see if the initializing phase completes -- not that it succeeded.

This adds an assertion to deploy_package and deploy_plugin that will more explicitly check that the initialization succeeded to help us weed through some of the random deployment errors that happen in test batteries.

@eob eob requested a review from GitOnUp August 24, 2023 19:34
@eob eob merged commit 5d3ee70 into main Aug 24, 2023
@eob eob deleted the deploy_plugin_test_fix branch August 24, 2023 20:22
Copy link
Copy Markdown
Contributor

@GitOnUp GitOnUp left a comment

Choose a reason for hiding this comment

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

🚢

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.

2 participants