Sort alphabetically if count is false (#300)

See https://gitlab.com/oersi/oersi-etl/-/issues/238
metafacture · May 12, 2023 · 4b0a8f3 · 4b0a8f3
1 parent 4d8bdbf
commit 4b0a8f3
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/metafix/src/main/java/org/metafacture/metafix/MetafixStreamAnalyzer.java b/metafix/src/main/java/org/metafacture/metafix/MetafixStreamAnalyzer.java
@@ -74,9 +74,9 @@ private TripleCount tripleCount() {
 
     private TripleSort tripleSort() {
         final TripleSort tripleSort = new TripleSort();
-        tripleSort.setNumeric(true);
+        tripleSort.setNumeric(count);
         tripleSort.setBy(Compare.OBJECT);
-        tripleSort.setOrder(Order.DECREASING);
+        tripleSort.setOrder(count ? Order.DECREASING : Order.INCREASING);
         return tripleSort;
     }
 

diff --git a/metafix/src/test/java/org/metafacture/metafix/MetafixListPathsTest.java b/metafix/src/test/java/org/metafacture/metafix/MetafixListPathsTest.java
@@ -55,9 +55,9 @@ public void testShouldListPaths() {
     public void testShouldListPathsNoCount() {
         lister.setCount(false);
         verify(
-            "c.*",
+            "a",
             "b.*",
-            "a");
+            "c.*");
     }
 
     @Test

diff --git a/metafix/src/test/java/org/metafacture/metafix/MetafixListValuesTest.java b/metafix/src/test/java/org/metafacture/metafix/MetafixListValuesTest.java
@@ -97,9 +97,9 @@ public void testShouldListPathsDontCount() {
         lister = new MetafixListValues("c");
         lister.setCount(false);
         verify(
-            "cC",
             "cA",
-            "cB");
+            "cB",
+            "cC");
     }
 
     private void processRecord() {
-Original file line number
+Diff line change
@@ Expand Up / @@ -55,9 +55,9 @@ public void testShouldListPaths() { @@
         public void testShouldListPathsNoCount() {
             lister.setCount(false);
             verify(
-                "c.*",
+                "a",
                 "b.*",
-                "a");
+                "c.*");
         }
         @Test
@@ Expand Down @@