Skip to content

Commit 25ff291

Browse files
authored
Tests: Remove queue overrides / limit concurrency (#433)
Ensure all backend variations are properly end-to-end tested by removing in-memory queue override; this currently requires limiting test concurrency to 1 because tests are currently sharing backend services.
1 parent 037e34c commit 25ff291

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

server/run-tests.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
# Run tests with various configurations:
44

5-
TEST_COMMAND="cargo test --all --all-features --all-targets"
5+
TEST_COMMAND="cargo test --all --all-features --all-targets -- --test-threads=1"
66

77
# Common variables:
88
export DATABASE_URL="postgresql://postgres:postgres@localhost:5432/postgres"

server/svix-server/tests/utils/mod.rs

+1-6
Original file line numberDiff line numberDiff line change
@@ -212,12 +212,7 @@ pub fn get_default_test_config() -> ConfigurationInner {
212212
let _ = dotenv::dotenv();
213213
let cfg = svix_server::cfg::load().unwrap();
214214

215-
// Change the queue type to in-memory. This is necessary so test workers don't pick up messages
216-
// from other tests whose threads then abort at the end of a test before associated database
217-
// transactions are complete.
218-
let mut cfg = cfg.as_ref().clone();
219-
cfg.queue_type = svix_server::cfg::QueueType::Memory;
220-
cfg
215+
cfg.as_ref().clone()
221216
}
222217

223218
pub fn start_svix_server() -> (TestClient, tokio::task::JoinHandle<()>) {

0 commit comments

Comments
 (0)