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

Add a unittest to address the issue of DM-16068

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      DRP F18-5
    • Team:
      Data Release Production

      Description

      Clearly we have no current test that would catch this, please add one (which, on the current w_2018_40 weekly would fail).  See DM-16068 for details.

        Attachments

          Issue Links

            Activity

            Hide
            lauren Lauren MacArthur added a comment -

            Ok, Jim, this should fail until you get DM-16068 fixed.  I had to add a small (205 KB) coadd meas version 2 catalog to tests/data/ for the purpose of this test.  We can update the comments about failing before merging once you get it to pass (although note that I couldn't actually test my asserts, so typos are a possibility)!

            Show
            lauren Lauren MacArthur added a comment - Ok, Jim, this should fail until you get DM-16068 fixed.  I had to add a small (205 KB) coadd meas version 2 catalog to tests/data/  for the purpose of this test.  We can update the comments about failing before merging once you get it to pass (although note that I couldn't actually test my asserts, so typos are a possibility)!
            Hide
            jbosch Jim Bosch added a comment -

            Looks good, thanks!  I've put your commit on a tickets/DM-16068 branch, so I think you can just go ahead and close this ticket.

            Show
            jbosch Jim Bosch added a comment - Looks good, thanks!  I've put your commit on a tickets/ DM-16068 branch, so I think you can just go ahead and close this ticket.

              People

              • Assignee:
                lauren Lauren MacArthur
                Reporter:
                lauren Lauren MacArthur
                Reviewers:
                Jim Bosch
                Watchers:
                Jim Bosch, John Parejko, Lauren MacArthur, Yusra AlSayyad
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel