• Icon: Story Story
    • Resolution: Fixed
    • Icon: Minor Minor
    • blueocean-plugin
    • None
    • 1.0-m10, 1.0-m11, 1.0-m12, 1.0-pre-beta-1, 1.0-beta-1

      Create a v1 acceptance test for testing the the multibranch flow:

      1. Create the multibranch job. Will need to have branches.
      2. Inspect structure of the Branches tab. Use a Nightwatch page-object for this.
      3. Setup an SSE listener in the test to listen for runs of one of the branches.
      4. Press the "run" widget on the branch being listened to.
      5. Use the async SSE events to move the test to a successful "done()".
      6. Verify that the "Queued" and "Started" Toast components appeared at the right time. This will be tricky hmmm.

          [JENKINS-36177] Acceptance test for Multi-Branch flow

          Tom FENNELLY created issue -
          Tom FENNELLY made changes -
          Epic Link New: JENKINS-36178 [ 172150 ]
          Tom FENNELLY made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          Michael Neale added a comment -

          These are all good things, but there have been some pretty major regressions that it would be nice to catch with a regular test suite rather than rely on this (eg around SSE)

          Michael Neale added a comment - These are all good things, but there have been some pretty major regressions that it would be nice to catch with a regular test suite rather than rely on this (eg around SSE)

          Tom FENNELLY added a comment -

          We'd catch those kinds of regressions if we had these kinda of tests but.

          Tom FENNELLY added a comment - We'd catch those kinds of regressions if we had these kinda of tests but.

          Michael Neale added a comment -

          some are really hard to test, that is for sure, and this seems pretty sensible (not saying shouldn't do it, but would be nice to find unit tests that can also cover the same where applicable).

          I suspect this is the best place to really test SSE, so we don't have a lot of choice.

          This is well worth doing

          Michael Neale added a comment - some are really hard to test, that is for sure, and this seems pretty sensible (not saying shouldn't do it, but would be nice to find unit tests that can also cover the same where applicable). I suspect this is the best place to really test SSE, so we don't have a lot of choice. This is well worth doing
          Michael Neale made changes -
          Priority Original: Major [ 3 ] New: Critical [ 2 ]

          Michael Neale added a comment -

          With this, I would like to have acceptance test run as part of the main build, I think it may be worth it.

          Michael Neale added a comment - With this, I would like to have acceptance test run as part of the main build, I think it may be worth it.
          Tom FENNELLY made changes -
          Status Original: In Progress [ 3 ] New: Open [ 1 ]
          Tom FENNELLY made changes -
          Description Original: Create a v1 acceptance test for testing the the multibranch flow:

          # Create the multibranch job. Will need to have branches.
          # Inspect structure of the Branches tab. Use a Nightwatch page-object for this.
          # Setup an SSE listener in the test to listen for runs of one of the branches.
          # Press the "run" widget on the branch being listened to.
          # Use the async SSE events to move the test to a successful "done()".
          New: Create a v1 acceptance test for testing the the multibranch flow:

          # Create the multibranch job. Will need to have branches.
          # Inspect structure of the Branches tab. Use a Nightwatch page-object for this.
          # Setup an SSE listener in the test to listen for runs of one of the branches.
          # Press the "run" widget on the branch being listened to.
          # Use the async SSE events to move the test to a successful "done()".
          # Verify that the "Queued" and "Started" Toast components appeared at the right time. This will be tricky hmmm.

            tscherler Thorsten Scherler
            tfennelly Tom FENNELLY
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: