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

Modify sconsUtils to use pytest for test execution

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: sconsUtils
    • Labels:
      None
    • Story Points:
      8
    • Team:
      Architecture

      Description

      This ticket is for modifying sconsUtils such that it uses pytest for test execution rather than python.
      This is part of improving the test execution environment such that we can capture output from tests in a JUnit format compatible with Jenkins, and so that we can report on skipped tests as well as test execution times.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Russell Owen can you please take another look at this PR? I've added the support for xdist since you last reviewed it. I've been able to do an entire build of lsst_sims and lsst_apps so the code is looking pretty solid now.

            Show
            tjenness Tim Jenness added a comment - Russell Owen can you please take another look at this PR? I've added the support for xdist since you last reviewed it. I've been able to do an entire build of lsst_sims and lsst_apps so the code is looking pretty solid now.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            I tested these branches as working building up to afw (afwdata was exlcuded). I've made some minor comments/questions on the remaining open PRs.

            Show
            jhoblitt Joshua Hoblitt added a comment - I tested these branches as working building up to afw (afwdata was exlcuded). I've made some minor comments/questions on the remaining open PRs.
            Hide
            tjenness Tim Jenness added a comment -

            Thanks. Please mark as reviewed. I've tested with lsst_ci and lsst_sims so things are working well. There are some blocker tickets still to be resolved.

            Show
            tjenness Tim Jenness added a comment - Thanks. Please mark as reviewed. I've tested with lsst_ci and lsst_sims so things are working well. There are some blocker tickets still to be resolved.
            Hide
            rowen Russell Owen added a comment -

            It looks great to me. I added a few small suggestions to the pull request for sconsUtils.

            Show
            rowen Russell Owen added a comment - It looks great to me. I added a few small suggestions to the pull request for sconsUtils.
            Hide
            tjenness Tim Jenness added a comment -

            Jenkins passed for lsst_sims, qserv and lsst_distrib/_ci. Merged.

            Show
            tjenness Tim Jenness added a comment - Jenkins passed for lsst_sims, qserv and lsst_distrib/_ci. Merged.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Joshua Hoblitt, Russell Owen
                Watchers:
                John Swinbank, Joshua Hoblitt, Pim Schellart [X] (Inactive), Russell Owen, Simon Krughoff, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: