feat: integration test suite #487
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
waynexia
force-pushed
the
integration-test
branch
from
692a5a5 to
e3564a2
Compare
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Codecov Report
@@ Coverage Diff @@
## develop #487 +/- ##
===========================================
- Coverage 86.41% 86.35% -0.06%
===========================================
Files 400 401 +1
Lines 50671 50769 +98
===========================================
+ Hits 43788 43843 +55
- Misses 6883 6926 +43
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
license-eye has totally checked 615 files.
| Valid | Invalid | Ignored | Fixed |
|---|---|---|---|
| 520 | 3 | 92 | 0 |
Click to see the invalid file list
- tests/runner/src/env.rs
- tests/runner/src/main.rs
- tests/runner/src/util.rs
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
|
Maybe better to mention that we need to clean the data directory before runing test. |
I plan to write a document in dev-guide to describe how to run and add integration test when this gets merged. I'll mention this here |
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
paomian
pushed a commit
to paomian/greptimedb
that referenced
this pull request
Oct 19, 2023
Signed-off-by: Ruihang Xia <waynestxia@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes Included
Add a SQL integration test suites and a group of basic SQLs from "get started" document. This test suite only supports local standalone mode now.
To run it, use
cd tests/runner cargo runNotes
The dependency
sqlnessis not published, I'm discussing publish procedure with CeresDB authors (it is derived from there). When things done I'll update it to crates.io registry.Known problem
The start command is
cargo run -- datanode start, which may cause zombie process when closing. It should be run from the binary directly.Due to the
DROP TABLEclause is not supported now, tests cannot clear its state and will result to "table already exist" when re-run it.