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

Modernize dax_imgserv to support pytest

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Data Access
    • Labels:
      None
    • Team:
      Architecture

      Description

      Support pytest.

        Attachments

          Issue Links

          blocks

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. DM-7600 Pytest and Flake8 integration with CI and sconsUtils

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-11514 Modify sconsUtils to use pytest for test execution

          • Done
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            There is one test file in this package that needs to be upgraded to the modern standard. It looks like a pretty simply port, with the main fix being to use proper test skipping. The tests do pass with pytest, albeit because they don't run because of the auth requirement, but we need to get the proper skip status out so that Jenkins can report the test skip properly.

            Show
            tjenness Tim Jenness added a comment - There is one test file in this package that needs to be upgraded to the modern standard. It looks like a pretty simply port, with the main fix being to use proper test skipping. The tests do pass with pytest, albeit because they don't run because of the auth requirement, but we need to get the proper skip status out so that Jenkins can report the test skip properly.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Brian Van Klaveren can you please take a look. I had to fix the isFits method to support python 3.

            Show
            tjenness Tim Jenness added a comment - Brian Van Klaveren can you please take a look. I had to fix the isFits method to support python 3.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Thank you. Merged.

            Show
            tjenness Tim Jenness added a comment - Thank you. Merged.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Brian Van Klaveren
              Watchers:
              Brian Van Klaveren, Fritz Mueller, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: