Skip to content

Commit 597aed3

Browse files
committed
Modify __repr__() methods to produce shorter and easier to read output
1 parent 4e965e5 commit 597aed3

File tree

4 files changed

+4
-7
lines changed

4 files changed

+4
-7
lines changed

fairgraph/kgproxy.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ def resolve(
151151
return obj
152152

153153
def __repr__(self):
154-
return "{self.__class__.__name__}(" "{self.classes!r}, {self.id!r})".format(self=self)
154+
return f"""{self.__class__.__name__}([{", ".join(cls.__name__ for cls in self.classes)}], id="{self.id.split('/')[-1]}")"""
155155

156156
def __eq__(self, other):
157157
return (

fairgraph/kgquery.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ def __init__(
7777
self.preferred_release_status = preferred_release_status
7878

7979
def __repr__(self):
80-
return "{self.__class__.__name__}(" "{self.classes!r}, {self.filter!r})".format(self=self)
80+
return f"""{self.__class__.__name__}([{", ".join(cls.__name__ for cls in self.classes)}], filter="{self.filter!r}")""".replace("https://kg.ebrains.eu/api/instances/", "")
8181

8282
def resolve(
8383
self,

fairgraph/queries.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ def __init__(
118118
raise ValueError("Sorting is only allowed on the root level of a query.")
119119

120120
def __repr__(self):
121-
return f"QueryProperty({self.path}, name={self.name})"
121+
return f"QueryProperty({self.path}, name={self.name}, reverse={self.reverse})"
122122

123123
def add_property(self, prop: QueryProperty):
124124
assert isinstance(prop, QueryProperty)

test/test_base.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -659,7 +659,4 @@ def test_properties(self):
659659
def test_repr(self):
660660
uri = "https://kg.ebrains.eu/api/instances/00000000-0000-0000-0000-000000001234"
661661
proxy = KGProxy(MockKGObject, uri)
662-
assert repr(proxy) == (
663-
"KGProxy([<class 'test.test_base.MockKGObject'>], "
664-
"'https://kg.ebrains.eu/api/instances/00000000-0000-0000-0000-000000001234')"
665-
)
662+
assert repr(proxy) == 'KGProxy([MockKGObject], id="00000000-0000-0000-0000-000000001234")'

0 commit comments

Comments
 (0)