TouchInstinct
/
checkstyle
306
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

improved test coverage of HeaderCheck 

added TODO comment about desired behaviour of HeaderCheck
This commit is contained in:
Lars Kühne 2002-11-10 12:20:51 +00:00
parent b041e53ae8
commit c2d424bf4c
1 changed files with 32 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/tests/com/puppycrawl/tools/checkstyle/HeaderCheckTest.java
View File

@ -2,6 +2,7 @@ package com.puppycrawl.tools.checkstyle;
import com.puppycrawl.tools.checkstyle.checks.HeaderCheck;
import com.puppycrawl.tools.checkstyle.checks.RegexpHeaderCheck;
import com.puppycrawl.tools.checkstyle.CheckstyleException;
public class HeaderCheckTest extends BaseCheckTestCase
{
@ -39,5 +40,36 @@ public class HeaderCheckTest extends BaseCheckTestCase
verify(c, fname, expected);
}
public void testNoHeader()
throws Exception
{
final CheckConfiguration checkConfig = new CheckConfiguration();
checkConfig.setClassname(HeaderCheck.class.getName());
// No header file specified
final Checker c = createChecker(checkConfig);
final String fname = getPath("InputScopeAnonInner.java");
// TODO: Is this really what we want or should we throw an Exception?
// Somehow checking nothing doesn't make too much sense...
final String[] expected = {
};
verify(c, fname, expected);
}
public void testIllegalArgs()
throws Exception
{
final CheckConfiguration checkConfig = new CheckConfiguration();
checkConfig.setClassname(HeaderCheck.class.getName());
checkConfig.addProperty("headerFile", getPath("nonexisting.file"));
try {
createChecker(checkConfig);
fail();
}
catch (CheckstyleException ex) {
// expected exception
}
}
}