Implement stable reuse hashes by sorting dictionary keys

Fixes #1553

Author: 0xced

src/Testcontainers/Configurations/Commons/JsonIgnoreRuntimeResourceLabels.cs

@@ -1,39 +1,20 @@
 namespace DotNet.Testcontainers.Configurations
 {
-  using System;
   using System.Collections.Generic;
   using System.Linq;
   using System.Text.Json;
-  using System.Text.Json.Serialization;
   using DotNet.Testcontainers.Clients;
   using DotNet.Testcontainers.Containers;
 
-  internal sealed class JsonIgnoreRuntimeResourceLabels : JsonConverter<IReadOnlyDictionary<string, string>>
+  internal sealed class JsonIgnoreRuntimeResourceLabels : JsonOrderedKeysConverter
   {
     private static readonly ISet<string> IgnoreLabels = new HashSet<string> { ResourceReaper.ResourceReaperSessionLabel, TestcontainersClient.TestcontainersVersionLabel, TestcontainersClient.TestcontainersSessionIdLabel };
 
-    public override bool CanConvert(Type typeToConvert)
-    {
-      return typeof(IEnumerable<KeyValuePair<string, string>>).IsAssignableFrom(typeToConvert);
-    }
-
-    public override IReadOnlyDictionary<string, string> Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
-    {
-      return JsonSerializer.Deserialize<IReadOnlyDictionary<string, string>>(ref reader);
-    }
-
     public override void Write(Utf8JsonWriter writer, IReadOnlyDictionary<string, string> value, JsonSerializerOptions options)
     {
       var labels = value.Where(label => !IgnoreLabels.Contains(label.Key)).ToDictionary(label => label.Key, label => label.Value);
 
-      writer.WriteStartObject();
-
-      foreach (var label in labels)
-      {
-        writer.WriteString(label.Key, label.Value);
-      }
-
-      writer.WriteEndObject();
+      base.Write(writer, labels, options);
     }
   }
 }

src/Testcontainers/Configurations/Commons/JsonOrderedKeysConverter.cs

@@ -0,0 +1,33 @@
+namespace DotNet.Testcontainers.Configurations
+{
+  using System;
+  using System.Collections.Generic;
+  using System.Linq;
+  using System.Text.Json;
+  using System.Text.Json.Serialization;
+
+  internal class JsonOrderedKeysConverter : JsonConverter<IReadOnlyDictionary<string, string>>
+  {
+    public override bool CanConvert(Type typeToConvert)
+    {
+      return typeof(IEnumerable<KeyValuePair<string, string>>).IsAssignableFrom(typeToConvert);
+    }
+
+    public override IReadOnlyDictionary<string, string> Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+    {
+      return JsonSerializer.Deserialize<IReadOnlyDictionary<string, string>>(ref reader);
+    }
+
+    public override void Write(Utf8JsonWriter writer, IReadOnlyDictionary<string, string> value, JsonSerializerOptions options)
+    {
+      writer.WriteStartObject();
+
+      foreach (var item in value.OrderBy(item => item.Key))
+      {
+        writer.WriteString(item.Key, item.Value);
+      }
+
+      writer.WriteEndObject();
+    }
+  }
+}

src/Testcontainers/Configurations/Commons/ResourceConfiguration.cs

@@ -14,6 +14,13 @@ namespace DotNet.Testcontainers.Configurations
   [PublicAPI]
   public class ResourceConfiguration<TCreateResourceEntity> : IResourceConfiguration<TCreateResourceEntity>
   {
+    private static readonly JsonSerializerOptions JsonSerializerOptions;
+
+    static ResourceConfiguration()
+    {
+      JsonSerializerOptions = new JsonSerializerOptions { Converters = { new JsonOrderedKeysConverter() } };
+    }
+
     /// <summary>
     /// Initializes a new instance of the <see cref="ResourceConfiguration{TCreateResourceEntity}" /> class.
     /// </summary>
@@ -88,7 +95,7 @@ protected ResourceConfiguration(IResourceConfiguration<TCreateResourceEntity> ol
     /// <inheritdoc />
     public virtual string GetReuseHash()
     {
-      var jsonUtf8Bytes = JsonSerializer.SerializeToUtf8Bytes(this, GetType());
+      var jsonUtf8Bytes = JsonSerializer.SerializeToUtf8Bytes(this, GetType(), JsonSerializerOptions);
 
 #if NET6_0_OR_GREATER
       return Convert.ToBase64String(SHA1.HashData(jsonUtf8Bytes));