• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Component/s: junit-plugin
• Labels:

  None
• Environment:

  Jenkins 2.263.1
  JUnit Plugin 1.48
  ECharts API Plugin 4.9.0-2
  Microsoft Edge Version 87.0.664.75 (Official build) (64-bit)

[JENKINS-64586] Test duration trend chart does not appear, because of jQuery version mismatch

Kalle Niemitalo created issue - 2021-01-10 00:55

Kalle Niemitalo made changes - 2021-01-10 00:55

Link

New: This issue relates to JENKINS-64585 [ JENKINS-64585 ]

Kalle Niemitalo made changes - 2021-01-10 01:27

Description

Original: After the JENKINS-62096 ECharts migration, the test duration trend chart at https://REDACTED/jenkins/job/REDACTED/job/master/327/testReport/history/ does not appear at all; there are no axes or legends or anything. The test result trend chart however works OK. The problem persists even if I comment out the {{$('[data-toggle="tooltip"]').tooltip();}} call that causes JENKINS-64585.

New: After the JENKINS-62096 ECharts migration, the test duration trend chart at https://REDACTED/jenkins/job/REDACTED/job/master/327/testReport/history/ does not appear at all; there are no axes or legends or anything. The test result trend chart however works OK. The problem persists even if I edit history.js as follows:  * For JENKINS-64585, comment out the {{$('[data-toggle="tooltip"]').tooltip();}} call.  * For JENKINS-64587, change the top-level statement to {{jQuery3(function ($) { … });}}. Therefore, the problem does not seem to be caused by those issues.

Kalle Niemitalo made changes - 2021-01-10 01:27

Link

New: This issue relates to JENKINS-64587 [ JENKINS-64587 ]

Kalle Niemitalo made changes - 2021-01-10 02:14

Description

Original: After the JENKINS-62096 ECharts migration, the test duration trend chart at https://REDACTED/jenkins/job/REDACTED/job/master/327/testReport/history/ does not appear at all; there are no axes or legends or anything. The test result trend chart however works OK. The problem persists even if I edit history.js as follows:  * For JENKINS-64585, comment out the {{$('[data-toggle="tooltip"]').tooltip();}} call.  * For JENKINS-64587, change the top-level statement to {{jQuery3(function ($) { … });}}. Therefore, the problem does not seem to be caused by those issues.

New: After the JENKINS-62096 ECharts migration, the test duration trend chart at https://REDACTED/jenkins/job/REDACTED/job/master/327/testReport/history/ does not appear at all; there are no axes or legends or anything. The test result trend chart however works OK. The problem persists even if I edit history.js as follows:  * For JENKINS-64585, comment out the {{$('[data-toggle="tooltip"]').tooltip();}} call.  * For JENKINS-64587, change the top-level statement to {code:javascript}(function ($, view) {     $(function () {         ...     }); })(jQuery3, view);{code} Therefore, the problem does not seem to be caused by those issues.

Kalle Niemitalo made changes - 2021-01-10 02:20

Link

New: This issue is caused by JENKINS-62096 [ JENKINS-62096 ]

Collapse comment: Kalle Niemitalo added a comment - 2021-01-10 02:46

Kalle Niemitalo added a comment - 2021-01-10 02:46

It seems the 'slid.bs.carousel' event handler is not called at all.

Expand comment: Kalle Niemitalo added a comment - 2021-01-10 02:46

Kalle Niemitalo added a comment - 2021-01-10 02:46 It seems the 'slid.bs.carousel' event handler is not called at all.

Collapse comment: Kalle Niemitalo added a comment - 2021-01-10 14:01, Edited by Kalle Niemitalo - 2021-01-10 14:07

Kalle Niemitalo added a comment - 2021-01-10 14:01 - edited

Both static/316c5f91/plugin/jquery3-api/webjars/jquery.min.js (jQuery v3.5.1) and adjuncts/316c5f91/org/kohsuke/stapler/jquery/jquery.full.js (jQuery JavaScript Library v1.12.4) have been loaded on the same web page. It seems the event handler is added using jquery.min.js but then looked up using jquery-full.js.

Expand comment: Kalle Niemitalo added a comment - 2021-01-10 14:01, Edited by Kalle Niemitalo - 2021-01-10 14:07

Kalle Niemitalo added a comment - 2021-01-10 14:01 - edited Both static/316c5f91/plugin/jquery3-api/webjars/jquery.min.js (jQuery v3.5.1) and adjuncts/316c5f91/org/kohsuke/stapler/jquery/jquery.full.js (jQuery JavaScript Library v1.12.4) have been loaded on the same web page. It seems the event handler is added using jquery.min.js but then looked up using jquery-full.js.

Collapse comment: Kalle Niemitalo added a comment - 2021-01-10 14:22

Kalle Niemitalo added a comment - 2021-01-10 14:22

Workaround: make history.js add the event handler by setting the onslid property in the DOM instead of using the jQuery event facility. That way, jQuery 1.12.4 will find the event handler.

        carousel.each(function () {
            this.onslid = function (e) {
                localStorage.setItem(carouselId, e.to);
                const chart = $(e.relatedTarget).find('>:first-child')[0].echart;
                if (chart) {
                    chart.resize();
                }
            };
        });

Expand comment: Kalle Niemitalo added a comment - 2021-01-10 14:22

Kalle Niemitalo added a comment - 2021-01-10 14:22 Workaround: make history.js add the event handler by setting the onslid property in the DOM instead of using the jQuery event facility. That way, jQuery 1.12.4 will find the event handler. carousel.each( function () { this .onslid = function (e) { localStorage.setItem(carouselId, e.to); const chart = $(e.relatedTarget).find( '>:first-child' )[0].echart; if (chart) { chart.resize(); } }; });

Kalle Niemitalo made changes - 2021-01-10 14:22

Summary

Original: Test duration trend chart does not appear

New: Test duration trend chart does not appear, because of jQuery version mismatch

Load more newer events