Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-31161

Update HTMLUnit in Jenkins Core

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Similar Issues:

      Description

      It is hoped to use modern JS libs for all of our new UI work (create item, 1st run plugin bundling experience etc). Jenkins Core currently uses old JS libs, mainly prototypejs. This library has been dead for years + we are using a forked version of it. We were hoping to get away from it in the newer work.
      We are also currently using a forked version of HtmlUnit in Jenkins Core. It's an old version (2.6 - latest is 2.18). This version of HtmlUnit doesn't support use of jQuery (and probably lots of other JS libs we would want to use) e.g. it complains about how the "isArray" method is defined in the lib. They claim that the latest version of HtmlUnit "fully supports" jQuery (I'm not convinced).
      The problem for us is that the Jenkins Core test harness relies heavily on HtmlUnit. There are many many tests written to run against the forked version, which means they fail when we try to do anything that that version of HtmlUnit doesn't like e.g. introduce jQuery.

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            I think this was fixed a while ago?

            Show
            jglick Jesse Glick added a comment - I think this was fixed a while ago?
            Hide
            tfennelly Tom FENNELLY added a comment -

            It was. Thanks for closing.

            Show
            tfennelly Tom FENNELLY added a comment - It was. Thanks for closing.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: recena
            Path:
            core/src/main/java/hudson/model/ItemGroupMixIn.java
            core/src/main/java/jenkins/model/item_category/ItemCategoryConfigurator.java
            http://jenkins-ci.org/commit/jenkins/2d0219f739030a45f7fce645b4e4e02ac7536574
            Log:
            Merge branch 'JENKINS-31162' into JENKINS-31161-withUI

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: recena Path: core/src/main/java/hudson/model/ItemGroupMixIn.java core/src/main/java/jenkins/model/item_category/ItemCategoryConfigurator.java http://jenkins-ci.org/commit/jenkins/2d0219f739030a45f7fce645b4e4e02ac7536574 Log: Merge branch ' JENKINS-31162 ' into JENKINS-31161 -withUI
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Manuel Recena
            Path:
            core/src/main/java/hudson/model/ItemGroupMixIn.java
            core/src/main/java/jenkins/model/item_category/Category.java
            core/src/main/java/jenkins/model/item_category/ItemCategory.java
            core/src/main/java/jenkins/model/item_category/ItemCategoryConfigurator.java
            http://jenkins-ci.org/commit/jenkins/480e41a356a584262881b99acfd0e3d8d316a12d
            Log:
            Merge remote-tracking branch 'origin/JENKINS-31162' into JENKINS-31161-withUI

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Manuel Recena Path: core/src/main/java/hudson/model/ItemGroupMixIn.java core/src/main/java/jenkins/model/item_category/Category.java core/src/main/java/jenkins/model/item_category/ItemCategory.java core/src/main/java/jenkins/model/item_category/ItemCategoryConfigurator.java http://jenkins-ci.org/commit/jenkins/480e41a356a584262881b99acfd0e3d8d316a12d Log: Merge remote-tracking branch 'origin/ JENKINS-31162 ' into JENKINS-31161 -withUI

              People

              Assignee:
              tfennelly Tom FENNELLY
              Reporter:
              hrmpw Patrick Wolf
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: