Skip to content

vdk-control-cli: use assert_click_status #1817

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
antoniivanov merged 3 commits into from
Apr 3, 2023
Merged

vdk-control-cli: use assert_click_status #1817

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3046ea3
vdk-control-cli: use assert_click_status
antoniivanov Mar 29, 2023
37d9588
Merge branch 'main' into person/aivanov/vdk-control-cli-asserts
antoniivanov Apr 3, 2023
7ec9f6d
Merge branch 'main' into person/aivanov/vdk-control-cli-asserts
antoniivanov Apr 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 10 additions & 29 deletions projects/vdk-control-cli/tests/vdk/internal/control/command_groups/job/test_list.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
from pytest_httpserver.pytest_plugin import PluginHTTPServer
from vdk.internal import test_utils
from vdk.internal.control.command_groups.job.list import list_command
from vdk.internal.test_utils import assert_click_status

test_utils.disable_vdk_authentication()

Expand Down Expand Up @@ -63,9 +64,7 @@ def test_list_default_all(httpserver: PluginHTTPServer, tmpdir: LocalPath):

runner = CliRunner()
result = runner.invoke(list_command, ["-t", "test-team", "-u", rest_api_url])
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)
assert (
"test-job" in result.output and "test-job-2" in result.output
), f"expected data not found in output: {result.output} "
Expand All @@ -92,19 +91,15 @@ def test_list_all(httpserver: PluginHTTPServer, tmpdir: LocalPath):

runner = CliRunner()
result = runner.invoke(list_command, ["-t", "test-team", "-a", "-u", rest_api_url])
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)
assert (
"test-job" in result.output
), f"expected data not found in output: {result.output} "

result = runner.invoke(
list_command, ["-t", "test-team", "--all", "-u", rest_api_url]
)
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)
assert (
"test-job" in result.output
), f"expected data not found in output: {result.output} "
Expand All @@ -114,9 +109,7 @@ def test_list_all(httpserver: PluginHTTPServer, tmpdir: LocalPath):
).respond_with_json(response)

result = runner.invoke(list_command, ["-u", rest_api_url])
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)
assert (
"test-job" in result.output
), f"expected data not found in output: {result.output} "
Expand All @@ -125,20 +118,14 @@ def test_list_all(httpserver: PluginHTTPServer, tmpdir: LocalPath):
def test_list_without_url(httpserver: PluginHTTPServer, tmpdir: LocalPath):
runner = CliRunner()
result = runner.invoke(list_command, ["-t", "test-team"])

assert (
result.exit_code == 2
), f"result exit code is not 2, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 2)
assert "what" in result.output and "why" in result.output


def test_list_with_empty_url(httpserver: PluginHTTPServer, tmpdir: LocalPath):
runner = CliRunner()
result = runner.invoke(list_command, ["-t", "test-team", "-u", ""])

assert (
result.exit_code == 2
), f"result exit code is not 2, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 2)
assert "what" in result.output and "why" in result.output


Expand All @@ -155,9 +142,7 @@ def test_list_with_json_output(httpserver: PluginHTTPServer, tmpdir: LocalPath):
result = runner.invoke(
list_command, ["-t", "test-team", "-u", rest_api_url, "-o", "json"]
)
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)

try:
json_result = json.loads(result.output)
Expand All @@ -175,9 +160,7 @@ def test_list_with_invalid_output(httpserver: PluginHTTPServer, tmpdir: LocalPat
result = runner.invoke(
list_command, ["-t", "test-team", "-u", rest_api_url, "-o", "invalid"]
)
assert (
result.exit_code == 2
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 2)
assert (
"Error: Invalid value for '--output'" in result.output
), f"expected data not found in output: {result.output}"
Expand Down Expand Up @@ -223,9 +206,7 @@ def get_comparing_values(self, request_query_string: bytes) -> tuple:

runner = CliRunner()
result = runner.invoke(list_command, ["-t", "test-team", "-a", "-u", rest_api_url])
assert (
result.exit_code == 0
), f"result exit code is not 0, result output: {result.output}, exc: {result.exc_info}"
assert_click_status(result, 0)
assert (
"test-job-1" in result.output and "test-job-120" in result.output
), f"expected data not found in output: {result.output}"