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

obs_base tests should not use daf_butler test configs

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_base
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      John Swinbank has noted that obs_base tests depend on a private butler config test file in DAF_BUTLER_DIR/tests/config. Those configs are not meant to be used outside of daf_butler since they contain unexpected formatters and storage classes designed specifically for internal daf_butler tests. obs_base should not be relying on those configurations. It should be sufficient for obs_base to use a default configuration made by Butler.makeRepo.

      I'm not entirely sure whether I should fix this or whether this should be handled by science pipelines.

        Attachments

          Issue Links

            Activity

            tjenness Tim Jenness created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Link This issue relates to DM-20994 [ DM-20994 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-21016 [ DM-21016 ]
            swinbank John Swinbank made changes -
            Assignee John Parejko [ parejkoj ]
            swinbank John Swinbank made changes -
            Team Alert Production [ 10300 ]
            tjenness Tim Jenness made changes -
            Assignee John Parejko [ parejkoj ] Tim Jenness [ tjenness ]
            tjenness Tim Jenness made changes -
            Team Alert Production [ 10300 ] Architecture [ 10304 ]
            tjenness Tim Jenness made changes -
            Story Points 1
            Urgent? off
            tjenness Tim Jenness made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            tjenness Tim Jenness made changes -
            Reviewers John Parejko [ parejkoj ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Parejkoj John Parejko made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, John Swinbank, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  CI Builds

                  No builds found.