Skip to content

Commit 2c93fde

Browse files
committed
Minor fix in StatefulTokenizerTest and TokenizerTextAnnotationBuilderTest to ensure cross-platform compatibility
1 parent 56d7b16 commit 2c93fde

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

tokenizer/src/test/java/edu/illinois/cs/cogcomp/nlp/tokenizer/StatefullTokenizerTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,8 @@ public void testStatefulTokenizerMultiline() {
152152
IntPair notOffsets = new IntPair(42, 45);
153153
assertEquals(notOffsets, tokenOffsets[notIndex]);
154154
int intolerantIndex = 14;
155-
IntPair intolerantOffsets = new IntPair(77, 87);
155+
int lineSepLength = System.lineSeparator().length();
156+
IntPair intolerantOffsets = new IntPair(76 + lineSepLength, 86 + lineSepLength);
156157
assertEquals(intolerantOffsets, tokenOffsets[intolerantIndex]);
157158
}
158159

tokenizer/src/test/java/edu/illinois/cs/cogcomp/nlp/tokenizer/TokenizerTextAnnotationBuilderTest.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,16 @@ public void testBuilder() {
3232
"Mr. Dawkins -- a liberal professor -- doesn't like fundamentalists. ";
3333
final String sentB = "He is intolerant of intolerance!";
3434

35-
final int refSentStartOffset = 71;
36-
final int refSentEndOffset = 103;
35+
String lineSep = System.lineSeparator();
36+
int lineSepLength = lineSep.length();
3737

38-
final int refTokStartOffset = 77;
38+
final int refSentStartOffset = 70 + lineSepLength;
39+
final int refSentEndOffset = 102 + lineSepLength;
40+
41+
final int refTokStartOffset = 76 + lineSepLength;
3942
final int refTokEndOffset = refTokStartOffset + 10;
4043

41-
final String text = sentA + System.lineSeparator() + sentB;
44+
final String text = sentA + lineSep + sentB;
4245

4346
TextAnnotation ta = bldr.createTextAnnotation("test", "test", text);
4447

0 commit comments

Comments
 (0)