Skip to content

DM-39563: Test now raises FileNotFoundError#48

Merged
timj merged 1 commit intomainfrom
tickets/DM-39563
Jun 8, 2023
Merged

DM-39563: Test now raises FileNotFoundError#48
timj merged 1 commit intomainfrom
tickets/DM-39563

Conversation

@timj
Copy link
Copy Markdown
Contributor

@timj timj commented Jun 8, 2023

Previously the repository index look up system was not being caught properly and so the RuntimeError printed through from the Config code. Now we catch this problem internally and allow the requested repo alias to be used. This changes the exception type being caught in the test.

Requires lsst/daf_butler#849

Previously the repository index look up system was not being
caught properly and so the RuntimeError printed through from
the Config code. Now we catch this problem internally and
allow the requested repo alias to be used. This changes
the exception type being caught in the test.
@timj timj requested a review from mfisherlevine June 8, 2023 00:10
Copy link
Copy Markdown

@andy-slac andy-slac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great.

@timj timj merged commit d3a4db9 into main Jun 8, 2023
@timj timj deleted the tickets/DM-39563 branch June 8, 2023 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants