jenkinsci / junit-plugin

Allows JUnit-format test results to be published
https://plugins.jenkins.io/junit
MIT License
76 stars 338 forks source link

[JENKINS-69658] - CSP compatibility for summary.jelly #654

Closed shlomomdahan closed 1 month ago

shlomomdahan commented 1 month ago

JENKINS-69658

This PR is an extension of https://github.com/jenkinsci/junit-plugin/pull/451 to fix the inline JS in the summary.jelly file.

The failureSummary.js file had to be changed so that it can handle event delegation as some of the elements we are targeting with the fix do not actually appear in the html until the parent is expanded.

Testing done

https://www.loom.com/share/796f46d313e34c8cbbaf85963c1ffb59?sid=20dc0f68-ddca-4f88-a977-1580816dcdab

https://www.loom.com/share/f7a8117e45a54da5a719599f0d1adb08?sid=8c812aef-9c9a-4a81-b662-c49229d68dbf

https://www.loom.com/share/6e3642baf5934fe59e2efdc3f3f0ec3b

Submitter checklist