https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #20 from Željko Filipin ---
Thanks Dan, I have tested this and it works great! :)
--
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
__
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
Dan Duvall changed:
What|Removed |Added
Status|PATCH_TO_REVIEW |RESOLVED
Resolution|---
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #18 from Gerrit Notification Bot ---
Change 160024 merged by jenkins-bot:
Stricter pending behavior for falsely passing steps
https://gerrit.wikimedia.org/r/160024
--
You are receiving this mail because:
You are the assignee for
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
Gerrit Notification Bot changed:
What|Removed |Added
Status|NEW |PATCH_TO_REVIEW
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #17 from Gerrit Notification Bot ---
Change 160024 had a related patch set uploaded by Dduvall:
Stricter pending behavior for falsely passing steps
https://gerrit.wikimedia.org/r/160024
--
You are receiving this mail because:
You
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #16 from Željko Filipin ---
(In reply to Dan Duvall from comment #15)
> I think I'd still rather just reimplement pending for our own Cucumber World
> (see comment 9) over monkey patching Cucumber::Pending (#1) or adding
> another d
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #15 from Dan Duvall ---
I think I'd still rather just reimplement pending for our own Cucumber World
(see comment 9) over monkey patching Cucumber::Pending (#1) or adding another
dependency (#2).
--
You are receiving this mail bec
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #14 from Željko Filipin ---
I see, that was what I misunderstood. Thanks. :)
Ok then, so we could do three things:
#1 patch Cucumber::Pending
#2 require RSpec::Core::Pending and use it
#3 report a bug for Cucumber::Pending sayin
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #13 from Dan Duvall ---
Right, but rspec-expectations is separate from rspec-core which is what
implements `describe`, `it`, `pending`, etc.
--
You are receiving this mail because:
You are the assignee for the bug.
You are on the
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #12 from Željko Filipin ---
As far as I understand it, Cucumber does use rspec, that is why we explicitly
use rspec-expectations[2].
1:
https://github.com/wikimedia/mediawiki-selenium/blob/master/mediawiki_selenium.gemspec#L26
--
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #11 from Dan Duvall ---
I'm not sure that cucumber even uses `rspec-core`. It might just use the
matchers. Either way, I think implementing our own helper is more inline with
the Cucumber Way—not that I think the Cucumber Way is gre
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #10 from Željko Filipin ---
I think just my Ruby-fu is failing me. See comments 6 and 7. RSpec::Pending
already does what I want, but looks like I am calling Cucumber::Pending here:
pending('bug #56081') do
true.should be(true)
e
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #9 from Dan Duvall ---
We could reimplement `#pending` in a world helper module and rescue/re-raise
the `Cucumber::Pending` exception as something else.
module MediawikiSelenium::Pending
class Failure < StandardError; end
def
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
Željko Filipin changed:
What|Removed |Added
CC||dduv...@wikimedia.org
Whitebo
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
Željko Filipin changed:
What|Removed |Added
Summary|Cucumber step should fail |Cucumber step should fail
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #7 from Željko Filipin ---
I am not sure how to call RSpec#pending from Cucumber, I have tried this:
RSpec::Core::Pending.pending('bug #56081')
but no luck:
uninitialized constant RSpec::Core (NameError)
--
You are receiving th
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #6 from Željko Filipin ---
I think the problem is that we are using Cucumber::Pending instead of
RSpec::Pending, and looks like Cucumber does not behave like RSpec.
--
You are receiving this mail because:
You are the assignee for
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #5 from Željko Filipin ---
But if change the expectation to the example from rspec documentation (so the
expectation inside the pending block no longer fails:
pending('bug #56081') do
true.should be(true)
end
I was
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #3 from Željko Filipin ---
If the step is marked as pending, the default Cucumber formatter (progress)
does not give a lot of information:
$ bundle exec cucumber features/font_selection.feature:23
Using the default profile...
...
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #4 from Željko Filipin ---
But formatter called "pretty" gives more information (... 'bug #56081
(Cucumber::Pending)' ...):
$ bundle exec cucumber -f pretty features/font_selection.feature:23
Using the default profile...
@commons
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #2 from Željko Filipin ---
Cucumber output if the step is not marked pending:
$ bundle exec cucumber features/font_selection.feature:23
Using the default profile...
F-
(::) failed steps (::)
expected: "sans-serif"
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
--- Comment #1 from Željko Filipin ---
The documentation says I am expecting the correct behavior:
https://github.com/rspec/rspec-core/blob/master/lib/rspec/core/pending.rb?source=cc#L50-L57
--
You are receiving this mail because:
You are th
https://bugzilla.wikimedia.org/show_bug.cgi?id=56243
Željko Filipin changed:
What|Removed |Added
Summary|Mark a scenario as failed |Cucumber step should fail
23 matches
Mail list logo