Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

Commit

Permalink
tests/test_shortcuts: Fix check for success on submit_results return
Browse files Browse the repository at this point in the history
Since submit_results now returns two values (success and testrun ID),
the test_basic for submit_results needs to be updated to reflect this,
otherwise the assertion will always return True.

Signed-off-by: Katie Worton <[email protected]>
  • Loading branch information
katieworton committed Nov 13, 2023
1 parent 80d16a8 commit 89e284b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/test_shortcuts.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def test_basic(self):
metadata = {"job_id": "12345", "a-metadata-field": "value"}
tests = {"testa": "pass", "testb": {"result": "pass", "log": "the log"}}
metrics = {"metrica": 42}
success = submit_results(
success, testrun_id = submit_results(
group_project_slug="my_group/my_project",
build_version="my_build",
env_slug="my_env",
Expand Down

0 comments on commit 89e284b

Please sign in to comment.