Skip to content

Commit

Permalink
Merge 'Centralize Rust integration and regression tests' from Sonny
Browse files Browse the repository at this point in the history
## What?
- centralized Rust integration and regression tests
- no new tests added
- no tests removed
- only refactored tests into modules and common utils
## Why?
- @penberg and I have a discussion [here](https://github.com/tursodataba
se/limbo/pull/694#discussion_r1921949665) about centralizing the
integration/ regression tests so that they are not scattered around.
- this is a PR to do that and some refactor of the existing tests to
make the structure easier to navigate + add new tests in the future.

Reviewed-by: Jussi Saurio (@jussisaurio)
Reviewed-by: Preston Thorpe (@PThorpe92)
Reviewed-by: Pere Diaz Bou (@pereman2)

Closes #753
  • Loading branch information
jussisaurio committed Jan 21, 2025
2 parents c282b23 + a090fb9 commit 5b00e41
Show file tree
Hide file tree
Showing 17 changed files with 737 additions and 724 deletions.
5 changes: 2 additions & 3 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ members = [
"macros",
"simulator",
"sqlite3",
"test", "extensions/percentile",
"tests",
"extensions/percentile",
]
exclude = ["perf/latency/limbo"]

Expand Down
5 changes: 0 additions & 5 deletions cli/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,3 @@ miette = { version = "7.4.0", features = ["fancy"] }

[features]
io_uring = ["limbo_core/io_uring"]

# not testing the cli on windows as rexpect does not support it.
[target.'cfg(not(windows))'.dev-dependencies]
assert_cmd = "^2"
rexpect = "0.6.0"
5 changes: 0 additions & 5 deletions test/README.md

This file was deleted.

Loading

0 comments on commit 5b00e41

Please sign in to comment.