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

Generator ignores all groups except last one #31

Closed
emilzse opened this issue Apr 27, 2020 · 3 comments · Fixed by #33
Closed

Generator ignores all groups except last one #31

emilzse opened this issue Apr 27, 2020 · 3 comments · Fixed by #33
Assignees
Labels
bug Something isn't working
Milestone

Comments

@emilzse
Copy link

emilzse commented Apr 27, 2020

Describe the bug
When having a regex with several groups it ignores all except the last one

To Reproduce
Steps to reproduce the behavior:

  1. "^(1)|(2)$"
RgxGen rgxGen = new RgxGen(regex);
StringIterator uniqueStrings = rgxGen.iterateUnique();
Set<String> strings = new HashSet<>();
uniqueStrings.forEachRemaining(strings::add);
return strings; // [2]

Expected behavior
It should return [1, 2]

Screenshots

Environment (please complete the following information):

  • RgxGen Version 1.1
@emilzse emilzse added the bug Something isn't working label Apr 27, 2020
@curious-odd-man curious-odd-man added this to the Version 1.2 milestone Apr 27, 2020
curious-odd-man added a commit that referenced this issue Apr 27, 2020
curious-odd-man added a commit that referenced this issue Apr 27, 2020
Regression test is added.
curious-odd-man added a commit that referenced this issue Apr 27, 2020
curious-odd-man added a commit that referenced this issue Apr 27, 2020
@curious-odd-man curious-odd-man linked a pull request Apr 27, 2020 that will close this issue
curious-odd-man added a commit that referenced this issue Apr 27, 2020
…cognized

#31.top level choich is not recognized
@curious-odd-man
Copy link
Owner

curious-odd-man commented Apr 27, 2020

Hello @emilzse .

It was an easy fix.
I'll keep ticket open for now, but later this week i'll deploy 1.2-SNAPSHOT, so you will be able to test yourself.

I'll post a notification then.

Best Regards!

@curious-odd-man
Copy link
Owner

Hello @emilzse.

Version 1.2 Snapshot is out live. Please refer to Readme how to use it, if you need that information.

@curious-odd-man
Copy link
Owner

Considering the issue resolved. Closing the issue due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants