arcanis/ahriman
1
0
Fork 0
mirror of https://github.com/arcan1s/ahriman.git synced 2026-01-17 11:43:41 +00:00
Code Issues Projects Releases Wiki Activity

fix: correct exception type on repository id comparison

Browse Source
This commit is contained in:
Evgenii Alekseev
2026-01-14 13:58:11 +02:00
parent 49cf91ea52
commit 00c4f32294
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/ahriman/models/repository_id.py
View File

@ -94,7 +94,7 @@ class RepositoryId:
TypeError: if other is different from RepositoryId type TypeError: if other is different from RepositoryId type
""" """
if not isinstance(other, RepositoryId): if not isinstance(other, RepositoryId):
raise ValueError(f"'<' not supported between instances of '{type(self)}' and '{type(other)}'") raise TypeError(f"'<' not supported between instances of '{type(self)}' and '{type(other)}'")
return (self.name, self.architecture) < (other.name, other.architecture) return (self.name, self.architecture) < (other.name, other.architecture)

4
tests/ahriman/models/test_repository_id.py
View File

@ -64,9 +64,9 @@ def test_lt() -> None:
def test_lt_invalid() -> None: def test_lt_invalid() -> None:
""" """
must raise ValueError if other is not valid repository id must raise TypeError if other is not valid repository id
""" """
with pytest.raises(ValueError): with pytest.raises(TypeError):
assert RepositoryId("x86_64", "a") < 42 assert RepositoryId("x86_64", "a") < 42