Skip to content

Commit

Permalink
SimpleAccessorNameNotationCheck_195
Browse files Browse the repository at this point in the history
  • Loading branch information
alexkravin authored and Daniil Yaroslavtsev committed Sep 1, 2014
1 parent f36b26c commit 3c7db16
Show file tree
Hide file tree
Showing 9 changed files with 48 additions and 45 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ public void correctWithoutPrefix() throws Exception
{
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
final String[] expected = {
"16: " + warningSetterMessage,
"21: " + warningGetterMessage,
"17: " + warningSetterMessage,
"22: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation1.java"), expected);
}
Expand All @@ -47,8 +47,8 @@ public void correctWithtPrefix() throws Exception
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
checkConfig.addAttribute("prefix", "m");
final String[] expected = {
"6: " + warningSetterMessage,
"11: " + warningGetterMessage,
"7: " + warningSetterMessage,
"12: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation1.java"), expected);
}
Expand All @@ -58,12 +58,12 @@ public void partlyWithoutPrefix() throws Exception
{
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
final String[] expected = {
"14: " + warningGetterMessage,
"19: " + warningSetterMessage,
"29: " + warningSetterMessage,
"34: " + warningGetterMessage,
"39: " + warningSetterMessage,
"44: " + warningGetterMessage,
"15: " + warningGetterMessage,
"20: " + warningSetterMessage,
"30: " + warningSetterMessage,
"35: " + warningGetterMessage,
"40: " + warningSetterMessage,
"45: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation2.java"), expected);
}
Expand All @@ -74,12 +74,12 @@ public void partlyWithPrefix() throws Exception
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
checkConfig.addAttribute("prefix", "m");
final String[] expected = {
"9: " + warningSetterMessage,
"14: " + warningGetterMessage,
"19: " + warningSetterMessage,
"24: " + warningGetterMessage,
"34: " + warningGetterMessage,
"39: " + warningSetterMessage,
"10: " + warningSetterMessage,
"15: " + warningGetterMessage,
"20: " + warningSetterMessage,
"25: " + warningGetterMessage,
"35: " + warningGetterMessage,
"40: " + warningSetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation2.java"), expected);
}
Expand All @@ -89,10 +89,10 @@ public void errorsWithoutPrefix() throws Exception
{
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
final String[] expected = {
"6: " + warningSetterMessage,
"11: " + warningGetterMessage,
"16: " + warningSetterMessage,
"21: " + warningGetterMessage,
"7: " + warningSetterMessage,
"12: " + warningGetterMessage,
"17: " + warningSetterMessage,
"22: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation3.java"), expected);
}
Expand All @@ -103,10 +103,10 @@ public void errorsWithPrefix() throws Exception
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
checkConfig.addAttribute("prefix", "m");
final String[] expected = {
"6: " + warningSetterMessage,
"11: " + warningGetterMessage,
"16: " + warningSetterMessage,
"21: " + warningGetterMessage,
"7: " + warningSetterMessage,
"12: " + warningGetterMessage,
"17: " + warningSetterMessage,
"22: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation3.java"), expected);
}
Expand All @@ -133,8 +133,8 @@ public void testBoolean() throws Exception
{
final DefaultConfiguration checkConfig = createCheckConfig(SimpleAccessorNameNotationCheck.class);
final String[] expected = {
"27: " + warningSetterMessage,
"32: " + warningGetterMessage,
"28: " + warningSetterMessage,
"33: " + warningGetterMessage,
};
verify(checkConfig, getPath("InputSimpleAccessorNameNotation5.java"), expected);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
package com.puppycrawl.tools.checkstyle.checks.coding;
public class InputSimpleAccesorNameNotation {
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation {
private int mFirst;
private int mSecond;
private int mA;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation1
{
private String field1;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation2
{
private String field1;
private int field2;

private String field3;
private String mField3;
private int mField4;

Expand All @@ -18,7 +19,7 @@ public String getFiel1() // !! in both case

public void setFiel2(String aField2) // !! in both case
{
field2 = aField2;
field3 = aField2;
}

public int getField2() //prefix "m" !!
Expand All @@ -38,7 +39,7 @@ public String getFiel3() // !! in both case

public void setFiel4(String aField4) // !! in both case
{
mField4 = aField4;
mField3 = aField4;
}

public int getField4() //prefix "" !!
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation3
{
private String field1;
private int mField2;

private String mField3;
public void setFiel1(String aField1) // !! in both case
{
field1 = aField1;
Expand All @@ -15,7 +16,7 @@ public String getFiel1() // !! in both case

public void setFie2(String aField2) // !! in both case
{
mField2 = aField2;
mField3 = aField2;
}

public int getFiel2() // !! in both case
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation4
{
private String field1;
Expand All @@ -22,6 +23,6 @@ public void setField2(int field2)
public int getField2()
{
String anyString = null;
return anyString;
return 4;
}
}
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
package com.github.sevntu.checkstyle.checks.coding;
public class InputSimpleAccessorNameNotation5
{
private boolean field1;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,18 +1,16 @@
package com.puppycrawl.tools.checkstyle.checks.coding;
package com.github.sevntu.checkstyle.checks.coding;

public class InputSimpleAccesorNameNotation7
public class InputSimpleAccessorNameNotation6
{
public static void
updateCheckSum(final InputStream metadataFile, File metadataFileSum, HashFunction hashFunction)
throws IOException
updateCheckSum(final String metadataFile, String metadataFileSum, String hashFunction)
throws Exception
{
HashCode hash = ByteStreams.hash(new InputSupplier<InputStream>()
{
@Override
public InputStream getInput() throws IOException
new Object() {
public String getInput() throws Exception
{
return metadataFile;
}
}, hashFunction);
};
}
}
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package com.puppycrawl.tools.checkstyle.checks.coding;
package com.github.sevntu.checkstyle.checks.coding;

public interface InputSimpleAccesorNameNotation8
public interface InputSimpleAccessorNameNotation7
{
String getSmth();
}

0 comments on commit 3c7db16

Please sign in to comment.