You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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 the isAllOf 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 and TEST_GROUP_2 part of the allowed groups, this case should return true and not false
Expected behavior
Operator EQ: must be exactly in - and only in - that particular group or set of groups
Operator IS_ALL_OF: must be in all of the specified groups
The text was updated successfully, but these errors were encountered:
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 groupsOperator
IS_ALL_OF
: must be in all of the specified groupsThe text was updated successfully, but these errors were encountered: