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

Add Rack to RSpec #625

Merged
merged 3 commits into from
Nov 15, 2018
Merged

Add Rack to RSpec #625

merged 3 commits into from
Nov 15, 2018

Conversation

delner
Copy link
Contributor

@delner delner commented Nov 7, 2018

This pull request converts the Rack minitests to RSpec.

@delner delner added dev/testing Involves testing processes (e.g. RSpec) dev/refactor Involves refactoring existing components labels Nov 7, 2018
@delner delner self-assigned this Nov 7, 2018
@delner delner force-pushed the refactor/add_rack_spec branch 2 times, most recently from 4660ad4 to 0e3deb0 Compare November 8, 2018 17:12
Copy link
Contributor

@pawelchcki pawelchcki left a comment

Choose a reason for hiding this comment

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

Great stuff!

@delner delner merged commit 1ec81df into 0.18-dev Nov 15, 2018
@delner delner deleted the refactor/add_rack_spec branch November 15, 2018 14:45
@mikhailov
Copy link

@delner would you be interested to consider Process.clock_gettime(Process::CLOCK_MONOTONIC) instead of Time.now?

@delner
Copy link
Contributor Author

delner commented Feb 6, 2020

Yeah, this has come up before in #424 and I'd be willing to consider it for the sake of improving accuracy. Ideally such an implementation would take minimal changes and not require the use of any other libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/refactor Involves refactoring existing components dev/testing Involves testing processes (e.g. RSpec)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants