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

Modify obs_lsstSim to support py.test

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_lsstSim
    • Labels:
      None

      Description

      Migrate the obs_lsstSim tests such that they run with the py.test test runner.

        Attachments

          Issue Links

            Activity

            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            I've switched from setUpClass to setUp for consistency with the other packages.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - I've switched from setUpClass to setUp for consistency with the other packages.
            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Passed Jenkins. https://ci.lsst.codes/job/stack-os-matrix/14875/label=centos-6,python=py2/console
            Hide
            tjenness Tim Jenness added a comment -

            Looks good. One quibble with the exception handling and a print that probably should not be there.

            Show
            tjenness Tim Jenness added a comment - Looks good. One quibble with the exception handling and a print that probably should not be there.
            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            Thanks for the touch up points. print removed. Now only catches RuntimeError.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Thanks for the touch up points. print removed. Now only catches RuntimeError.
            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            Merged to master.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Merged to master.

              People

              • Assignee:
                wmwood-vasey Michael Wood-Vasey
                Reporter:
                wmwood-vasey Michael Wood-Vasey
                Reviewers:
                Tim Jenness
                Watchers:
                Michael Wood-Vasey, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel