Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-20814

Fix broken tests in meas_extensions_scarlet

    Details

      Description

      meas_extensions_scarlet tests are missing the setup_module function. For some reason this did not cause the tests to fail when built locally or with Jenkins, but did prevent the weekly build.

        Attachments

          Activity

          Hide
          fred3m Fred Moolekamp added a comment -

          Hi Tim,
          Would you mind reviewing this ticket so that I can merge it and Fabio can rebuild the the weekly?

          Thanks,
          -Fred

          Show
          fred3m Fred Moolekamp added a comment - Hi Tim, Would you mind reviewing this ticket so that I can merge it and Fabio can rebuild the the weekly? Thanks, -Fred
          Show
          fred3m Fred Moolekamp added a comment - successful Jenkins build: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30197/pipeline
          Hide
          ktl Kian-Tat Lim added a comment -

          Looks complete and correct. Thanks for adding these in. (I believe the lack of these doesn't always cause failures locally or in Jenkins because the underlying problem is a race condition when running multi-process pytest.)

          Show
          ktl Kian-Tat Lim added a comment - Looks complete and correct. Thanks for adding these in. (I believe the lack of these doesn't always cause failures locally or in Jenkins because the underlying problem is a race condition when running multi-process pytest.)
          Hide
          fred3m Fred Moolekamp added a comment -

          Thanks for the review Kian-Tat Lim. Changes merged.

          Show
          fred3m Fred Moolekamp added a comment - Thanks for the review Kian-Tat Lim . Changes merged.

            People

            • Assignee:
              fred3m Fred Moolekamp
              Reporter:
              fred3m Fred Moolekamp
              Reviewers:
              Tim Jenness
              Watchers:
              Fred Moolekamp, Kian-Tat Lim, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel