Adjust changelog and tests

Author: fsktom

CHANGELOG.md

@@ -14,7 +14,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) a
 - [[#161](https://github.com/plotly/plotly.rs/pull/161)] Added `Axis` `scaleanchor` settter.
 - [[#159](https://github.com/plotly/plotly.rs/pull/159)] Make `heat_map` module public to expose `Smoothing enum`.
 - [[#153](https://github.com/plotly/plotly.rs/pull/153)] Added `LayoutScene`.
-- [[#154](https://github.com/plotly/plotly.rs/pull/154)] Improve ergonomics of `Title` struct: `new` method now takes no arguments as per other structs, whilst a new `with_text()` constructor is added for convenience. Where other structs contain a `Title`, users can now call the `title()` method with anything that `impl`s `Into<Title>`, viz. `String`, `&String`, `&str` and `Title`.
+- [[#154](https://github.com/plotly/plotly.rs/pull/154)] Improve ergonomics of `Title` and `LegendGroupTitle` structs: `new` method now takes no arguments as per other structs, whilst a new `with_text()` constructor is added for convenience. Where other structs contain a `Title` (and `LegendGroupTitle`), users  can now call the `title()` (and `legend_group_title()`) method with anything that `impl`s `Into<Title>`, viz. `String`, `&String`, `&str` and `Title`.
 
 ## [0.8.4] - 2023-07-09
 ### Added

plotly/src/layout/update_menu.rs

@@ -246,10 +246,7 @@ mod tests {
     use serde_json::{json, to_value};
 
     use super::*;
-    use crate::{
-        common::{Title, Visible},
-        Layout,
-    };
+    use crate::{common::Visible, Layout};
 
     #[test]
     fn test_serialize_button_method() {
@@ -315,7 +312,7 @@ mod tests {
                 Visible::True,
                 Visible::False,
             ]))
-            .push_relayout(Layout::modify_title(Title::with_text("Hello".to_string())))
+            .push_relayout(Layout::modify_title("Hello"))
             .push_relayout(Layout::modify_width(20))
             .build();
 

plotly/src/plot.rs

@@ -463,7 +463,7 @@ impl Plot {
     fn show_with_default_app(temp_path: &str) {
         use std::process::Command;
         Command::new("open")
-            .args(&[temp_path])
+            .args([temp_path])
             .output()
             .expect(DEFAULT_HTML_APP_NOT_FOUND);
     }
@@ -597,9 +597,7 @@ mod tests {
     #[test]
     fn test_layout_to_json() {
         let mut plot = create_test_plot();
-        let layout = Layout::new()
-            .title("TestTitle")
-            .title(crate::common::Title::with_text("TestTitle"));
+        let layout = Layout::new().title("TestTitle");
         plot.set_layout(layout);
 
         let expected = json!({

plotly/src/traces/heat_map.rs

@@ -206,7 +206,7 @@ mod tests {
         .hover_template_array(vec!["tmpl1", "tmpl2"])
         .hover_text(vec!["hov", "er"])
         .legend_group("1")
-        .legend_group_title(LegendGroupTitle::new("Legend Group Title"))
+        .legend_group_title("Legend Group Title")
         .name("name")
         .opacity(0.99)
         .reverse_scale(false)