Skip to content

Commit bf81cfe

Browse files
committed
Fix observations
1 parent 4842d10 commit bf81cfe

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

pydough/metadata/collections/collection_metadata.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ def __init__(
4848
PropertyMetadata,
4949
)
5050

51-
is_valid_name.verify(name, f"name {name!r}")
51+
is_valid_name.verify(name, f"collection name {name!r}")
5252
HasType(GraphMetadata).verify(graph, f"graph {name!r}")
5353

5454
self._graph: GraphMetadata = graph

tests/test_metadata_errors.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,12 +85,12 @@ def test_missing_property(get_sample_graph: graph_fetcher) -> None:
8585
),
8686
pytest.param(
8787
"BAD_COLLECTION_NAME_1",
88-
"name '0' must be a string that is a valid Python identifier",
88+
"collection name '0' must be a string that is a valid Python identifier",
8989
id="BAD_COLLECTION_NAME_1",
9090
),
9191
pytest.param(
9292
"BAD_COLLECTION_NAME_2",
93-
"name 'Invalid name' must be a string that is a valid Python identifier",
93+
"collection name 'Invalid name' must be a string that is a valid Python identifier",
9494
id="BAD_COLLECTION_NAME_2",
9595
),
9696
pytest.param(
@@ -100,12 +100,12 @@ def test_missing_property(get_sample_graph: graph_fetcher) -> None:
100100
),
101101
pytest.param(
102102
"BAD_PROPERTY_NAME_2",
103-
"name '0' must be a string that is a valid Python identifier",
103+
"collection name '0' must be a string that is a valid Python identifier",
104104
id="BAD_PROPERTY_NAME_2",
105105
),
106106
pytest.param(
107107
"BAD_TABLE_NAME",
108-
"name 'invalid table_name' must be a string that is a valid Python identifier",
108+
"collection name 'invalid table_name' must be a string that is a valid Python identifier",
109109
id="BAD_TABLE_NAME",
110110
),
111111
pytest.param(

0 commit comments

Comments
 (0)