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

Modify ip_isr tests to support pytest

    Details

      Description

      This ticket is for the work of migrating the ip_isr tests such that they run with the py.test test runner.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            I've updated all the tests and it's running in jenkins now. Please take a look.

            Show
            Parejkoj John Parejko added a comment - I've updated all the tests and it's running in jenkins now. Please take a look.
            Hide
            tjenness Tim Jenness added a comment -

            Did you run the tests in a single pytest process running them in both orders?

            Show
            tjenness Tim Jenness added a comment - Did you run the tests in a single pytest process running them in both orders?
            Hide
            Parejkoj John Parejko added a comment -

            I believe so.

            Show
            Parejkoj John Parejko added a comment - I believe so.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks good. Some minor questions / comments on PR.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks good. Some minor questions / comments on PR.
            Hide
            tjenness Tim Jenness added a comment -

            Yes, reversing works fine. "63 passed". Please add "tests/.cache" to the gitignore file.

            Show
            tjenness Tim Jenness added a comment - Yes, reversing works fine. "63 passed". Please add "tests/.cache" to the gitignore file.
            Hide
            Parejkoj John Parejko added a comment -

            Responded to review comments and merged. ip_isr should be ready for py.test.

            Show
            Parejkoj John Parejko added a comment - Responded to review comments and merged. ip_isr should be ready for py.test.

              People

              • Assignee:
                krughoff Simon Krughoff
                Reporter:
                Parejkoj John Parejko
                Reviewers:
                Pim Schellart [X] (Inactive)
                Watchers:
                John Parejko, Pim Schellart [X] (Inactive), Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel