CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Sorry I closed my previous PR by accident. The reason I didn't use GrayReleaseRuleItemTransformer#batchTransformFromJSON is once we transform it into object, the object's toString() method is not suitable for the contains method and still requires us to perform string manipulation.

I have read the CLA Document and I hereby sign the CLA

recheck

Codecov Report

Merging #4599 (0473cd6) into master (75ae5cc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4599   +/-   ##
=========================================
  Coverage     47.23%   47.23%           
  Complexity     1649     1649           
=========================================
  Files           347      347           
  Lines         10631    10631           
  Branches       1053     1053           
=========================================
  Hits           5022     5022           
  Misses         5306     5306           
  Partials        303      303           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

The unit test failed, which seems related to the line of code

Set<GrayReleaseRuleItemDTO> contextRules = GrayReleaseRuleItemTransformer.batchTransformFromJSON(context.substring(context.indexOf("["),context.lastIndexOf("]")+1));

The unit test failed, which seems related to the line of code

Set<GrayReleaseRuleItemDTO> contextRules = GrayReleaseRuleItemTransformer.batchTransformFromJSON(context.substring(context.indexOf("["),context.lastIndexOf("]")+1));

Yeah that seems to be where the problem lies. I have changed it to transfer the context to a map struct to avoid hardcoding. Now it should work fine.