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

Add unit test for IsrTask

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Invalid
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_isr

      Description

      ip_isr has many unit tests, but none tests IsrTask. I suggest adding such a task, likely using the data in obs_test

        Attachments

          Issue Links

          duplicates

          Improvement - An improvement or enhancement to an existing feature or task. DM-15683 Add tests for ip_isr/isrTask

          • Done
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-6147 Set SUSPECT mask in ISR task and make saturation a double

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Epic Link DM-5691 [ 23683 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-6147 [ DM-6147 ]
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            A useful refactor to simplify testing isrTask.run(), would be to have the stuff inside the for amp in ccd: live in a do_amp() method. do_amp() can then be tested separately.

            Show
            Parejkoj John Parejko added a comment - A useful refactor to simplify testing isrTask.run() , would be to have the stuff inside the for amp in ccd: live in a do_amp() method. do_amp() can then be tested separately.
            krughoff Simon Krughoff made changes -
            Epic Link DM-5691 [ 23683 ] DM-7362 [ 26448 ]
            krughoff Simon Krughoff made changes -
            Epic Link DM-7362 [ 26448 ] DM-8472 [ 28104 ]
            krughoff Simon Krughoff made changes -
            Epic Link DM-8472 [ 28104 ] DM-9680 [ 30785 ]
            swinbank John Swinbank made changes -
            Epic Link DM-9680 [ 30785 ] DM-10068 [ 31628 ]
            swinbank John Swinbank made changes -
            Epic Link DM-10068 [ 31628 ] DM-11798 [ 34281 ]
            swinbank John Swinbank made changes -
            Epic Link DM-11798 [ 34281 ] DM-12728 [ 36327 ]
            swinbank John Swinbank made changes -
            Epic Link DM-12728 [ 36327 ] DM-14447 [ 80385 ]
            yusra Yusra AlSayyad made changes -
            Watchers John Parejko, Russell Owen [ John Parejko, Russell Owen ] Christopher Waters, John Parejko, Russell Owen [ Christopher Waters, John Parejko, Russell Owen ]
            yusra Yusra AlSayyad made changes -
            Risk Score 0
            czw Christopher Waters made changes -
            Link This issue duplicates DM-15683 [ DM-15683 ]
            Hide
            Permalink
            czw Christopher Waters added a comment -

            Closing as a duplicate of DM-15683

            Show
            czw Christopher Waters added a comment - Closing as a duplicate of DM-15683
            czw Christopher Waters made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Done [ 10002 ]
            Hide
            Permalink
            czw Christopher Waters added a comment -

            Marking invalid, as DM-15683 is subsuming this work (and that will eventually be marked "Done").

            Show
            czw Christopher Waters added a comment - Marking invalid, as DM-15683 is subsuming this work (and that will eventually be marked "Done").
            czw Christopher Waters made changes -
            Status Done [ 10002 ] Invalid [ 11005 ]

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              rowen Russell Owen
              Watchers:
              Christopher Waters, John Parejko, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.