Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Omnibus test change branch #58

Merged
merged 4 commits into from
Dec 9, 2022
Merged

Conversation

michaelglass
Copy link
Contributor

@michaelglass michaelglass commented Dec 9, 2022

this is an omnibus branch included a number of test improvements:

if you want to review the code properly, it's going to be easier looking at those individual PRs

@michaelglass michaelglass enabled auto-merge (squash) December 9, 2022 10:36
* include fork of rspec-snapshot that fails the build when the snapshot is written

* confirm missing snapshots fail the build

* Revert "confirm missing snapshots fail the build"

This reverts commit b4e3dd2.
* graceful exit within abq was masking rspec tests on older versions of rspec

* add an additional exit safeguard

* check num tests vs expected tests

* sanitize backtrace tweaks

* regenerate snapshots

* break test to ensure bin/check_num_tests.rb works

* Revert "break test to ensure bin/check_num_tests.rb works"

This reverts commit 1da32a2.
* use captain cli

* unset RWX_ACCESS_TOKEN to be compatible with captain-cli
@michaelglass michaelglass force-pushed the omnibus-test-change-branch branch from b7ed95f to a28a78f Compare December 9, 2022 14:25
Copy link
Member

@tonywok tonywok left a comment

Choose a reason for hiding this comment

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

🔒

@michaelglass michaelglass merged commit 00e5559 into main Dec 9, 2022
@michaelglass michaelglass deleted the omnibus-test-change-branch branch December 9, 2022 14:27
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