Branch: 
refs/heads/dependabot/maven/org.jenkins-ci.modules-instance-identity-116.vf8f487400980
  Home:   https://github.com/jenkinsci/jenkins
  Commit: ec05f0db6dc2cb53bf038046728232f35f941723
      
https://github.com/jenkinsci/jenkins/commit/ec05f0db6dc2cb53bf038046728232f35f941723
  Author: James Nord <jtn...@users.noreply.github.com>
  Date:   2022-07-01 (Fri, 01 Jul 2022)

  Changed paths:
    M test/src/test/java/hudson/PluginTest.java
    M test/src/test/java/hudson/cli/ListPluginsCommandTest.java
    M test/src/test/java/hudson/model/UsageStatisticsTest.java
    M test/src/test/java/jenkins/I18nTest.java

  Log Message:
  -----------
  Fix tests that where failing due to incorrect assumptions

now that we actually have the plugins as expected in JenkinsRule 4 tests
where failing for a variety of reasons

1. loading a detached plugin would downgrade the version requiring a
   restart and failing the test
2. the test was checking that nothing was installed

installing detached plugins has now been skipped where the plugin being
installed was already available due to being a maven dependency.  This
also means we only have onve version of the plugin on the classpath -
prior to this both the classes from the maven dependency and the classes
from the plugin would have been on the uber classpath creating
interesting results.

tests have been updated where there are changes between the detached
version of the plugin and the plugin from the maven dependency.

Where the test was making invalid assumptions this has also been fixed.


-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/jenkins/push/refs/heads/dependabot/maven/org.jenkins-ci.modules-instance-identity-116.vf8f487400980/ec6b75-ec05f0%40github.com.

Reply via email to