Skip to content

perform_database_query_matcher should be truthy as long as one query matches#8471

Merged
mgrunberg merged 2 commits intomasterfrom
fix-perform_database_query_matcher
Sep 4, 2024
Merged

perform_database_query_matcher should be truthy as long as one query matches#8471
mgrunberg merged 2 commits intomasterfrom
fix-perform_database_query_matcher

Conversation

@mgrunberg
Copy link
Contributor

Found while working in #8470

Before this commit the order or SQL queries matter:

  • match, not match

results in a false matching when it should be true

matches

Before this commit the order or SQL queries matter:

- match, not match

results in a false matching when it should be true
@codecov
Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.11%. Comparing base (7117d59) to head (8185309).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8471   +/-   ##
=======================================
  Coverage   99.11%   99.11%           
=======================================
  Files         141      141           
  Lines        4069     4069           
=======================================
  Hits         4033     4033           
  Misses         36       36           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Co-authored-by: Javier Julio <JJfutbol@gmail.com>
@mgrunberg mgrunberg merged commit 1477e90 into master Sep 4, 2024
@mgrunberg mgrunberg deleted the fix-perform_database_query_matcher branch September 4, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants