add a few simple unittests and disable failing tests on MacOs

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

Author: andrei-ng

plotly/src/common/mod.rs

@@ -2210,6 +2210,15 @@ mod tests {
         assert_eq!(to_value(Reference::Paper).unwrap(), json!("paper"));
     }
 
+    #[test]
+    #[rustfmt::skip]
+    fn test_serialize_legend_group_title() {
+        assert_eq!(to_value(LegendGroupTitle::new()).unwrap(), json!({}));
+        assert_eq!(to_value(LegendGroupTitle::with_text("title_str").font(Font::default())).unwrap(), json!({"font": {}, "text": "title_str"}));
+        assert_eq!(to_value(LegendGroupTitle::from(String::from("title_string"))).unwrap(), json!({"text" : "title_string"}));
+        assert_eq!(to_value(LegendGroupTitle::from(&String::from("title_string"))).unwrap(), json!({"text" : "title_string"}));
+    }
+
     #[test]
     fn test_serialize_pad() {
         let pad = Pad::new(1, 2, 3);

plotly/src/layout/themes.rs

@@ -165,6 +165,31 @@ mod tests {
     use super::*;
     use crate::*;
 
+    #[test]
+    fn test_plotly_default() {
+        let template = &*DEFAULT;
+        let layout = Layout::new().template(template);
+        let mut plot = Plot::new();
+        plot.set_layout(layout);
+        plot.add_trace(Bar::new(vec![0], vec![1]));
+
+        let expected = r##"{"template":{"layout":{}}}"##; // etc...
+        assert!(plot.to_json().contains(expected));
+    }
+
+    #[test]
+    fn test_plotly_white() {
+        let template = &*PLOTLY_WHITE;
+        let layout = Layout::new().template(template);
+        let mut plot = Plot::new();
+        plot.set_layout(layout);
+        plot.add_trace(Bar::new(vec![0], vec![1]));
+        dbg!(plot.to_json());
+
+        let expected = r##"{"template":{"layout":{"title":{"x":0.05},"font":{"color":"#2a3f5f"}"##; // etc...
+        assert!(plot.to_json().contains(expected));
+    }
+
     #[test]
     fn test_plotly_dark() {
         let template = &*PLOTLY_DARK;

plotly/src/plot.rs

@@ -651,8 +651,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "windows"))]
+    // This seems to fail unpredictably on MacOs.
     #[test]
+    #[cfg(not(all(target_os = "windows", target_os = "macos")))]
     #[cfg(feature = "kaleido")]
     fn test_save_to_png() {
         let plot = create_test_plot();
@@ -663,8 +664,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "windows"))]
+    // This seems to fail unpredictably on MacOs.
     #[test]
+    #[cfg(not(all(target_os = "windows", target_os = "macos")))]
     #[cfg(feature = "kaleido")]
     fn test_save_to_jpeg() {
         let plot = create_test_plot();
@@ -675,8 +677,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "windows"))]
+    // This seems to fail unpredictably on MacOs.
     #[test]
+    #[cfg(not(all(target_os = "windows", target_os = "macos")))]
     #[cfg(feature = "kaleido")]
     fn test_save_to_svg() {
         let plot = create_test_plot();
@@ -687,8 +690,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
+    // This seems to fail unpredictably on MacOs.
     #[test]
-    #[ignore] // This seems to fail unpredictably on MacOs.
+    #[ignore]
     #[cfg(feature = "kaleido")]
     fn test_save_to_eps() {
         let plot = create_test_plot();
@@ -699,8 +703,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "windows"))]
+    // This seems to fail unpredictably on MacOs.
     #[test]
+    #[cfg(not(all(target_os = "windows", target_os = "macos")))]
     #[cfg(feature = "kaleido")]
     fn test_save_to_pdf() {
         let plot = create_test_plot();
@@ -711,8 +716,9 @@ mod tests {
         assert!(!dst.exists());
     }
 
-    #[cfg(not(target_os = "windows"))]
+    // This seems to fail unpredictably on MacOs.
     #[test]
+    #[cfg(not(all(target_os = "windows", target_os = "macos")))]
     #[cfg(feature = "kaleido")]
     fn test_save_to_webp() {
         let plot = create_test_plot();