Skip to content

GROOVY-12187: Add SqlInjectionChecker type checker to detect quoted G…#2731

Open
paulk-asert wants to merge 1 commit into
apache:masterfrom
paulk-asert:groovy12187
Open

GROOVY-12187: Add SqlInjectionChecker type checker to detect quoted G…#2731
paulk-asert wants to merge 1 commit into
apache:masterfrom
paulk-asert:groovy12187

Conversation

@paulk-asert

Copy link
Copy Markdown
Contributor

…String interpolation in SQL queries

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 67.05882% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.2725%. Comparing base (3240fcf) to head (46257c8).

Files with missing lines Patch % Lines
...ovy/groovy/typecheckers/SqlInjectionChecker.groovy 67.0588% 3 Missing and 25 partials ⚠️
Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##               master      #2731        +/-   ##
==================================================
- Coverage     69.2732%   69.2725%   -0.0007%     
- Complexity      34769      34810        +41     
==================================================
  Files            1542       1543         +1     
  Lines          130456     130541        +85     
  Branches        23786      23816        +30     
==================================================
+ Hits            90371      90429        +58     
- Misses          31962      31966         +4     
- Partials         8123       8146        +23     
Files with missing lines Coverage Δ
...ovy/groovy/typecheckers/SqlInjectionChecker.groovy 67.0588% <67.0588%> (ø)

... and 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@testlens-app

testlens-app Bot commented Jul 23, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 46257c8
▶️ Tests: 105752 executed
⚪️ Checks: 31/31 completed


Learn more about TestLens at testlens.app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants