Cucumber has built in syntax check:
$ cucumber -h
(...)
-d, --dry-run Invokes formatters without executing the steps.
This also omits the loading of your support/env.rb file if it exists.(...)
We should run it for all repositories that contain Cucumber code[1].
1: https://github.com/wikimedia/mediawiki-selenium
Version: wmf-deployment
Severity: normal
Whiteboard: zfilipin
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | hashar | T119138 [keyresult] Migrate majority of CI jobs to Nodepool (part 2) | |||
| Duplicate | None | T136249 Track what tests we have left to convert to nodepool | |||
| Resolved | hashar | T119140 [keyresult] Migrate as many misc CI jobs as possible to Nodepool | |||
| Resolved | hashar | T114315 [keyresult] Migrate majority of CI jobs to Nodepool (part 1) | |||
| Declined | None | T58251 Cucumber linter should run for all repositories that contain Cucumber code | |||
| Resolved | zeljkofilipin | T117993 Add Rakefile to repositories with Ruby code | |||
| Resolved | zeljkofilipin | T114860 Move Bundler Jenkins jobs to Nodepool instances | |||
| Resolved | • dduvall | T104024 Define a CI entry point for Ruby code | |||
| Duplicate | zeljkofilipin | T114262 Delete ruby2.0lint job and only run bundle-rubocop job for repositories with Ruby code |
UniversalLanguageSelector/tests/browser$ cucumber --dry-run
42 scenarios (42 skipped)
317 steps (317 skipped)
0m0.600s
"cucumber -d" runs fine for:
browsertests, Flow, UniversalLanguageSelector
and fails with an error message similar to this for the rest of the repos:
$ cucumber -d
uninitialized constant ArticlePage::PageObject (NameError)
/home/z/project/CirrusSearch/tests/browser/features/support/pages/article_page.rb:2:in `<class:ArticlePage>'
/home/z/project/CirrusSearch/tests/browser/features/support/pages/article_page.rb:1:in `<top (required)>'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/rb_support/rb_language.rb:122:in `load'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/rb_support/rb_language.rb:122:in `load_code_file'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime/support_code.rb:180:in `load_file'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime/support_code.rb:83:in `block in load_files!'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime/support_code.rb:82:in `each'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime/support_code.rb:82:in `load_files!'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime.rb:183:in `load_step_definitions'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/runtime.rb:42:in `run!'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/lib/cucumber/cli/main.rb:47:in `execute!'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/gems/cucumber-1.3.8/bin/cucumber:13:in `<top (required)>'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/bin/cucumber:23:in `load'
/home/z/.rvm/gems/ruby-2.0.0-p247@CirrusSearch/bin/cucumber:23:in `<main>'
As the documentation says, -d option does not load env.rb file, and that causes the problem.
Looks like page-object gem is not required in some repos. This works for all repos (page-object gem has to be installed):
cucumber -d -r page-object
Don't you need to use bundler to run it ? Supposedly page object would be provided so:
bundle install
bundle exec cucumber -d
What is the use case to run that command ? The browser test jobs run cucumber already so that sounds redundant to me.
Was just wondering since that task has been opened back in October 2013. I guess the cucumber linting can be added to a future rake test entry point.
I have stumbled upon this task while cleaning up browser tests board. I was forgotten for a while, hence no activity. :(
We have a Rakefile entry point now ( T117993 ). So the cucumber linter can be added as needed.
This is 2+years old task. The main use case for cucumber is for the selenium tests and we are migrating to webdriver.io / Mocha