PolymerLabs / arcs

Arcs
BSD 3-Clause "New" or "Revised" License
56 stars 35 forks source link

AssertVariableOrdering needs to check all permutations for group items. #7094

Closed copybara-service[bot] closed 3 years ago

copybara-service[bot] commented 3 years ago

AssertVariableOrdering needs to check all permutations for group items.

With the current approach, an input of [a,x,a,y] will fail against grp(seq(a,y), seq(a,x)) because the first sequence will match the first 'a' and the 'y' at the end, leaving [x,a] which won't match seq(a,x).