Add tests (#74)

* partial test coverage of common/mod.rs

* finish tests for common.rs

* start layout tests

* finish tests for enums

* just a few more structs to go for layout...

* more tests

* make tests pass

* update changelog

Author: mfreeborn

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Support for [configuration options](https://plotly.com/javascript/configuration-options/)
 - Support for layout templates and pre-defined themes
 - Support for WASM environments
+- Lots of tests
 ### Changed
 - Improve implementation of `private::NumOrString` to support more primitive types ([Issue 
 #47](https://github.com/igiagkiozis/plotly/issues/47))

plotly/examples/basic_charts.rs

@@ -2,7 +2,7 @@ use itertools_num::linspace;
 use plotly::common::{
     ColorScale, ColorScalePalette, DashType, Fill, Font, Line, LineShape, Marker, Mode, Title,
 };
-use plotly::layout::{Axis, BarMode, Layout, Legend, TicksDirection};
+use plotly::layout::{Axis, BarMode, Layout, Legend, TicksDirection, TraceOrder};
 use plotly::{Bar, NamedColor, Plot, Rgb, Rgba, Scatter, ScatterPolar};
 use rand_distr::{Distribution, Normal, Uniform};
 
@@ -394,7 +394,7 @@ fn line_shape_options_for_interpolation(show: bool) {
     let layout = Layout::new().legend(
         Legend::new()
             .y(0.5)
-            .trace_order("reversed")
+            .trace_order(TraceOrder::Reversed)
             .font(Font::new().size(16)),
     );
     plot.set_layout(layout);
@@ -445,7 +445,7 @@ fn line_dash(show: bool) {
         .legend(
             Legend::new()
                 .y(0.5)
-                .trace_order("reversed")
+                .trace_order(TraceOrder::Reversed)
                 .font(Font::new().size(16)),
         )
         .x_axis(Axis::new().range(vec![0.95, 5.05]).auto_range(false))

plotly/examples/fundamentals.rs

@@ -1,5 +1,5 @@
 use itertools_num::linspace;
-use plotly::common::{Fill, Font, Mode};
+use plotly::common::{DashType, Fill, Font, Mode};
 use plotly::layout::{
     Axis, GridPattern, Layout, LayoutGrid, Margin, Shape, ShapeLayer, ShapeLine, ShapeType,
 };
@@ -56,7 +56,7 @@ fn vertical_and_horizontal_lines_positioned_relative_to_axes(show: bool) {
                 ShapeLine::new()
                     .color(NamedColor::LightSeaGreen)
                     .width(3.)
-                    .dash("dashdot"),
+                    .dash(DashType::DashDot),
             ),
     );
     layout.add_shape(
@@ -70,7 +70,7 @@ fn vertical_and_horizontal_lines_positioned_relative_to_axes(show: bool) {
                 ShapeLine::new()
                     .color(NamedColor::MediumPurple)
                     .width(3.)
-                    .dash("dot"),
+                    .dash(DashType::Dot),
             ),
     );
 

plotly/examples/subplots.rs

@@ -1,5 +1,5 @@
-use plotly::common::{Font, Side, Title};
-use plotly::layout::{Axis, GridPattern, Layout, LayoutGrid, Legend, RowOrder};
+use plotly::common::{AxisSide, Font, Title};
+use plotly::layout::{Axis, GridPattern, Layout, LayoutGrid, Legend, RowOrder, TraceOrder};
 use plotly::{Plot, Rgb, Scatter};
 
 // Subplots
@@ -124,7 +124,7 @@ fn stacked_subplots_with_shared_x_axis(show: bool) {
 
     let layout = Layout::new()
         .y_axis(Axis::new().domain(&[0., 0.33]))
-        .legend(Legend::new().trace_order("reversed"))
+        .legend(Legend::new().trace_order(TraceOrder::Reversed))
         .y_axis2(Axis::new().domain(&[0.33, 0.66]))
         .y_axis3(Axis::new().domain(&[0.66, 1.]));
     plot.set_layout(layout);
@@ -197,7 +197,7 @@ fn two_y_axes(show: bool) {
                 .title(Title::new("yaxis2 title").font(Font::new().color(Rgb::new(148, 103, 189))))
                 .tick_font(Font::new().color(Rgb::new(148, 103, 189)))
                 .overlaying("y")
-                .side(Side::Right),
+                .side(AxisSide::Right),
         );
     plot.set_layout(layout);
     if show {
@@ -235,7 +235,7 @@ fn multiple_axes(show: bool) {
                 .tick_font(Font::new().color("#ff7f0e"))
                 .anchor("free")
                 .overlaying("y")
-                .side(Side::Left)
+                .side(AxisSide::Left)
                 .position(0.15),
         )
         .y_axis3(
@@ -244,15 +244,15 @@ fn multiple_axes(show: bool) {
                 .tick_font(Font::new().color("#d62728"))
                 .anchor("x")
                 .overlaying("y")
-                .side(Side::Right),
+                .side(AxisSide::Right),
         )
         .y_axis4(
             Axis::new()
                 .title(Title::new("yaxis4 title").font(Font::new().color("#9467bd")))
                 .tick_font(Font::new().color("#9467bd"))
                 .anchor("free")
                 .overlaying("y")
-                .side(Side::Right)
+                .side(AxisSide::Right)
                 .position(0.85),
         );
     plot.set_layout(layout);