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

meas_base still uses eups in tests

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture

      Description

      tests/centroid.py uses EUPS to determine the location of the data file used by the test. This needs to be fixed to use a location relative to the test file.

        Attachments

          Issue Links

          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-2636 Update code to use the function provided in DM-2635

          • Done

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-44 Remove build system dependencies from tests

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Link This issue relates to DM-2636 [ DM-2636 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to RFC-44 [ RFC-44 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Simple change to one of the tests. Passes the CI (job #1344).

            Show
            tjenness Tim Jenness added a comment - Simple change to one of the tests. Passes the CI (job #1344).
            tjenness Tim Jenness made changes -
            Reviewers Joshua Hoblitt [ jhoblitt ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            Hide
            Permalink
            jhoblitt Joshua Hoblitt added a comment -

            This seems reasonable since the test data is contained in the repo itself.

            Show
            jhoblitt Joshua Hoblitt added a comment - This seems reasonable since the test data is contained in the repo itself.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Merged.

            Show
            tjenness Tim Jenness added a comment - Merged.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status In Review [ 10004 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to SIM-1264 [ SIM-1264 ]
            tjenness Tim Jenness made changes -
            Team SQuaRE [ 10302 ] Architecture [ 10304 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Joshua Hoblitt
              Watchers:
              Joshua Hoblitt, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.