From ac968ee3d51616de3bca8e7b3ce778f9c6c3b9ed Mon Sep 17 00:00:00 2001 From: giswqs Date: Sat, 24 Oct 2026 12:20:49 -0400 Subject: [PATCH] test(ui): give the connection-chooser smoke test its own database The smoke tests share one process, and passwords typed for an address are remembered process-wide. Another test leaving one for the shared address made the project open without the chooser now and then (main CI, and 1 in 72 runs here under load; 0 in 144 with this change). --- crates/ui-egui/tests/remembered_database/mod.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crates/ui-egui/tests/remembered_database/mod.rs b/crates/ui-egui/tests/remembered_database/mod.rs index 41c95ba41..bc3832322 100644 --- a/crates/ui-egui/tests/remembered_database/mod.rs +++ b/crates/ui-egui/tests/remembered_database/mod.rs @@ -83,6 +83,9 @@ fn remembered_database_failed_store_has_visible_notice() { #[test] fn remembered_database_chooser_cancel_selection_and_project_scope() { + // Its own database: passwords for an address are remembered process-wide, and one the other + // tests leave for the shared address would open the project without asking which to use. + const ADDRESS: &str = "postgres:host=127.0.0.1 port=1 dbname=ui_remembered_chooser user=smoke sslmode=disable"; let store = Arc::new(MemorySecrets::new()); let mut seed = with_store(store.clone()); for name in ["First database", "Second database"] {