-
Notifications
You must be signed in to change notification settings - Fork 85
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
Rewrite transaction test #2053
Rewrite transaction test #2053
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #2053 +/- ##
==========================================
+ Coverage 90.23% 90.24% +0.01%
==========================================
Files 950 950
Lines 33776 33776
==========================================
+ Hits 30478 30482 +4
+ Misses 3298 3294 -4 ☔ View full report in Codecov by Sentry. |
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.
Let's remove all cpp tests that have been converted
test/test_files/transaction/create_rel/insert_rels_to_small_list.test
Outdated
Show resolved
Hide resolved
@@ -5,6 +5,7 @@ | |||
#include "common/exception/test.h" | |||
#include "common/string_utils.h" | |||
#include "spdlog/spdlog.h" | |||
#include "test_helper/test_helper.h" |
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.
this must be a duplicate include
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.
it needs to be included
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.
Address comments and rebase before you merge.
Also, update our documentation on testing framework after you merge the PR.
I have read and agree to the terms under CLA.md.
I have made the following changes:
Expand multiple connections to support default connection if not create connection previously
-use case:
-CREATE_CONNECTION conn_write
-STATEMENT [conn_write] BEGIN WRITE TRANSACTION
---- ok
-STATEMENT BEGIN READ TRANSACTION
---- ok
Expand the test framework to support batch statements; the file is loaded from KUZU_ROOT_PATH/test/statements
-use case:
-BATCH_STATEMENTS <FILE>:xx.cypher
---- ok
Rewrite the following tests to test_file