Skip to content

Commit 70ccf5d

Browse files
committed
fix(test): actually specify codecov token when uploading coverage
So far we were relying on the environment variable being present, but it seems due to a recent change the token actually needs to be passed via a flag to the uploader. Signed-off-by: Patrick Roy <[email protected]>
1 parent 4b6c9b3 commit 70ccf5d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/integration_tests/build/test_coverage.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,14 +95,14 @@ def test_coverage(monkeypatch):
9595
if not branch:
9696
branch = utils.check_output("git rev-parse --abbrev-ref HEAD").stdout
9797

98-
codecov_cmd = f"codecov -f {lcov_file} -F {global_props.host_linux_version}-{global_props.instance}"
98+
codecov_cmd = f"codecov -f {lcov_file} -F {global_props.host_linux_version}-{global_props.instance} -t {os.environ['CODECOV_TOKEN']}"
9999

100100
if pr_number and pr_number != "false":
101101
codecov_cmd += f" -P {pr_number}"
102102
else:
103103
codecov_cmd += f" -B {branch}"
104104

105-
utils.check_output(codecov_cmd)
105+
print(utils.check_output(codecov_cmd))
106106
else:
107107
warnings.warn(
108108
"Not uploading coverage report due to missing CODECOV_TOKEN environment variable"

0 commit comments

Comments
 (0)