feat: don't require sequenceId when executing a query

wayfair-incubator / gqmock

Project generated via @wayfair-incubator oss-template

MIT License

6 stars 2 forks source link

feat: don't require sequenceId when executing a query #28

Closed mjfaga closed 1 year ago

mjfaga commented 1 year ago

Description

Sequence Ids shouldn't be required when executing a query. In this state, we want to allow folks to use the mock server like they would any other mock server. In this state, they will just never be able to look up and leverage a seed. The mock server logs this in it's output.

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 #28 (5cce051) into main (becda79) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   94.26%   94.26%           
=======================================
  Files          13       13           
  Lines         471      471           
  Branches      143      144    +1     
=======================================
  Hits          444      444           
  Misses         27       27

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