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

CI: multiverse - unpin webrick #2143

Merged
merged 2 commits into from
Jul 26, 2023
Merged

CI: multiverse - unpin webrick #2143

merged 2 commits into from
Jul 26, 2023

Conversation

fallwith
Copy link
Contributor

@fallwith fallwith commented Jul 25, 2023

  • Update the fake server based classes used for unit tests to run on the latest version of WEBrick (previously we were not compatible with v1.8.0+ and locked to v1.7.0)
  • Update the fake server class to respect ENV['DEBUG']. Going forward if there are any errors happening that are fake server related, we need only to set the DEBUG environment variable to see what is going on.
  • Update the fake collector class to stop attempting to call #rewind on objects that don't support it (for improved WEBrick compatibility)

* update FakeServer to respect `ENV['DEBUG']` and use $stdout instead of
  /dev/null for all output when set
* update FakeCollector to not attempt `#rewind` on objects that don't
  support it
@fallwith fallwith marked this pull request as ready for review July 26, 2023 00:48
@github-actions
Copy link

SimpleCov Report

Coverage Threshold
Line 94.26% 94%
Branch 85.75% 85%

@fallwith
Copy link
Contributor Author

@fallwith
Copy link
Contributor Author

resolves #1774

@fallwith fallwith merged commit 4a45984 into dev Jul 26, 2023
115 checks passed
@fallwith fallwith deleted the multiverse_unpin_webrick branch July 26, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants