Skip to content

Commit 5c35f36

Browse files
committed
Use \t|\t in analyzer output templates instead of \t (#300)
See https://gitlab.com/oersi/oersi-etl/-/issues/238
1 parent 1d0fe7b commit 5c35f36

File tree

4 files changed

+25
-25
lines changed

4 files changed

+25
-25
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
public class MetafixListPaths extends MetafixStreamAnalyzer {
3838

3939
public MetafixListPaths() {
40-
super("nothing()", Compare.PREDICATE, "${s}\t ${o}", "${p}");
40+
super("nothing()", Compare.PREDICATE, "${s}\t|\t${o}", "${p}");
4141
setIndex(false);
4242
}
4343

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
public class MetafixListValues extends MetafixStreamAnalyzer {
3737

3838
public MetafixListValues(final String path) {
39-
super(fix(path), Compare.OBJECT, "${o}\t ${s}", "${o}");
39+
super(fix(path), Compare.OBJECT, "${o}\t|\t${s}", "${o}");
4040
}
4141

4242
private static String fix(final String path) {

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

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,9 @@ public MetafixListPathsTest() {
4646
@Test
4747
public void testShouldListPaths() {
4848
verify(
49-
"c.*\t 3",
50-
"b.*\t 2",
51-
"a\t 1");
49+
"c.*\t|\t3",
50+
"b.*\t|\t2",
51+
"a\t|\t1");
5252
}
5353

5454
@Test
@@ -64,12 +64,12 @@ public void testShouldListPathsNoCount() {
6464
public void testShouldListPathsUseIndex() {
6565
lister.setIndex(true);
6666
verify(
67-
"a\t 1",
68-
"b.1\t 1",
69-
"b.2\t 1",
70-
"c.1\t 1",
71-
"c.2\t 1",
72-
"c.3\t 1");
67+
"a\t|\t1",
68+
"b.1\t|\t1",
69+
"b.2\t|\t1",
70+
"c.1\t|\t1",
71+
"c.2\t|\t1",
72+
"c.3\t|\t1");
7373
}
7474

7575
@Test
@@ -88,9 +88,9 @@ public void testShouldListPathsNoCountUseIndex() {
8888
@Test
8989
public void testShouldListPathsSortedByFrequency() {
9090
verify(
91-
"c.*\t 3",
92-
"b.*\t 2",
93-
"a\t 1");
91+
"c.*\t|\t3",
92+
"b.*\t|\t2",
93+
"a\t|\t1");
9494
}
9595

9696
private void processRecord() {

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

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,51 +45,51 @@ public MetafixListValuesTest() {
4545
@Test
4646
public void testShouldListPathsSimple() {
4747
lister = new MetafixListValues("a");
48-
verify("2\t aA");
48+
verify("2\t|\taA");
4949
}
5050

5151
@Test
5252
public void testShouldListPathsWildcard() {
5353
lister = new MetafixListValues("a.*");
54-
verify("2\t aA");
54+
verify("2\t|\taA");
5555
}
5656

5757
@Test
5858
public void testShouldListPathsIndex() {
5959
lister = new MetafixListValues("a.1");
60-
verify("1\t aA");
60+
verify("1\t|\taA");
6161
}
6262

6363
@Test
6464
public void testShouldListPathsTwoValues() {
6565
lister = new MetafixListValues("b");
6666
verify(
67-
"2\t bB",
68-
"1\t bA");
67+
"2\t|\tbB",
68+
"1\t|\tbA");
6969
}
7070

7171
@Test
7272
public void testShouldListPathsTwoValuesWildcard() {
7373
lister = new MetafixListValues("b.*");
7474
verify(
75-
"2\t bB",
76-
"1\t bA");
75+
"2\t|\tbB",
76+
"1\t|\tbA");
7777
}
7878

7979
@Test
8080
public void testShouldListPathsTwoValuesIndex() {
8181
lister = new MetafixListValues("b.1");
8282
verify(
83-
"1\t bB");
83+
"1\t|\tbB");
8484
}
8585

8686
@Test
8787
public void testShouldListPathsSortCount() {
8888
lister = new MetafixListValues("c");
8989
verify(
90-
"3\t cC",
91-
"1\t cA",
92-
"1\t cB");
90+
"3\t|\tcC",
91+
"1\t|\tcA",
92+
"1\t|\tcB");
9393
}
9494

9595
@Test

0 commit comments

Comments
 (0)