Fix BPNL Group validation operators logic

[bmg13]

WHAT

Fixes some of the existing BPN Group validation operators' scope and proposes the removal of potentially redundant ones.

WHY

Too keep the expected behaviour of some operators, specially the isAllOf since was not validating properly (switched the assigned and allowed groups in the validation).

FURTHER NOTES

The suggestions are based on expected behaviour (isAllOf should check if all the allowed groups are present in the assigned groups) and potential improvements (eq e neq could maybe be removed since they do not seem to add much value when isAllOf, isAnyOf and isNoneOf exist).

Changed the BpnGroupHolder properties to Sets simply to easily avoid repeated entries.

Tests updated accordingly.

Closes #1665

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud