Skip to content

Commit

Permalink
fix ruff TD rules
Browse files Browse the repository at this point in the history
  • Loading branch information
nstarman committed Jun 6, 2023
1 parent d0e99dc commit 684adee
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
3 changes: 3 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,9 @@ ignore = [
"N802",
"PD",
"RET505", "RET506", "RET507",
# flake8-todo (TD)
"TD002", # Missing author in TODO
"TD003", # Missing issue link on the line following this TODO
]

[tool.ruff.per-file-ignores]
Expand Down
4 changes: 2 additions & 2 deletions tests/classy/test_components.py
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ def test_wrapper_is_compliant(self, wrapper):
if hasattr(super(), "test_wrapper_is_compliant"):
super().test_wrapper_is_compliant(wrapper)

# FIXME: this should be an instance
# TODO: this should be an instance
with pytest.raises(NotImplementedError):
assert isinstance(wrapper, NeutrinoComponent)

Expand Down Expand Up @@ -172,7 +172,7 @@ def test_wrapper_is_compliant(self, wrapper):
if hasattr(super(), "test_wrapper_is_compliant"):
super().test_wrapper_is_compliant(wrapper)

# FIXME: should be a DarkMatterComponent instance
# TODO: should be a DarkMatterComponent instance
assert isinstance(wrapper, DarkMatterComponent)

def test_Omega_dm0(self, wrapper, cosmo):
Expand Down
4 changes: 2 additions & 2 deletions tests/classy/test_standard.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,10 @@ def test_wrapper_is_compliant(self, wrapper):
"""Test that StandardCosmologyWrapper is a StandardCosmologyWrapper."""
super().test_wrapper_is_compliant(wrapper)

# FIXME: it should be an instance
# TODO: it should be an instance
with pytest.raises(NotImplementedError):
assert isinstance(wrapper, StandardCosmology)

# FIXME: it should be an instance
# TODO: it should be an instance
with pytest.raises(NotImplementedError):
assert isinstance(wrapper, StandardCosmologyWrapperAPI)

0 comments on commit 684adee

Please sign in to comment.