Skip to content

Commit

Permalink
Update docstrings with the name/headers rename
Browse files Browse the repository at this point in the history
  • Loading branch information
maxhutch committed Jan 9, 2020
1 parent 1df3731 commit bf8ac4a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/citrine/resources/ara_definition.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def __init__(self, *, name: str, description: str, datasets: List[UUID],
headers = [x.headers for x in variables]
dup_headers = self._get_dups(headers)
if len(dup_headers) > 0:
raise ValueError("Multiple variables defined these output_names,"
raise ValueError("Multiple variables defined these headers,"
" which much be unique: {}".format(dup_headers))

missing_variables = [x.data_source for x in columns if x.data_source not in names]
Expand Down
3 changes: 2 additions & 1 deletion tests/resources/test_ara_definition.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def test_init_ara_definition():


def test_dup_names():
"""Make sure that variable short_name and output_name are unique across an ara definition"""
"""Make sure that variable name and headers are unique across an ara definition"""
with pytest.raises(ValueError) as excinfo:
AraDefinition(
name="foo", description="bar", datasets=[], rows=[], columns=[],
Expand All @@ -84,6 +84,7 @@ def test_dup_names():


def test_missing_variable():
"""Make sure that every data_source matches a name of a variable"""
with pytest.raises(ValueError) as excinfo:
AraDefinition(
name="foo", description="bar", datasets=[], rows=[], variables=[],
Expand Down

0 comments on commit bf8ac4a

Please sign in to comment.