Add tests for ProgressUpdateBuilder

src/error.rs

@@ -101,7 +101,7 @@ pub enum Client {
     },
 
     /// The 'Job' is malformed.
-    #[error("progress update hash is malformed: {desc}")]
+    #[error("progress update is malformed: {desc}")]
     ProgressUpdateMalformed {
         /// Details on what is missing or incorrect about an instance of 'ProgressUpdate'
         desc: String,

src/proto/single/mod.rs

@@ -387,4 +387,77 @@ mod test {
         assert_ne!(job1.jid, job2.jid);
         assert_ne!(job1.created_at, job2.created_at);
     }
+
+    #[test]
+    fn test_progress_updaet_needs_jid() {
+        let result = ProgressUpdateBuilder::default().build();
+        let err = result.unwrap_err();
+        assert_eq!(
+            err.to_string(),
+            "progress update is malformed: `jid` must be initialized"
+        )
+    }
+
+    #[test]
+    fn test_percent_validated_on_progress_update() {
+        let tracked = utils::gen_random_jid();
+        let result = ProgressUpdateBuilder::default()
+            .jid(tracked)
+            .percent(120)
+            .build();
+        let err = result.unwrap_err();
+        assert_eq!(
+            err.to_string(),
+            "progress update is malformed: `percent` indicates job execution progress and should be in the range from 0 to 100 inclusive"
+        )
+    }
+
+    #[test]
+    fn test_progress_update_can_be_created_with_builder() {
+        let tracked = utils::gen_random_jid();
+        let progress = ProgressUpdateBuilder::default()
+            .jid(tracked.clone())
+            .build()
+            .unwrap();
+
+        assert_eq!(progress.jid, tracked);
+        assert!(progress.desc.is_none());
+        assert!(progress.percent.is_none());
+        assert!(progress.reserve_until.is_none());
+
+        let extra_time_needed = chrono::Duration::seconds(300);
+        let extend = Utc::now() + extra_time_needed;
+        let progress = ProgressUpdateBuilder::default()
+            .jid(tracked.clone())
+            .desc("Resizing the image...".to_string())
+            .percent(67)
+            .reserve_until(extend)
+            .build()
+            .unwrap();
+        let serialized = serde_json::to_string(&progress).unwrap();
+        assert!(serialized.contains("jid"));
+        assert!(serialized.contains(&tracked));
+
+        assert!(serialized.contains("desc"));
+        assert!(serialized.contains("Resizing the image..."));
+
+        assert!(serialized.contains("percent"));
+        assert!(serialized.contains("67"));
+
+        assert!(serialized.contains("reserve_until"));
+        assert!(serialized.contains(&extend.to_rfc3339_opts(chrono::SecondsFormat::Nanos, true)));
+
+        let progress = ProgressUpdateBuilder::default()
+            .jid(tracked.clone())
+            .build()
+            .unwrap();
+        let serialized =serde_json::to_string(&progress).unwrap();
+
+        assert!(serialized.contains("jid"));
+        assert!(serialized.contains(&tracked));
+
+        assert!(!serialized.contains("percent"));
+        assert!(!serialized.contains("desc"));
+        assert!(!serialized.contains("reserve_until"));
+    }
 }