Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add entries method for looping through table rows #413

Merged
merged 2 commits into from
Nov 17, 2020

Conversation

zabil
Copy link
Member

@zabil zabil commented Nov 13, 2020

Fixes: #412

Usage:

step("Almost all words have vowels <table>", function(table) {
  table.entries((entry) => {
    assert.equal(numberOfVowels(entry["Word"]), parseInt(entry["Vowel Count"]));
  });
});

step("Almost all words have vowels <table>", function(table) {
  table.entries((entry) => {
    const { Word } = entry;
  });
});

Signed-off-by: Zabil Cheriya Maliackal zabilcm@gmail.com

Usage:
        table.entries(entry => console.log(entry["column name"]);

Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
Signed-off-by: Zabil Cheriya Maliackal <zabilcm@gmail.com>
@zabil zabil added feature ReleaseCandidate If a PR is tagged with this label, after merging it will be released labels Nov 16, 2020
@gaugebot
Copy link

gaugebot bot commented Nov 16, 2020

@zabil Thank you for contributing to gauge-js. Your pull request has been labeled as a release candidate 🎉🎉.

Merging this PR will trigger a release.

Please bump up the version as part of this PR.

Instructions to bump the version can found at CONTRIBUTING.md

If the CONTRIBUTING.md file does not exist or does not include instructions about bumping up the version, please looks previous commits in git history to see what changes need to be done.

@zabil zabil merged commit 1842916 into getgauge:master Nov 17, 2020
@zabil zabil deleted the tableentries branch April 21, 2022 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature ReleaseCandidate If a PR is tagged with this label, after merging it will be released
Development

Successfully merging this pull request may close these issues.

Feature request: access table data by column name instead of column index
2 participants