Skip to content

Commit 52582db

Browse files
author
Nina Kahr
committed
PA-655: Change get to list in website cli. by Filip, Nina
1 parent 1b17785 commit 52582db

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

cli/website.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@ def get(
4646
)
4747
):
4848
"""If no domain name is specified, list all domains. Otherwise get details for specified domain"""
49-
websites = Website().get()
49+
# TODO: implement get if domain_name is provided
50+
51+
websites = Website().list()
5052
typer.echo(
5153
snakesay(
5254
f"You have {len(websites)} website(s). "

tests/test_cli_website.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ def test_create_with_domain_and_command_creates_it(mocker):
9393

9494
def test_get_with_no_domain_lists_websites(mocker, website_info):
9595
mock_website = mocker.patch("cli.website.Website")
96-
mock_website.return_value.get.return_value = [website_info]
96+
mock_website.return_value.list.return_value = [website_info]
9797

9898
result = runner.invoke(
9999
app,
@@ -102,7 +102,7 @@ def test_get_with_no_domain_lists_websites(mocker, website_info):
102102
],
103103
)
104104
assert result.exit_code == 0
105-
mock_website.return_value.get.assert_called_once()
105+
mock_website.return_value.list.assert_called_once()
106106
assert "You have 1 website(s). " in result.stdout
107107
assert "foo.bar.com" in result.stdout
108108

0 commit comments

Comments
 (0)