Skip to content

Explicitly call root test runner class to avoid a confusing error when test runner gem is not loaded #120

Merged
ArturT merged 6 commits intomasterfrom
prefix-other-gem-const
Jul 18, 2020
Merged

Explicitly call root test runner class to avoid a confusing error when test runner gem is not loaded #120
ArturT merged 6 commits intomasterfrom
prefix-other-gem-const

Conversation

@ArturT
Copy link
Copy Markdown
Member

@ArturT ArturT commented Jul 18, 2020

Explicitly call root test runner class to avoid a confusing error when test runner gem is not loaded

why

These changes will help us see more explicit and clear errors when test runner gem like RSpec is not loaded properly.
For instance, it could help avoid misunderstanding of error like #119

changes applied

  • Use ::RSpec instead of RSpec to be more explicit about using gem RSpec
  • Use ::Cucumber instead of Cucumber
  • Use ::Minitest instead of Minitest
  • Use ::Test::Unit instead of Test::Unit
  • Use ::Turnip instead of Turnip

@ArturT ArturT merged commit 63acdeb into master Jul 18, 2020
@ArturT ArturT deleted the prefix-other-gem-const branch July 18, 2020 21:16
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.

1 participant