Skip to content

Commit 4b0a8f3

Browse files
committed
Sort alphabetically if count is false (#300)
See https://gitlab.com/oersi/oersi-etl/-/issues/238
1 parent 4d8bdbf commit 4b0a8f3

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

metafix/src/main/java/org/metafacture/metafix/MetafixStreamAnalyzer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,9 @@ private TripleCount tripleCount() {
7474

7575
private TripleSort tripleSort() {
7676
final TripleSort tripleSort = new TripleSort();
77-
tripleSort.setNumeric(true);
77+
tripleSort.setNumeric(count);
7878
tripleSort.setBy(Compare.OBJECT);
79-
tripleSort.setOrder(Order.DECREASING);
79+
tripleSort.setOrder(count ? Order.DECREASING : Order.INCREASING);
8080
return tripleSort;
8181
}
8282

metafix/src/test/java/org/metafacture/metafix/MetafixListPathsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,9 @@ public void testShouldListPaths() {
5555
public void testShouldListPathsNoCount() {
5656
lister.setCount(false);
5757
verify(
58-
"c.*",
58+
"a",
5959
"b.*",
60-
"a");
60+
"c.*");
6161
}
6262

6363
@Test

metafix/src/test/java/org/metafacture/metafix/MetafixListValuesTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,9 @@ public void testShouldListPathsDontCount() {
9797
lister = new MetafixListValues("c");
9898
lister.setCount(false);
9999
verify(
100-
"cC",
101100
"cA",
102-
"cB");
101+
"cB",
102+
"cC");
103103
}
104104

105105
private void processRecord() {

0 commit comments

Comments
 (0)