Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,12 @@ gemspec
if ENV['CUCUMBER_RUBY_CORE']
gem 'cucumber-core', path: ENV['CUCUMBER_RUBY_CORE']
elsif !ENV['CUCUMBER_USE_RELEASED_GEMS']
gem 'cucumber-core', github: 'cucumber/cucumber-ruby-core', branch: 'gherkin-7'
gem 'cucumber-core', github: 'cucumber/cucumber-ruby-core', branch: 'gherkin-8'
end

if ENV['CUCUMBER_RUBY']
gem 'cucumber', path: ENV['CUCUMBER_RUBY']
elsif !ENV['CUCUMBER_USE_RELEASED_GEMS']
gem 'cucumber', github: 'cucumber/cucumber-ruby', branch: 'gherkin-7'
gem 'cucumber', github: 'cucumber/cucumber-ruby', branch: 'gherkin-8'
end

2 changes: 1 addition & 1 deletion cucumber-wire.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Gem::Specification.new do |s|
s.required_ruby_version = '>= 2.3'

s.add_dependency 'cucumber-core', '~> 5.0', '>= 5.0.0'
s.add_dependency 'cucumber-expressions', '~> 8.0', '>= 8.0.0'
s.add_dependency 'cucumber-expressions', '~> 8.0', '>= 8.0.2'

s.add_development_dependency 'cucumber', '~> 3.1', '>= 3.1.2'
s.add_development_dependency 'rake', '~> 12.3', '>= 12.3.3'
Expand Down
4 changes: 2 additions & 2 deletions features/snippets_message.feature
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ Feature: Snippets message
Given there is a wire server running on port 54321 which understands the following protocol:
| request | response |
| ["step_matches",{"name_to_match":"we're all wired"}] | ["success",[]] |
| ["snippet_text",{"step_keyword":"Given","multiline_arg_class":"","step_name":"we're all wired"}] | ["success","foo()\n bar;\nbaz"] |
| ["snippet_text",{"step_keyword":"Given","multiline_arg_class":"","step_name":"we're all wired"}] | ["success","foo()\n\tbar;\nbaz"] |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bad idea, the \t is not considered as a real tab, I'll fix that ...

| ["begin_scenario"] | ["success"] |
| ["end_scenario"] | ["success"] |
When I run `cucumber -f pretty`
Expand All @@ -41,7 +41,7 @@ Feature: Snippets message
And the output should contain:
"""
foo()
bar;
\tbar;
baz

"""
4 changes: 2 additions & 2 deletions lib/cucumber/wire/add_hooks_filter.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ def test_case(test_case)
end

def before_hook(test_case)
# TODO: is this dependency on Cucumber::Hooks OK? Feels a bit internal..
# TODO: how do we express the location of the hook? Should we create one hook per connection so we can use the host:port of the connection?
# TODO: is this dependency on Cucumber::Hooks OK? Feels a bit internal..
# TODO: how do we express the location of the hook? Should we create one hook per connection so we can use the host:port of the connection?
Cucumber::Hooks.before_hook(Core::Test::Location.new('TODO:wire')) do
connections.begin_scenario(test_case)
end
Expand Down
2 changes: 1 addition & 1 deletion lib/cucumber/wire/connections.rb
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ def initialize(connections, configuration, registry)
def find_match(test_step)
matches = step_matches(test_step.name)
return unless matches.any?
# TODO: handle ambiguous matches (push to cucumber?)
# TODO: handle ambiguous matches (push to cucumber?)
matches.first
end

Expand Down
2 changes: 1 addition & 1 deletion lib/cucumber/wire/protocol/requests.rb
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def handle_pending(message)
end

def handle_diff!(tables)
# TODO: figure out if / how we could get a location for a table from the wire (or make a null location)
# TODO: figure out if / how we could get a location for a table from the wire (or make a null location)
location = Core::Test::Location.new(__FILE__, __LINE__)
table1 = table(tables[0], location)
table2 = table(tables[1], location)
Expand Down