You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/08/03 13:59:28 UTC

svn commit: r1509967 - /commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java

Author: sebb
Date: Sat Aug  3 11:59:28 2013
New Revision: 1509967

URL: http://svn.apache.org/r1509967
Log:
Fix up TODOs

Modified:
    commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java

Modified: commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java?rev=1509967&r1=1509966&r2=1509967&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java (original)
+++ commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java Sat Aug  3 11:59:28 2013
@@ -324,37 +324,37 @@ public class CSVLexerTest {
         assertThat(lexer.nextToken(new Token()), hasContent("NotEscaped"));
     }
 
-    @Test // TODO is this correct? Do we expect TAB to be un/escaped?
+    @Test // TODO is this correct? Do we expect <esc>TAB to be unescaped?
     public void testEscapedTab() throws Exception {
         final Lexer lexer = getLexer("character\\" + TAB + "Escaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + TAB + "Escaped"));
     }
 
-    @Test // TODO is this correct? Do we expect TAB to be un/escaped?
+    @Test
     public void testTab() throws Exception {
         final Lexer lexer = getLexer("character" + TAB + "NotEscaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + TAB + "NotEscaped"));
     }
 
-    @Test // TODO is this correct? Do we expect BACKSPACE to be un/escaped?
+    @Test // TODO is this correct? Do we expect <esc>BACKSPACE to be unescaped?
     public void testEscapedBackspace() throws Exception {
         final Lexer lexer = getLexer("character\\" + BACKSPACE + "Escaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + BACKSPACE + "Escaped"));
     }
 
-    @Test // TODO is this correct? Do we expect BACKSPACE to be un/escaped?
+    @Test
     public void testBackspace() throws Exception {
         final Lexer lexer = getLexer("character" + BACKSPACE + "NotEscaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + BACKSPACE + "NotEscaped"));
     }
 
-    @Test // TODO is this correct? Do we expect FF to be un/escaped?
+    @Test // TODO is this correct? Do we expect <esc>FF to be unescaped?
     public void testEscapedFF() throws Exception {
         final Lexer lexer = getLexer("character\\" + FF + "Escaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + FF + "Escaped"));
     }
 
-    @Test // TODO is this correct? Do we expect FF to be un/escaped?
+    @Test
     public void testFF() throws Exception {
         final Lexer lexer = getLexer("character" + FF + "NotEscaped", formatWithEscaping);
         assertThat(lexer.nextToken(new Token()), hasContent("character" + FF + "NotEscaped"));



Re: svn commit: r1509967 - /commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java

Posted by Benedikt Ritter <br...@apache.org>.
Thanks


2013/8/3 <se...@apache.org>

> Author: sebb
> Date: Sat Aug  3 11:59:28 2013
> New Revision: 1509967
>
> URL: http://svn.apache.org/r1509967
> Log:
> Fix up TODOs
>
> Modified:
>
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java
>
> Modified:
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java
> URL:
> http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java?rev=1509967&r1=1509966&r2=1509967&view=diff
>
> ==============================================================================
> ---
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java
> (original)
> +++
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/CSVLexerTest.java
> Sat Aug  3 11:59:28 2013
> @@ -324,37 +324,37 @@ public class CSVLexerTest {
>          assertThat(lexer.nextToken(new Token()),
> hasContent("NotEscaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect TAB to be un/escaped?
> +    @Test // TODO is this correct? Do we expect <esc>TAB to be unescaped?
>      public void testEscapedTab() throws Exception {
>          final Lexer lexer = getLexer("character\\" + TAB + "Escaped",
> formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> TAB + "Escaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect TAB to be un/escaped?
> +    @Test
>      public void testTab() throws Exception {
>          final Lexer lexer = getLexer("character" + TAB + "NotEscaped",
> formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> TAB + "NotEscaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect BACKSPACE to be
> un/escaped?
> +    @Test // TODO is this correct? Do we expect <esc>BACKSPACE to be
> unescaped?
>      public void testEscapedBackspace() throws Exception {
>          final Lexer lexer = getLexer("character\\" + BACKSPACE +
> "Escaped", formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> BACKSPACE + "Escaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect BACKSPACE to be
> un/escaped?
> +    @Test
>      public void testBackspace() throws Exception {
>          final Lexer lexer = getLexer("character" + BACKSPACE +
> "NotEscaped", formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> BACKSPACE + "NotEscaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect FF to be un/escaped?
> +    @Test // TODO is this correct? Do we expect <esc>FF to be unescaped?
>      public void testEscapedFF() throws Exception {
>          final Lexer lexer = getLexer("character\\" + FF + "Escaped",
> formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> FF + "Escaped"));
>      }
>
> -    @Test // TODO is this correct? Do we expect FF to be un/escaped?
> +    @Test
>      public void testFF() throws Exception {
>          final Lexer lexer = getLexer("character" + FF + "NotEscaped",
> formatWithEscaping);
>          assertThat(lexer.nextToken(new Token()), hasContent("character" +
> FF + "NotEscaped"));
>
>
>


-- 
http://people.apache.org/~britter/
http://www.systemoutprintln.de/
http://twitter.com/BenediktRitter
http://github.com/britter