Skip to content

Commit 6208f0c

Browse files
committed
Fix up TODOs
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/csv/trunk@1509967 13f79535-47bb-0310-9956-ffa450edef68
1 parent 3fd92ec commit 6208f0c

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

src/test/java/org/apache/commons/csv/CSVLexerTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -324,37 +324,37 @@ public void testLF() throws Exception {
324324
assertThat(lexer.nextToken(new Token()), hasContent("NotEscaped"));
325325
}
326326

327-
@Test // TODO is this correct? Do we expect TAB to be un/escaped?
327+
@Test // TODO is this correct? Do we expect <esc>TAB to be unescaped?
328328
public void testEscapedTab() throws Exception {
329329
final Lexer lexer = getLexer("character\\" + TAB + "Escaped", formatWithEscaping);
330330
assertThat(lexer.nextToken(new Token()), hasContent("character" + TAB + "Escaped"));
331331
}
332332

333-
@Test // TODO is this correct? Do we expect TAB to be un/escaped?
333+
@Test
334334
public void testTab() throws Exception {
335335
final Lexer lexer = getLexer("character" + TAB + "NotEscaped", formatWithEscaping);
336336
assertThat(lexer.nextToken(new Token()), hasContent("character" + TAB + "NotEscaped"));
337337
}
338338

339-
@Test // TODO is this correct? Do we expect BACKSPACE to be un/escaped?
339+
@Test // TODO is this correct? Do we expect <esc>BACKSPACE to be unescaped?
340340
public void testEscapedBackspace() throws Exception {
341341
final Lexer lexer = getLexer("character\\" + BACKSPACE + "Escaped", formatWithEscaping);
342342
assertThat(lexer.nextToken(new Token()), hasContent("character" + BACKSPACE + "Escaped"));
343343
}
344344

345-
@Test // TODO is this correct? Do we expect BACKSPACE to be un/escaped?
345+
@Test
346346
public void testBackspace() throws Exception {
347347
final Lexer lexer = getLexer("character" + BACKSPACE + "NotEscaped", formatWithEscaping);
348348
assertThat(lexer.nextToken(new Token()), hasContent("character" + BACKSPACE + "NotEscaped"));
349349
}
350350

351-
@Test // TODO is this correct? Do we expect FF to be un/escaped?
351+
@Test // TODO is this correct? Do we expect <esc>FF to be unescaped?
352352
public void testEscapedFF() throws Exception {
353353
final Lexer lexer = getLexer("character\\" + FF + "Escaped", formatWithEscaping);
354354
assertThat(lexer.nextToken(new Token()), hasContent("character" + FF + "Escaped"));
355355
}
356356

357-
@Test // TODO is this correct? Do we expect FF to be un/escaped?
357+
@Test
358358
public void testFF() throws Exception {
359359
final Lexer lexer = getLexer("character" + FF + "NotEscaped", formatWithEscaping);
360360
assertThat(lexer.nextToken(new Token()), hasContent("character" + FF + "NotEscaped"));

0 commit comments

Comments
 (0)