chore: Ensure non-deterministic map key order doesn't break things

Author: mindok

lib/chart/mapping.ex

@@ -152,7 +152,11 @@ defmodule Contex.Mapping do
         :ok
 
       mappings ->
-        mapping_string = Enum.map_join(mappings, ", ", &"\"#{&1}\"")
+        mapping_string =
+          mappings
+          |> Enum.sort()
+          |> Enum.map_join(", ", &"\"#{&1}\"")
+
         raise "Required mapping(s) #{mapping_string} not included in column map."
     end
   end

test/contex_bar_chart_test.exs

@@ -232,13 +232,9 @@ defmodule ContexBarChartTest do
                [51.429, 58.0, 61.0, 68.571],
                [68.571, 58.0, 61.0, 0.0]
              ] ==
-               Stream.map(rects_map, &Map.delete(&1, :title))
-               |> Stream.map(&Enum.unzip/1)
-               |> Stream.map(fn value ->
-                 elem(value, 1)
-               end)
-               |> Enum.map(fn value ->
-                 Enum.map(value, string_to_rounded_float)
+               Enum.map(rects_map, fn rect ->
+                 [rect.height, rect.width, rect.x, rect.y]
+                 |> Enum.map(string_to_rounded_float)
                end)
 
       assert ["10", "20", "30", "40"] ==

test/contex_continuous_log_scale_test.exs

@@ -9,9 +9,9 @@ defmodule ContinuousLogScaleTest do
     test "defaults" do
       assert %Contex.ContinuousLogScale{
                custom_tick_formatter: nil,
-               domain: {0.0, 1.0},
+               domain: {+0.0, 1.0},
                tick_positions: [
-                 0.0,
+                 +0.0,
                  0.1,
                  0.2,
                  _,

test/contex_gantt_chart_test.exs

@@ -204,12 +204,9 @@ defmodule ContexGanttChartTest do
                [28.0, 15.0, 51.25, 61.0],
                [28.0, 30.0, 81.25, 91.0]
              ] ==
-               Stream.map(rects_map, &Enum.unzip/1)
-               |> Stream.map(fn value ->
-                 elem(value, 1)
-               end)
-               |> Enum.map(fn value ->
-                 Enum.map(value, string_to_rounded_float)
+               Enum.map(rects_map, fn rect ->
+                 [rect.height, rect.width, rect.x, rect.y]
+                 |> Enum.map(string_to_rounded_float)
                end)
 
       labels =

test/contex_scale_utils_test.exs

@@ -8,8 +8,8 @@ defmodule ScaleUtilsTest do
     test "with fixed intervals" do
       assert %{
                display_decimals: 0,
-               ticks: [0.0, 20.0, 40.0, 60.0, 80.0, 100.0],
-               nice_domain: {0.0, 100.0}
+               ticks: [+0.0, 20.0, 40.0, 60.0, 80.0, 100.0],
+               nice_domain: {+0.0, 100.0}
              } =
                ScaleUtils.compute_nice_settings(
                  0,
@@ -23,7 +23,7 @@ defmodule ScaleUtilsTest do
       assert %{
                display_decimals: 0,
                ticks: [0, 10, 50],
-               nice_domain: {0.0, 80.0}
+               nice_domain: {+0.0, 80.0}
              } =
                ScaleUtils.compute_nice_settings(
                  0,