Closed rafaelmag110 closed 2 months ago
Can I assign this issue to me? @ndr-brt @rafaelmag110
Please have a look at: https://github.com/eclipse-tractusx/sig-release/issues/775#issuecomment-2242235805, there is an additional issue with operators
The issue was solved on the following Pull Request: https://github.com/eclipse-tractusx/tractusx-edc/pull/1456
Describe the bug
Using the
isAllOf
operator in a policy, the validation fails if a BPNL belongs to more groups than the ones specified in the policy.In the BPNL validation extension, the validation function evaluates the
EQ
and theisAllOf
operators in the same way, however even the documentation states they should evaluate the groups differently, as shown in the expected behaviour.Also, in the
BusinessPartnerGroupFunctionTest
class, the following circumstance is tested:Arguments.of("Overlapping groups", IS_ALL_OF, List.of(TEST_GROUP_1, TEST_GROUP_2, "different-group", "another-different-group"), false)
And being
TEST_GROUP_1
andTEST_GROUP_2
part of the allowed groups, this case should returntrue
and notfalse
Expected behavior
Operator
EQ
: must be exactly in - and only in - that particular group or set of groups OperatorIS_ALL_OF
: must be in all of the specified groups