Merge pull request #68 from qwandor/lifetimes

Lifetime of new Point shouldn't be tied to lifetime of measurement
driftluo · Jan 17, 2025 · 0e63c06 · 0e63c06
2 parents 73409e1 + 4e0054f
commit 0e63c06
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/keys.rs b/src/keys.rs
@@ -35,8 +35,8 @@ pub struct Point<'a> {
 
 impl<'a> Point<'a> {
     /// Create a new point
-    pub fn new(measurement: &str) -> Point {
-        Point {
+    pub fn new(measurement: &'_ str) -> Self {
+        Self {
             measurement: String::from(measurement),
             tags: HashMap::new(),
             fields: HashMap::new(),

diff --git a/tests/client_test.rs b/tests/client_test.rs
@@ -33,6 +33,7 @@ fn create_and_delete_measurement() {
 
         tokio::spawn(client.write_point(point, Some(Precision::Seconds), None))
             .await
+            .unwrap()
             .unwrap();
 
         client.drop_measurement("temporary").await.unwrap();
@@ -62,6 +63,7 @@ fn use_points() {
 
         tokio::spawn(client.write_points(points, Some(Precision::Seconds), None))
             .await
+            .unwrap()
             .unwrap();
 
         sleep(Duration::from_secs(3));