rasing an exception prints out all stacktrace, which is not practical nor helpful. raising click.Exception only prints out the exception msg

Author: dana-yaish

codecov_cli/helpers/request.py

@@ -2,6 +2,7 @@
 import uuid
 from time import sleep
 
+import click
 import requests
 
 from codecov_cli.types import RequestError, RequestResult
@@ -46,9 +47,11 @@ def send_post_request(
 
 def get_token_header_or_fail(token: uuid.UUID) -> dict:
     if token is None:
-        raise Exception("Codecov token not found.")
+        raise click.ClickException(
+            "Codecov token not found. Please provide Codecov token with -t flag."
+        )
     if not isinstance(token, uuid.UUID):
-        raise Exception(f"Token must be UUID. Received {type(token)}")
+        raise click.ClickException(f"Token must be UUID. Received {type(token)}")
     return {"Authorization": f"token {token.hex}"}
 
 

tests/helpers/test_request.py

@@ -57,7 +57,10 @@ def test_get_token_header_or_fail():
     with pytest.raises(Exception) as e:
         get_token_header_or_fail(token)
 
-    assert str(e.value) == "Codecov token not found."
+    assert (
+        str(e.value)
+        == "Codecov token not found. Please provide Codecov token with -t flag."
+    )
 
     # Test with an invalid token type
     token = "invalid_token"