Default group sequence does not honour group hierarchy

Description

Assuming the following

The following code returns as expected

But the following fails

Note that the actual test used group sequence provider but that's not relevant I think.

Environment

None

Activity

Show:
Scott Marlow
February 17, 2016, 6:51 PM

Hi Gunnar, will do.

Scott Marlow
February 17, 2016, 8:35 PM

WildFly pull request is at https://github.com/wildfly/wildfly/pull/8703, after that passes, will start the EAP pull request.

Scott Marlow
February 17, 2016, 9:08 PM

https://github.com/wildfly/wildfly/pull/8703 is failing in a validation test on WildFly. Error is javax.validation.GroupDefinitionException: HV000048: Unable to expand group sequence."

Gunnar Morling
February 18, 2016, 8:20 AM

Ah, so the fix actually reveals an issue with the default group sequence provider used in the tests.

It isn't save for repeated invocations (which it has to be) and adds more and more elements to a sequence object which it holds in a field as it is invoked several times. I'll send a PR with the HV update and the fix for that.

Gunnar Morling
February 18, 2016, 9:22 AM

, I've filed PR https://github.com/wildfly/wildfly/pull/8706 based on top of yours which also fixes that issue in the test suite.

Assignee

Gunnar Morling

Reporter

Emmanuel Bernard

Labels

None

Feedback Requested

None

Feedback Requested By

None

backPortable

None

Suitable for new contributors

None

backportDecision

None

backportReEvaluate

None

Components

Fix versions

Priority

Blocker
Configure