Merge pull request #14 from cowprotocol/tracing_spans

Switch to tracing logs to allow for log spans per order
charlesndalton · Oct 22, 2023 · 5890436 · 5890436
2 parents 0dfd83e + 74155bd
commit 5890436
Show file tree

Hide file tree

Showing 4 changed files with 192 additions and 106 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -17,10 +17,11 @@ serde = "1.0"
 serde_json = "1.0"
 hex = "0.4.3"
 log = "0.4"
-env_logger = "0.8.4"
 anyhow = "1.0.61"
 ethers = { version = "0.17.0", features = ["abigen", "openssl"] }
 url = { version = "2.2.2" }
+tracing = "0.1.37"
+tracing-subscriber = { version = "0.3.17", features = ["env-filter"] }
 
 [dev-dependencies]
 rand = { version = "0.8.4", features = ["min_const_gen"] }
diff --git a/src/ethereum_client.rs b/src/ethereum_client.rs
@@ -241,7 +241,7 @@ mod tests {
             .await
             .expect("Unable to get requested swaps");
 
-        assert!(requested_swaps.len() > 0);
+        assert!(!requested_swaps.is_empty());
     }
 
     #[test]
-Original file line number
+Diff line change
@@ Expand Up / @@ -241,7 +241,7 @@ mod tests { @@
                 .await
                 .expect("Unable to get requested swaps");
-            assert!(requested_swaps.len() > 0);
+            assert!(!requested_swaps.is_empty());
         }
         #[test]
@@ Expand Down @@