Skip to content

PR #15824

PR #15824 #2305

Triggered via dynamic January 27, 2025 08:54
Status Success
Total duration 1m 47s
Artifacts

codeql

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

Annotations

8 errors and 54 warnings
ProgramLifecycleHttpHandlerInternalTest.testGetRunRecordMeta: ProgramLifecycleHttpHandlerInternalTest#L78
Problem accessing: /v3Internal/namespaces/testnamespace1/apps/SleepWorkflowApp/versions/-SNAPSHOT/workflows/SleepWorkflow/runs/9f6f6842-dc8e-11ef-aad4-4e40ed74e3e3. Reason: Not Found
ProgramLifecycleHttpHandlerTest.testProgramStartStopStatusErrors: ProgramLifecycleHttpHandlerTest#L380
Wrong response code with message expected:<409> but was:<200>
ProgramLifecycleHttpHandlerTest.testProgramList: ProgramLifecycleHttpHandlerTest#L843
expected:<[[]]> but was:<[[{"type":"Workflow","app":"App","name":"NoOpWorkflow","description":"NoOp Workflow description"},{"type":"Workflow","app":"AppWithWorkflow","name":"SampleWorkflow","description":"SampleWorkflow description"}]]>
ProgramLifecycleHttpHandlerTest.testVersionedProgramStartStopStatus: ProgramLifecycleHttpHandlerTest#L321
Wrong response code with message expected:<409> but was:<200>
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewRunner.java#L116
Abbreviation in name 'levelDBTableService' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewRunner.java#L137
Abbreviation in name 'levelDBTableService' must contain no more than '1' consecutive capital letters.
com.puppycrawl.tools.checkstyle.checks.design.OneTopLevelClassCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ProgramNotificationSubscriberService.java#L251
Top-level class ProgramNotificationSingleTopicSubscriberService has to reside in its own source file.
com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/AbstractProgramLifecycleService.java#L206
All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '135'.
com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck: cdap-app-fabric/src/main/java/io/cdap/cdap/app/guice/AppFabricServiceRuntimeModule.java#L123
Wrong lexicographical order for 'io.cdap.cdap.internal.app.services.FlowControlService' import. Should be before 'io.cdap.cdap.internal.app.services.RunRecordCorrectorService'.
com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/capability/CapabilityManagementServiceTest.java#L147
Distance between variable 'programId' 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-app-fabric/src/test/java/io/cdap/cdap/internal/capability/CapabilityManagementServiceTest.java#L236
Distance between variable 'programId' 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.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/capability/CapabilityManagementServiceTest.java#L292
Distance between variable 'programId' 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.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/capability/CapabilityManagementServiceTest.java#L383
Distance between variable 'programId' 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.coding.VariableDeclarationUsageDistanceCheck: cdap-app-fabric/src/test/java/io/cdap/cdap/internal/AppFabricClient.java#L241
Distance between variable 'responder' 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-app-fabric/src/test/java/io/cdap/cdap/internal/AppFabricClient.java#L263
Distance between variable 'responder' 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).