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

Deprecate PageObject constructor receiving the Injector as parameter

XMLWordPrintable

      See https://github.com/jenkinsci/acceptance-test-harness/pull/631 and https://github.com/jenkinsci/acceptance-test-harness/pull/632.
       
      As part of those PRs I've found that many classes extending `PageObject` are using the constructor receiving `Injector` as parameter. This constructor should be deprecated and those classes should use the other constructor receiving the `context`.

      Motivation of the change

      • If more than one instances are involved in the test (SSO e.g.) using the first constructor makes the injector retrieve the Jenkins instance, and it always retrieves the first one registered.
      • Using the constructor with `context` stores the injector as well, so we don't lose anything

            Unassigned Unassigned
            fcojfernandez Francisco Fernández
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: