fix: resolve matching arguments correctly + support nested arguments

wayfair-incubator / gqmock

Project generated via @wayfair-incubator oss-template

MIT License

6 stars 2 forks source link

fix: resolve matching arguments correctly + support nested arguments #30

Closed mjfaga closed 1 year ago

mjfaga commented 1 year ago

Description

Nested arguments weren't previously supported. Now we recursively loop through arguments and compare values
Refactored to loop over the match arguments (instead of the execution arguments) to better support partial argument matching and the default {} matching argument values.

Type of Change

[x] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] Functionality is documented
[x] All code style checks pass
[x] New code contribution is covered by automated tests
[x] All new and existing tests pass

codecov[bot] commented 1 year ago

Codecov Report

Merging #30 (ce56a5b) into main (0ff146e) will increase coverage by 0.09%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   94.35%   94.44%   +0.09%     
==========================================
  Files          13       13              
  Lines         478      486       +8     
  Branches      147      147              
==========================================
+ Hits          451      459       +8     
  Misses         27       27

Files Changed	Coverage Δ
src/seed/SeedManager.ts	`100.00% <100.00%> (ø)`