SONARJAVA-6454 : Rule S8714 - implement quickfix#5663

Draft

rombirli wants to merge 27 commits into

masterfrom

rombirli/sonarjava-6406-create-rule-S8714

Commits on Jun 8, 2026

quickfix skeleton
rombirli
committed
quickfix skeleton 2
rombirli
committed
QuickFix skleton part3
rombirli
committed
Initial quickfixes implementation, arguments not aware
rombirli
committed
Fix for junit assertDoesntThrow (doesnt takes any throwable as arg)
rombirli
committed
Junit arguments aware
rombirli
committed
Add quickfix arguments for assertJ
rombirli
committed
Fix runtime issues (findfirst) and missing class names for String and Supplier
rombirli
committed
Filter out fail method invocation from replacement
rombirli
committed
Fix quickfix fail filterign & try to test quickfix
rombirli
committed
Improve test (still failing), add comment how to generate quickfix correctly
rombirli
committed
Refine implementation plan for 3 steps replacement for excluding fail method invocation
rombirli
committed
3 steps edit implementation
rombirli
committed
Fix tests, try to remove semicolon after fail in quickfix (dont work)
rombirli
committed
Fix : report only on junit5 tests / assertJ fail
rombirli
committed
remove import *
rombirli
committed
NonCompliant -> Noncompliant
rombirli
committed
Support Supplier<String> as argument for junit fail
rombirli
committed
Remove import * in TryCatchUtils.java
rombirli
committed
Remove import * in TryCatchUtils.java
rombirli
committed
Avoid NPE crash when there is no catch
rombirli
committed
Clarify intentionality of sample changes
rombirli
committed
Fix autoscan
rombirli
committed
FIX (Address comments, refactor, ...) TryCatchUtils and unit test it
rombirli
committed
Address comment : "The fourth argument is a function of the third. Let's remove it."
rombirli
committed
Fix QG : use !isEmpty() instead of size() > 0
rombirli
committed
Remove Leftover debug System.out.println in new test
rombirli
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

SONARJAVA-6454 : Rule S8714 - implement quickfix#5663

SONARJAVA-6454 : Rule S8714 - implement quickfix#5663
rombirli wants to merge 27 commits into
masterfrom
rombirli/sonarjava-6406-create-rule-S8714

Commits on Jun 8, 2026

quickfix skeleton

quickfix skeleton 2

QuickFix skleton part3

Initial quickfixes implementation, arguments not aware

Fix for junit assertDoesntThrow (doesnt takes any throwable as arg)

Junit arguments aware

Add quickfix arguments for assertJ

Fix runtime issues (findfirst) and missing class names for String and Supplier

Filter out fail method invocation from replacement

Fix quickfix fail filterign & try to test quickfix

Improve test (still failing), add comment how to generate quickfix correctly

Refine implementation plan for 3 steps replacement for excluding fail method invocation

3 steps edit implementation

Fix tests, try to remove semicolon after fail in quickfix (dont work)

Fix : report only on junit5 tests / assertJ fail

remove import *

NonCompliant -> Noncompliant

Support Supplier<String> as argument for junit fail

Remove import * in TryCatchUtils.java

Remove import * in TryCatchUtils.java

Avoid NPE crash when there is no catch

Clarify intentionality of sample changes

Fix autoscan

FIX (Address comments, refactor, ...) TryCatchUtils and unit test it

Address comment : "The fourth argument is a function of the third. Let's remove it."

Fix QG : use !isEmpty() instead of size() > 0

Remove Leftover debug System.out.println in new test