Skip to content

Speed up Refaster bug checker #2641

Speed up Refaster bug checker

Speed up Refaster bug checker #2641

Triggered via pull request December 24, 2024 12:35
Status Success
Total duration 7m 44s
Artifacts

build.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

51 warnings
A change can be made to line 83 without causing a test to fail: refaster-support/src/main/java/tech/picnic/errorprone/refaster/AnnotatedCompositeCodeTransformer.java#L83
replaced return value with null for withTransformers (no tests cover this line NullReturnValsMutator)
2 different changes can be made to line 147 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L147
removed conditional - replaced equality check with false (covered by 19 tests RemoveConditionalMutator_EQUAL_ELSE) removed conditional - replaced equality check with true (covered by 19 tests RemoveConditionalMutator_EQUAL_IF)
A change can be made to line 203 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L203
replaced return value with "" for treeKindToString (covered by 19 tests EmptyObjectReturnValsMutator)
A change can be made to line 163 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L163
removed conditional - replaced equality check with true (covered by 19 tests RemoveConditionalMutator_EQUAL_IF)
A change can be made to line 140 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L140
removed conditional - replaced equality check with false (covered by 19 tests RemoveConditionalMutator_EQUAL_ELSE)
2 different changes can be made to line 166 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L166
removed conditional - replaced equality check with false (no tests cover this line RemoveConditionalMutator_EQUAL_ELSE) removed conditional - replaced equality check with true (no tests cover this line RemoveConditionalMutator_EQUAL_IF)
2 different changes can be made to line 137 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L137
removed conditional - replaced equality check with false (no tests cover this line RemoveConditionalMutator_EQUAL_ELSE) removed conditional - replaced equality check with true (no tests cover this line RemoveConditionalMutator_EQUAL_IF)
A change can be made to line 135 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L135
removed conditional - replaced equality check with false (covered by 19 tests RemoveConditionalMutator_EQUAL_ELSE)
A change can be made to line 424 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L424
replaced return value with Collections.emptyList for copy (no tests cover this line EmptyObjectReturnValsMutator)
A change can be made to line 331 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L331
removed conditional - replaced equality check with false (covered by 19 tests RemoveConditionalMutator_EQUAL_ELSE)
A change can be made to line 386 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L386
replaced return value with null for visitCompoundAssignment (no tests cover this line NullReturnValsMutator)
A change can be made to line 392 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L392
replaced return value with null for visitUnary (no tests cover this line NullReturnValsMutator)
2 different changes can be made to line 413 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L413
removed conditional - replaced equality check with false (no tests cover this line RemoveConditionalMutator_EQUAL_ELSE) removed conditional - replaced equality check with true (no tests cover this line RemoveConditionalMutator_EQUAL_IF)
A change can be made to line 336 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L336
removed conditional - replaced equality check with false (covered by 19 tests RemoveConditionalMutator_EQUAL_ELSE)
A change can be made to line 349 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L349
replaced return value with "" for getIdentifier (no tests cover this line EmptyObjectReturnValsMutator)
A change can be made to line 379 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L379
replaced return value with null for visitAssignment (no tests cover this line NullReturnValsMutator)
2 different changes can be made to line 409 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L409
removed conditional - replaced equality check with false (no tests cover this line RemoveConditionalMutator_EQUAL_ELSE) removed conditional - replaced equality check with true (no tests cover this line RemoveConditionalMutator_EQUAL_IF)
A change can be made to line 398 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L398
replaced return value with null for visitBinary (covered by 19 tests NullReturnValsMutator)
5 different changes can be made to line 354 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L354
removed call to substring (no tests cover this line RemoveChainedCallsMutator) changed conditional boundary (no tests cover this line ConditionalsBoundaryMutator) Replaced integer addition with subtraction (no tests cover this line MathMutator) removed conditional - replaced comparison check with false (no tests cover this line RemoveConditionalMutator_ORDER_ELSE) replaced return value with "" for getSimpleName (no tests cover this line EmptyObjectReturnValsMutator)
A change can be made to line 49 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/Node.java#L49
removed conditional - replaced equality check with false (covered by 308 tests RemoveConditionalMutator_EQUAL_ELSE)
2 different changes can be made to line 59 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/Node.java#L59
changed conditional boundary (covered by 308 tests ConditionalsBoundaryMutator) removed conditional - replaced comparison check with false (covered by 308 tests RemoveConditionalMutator_ORDER_ELSE)
A change can be made to line 487 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L487
replaced return value with null for visitAssignment (no tests cover this line NullReturnValsMutator)
A change can be made to line 506 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L506
replaced return value with null for visitBinary (covered by 19 tests NullReturnValsMutator)
A change can be made to line 494 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L494
replaced return value with null for visitCompoundAssignment (no tests cover this line NullReturnValsMutator)
2 different changes can be made to line 461 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L461
swapped parameters 1 and 2 in call to reduce (covered by 19 tests ParamSwapMutator) replaced return value with null for visitVariable (covered by 19 tests NullReturnValsMutator)
A change can be made to line 500 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L500
replaced return value with null for visitUnary (no tests cover this line NullReturnValsMutator)
A change can be made to line 455 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L455
replaced return value with null for visitMethod (covered by 19 tests NullReturnValsMutator)
A change can be made to line 446 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L446
replaced return value with null for visitClass (covered by 19 tests NullReturnValsMutator)
A change can be made to line 106 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/Node.java#L106
removed conditional - replaced equality check with false (covered by 25 tests RemoveConditionalMutator_EQUAL_ELSE)
A change can be made to line 100 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/Node.java#L100
removed call to java/util/List::sort (covered by 25 tests VoidMethodCallMutator)
A change can be made to line 282 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L282
replaced return value with null for getClassIdent (no tests cover this line NullReturnValsMutator)
A change can be made to line 287 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L287
replaced return value with "" for getTopLevelClass (no tests cover this line EmptyObjectReturnValsMutator)
A change can be made to line 278 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L278
replaced return value with null for getTemplateStatements (no tests cover this line NullReturnValsMutator)
A change can be made to line 266 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L266
replaced boolean return with false for isUClassIdent (covered by 19 tests BooleanFalseReturnValsMutator)
A change can be made to line 291 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/RefasterRuleSelector.java#L291
replaced return value with null for getExpressions (no tests cover this line NullReturnValsMutator)
A change can be made to line 104 without causing a test to fail: refaster-runner/src/main/java/tech/picnic/errorprone/refaster/runner/Refaster.java#L104
replaced return value with null for matchCompilationUnit (no tests cover this line NullReturnValsMutator)