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

            Activity

            Hide
            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
            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
            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: