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

Make rspec run all specs #441

Merged
merged 4 commits into from
Jul 5, 2017
Merged

Make rspec run all specs #441

merged 4 commits into from
Jul 5, 2017

Conversation

mvz
Copy link
Contributor

@mvz mvz commented Jun 30, 2017

Summary

These changes make the RSpec specs run to completion.

Details

These are some small fixes so RSpec doesn't abort prematurely:

  • Load deprecated method module because it is still included and its methods are still tested (and sometimes used).
  • Fix initialization of CommandMonitor; it always needs an announcer.
  • Fix some left-over instances of the moved #run method.
  • Load spec_helper in all spec files. This is mainly important when checking spec failures by running individual files.

Motivation and Context

This is a small part of #434 that hopefully can be merged more quickly so there is a basis for making the final fixes to the specs and then fixing the cucumber scenarios.

@mvz mvz mentioned this pull request Jun 30, 2017
@maxmeyer maxmeyer merged commit ec27803 into cucumber:master Jul 5, 2017
@mvz mvz deleted the make-rspec-run-all-specs branch July 7, 2017 08:52
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