Skip to content

PR #15648

PR #15648 #1487

Triggered via dynamic July 19, 2024 12:23
Status Success
Total duration 1m 56s
Artifacts

codeql

on: dynamic
Matrix: analyze
Fit to window
Zoom out
Zoom in

Annotations

131 warnings
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/RemoteAppStateStoreTest.java#L51
Wrong lexicographical order for 'java.io.IOException' import. Should be before 'org.mockito.Mockito'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/RemoteAppStateStoreTest.java#L52
Wrong lexicographical order for 'java.nio.charset.StandardCharsets' import. Should be before 'org.mockito.Mockito'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/RemoteAppStateStoreTest.java#L53
Wrong lexicographical order for 'java.util.Base64' import. Should be before 'org.mockito.Mockito'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/runtime/RemoteAppStateStoreTest.java#L54
Wrong lexicographical order for 'java.util.Optional' import. Should be before 'org.mockito.Mockito'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/capability/AutoInstallTest.java#L84
Distance between variable 'ca' declaration and its first usage is 11, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric-tests/src/test/java/io/cdap/cdap/internal/app/runtime/batch/DynamicPartitionerWithAvroTest.java#L138
Distance between variable 'app' declaration and its first usage is 7, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-common/src/test/java/io/cdap/cdap/io/SchemaTest.java#L195
Distance between variable 'schema' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-common/src/test/java/io/cdap/cdap/io/SchemaTest.java#L261
Abbreviation in name 'testParseFlatSQL' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-common/src/test/java/io/cdap/cdap/io/SchemaTest.java#L296
Abbreviation in name 'testNestedSQL' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-common/src/test/java/io/cdap/cdap/io/SchemaTest.java#L340
Abbreviation in name 'testParseSQLWithWhitespace' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-common/src/test/java/io/cdap/cdap/io/SchemaTest.java#L358
Abbreviation in name 'testInvalidSQL' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L39
Extra separation in import group before 'io.netty.handler.ssl.util.InsecureTrustManagerFactory'
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L39
Wrong lexicographical order for 'io.netty.handler.ssl.util.InsecureTrustManagerFactory' import. Should be before 'javax.net.ssl.SSLContext'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L96
Distance between variable 'sConfiguration' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L101
Distance between variable 'discoveryServiceClient' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterHttpTest.java#L103
Distance between variable 'userIdentityExtractor' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterPipelineTest.java#L150
Abbreviation in name 'testDeployNTimes' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L89
Extra separation in import group before 'io.netty.handler.ssl.util.InsecureTrustManagerFactory'
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L89
Wrong lexicographical order for 'io.netty.handler.ssl.util.InsecureTrustManagerFactory' import. Should be before 'javax.ws.rs.PathParam'.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L142
Abbreviation in name 'getHTTPClient' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L144
Abbreviation in name 'getAsyncHTTPClient' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L148
Abbreviation in name 'resolveURI' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-gateway/src/test/java/io/cdap/cdap/gateway/router/NettyRouterTestBase.java#L593
Abbreviation in name 'openURL' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/JAASAuthenticationHandler.java#L29
Abbreviation in name 'JAASAuthenticationHandler' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/AbstractAuthenticationHandler.java#L58
Comment has incorrect indentation level 0, expected is 4, indentation should be the same level as line 59.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/GrantAccessToken.java#L137
Distance between variable 'token' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/JASPIAuthenticationHandler.java#L46
Abbreviation in name 'JASPIAuthenticationHandler' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/ExternalAuthenticationServer.java#L44
Wrong lexicographical order for 'org.eclipse.jetty.server.Server' import. Should be before 'org.eclipse.jetty.server.ServerConnector'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/ExternalAuthenticationServer.java#L160
Distance between variable 'keyStoreType' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/ExternalAuthenticationServer.java#L163
Distance between variable 'keyPassword' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/LDAPLoginModule.java#L42
Abbreviation in name 'LDAPLoginModule' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/LDAPLoginModule.java#L49
Abbreviation in name 'TrustAllSSLSocketFactory' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck: cdap-security/src/main/java/io/cdap/cdap/security/server/LDAPLoginModule.java#L71
WhitespaceAround: '}' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/MTLSUserIdentity.java#L33
Abbreviation in name 'MTLSUserIdentity' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/MTLSUserIdentity.java#L45
Abbreviation in name 'getX509PrincipalCN' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/ExternalMTLSAuthenticationServerTest.java#L74
Abbreviation in name 'ExternalMTLSAuthenticationServerTest' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/ExternalMTLSAuthenticationServerTest.java#L93
Distance between variable 'sConf' declaration and its first usage is 11, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/MTLSLoginService.java#L30
Wrong lexicographical order for 'javax.servlet.ServletRequest' import. Should be before 'org.slf4j.LoggerFactory'.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-security/src/test/java/io/cdap/cdap/security/server/MTLSLoginService.java#L38
Abbreviation in name 'MTLSLoginService' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-security/src/test/java/io/cdap/cdap/security/encryption/guice/LazyDelegateAeadCipherTest.java#L26
Wrong lexicographical order for 'io.cdap.cdap.security.spi.encryption.AeadCipherContext' import. Should be before 'io.cdap.cdap.security.spi.encryption.AeadCipherCryptor'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-unit-test/src/test/java/io/cdap/cdap/admin/AdminAppTestRun.java#L92
Distance between variable 'bFormat' declaration and its first usage is 13, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-unit-test/src/test/java/io/cdap/cdap/admin/AdminAppTestRun.java#L120
Abbreviation in name 'newBPath' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-unit-test/src/test/java/io/cdap/cdap/admin/AdminAppTestRun.java#L151
Abbreviation in name 'serviceURI' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-unit-test/src/test/java/io/cdap/cdap/admin/AdminAppTestRun.java#L250
Abbreviation in name 'namespaceXMeta' must contain no more than '1' consecutive capital letters.