diegoholiveira / jsonlogic

Go Lang implementation of JsonLogic

MIT License

159 stars 48 forks source link

Issue Fix #56 #57

Closed vivekagrawal2810 closed 2 years ago

vivekagrawal2810 commented 2 years ago

solveVars func call for getting conditions should be called for each value of an array which is the first argument of an array. Similar logic is written javascript library as well. Second Rule which uses in operator will get evaluated properly if above solveVars func returns correct rule evaluated string value.

Also added the test case to test the fix. All other test cases passed successfully.

codecov[bot] commented 2 years ago

Codecov Report

Merging #57 (9862c50) into master (d34d6a7) will decrease coverage by 0.00%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   96.00%   95.99%   -0.01%     
==========================================
  Files           9        9              
  Lines         925      924       -1     
==========================================
- Hits          888      887       -1     
  Misses         25       25              
  Partials       12       12