Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Address issue where the not_to(contain) not run correctly #1246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 19, 2023

Conversation

lwasylow
Copy link
Member

Address issue where the not_to(contain) executes a negated matcher wi…thout negated flag set causing refcursor compare sql generated inclusion comparision.
fixes #1245

…thout negated flag set causing refcursor compare sql generated inclusion comparision.
@lwasylow lwasylow requested a review from jgebal March 15, 2023 06:12
@github-actions
Copy link
Contributor

github-actions bot commented Mar 15, 2023

Unit Test Results

       1 files         2 suites   5m 8s ⏱️
1 034 tests 1 030 ✔️ 4 💤 0
1 037 runs  1 029 ✔️ 8 💤 0

Results for commit 8460a30.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02 🎉

Comparison is base (5cf7f34) 94.86% compared to head (8460a30) 94.88%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1246      +/-   ##
===========================================
+ Coverage    94.86%   94.88%   +0.02%     
===========================================
  Files          108      108              
  Lines         5105     5106       +1     
  Branches        23       23              
===========================================
+ Hits          4843     4845       +2     
+ Misses         255      254       -1     
  Partials         7        7              
Flag Coverage Δ
11XE 94.25% <100.00%> (+0.08%) ⬆️
12.1EE 94.31% <100.00%> (+0.04%) ⬆️
18XE 94.50% <100.00%> (+0.02%) ⬆️
21XE 94.68% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
source/expectations/matchers/ut_contain.tpb 88.23% <100.00%> (+0.73%) ⬆️
source/expectations/ut_expectation.tpb 99.37% <100.00%> (+0.62%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@jgebal jgebal merged commit 6d77183 into develop Mar 19, 2023
@jgebal jgebal deleted the bug/1245_not_to_contains_fails branch March 19, 2023 16:55
@jgebal jgebal added this to the v3.1.14 milestone May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

not_to(contain) fails while not_to_contain passes
2 participants