Fix issue with Initial values

[davidjgonzalez]

Fixes #1215

Description

Issue with predicate group assignmnent. This was because the isReady() check in initGroup relied on the group Id making a cyclical-ish condition. Removed the isReady check in initGroup.. the only downside is groupId may not always be sequential (if there's a !isReady predicate between two "ready" predicates) however this doesnt have any material impact.

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Project coverage is 27.99%. Comparing base (3ed3d6b) to head (c4b6e1a).
Report is 6 commits behind head on develop.

Files with missing lines	Patch %	Lines
...share/components/predicates/AbstractPredicate.java	0.00%	19 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             develop    #1218      +/-   ##
=============================================
- Coverage      27.99%   27.99%   -0.01%     
- Complexity       552      553       +1     
=============================================
  Files            182      182              
  Lines           4772     4769       -3     
  Branches         680      679       -1     
=============================================
- Hits            1336     1335       -1     
+ Misses          3246     3245       -1     
+ Partials         190      189       -1     

Flag	Coverage Δ
unittests	27.99% <0.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.