Revision: 6035
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=6035&view=rev
Author:   gulp21-1
Date:     2011-12-13 18:13:13 +0000 (Tue, 13 Dec 2011)
Log Message:
-----------
fixed problems with unit tests caused by WORD_REPEAT_BEGINNING_RULE

Modified Paths:
--------------
    
trunk/JLanguageTool/src/java/org/languagetool/rules/WordRepeatBeginningRule.java
    trunk/JLanguageTool/src/test/org/languagetool/JLanguageToolTest.java
    
trunk/JLanguageTool/src/test/org/languagetool/rules/de/GermanWordRepeatBeginningRuleTest.java

Modified: 
trunk/JLanguageTool/src/java/org/languagetool/rules/WordRepeatBeginningRule.java
===================================================================
--- 
trunk/JLanguageTool/src/java/org/languagetool/rules/WordRepeatBeginningRule.java
    2011-12-12 22:06:33 UTC (rev 6034)
+++ 
trunk/JLanguageTool/src/java/org/languagetool/rules/WordRepeatBeginningRule.java
    2011-12-13 18:13:13 UTC (rev 6035)
@@ -111,7 +111,8 @@
 
   @Override
   public void reset() {
-    // nothing
+    lastToken = "";
+    beforeLastToken = "";
   }
 
 }

Modified: trunk/JLanguageTool/src/test/org/languagetool/JLanguageToolTest.java
===================================================================
--- trunk/JLanguageTool/src/test/org/languagetool/JLanguageToolTest.java        
2011-12-12 22:06:33 UTC (rev 6034)
+++ trunk/JLanguageTool/src/test/org/languagetool/JLanguageToolTest.java        
2011-12-13 18:13:13 UTC (rev 6035)
@@ -53,7 +53,7 @@
     assertEquals(0, tool.check("A test that should not give errors.").size());
     assertEquals(1, tool.check("A test test that should give errors.").size());
     assertEquals(0, tool.check("I can give you more a detailed 
description.").size());
-    assertEquals(9, tool.getAllRules().size());
+    assertEquals(10, tool.getAllRules().size());
     tool.activateDefaultPatternRules();
     assertTrue(tool.getAllRules().size() > 3);
     assertEquals(1, tool.check("I can give you more a detailed 
description.").size());

Modified: 
trunk/JLanguageTool/src/test/org/languagetool/rules/de/GermanWordRepeatBeginningRuleTest.java
===================================================================
--- 
trunk/JLanguageTool/src/test/org/languagetool/rules/de/GermanWordRepeatBeginningRuleTest.java
       2011-12-12 22:06:33 UTC (rev 6034)
+++ 
trunk/JLanguageTool/src/test/org/languagetool/rules/de/GermanWordRepeatBeginningRuleTest.java
       2011-12-13 18:13:13 UTC (rev 6035)
@@ -38,8 +38,8 @@
     // errors:
     assertEquals(1, langTool.check("Er ist nett. Er heißt Max. Er ist 
11.").size());
     assertEquals(1, langTool.check("Außerdem kommt er. Außerdem kommt 
sie.").size());
-    // TODO:
-    //assertEquals(0, langTool.check("Außerdem ist das ein neuer 
Text.").size());
+    // this used to cause false alarms because reset() was not implemented
+    assertEquals(0, langTool.check("Außerdem ist das ein neuer Text.").size());
   }
 
 }

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Systems Optimization Self Assessment
Improve efficiency and utilization of IT resources. Drive out cost and 
improve service delivery. Take 5 minutes to use this Systems Optimization 
Self Assessment. http://www.accelacomm.com/jaw/sdnl/114/51450054/
_______________________________________________
Languagetool-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-cvs

Reply via email to