ensure sorted config

ppgia-unifor · Jan 2, 2024 · 5a70c55 · 5a70c55
1 parent a5e5140
commit 5a70c55
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/main/java/br/unifor/ppgia/resiliencebench/resources/scenario/Scenario.java b/src/main/java/br/unifor/ppgia/resiliencebench/resources/scenario/Scenario.java
@@ -27,9 +27,10 @@ public Scenario() {
   @Override
   public String toString() {
     var delimiter = ".";
-    var patternConfigSerialized = spec.getPatternConfig().entrySet().stream()
-            .map(e -> e.getKey() + "-" + e.getValue().toString())
-            .collect(joining(","));
+    var keys = spec.getPatternConfig().keySet().stream().sorted(String::compareTo);
+    var patternConfigSerialized = keys
+            .map(k ->  k + "-" + spec.getPatternConfig().get(k))
+            .collect(joining("."));
     if (patternConfigSerialized.isEmpty()) {
       patternConfigSerialized = "none";
     }

diff --git a/src/test/java/br/unifor/ppgia/resiliencebench/resources/scenario/ScenarioTest.java b/src/test/java/br/unifor/ppgia/resiliencebench/resources/scenario/ScenarioTest.java
@@ -16,12 +16,12 @@ void test_name_with_all_attributes() {
     var spec = new ScenarioSpec(
             "target-service-name",
             "source-service-name",
-            Map.of("maxAttempts", 10),
+            Map.of("za", 1.1, "maxAttempts", 10, "perRetryTimeout", 100),
             new ScenarioWorkload("workloadName", 100),
             new ScenarioFaultTemplate(25, new DelayFault(100))
     );
     var scenario = new Scenario(spec);
-    assertEquals("source-service-name.target-service-name.maxattempts-10.workloadname-100.delay-100ms-25p", scenario.toString());
+    assertEquals("source-service-name.target-service-name.maxattempts-10.perretrytimeout-100.za-1.1.workloadname-100.delay-100ms-25p", scenario.toString());
     assertTrue(scenario.toString().matches(regex));
   }