Merge pull request #316 from zeenix/test-fixes

✅ 2 small fixes/improvements to tests

Author: zeenix

tests/fdo.rs

@@ -20,7 +20,7 @@ use zbus::{
 
 #[tokio::test(flavor = "multi_thread", worker_threads = 2)]
 #[instrument]
-#[timeout(15000)]
+#[timeout(30000)]
 async fn name_ownership_changes() {
     busd::tracing_subscriber::init();
 
@@ -216,7 +216,7 @@ async fn name_ownership_changes_client(address: &str, tx: Sender<()>) -> anyhow:
 
     let mut unique_name_signaled = false;
     let mut well_known_name_signaled = false;
-    while !unique_name_signaled && !well_known_name_signaled {
+    while !unique_name_signaled || !well_known_name_signaled {
         let changed = name_changed_stream.next().await.unwrap();
         if *changed.args()?.name() == *conn2_unique_name {
             ensure!(

tests/greet.rs

@@ -23,7 +23,7 @@ use zbus::{
 
 #[tokio::test(flavor = "multi_thread", worker_threads = 2)]
 #[instrument]
-#[timeout(15000)]
+#[timeout(30000)]
 async fn greet() {
     busd::tracing_subscriber::init();
 

tests/monitor.rs

@@ -15,7 +15,7 @@ use zbus::{
 
 #[tokio::test(flavor = "multi_thread", worker_threads = 2)]
 #[instrument]
-#[timeout(15000)]
+#[timeout(30000)]
 async fn become_monitor() {
     busd::tracing_subscriber::init();
 

tests/multiple_conns.rs

@@ -13,7 +13,7 @@ use zbus::connection;
 
 #[tokio::test(flavor = "multi_thread", worker_threads = 8)]
 #[instrument]
-#[timeout(15000)]
+#[timeout(30000)]
 async fn multi_conenct() {
     busd::tracing_subscriber::init();