diff options
author | vasylieva <valeria.vasylieva@gmail.com> | 2017-07-26 11:02:07 +0300 |
---|---|---|
committer | Roman Ivanov <ivanov-jr@mail.ru> | 2017-07-26 13:36:08 -0700 |
commit | 0093555b4a1487ee48d0b49dd9a59c53d9c730da (patch) | |
tree | 0b92e7ac118048b7c372297976582d7f476cc023 /config/intellij-idea-inspections.xml | |
parent | 66ae75b11025247efea7ec0b4d89cd85ecd14e5c (diff) | |
download | checkstyle-0093555b4a1487ee48d0b49dd9a59c53d9c730da.tar.gz |
Issue #4801: fix some violations from eleventh part of idea violations
Diffstat (limited to 'config/intellij-idea-inspections.xml')
-rw-r--r-- | config/intellij-idea-inspections.xml | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/config/intellij-idea-inspections.xml b/config/intellij-idea-inspections.xml index 00934b10e..b59b8db4a 100644 --- a/config/intellij-idea-inspections.xml +++ b/config/intellij-idea-inspections.xml @@ -2132,6 +2132,8 @@ <option value="HtmlTagCanBeJavadocTag" /> <!-- we need to work with AntClassLoader, there is no way to avoid this --> <option value="ClassLoaderInstantiation" /> + <!-- There are asserts in DetailASTTest#checkNode, but idea does not see them --> + <option value="JUnitTestMethodWithNoAssertions" /> </list> </option> </inspection_tool> @@ -2203,8 +2205,8 @@ </inspection_tool> <inspection_tool class="TestMethodInProductCode" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="TestMethodIsPublicVoidNoArg" enabled="true" level="ERROR" enabled_by_default="true" /> - <inspection_tool class="TestMethodWithoutAssertion" enabled="false" level="ERROR" enabled_by_default="false"> - <option name="assertionMethods" value="org.junit.Assert,assert.*|fail.*,junit.framework.Assert,assert.*|fail.*,org.mockito.Mockito,verify.*,org.mockito.InOrder,verify,org.junit.rules.ExpectedException,expect.*,org.hamcrest.MatcherAssert,assertThat" /> + <inspection_tool class="TestMethodWithoutAssertion" enabled="true" level="ERROR" enabled_by_default="true"> + <option name="assertionMethods" value="org.junit.Assert,assert.*|fail.*,junit.framework.Assert,assert.*|fail.*,org.mockito.Mockito,verify.*,org.mockito.InOrder,verify,org.junit.rules.ExpectedException,expect.*,org.hamcrest.MatcherAssert,assertThat,com.puppycrawl.tools.checkstyle.BaseCheckTestSupport,verify.*,com.puppycrawl.tools.checkstyle.AbstractModuleTestSupport,verify.*,com.puppycrawl.tools.checkstyle.AbstractTreeTestSupport,verify.*,com.google.checkstyle.test.base.BaseCheckTestSupport,verify.*,com.puppycrawl.tools.checkstyle.internal.TestUtils,assert.*,com.puppycrawl.tools.checkstyle.grammars.AstRegressionTest.AssertGeneratedJavaLexer,verify.*,nl.jqno.equalsverifier.EqualsVerifier,verify.*,org.powermock.api.mockito.PowerMockito,verify.*" /> <option name="assertKeywordIsAssertion" value="false" /> </inspection_tool> <inspection_tool class="TestNGDataProvider" enabled="true" level="ERROR" enabled_by_default="true" /> @@ -2260,6 +2262,7 @@ </inspection_tool> <inspection_tool class="TrailingSpacesInProperty" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="TransientFieldInNonSerializableClass" enabled="true" level="ERROR" enabled_by_default="true" /> + <!-- conflicts with FindBugs --> <inspection_tool class="TransientFieldNotInitialized" enabled="false" level="ERROR" enabled_by_default="false" /> <inspection_tool class="TrivialConditionalJS" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="TrivialIf" enabled="true" level="ERROR" enabled_by_default="true" /> @@ -2325,6 +2328,7 @@ <inspection_tool class="UnnecessaryConstructor" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="UnnecessaryContinue" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="UnnecessaryContinueJS" enabled="true" level="ERROR" enabled_by_default="true" /> + <!-- conflicts with other tools --> <inspection_tool class="UnnecessaryDefault" enabled="false" level="ERROR" enabled_by_default="false" /> <inspection_tool class="UnnecessaryEnumModifier" enabled="true" level="ERROR" enabled_by_default="true" /> <inspection_tool class="UnnecessaryExplicitNumericCast" enabled="true" level="ERROR" enabled_by_default="true" /> |