Skip to content

Conversation

@mathomp4
Copy link
Member

Closes #369

This just cleans up a bit for when a components.yaml doesn't have a fixture.

Before:

...
  File "/Users/mathomp4/.homebrew/brew/Cellar/mepo/2.4.0/libexec/lib/python3.12/site-packages/mepo/registry.py", line 49, in __validate
    assert num_fixtures == 1
           ^^^^^^^^^^^^^^^^^
AssertionError

Now:

...
  File "/Users/mathomp4/mepo/src/mepo/registry.py", line 51, in __validate
    raise ValueError("At least one fixture must be defined in the registry")
ValueError: At least one fixture must be defined in the registry

I'll let @pchakraborty see if this should be a ValueError or something different.

@mathomp4 mathomp4 self-assigned this Jun 16, 2025
@mathomp4 mathomp4 requested a review from a team as a code owner June 16, 2025 14:21
@mathomp4 mathomp4 added the bugfix This fixes a bug label Jun 16, 2025
@mathomp4 mathomp4 changed the title Fixes #369. Clarify no fixture assert Fixes #369. Clarify fixture assertions Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant