Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Issue #16 Support for table-based steps - current tests pass #26

Merged
merged 1 commit into from

3 participants

@kris-at-tout

Very simple change to make tables in scenario outlines work for both text and numerical values.

@indexzero
Owner

@pacovell Thoughts? I should just give you admin access here >.<

@pacovell

Great, thank you - I will review ASAP

@pacovell pacovell merged commit 9f4b73f into nodejitsu:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/kyuri/runners/cucumber.js
View
2  lib/kyuri/runners/cucumber.js
@@ -106,7 +106,7 @@ Cucumber.prototype.run = function (features, steps, callback) {
Object.keys(step).forEach(function (i) {
exampleStep[i] = step[i].slice(0); // copy the array
scenario.exampleVariables.forEach(function (variable) {
- exampleStep[i][1] = exampleStep[i][1].replace('\<' + variable + '\>', example[variable]);
+ exampleStep[i][1] = exampleStep[i][1].replace('\<' + variable + '\>', '"' + example[variable] + '"');
});
});
steps.push(exampleStep);
Something went wrong with that request. Please try again.