search

mickhansen / graphql-sequelize

GraphQL & Relay for MySQL & Postgres via Sequelize
MIT License
1.9k stars 172 forks source link

replaceWhereOperators helper should not mutate original argument (#624) #625

Closed zandroid closed 6 years ago

zandroid commented 6 years ago

Add test and fix the problem #624

codecov-io commented 6 years ago

Codecov Report

Merging #625 into master will decrease coverage by 0.22%. The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #625      +/-   ##
==========================================
- Coverage    91.7%   91.48%   -0.23%     
==========================================
  Files          13       13              
  Lines         410      411       +1     
==========================================
  Hits          376      376              
- Misses         34       35       +1
Impacted Files Coverage Δ
src/replaceWhereOperators.js 93.33% <85.71%> (-6.67%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 80e5d8b...f2fedd4. Read the comment docs.