try one more time on macos

Signed-off-by: Andrei Gherghescu <[email protected]>

Author: andrei-ng

plotly/src/plot.rs

@@ -761,7 +761,7 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "macos"))]
+    // #[cfg(not(target_os = "macos"))]
     #[test]
     #[cfg(feature = "kaleido")]
     fn test_save_to_jpeg() {
@@ -773,7 +773,7 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "macos"))]
+    // #[cfg(not(target_os = "macos"))]
     #[test]
     #[cfg(feature = "kaleido")]
     fn test_save_to_svg() {
@@ -822,7 +822,6 @@ mod tests {
     }
 
     #[test]
-    #[ignore] // Fails in the CI
     #[cfg(not(target_os = "macos"))]
     #[cfg(feature = "kaleido")]
     fn test_image_to_base64() {
@@ -851,7 +850,6 @@ mod tests {
     }
 
     #[test]
-    #[ignore] // Fails in the CI
     #[cfg(not(target_os = "macos"))]
     #[cfg(feature = "kaleido")]
     fn test_image_to_svg_string() {

plotly_kaleido/src/lib.rs

@@ -198,7 +198,10 @@ impl Kaleido {
                 "--disable-software-rasterizer",
                 "--single-process",
                 "--disable-gpu",
-                "--timeout 2000",
+                "--no-sandbox",
+                "--allow-silent-push",
+                "-allow-third-party-modules",
+                "--allow-running-insecure-content",
             ])
             .stdin(Stdio::piped())
             .stdout(Stdio::piped())
@@ -227,16 +230,29 @@ impl Kaleido {
         let output_lines = BufReader::new(process.stdout.take().unwrap()).lines();
         for line in output_lines.map_while(Result::ok) {
             let res = KaleidoResult::from(line.as_str());
-            if let Some(image_data) = res.result {
-                // TODO: this should be refactored
-                // The assumption is that  KaleidoResult contains a single image.
-                // We should end the loop on the first valid one.
-                // If that is not the case, prior implementation would have returned the last
-                // valid image
-                return Ok(image_data);
+            match res.result {
+                Some(image_data) => {
+                    // TODO: this should be refactored
+                    // The assumption is that  KaleidoResult contains a single image.
+                    // We should end the loop on the first valid one.
+                    // If that is not the case, prior implementation would have returned the last
+                    // valid image
+                    return Ok(image_data);
+                }
+                None => {
+                    println!("empty line from Kaleido");
+                }
             }
         }
 
+        // Don't eat up Kaleido/Chromiu erros but show them in the terminal
+        let stderr = process.stderr.take().unwrap();
+        let stderr_lines = BufReader::new(stderr).lines();
+        for line in stderr_lines {
+            let line = line.unwrap();
+            eprintln!("{}", line);
+        }
+
         Ok(String::default())
     }
 }
@@ -355,18 +371,18 @@ mod tests {
         let dst = PathBuf::from("example.pdf");
         let r = k.save(dst.as_path(), &test_plot, "pdf", 1200, 900, 4.5);
         assert!(r.is_ok());
-        assert!(std::fs::remove_file(dst.as_path()).is_ok());
+        // assert!(std::fs::remove_file(dst.as_path()).is_ok());
     }
 
     // This generates empty eps files for some reason
     #[test]
-    #[ignore]
+    // #[ignore]
     fn test_save_eps() {
         let test_plot = create_test_plot();
         let k = Kaleido::new();
         let dst = PathBuf::from("example.eps");
         let r = k.save(dst.as_path(), &test_plot, "eps", 1200, 900, 4.5);
         assert!(r.is_ok());
-        assert!(std::fs::remove_file(dst.as_path()).is_ok());
+        // assert!(std::fs::remove_file(dst.as_path()).is_ok());
     }
 }