Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JENKINS-73941 - Option to hide "Use Groovy Sandbox" for users without Administer permission globally in the system #584

Merged
merged 27 commits into from
Oct 25, 2024

JENKINS-73941 - Add forceSandbox logic to SecureGroovyScript

4e97712
Select commit
Loading
Failed to load commit list.
Merged

JENKINS-73941 - Option to hide "Use Groovy Sandbox" for users without Administer permission globally in the system #584

JENKINS-73941 - Add forceSandbox logic to SecureGroovyScript
4e97712
Select commit
Loading
Failed to load commit list.
ci.jenkins.io / Code Coverage succeeded Oct 25, 2024 in 0s

Line Coverage: 100.00% (+13.97%), Branch Coverage: 85.29% (+10.14%)

Summary for modified lines

  • 40 lines have been modified
  • all lines are covered
  • 4 lines are covered only partially

Overview by baseline

Quality Gates Summary

No active quality gates.

Details

Project coverage details

Module Coverage Package Coverage File Coverage Class Coverage Method Coverage Line Coverage Branch Coverage Instruction Coverage
🐾 Overall project 100.00% (1/1) 100.00% (5/5) 93.94% (31/33) 92.19% (59/64) 85.28% (423/496) 87.33% (1717/1966) 78.86% (910/1154) 87.63% (8258/9424)
📈 Overall project (difference to reference job) +0.00% +0.00% +0.00% +0.00% +0.32% ⬆️ +0.17% ⬆️ +0.31% ⬆️ +0.11% ⬆️
🐾 Modified files 100.00% (1/1) 100.00% (2/2) 100.00% (4/4) 95.24% (20/21) 86.50% (141/163) 86.03% (659/766) 75.15% (251/334) 86.31% (2882/3339)
📈 Modified files (difference to reference job) n/a n/a n/a n/a n/a +0.47% ⬆️ +1.45% ⬆️ n/a
🐾 Modified code lines 100.00% (1/1) 100.00% (2/2) 100.00% (4/4) n/a n/a 100.00% (18/18) 85.29% (29/34) n/a
📈 Modified code lines (difference to modified files) n/a n/a n/a n/a n/a +13.97% ⬆️ +10.14% ⬆️ n/a
🐾 Indirect changes 100.00% (1/1) 100.00% (1/1) 100.00% (1/1) n/a n/a 100.00% (1/1) 100.00% (1/1) n/a

Annotations

Check warning on line 470 in src/main/java/org/jenkinsci/plugins/scriptsecurity/sandbox/groovy/SecureGroovyScript.java

See this annotation in the file changed.

@ci-jenkins-io ci-jenkins-io / Code Coverage

Partially covered line

Line 470 is only partially covered, 3 branches are missing

Check warning on line 900 in src/main/java/org/jenkinsci/plugins/scriptsecurity/scripts/ScriptApproval.java

See this annotation in the file changed.

@ci-jenkins-io ci-jenkins-io / Code Coverage

Partially covered line

Line 900 is only partially covered, 2 branches are missing