XMLWordPrintable

## Details

• Type: Improvement
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
None
• Story Points:
12
• Sprint:
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
• Team:
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.

## Activity

Hide
Christopher Waters added a comment -

This ticket will resolve these.

Show
Christopher Waters added a comment - This ticket will resolve these.
Hide
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.

Show
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.
Hide
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.

Show
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.
Hide
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.

Show
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

• Assignee:
Christopher Waters
Reporter:
Christopher Waters
Reviewers:
John Parejko
Watchers:
Andrés Alejandro Plazas Malagón, Christopher Waters, John Parejko, Russell Owen, Simon Krughoff, Yusra AlSayyad