Skip to content
Permalink
Browse files

Fixed JENKINS-19157 - Make st:bind work when Jenkins has jQuery plugi…

…n installed

See JENKINS-18867 and JENKINS-18875 for more context

Testing: unit tests continue to pass. unable to reproduce this locally
  • Loading branch information
nullin committed Aug 16, 2013
1 parent c01349a commit 4767eb1660876a2d8498429e2e8220c3df87ff23
Showing with 6 additions and 7 deletions.
  1. +3 −0 README
  2. +3 −7 src/main/resources/hudson/plugins/testng/results/PackageResult/reportDetail.groovy
3 README
@@ -16,6 +16,9 @@ graph and all details about which tests that failed are also presented.
Release Notes
-------

### upcoming
* Fixed: JENKINS-19157 - Make st:bind work when Jenkins has jQuery plugin installed

### v1.4
* Bumped up supported Jenkins version to v1.526 or later
* Integrated TestNG plugin results into email-ext plugin
@@ -8,16 +8,12 @@ t = namespace("/lib/hudson")
st = namespace("jelly:stapler")

script(src:"${app.rootUrl}/plugin/testng-plugin/js/toggle_table.js")

//see https://issues.jenkins-ci.org/browse/JENKINS-18867 & https://issues.jenkins-ci.org/browse/JENKINS-18875
st.bind(var:"thisPkgResult", value:my)
script() {
text("//Loads data for all the methods")
text("\nfunction showAllExecMthds() {")
text("\nvar foo = ")
// apparently, calling st.bind spits out the required information
// directly to output stream. So, no need to wrap it in text().
// in fact, that approach doesn't work
st.bind(value:my)
text("\nfoo.getAllSortedTestMethodsByStartTime(function(t) {")
text("\nthisPkgResult.getAllSortedTestMethodsByStartTime(function(t) {")
text("\ndocument.getElementById('sortedMethods').innerHTML = t.responseObject();")
text("\n})")
text("\ndocument.getElementById(\"showAllLink\").style.display = \"none\"; }")

0 comments on commit 4767eb1

Please sign in to comment.
You can’t perform that action at this time.