-
Notifications
You must be signed in to change notification settings - Fork 883
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
Add hypercore chunk tests for triggers #7483
Conversation
1a7f4b1
to
1510ded
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7483 +/- ##
==========================================
+ Coverage 80.06% 82.18% +2.11%
==========================================
Files 190 230 +40
Lines 37181 43188 +6007
Branches 9450 10867 +1417
==========================================
+ Hits 29770 35495 +5725
- Misses 2997 3371 +374
+ Partials 4414 4322 -92 ☔ View full report in Codecov by Sentry. |
@gayyappan, @erimatnor: please review this pull request.
|
1510ded
to
22e974d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one minor question.
22e974d
to
7c4a55b
Compare
12be248
to
e727459
Compare
Add tests for statement and row triggers on hypercore chunks. We test `INSERT`, `UPDATE`, `DELETE`, and `TRUNCATE` triggers, where applicable. We check that the `OLD` and `NEW` values are correct as well as that the statement and row triggers are executed the correct number of times.
e727459
to
17d35ed
Compare
Add tests for statement and row triggers on hypercore chunks. We test
INSERT
,UPDATE
,DELETE
, andTRUNCATE
triggers, where applicable. We check that theOLD
andNEW
values are correct as well as that the statement and row triggers are executed the correct number of times.Disable-check: force-changelog-file