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

Add tests for ip_isr/isrTask

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • ip_isr
    • None
    • 12
    • DRP F18-4, DRP F18-5, DRP F18-6, DRP S19-1, DRP S19-2, DRP S19-3, DRP S19-4, DRP S19-5, DRP S19-6a
    • Data Release Production

    Description

      The current isrTask is poorly covered by the current tests, and there are a number of other ISR functions with no tests.  Fix this.

      Attachments

        Issue Links

          Activity

            This ticket will resolve these.

            czw Christopher Waters added a comment - This ticket will resolve these.

            I'm beginning work on this issue, using the DM-6640 `ip_isr` as a basis, under the assumption that that will be merged first.

            czw Christopher Waters added a comment - I'm beginning work on this issue, using the DM-6640 `ip_isr` as a basis, under the assumption that that will be merged first.

            These tests have finally converged with the addition of the `ip_isr/isrMock.py` code to supply "realistic" simulated data to check.  Running these tests uncovered some bugs in the ISR code, which has now been corrected.

            czw Christopher Waters added a comment - These tests have finally converged with the addition of the `ip_isr/isrMock.py` code to supply "realistic" simulated data to check.  Running these tests uncovered some bugs in the ISR code, which has now been corrected.

            Please file a ticket describing the missing tests (places where the current test is just "did the Task not raise or return None", etc.), so that we do not forget that some of those tests are not functionality tests. If its possible to list such tests in that ticket, that would be useful.

            Parejkoj John Parejko added a comment - Please file a ticket describing the missing tests (places where the current test is just "did the Task not raise or return None", etc.), so that we do not forget that some of those tests are not functionality tests. If its possible to list such tests in that ticket, that would be useful.

            People

              czw Christopher Waters
              czw Christopher Waters
              John Parejko
              Andrés Alejandro Plazas Malagón, Christopher Waters, John Parejko, Russell Owen, Simon Krughoff (Inactive), Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.