diff options
author | rnveach <rveach02@gmail.com> | 2015-10-18 13:17:57 -0400 |
---|---|---|
committer | Roman Ivanov <ivanov-jr@mail.ru> | 2015-10-19 12:36:38 -0400 |
commit | 375dbf134ceaca9cf00fc3c220d8e32dc03ca099 (patch) | |
tree | 38985cbb62cfde5174d4fa61d19994e230ec1d04 /src/test/resources/com | |
parent | beb0ba7574c0e3bc31f7b946b826ef3938e67309 (diff) | |
download | checkstyle-375dbf134ceaca9cf00fc3c220d8e32dc03ca099.tar.gz |
Issue #2161: unify test input locations for whitespace package
Diffstat (limited to 'src/test/resources/com')
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputBraces.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/InputBraces.java) | 6 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputDoWhileWhitespaceAround.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputDoWhileWhitespaceAround.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputEmptyTypesAndCycles.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputEmptyTypesAndCycles.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputGenerics.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/InputGenerics.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhitespaceAfterArrayDeclarations.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhitespaceAfterArrayDeclarations.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputParenPad.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputParenPad.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSimple.java | 225 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSwitchWhitespaceAround.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputSwitchWhitespaceAround.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputWhitespace.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/InputWhitespace.java) | 2 | ||||
-rw-r--r-- | src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadWithSpace.java (renamed from src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/ParenPadWithSpace.java) | 2 |
11 files changed, 235 insertions, 14 deletions
diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/InputBraces.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputBraces.java index 762a5714d..54ef42ba2 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/InputBraces.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputBraces.java @@ -2,12 +2,8 @@ // Test case file for checkstyle. // Created: 2001 //////////////////////////////////////////////////////////////////////////////// -package com.puppycrawl.tools.checkstyle; +package com.puppycrawl.tools.checkstyle.checks.whitespace; -/** - * Test case for correct use of braces. - * @author Oliver Burn - **/ class InputBraces { /** @return helper func **/ diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputDoWhileWhitespaceAround.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputDoWhileWhitespaceAround.java index c500336ad..6c05bfdcc 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputDoWhileWhitespaceAround.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputDoWhileWhitespaceAround.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; public class InputDoWhileWhitespaceAround { diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputEmptyTypesAndCycles.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputEmptyTypesAndCycles.java index 5c55cee75..6a436ed16 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputEmptyTypesAndCycles.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputEmptyTypesAndCycles.java @@ -2,7 +2,7 @@ // Test case file for checkstyle. // Created: 2001 //////////////////////////////////////////////////////////////////////////////// -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; import java.lang.annotation.Documented; import java.lang.annotation.ElementType; diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/InputGenerics.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputGenerics.java index 0361c0fa5..d1a2f6984 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/InputGenerics.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputGenerics.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle; +package com.puppycrawl.tools.checkstyle.checks.whitespace; import java.util.Collection; import java.util.Map; diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java index dacc9a3a9..2f94861c9 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhiteSpaceAfterCheckFormerNpe.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; public class InputNoWhiteSpaceAfterCheckFormerNpe { diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhitespaceAfterArrayDeclarations.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhitespaceAfterArrayDeclarations.java index 2011eb738..a4942ffc7 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputNoWhitespaceAfterArrayDeclarations.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputNoWhitespaceAfterArrayDeclarations.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; public class InputNoWhitespaceAfterArrayDeclarations { diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputParenPad.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputParenPad.java index ec627cff9..b3c08ea8b 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputParenPad.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputParenPad.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; import java.io.IOException; import java.io.StringWriter; diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSimple.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSimple.java new file mode 100644 index 000000000..073be3652 --- /dev/null +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSimple.java @@ -0,0 +1,225 @@ +//////////////////////////////////////////////////////////////////////////////// +// Test case file for checkstyle. +// Created: Feb-2001 +// Ignore error +//////////////////////////////////////////////////////////////////////////////// +package com.puppycrawl.tools.checkstyle.checks.whitespace; +import java.io.*; +/** + * Contains simple mistakes: + * - Long lines + * - Tabs + * - Format of variables and parameters + * - Order of modifiers + * @author Oliver Burn + **/ +final class InputSimple +{ + // Long line ---------------------------------------------------------------- + // Contains a tab -> <- + // Contains trailing whitespace -> + + // Name format tests + // + /** Invalid format **/ + public static final int badConstant = 2; + /** Valid format **/ + public static final int MAX_ROWS = 2; + + /** Invalid format **/ + private static int badStatic = 2; + /** Valid format **/ + private static int sNumCreated = 0; + + /** Invalid format **/ + private int badMember = 2; + /** Valid format **/ + private int mNumCreated1 = 0; + /** Valid format **/ + protected int mNumCreated2 = 0; + + /** commas are wrong **/ + private int[] mInts = new int[] {1,2, 3, + 4}; + + // + // Accessor tests + // + /** should be private **/ + public static int sTest1; + /** should be private **/ + protected static int sTest3; + /** should be private **/ + static int sTest2; + + /** should be private **/ + int mTest1; + /** should be private **/ + public int mTest2; + + // + // Parameter name format tests + // + + /** + * @return hack + * @param badFormat1 bad format + * @param badFormat2 bad format + * @param badFormat3 bad format + * @throws java.lang.Exception abc + **/ + int test1(int badFormat1,int badFormat2, + final int badFormat3) + throws java.lang.Exception + { + return 0; + } + + /** method that is 20 lines long **/ + private void longMethod() + { + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + } + + /** constructor that is 10 lines long **/ + private InputSimple() + { + // a line + // a line + // a line + // a line + // a line + // a line + // a line + // a line + } + + /** test local variables */ + private void localVariables() + { + // normal decl + int abc = 0; + int ABC = 0; + + // final decls + final int cde = 0; + final int CDE = 0; + + // decl in for loop init statement + for (int k = 0; k < 1; k++) + { + String innerBlockVariable = ""; + } + for (int I = 0; I < 1; I++) + { + String InnerBlockVariable = ""; + } + } + + /** test method pattern */ + void ALL_UPPERCASE_METHOD() + { + } + + /** test illegal constant **/ + private static final int BAD__NAME = 3; + + // A very, very long line that is OK because it matches the regexp "^.*is OK.*regexp.*$" + // long line that has a tab -> <- and would be OK if tab counted as 1 char + // tabs that count as one char because of their position -> <- -> <-, OK + + /** some lines to test the error column after tabs */ + void errorColumnAfterTabs() + { + // with tab-width 8 all statements below start at the same column, + // with different combinations of ' ' and '\t' before the statement + int tab0 =1; + int tab1 =1; + int tab2 =1; + int tab3 =1; + int tab4 =1; + int tab5 =1; + } + + // FIXME: + /* FIXME: a + * FIXME: + * TODO + */ + /* NOTHING */ + /* YES */ /* FIXME: x */ /* YES!! */ + + /** test long comments **/ + void veryLong() + { + /* + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + blah blah blah blah + enough talk */ + } + + /** + * @see to lazy to document all args. Testing excessive # args + **/ + void toManyArgs(int aArg1, int aArg2, int aArg3, int aArg4, int aArg5, + int aArg6, int aArg7, int aArg8, int aArg9) + { + } +} + +/** Test class for variable naming in for each clauses. */ +class InputSimple2 +{ + /** Some more Javadoc. */ + public void doSomething() + { + //"O" should be named "o" + for (Object O : new java.util.ArrayList()) + { + + } + } +} + +/** Test enum for member naming check */ +enum MyEnum1 +{ + /** ABC constant */ + ABC, + + /** XYZ constant */ + XYZ; + + /** Should be mSomeMemeber */ + private int someMember; +} diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputSwitchWhitespaceAround.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSwitchWhitespaceAround.java index 867166a63..81bb692b0 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/InputSwitchWhitespaceAround.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputSwitchWhitespaceAround.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; public class InputSwitchWhitespaceAround { diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/InputWhitespace.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputWhitespace.java index 75df80731..fa45adc8b 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/InputWhitespace.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/InputWhitespace.java @@ -4,7 +4,7 @@ //////////////////////////////////////////////////////////////////////////////// package com . puppycrawl .tools. - checkstyle; + checkstyle.checks.whitespace; /** * Class for testing whitespace issues. diff --git a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/ParenPadWithSpace.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadWithSpace.java index 07b452d81..c154596fb 100644 --- a/src/test/resources/com/puppycrawl/tools/checkstyle/whitespace/ParenPadWithSpace.java +++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadWithSpace.java @@ -1,4 +1,4 @@ -package com.puppycrawl.tools.checkstyle.whitespace; +package com.puppycrawl.tools.checkstyle.checks.whitespace; public class ParenPadWithSpace { |