Skip to content

Added missed path in table-driven scenarios fix.#2807

Merged
sriv merged 1 commit intogetgauge:masterfrom
system-verification:feature/bug-fix-table-driven-scenarios
Dec 9, 2025
Merged

Added missed path in table-driven scenarios fix.#2807
sriv merged 1 commit intogetgauge:masterfrom
system-verification:feature/bug-fix-table-driven-scenarios

Conversation

@jensakejohansson
Copy link
Copy Markdown
Contributor

@jensakejohansson jensakejohansson commented Dec 8, 2025

Hi @sriv

I found glitch in #2791

When running this spec:

# Specification Heading 3
   |word1|count1|
   |-----|------|
   |a    |1     |
   |c    |0     |

## Vowel counts in single word3
   |word3|count3|
   |-----|------|
   |g    |0     |
   |a    |1     |
* Vowels in English language are "aeiou".
* The word <word1> has <count1> vowels.
* The word <word3> has <count3> vowels.
* Stop here

## Vowel counts in single word4
   |word4|count4|
   |-----|------|
   |g    |0     |
   |a    |1     |
* Vowels in English language are "aeiou".
* The word <word4> has <count4> vowels.

The scenario "Vowel counts in single word4" was 'duplicated' instead of reported as table driven:

image

WIth this PR that path is covered too and the reporting turns out as expected:

image

I have no test. I tried to write something that I felt was actually useful, not just there for the sake of being there, but didn't manage.

I see that the updated plugins got released. I've be running this build quite a lot. Maybe we should also release Gauge if you feel it is safe enough?

Signed-off-by: jensakejohansson <jens.johansson@systemverification.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

Benchmark Results

java_simple_multithreaded.csv

Commit CPU Memory Time ExitCode
bb872e3 33% 67488 0:11.24 0
bc05ae1 32% 65608 0:11.87 0
7d3f2cf 30% 65828 0:12.41 0
33dfb43 38% 65552 0:09.94 0

java_maven_multithreaded.csv

Commit CPU Memory Time ExitCode
bb872e3 68% 204612 0:16.22 0
bc05ae1 58% 206016 0:18.52 0
7d3f2cf 43% 180196 0:23.88 0
33dfb43 51% 178848 0:20.21 0

java_simple_serial.csv

Commit CPU Memory Time ExitCode
bb872e3 36% 67452 0:17.74 0
bc05ae1 45% 62876 0:14.33 0
7d3f2cf 43% 65616 0:14.85 0
33dfb43 44% 65580 0:14.35 0

java_gradle_multithreaded.csv

Commit CPU Memory Time ExitCode
bb872e3 9% 121168 0:24.11 0
bc05ae1 9% 124120 0:24.78 0
7d3f2cf 182% 580952 0:28.28 0
33dfb43 205% 545636 0:24.46 0

java_simple_parallel.csv

Commit CPU Memory Time ExitCode
bb872e3 20% 67792 0:27.00 0
bc05ae1 19% 67884 0:30.30 0
7d3f2cf 20% 67500 0:26.37 0
33dfb43 22% 68124 0:24.19 0

java_maven_serial.csv

Commit CPU Memory Time ExitCode
bb872e3 76% 220396 0:17.73 0
bc05ae1 78% 221940 0:17.37 0
7d3f2cf 48% 179544 0:27.44 0
33dfb43 58% 179708 0:22.96 0

java_gradle_parallel.csv

Commit CPU Memory Time ExitCode
bb872e3 6% 123416 0:42.18 0
bc05ae1 5% 115312 0:43.77 0
7d3f2cf 115% 557904 0:44.08 0
33dfb43 120% 569860 0:43.94 0

java_gradle_serial.csv

Commit CPU Memory Time ExitCode
bb872e3 9% 101224 0:25.11 0
bc05ae1 9% 106956 0:27.26 0
7d3f2cf 180% 559464 0:29.04 0
33dfb43 200% 553880 0:26.06 0

java_maven_parallel.csv

Commit CPU Memory Time ExitCode
bb872e3 36% 196088 0:33.94 0
bc05ae1 34% 194564 0:36.99 0
7d3f2cf 29% 181836 0:46.20 0
33dfb43 32% 178740 0:36.68 0

Notes

  • The results above are generated by running against seed projects in https://github.com/getgauge/gauge-benchmark
  • These results are not persisted, but on merging to master the benchmark will be rerun.
  • These benchmark are run in Github Actions' agents, which are virtualized. Results are not to be taken as actual benchmarks.Rather, these are indicative numbers and make sense for comparison.

See Workflow log for more details.

@gaugebot
Copy link
Copy Markdown

gaugebot bot commented Dec 9, 2025

@jensakejohansson Thank you for contributing to gauge. 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.

@sriv sriv merged commit 7ea39af into getgauge:master Dec 9, 2025
28 checks passed
@sriv
Copy link
Copy Markdown
Member

sriv commented Dec 9, 2025

the version is already bumped from a previous PR (that was supposed to be released!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants