test: Change tests for new changes

calvin-codecov · calvin-codecov · commit 31f1161a058e · 2026-03-18T16:58:24.000-07:00
diff --git a/Makefile b/Makefile
@@ -23,6 +23,9 @@ lint:
 	make lint.install
 	make lint.run
 
+test:
+	cd codecov-cli && uv run pytest --cov --junitxml=junit.xml
+
 command_dump:
 	cd prevent-cli && uv run command_dump.py
 
diff --git a/codecov-cli/tests/commands/test_invoke_upload.py b/codecov-cli/tests/commands/test_invoke_upload.py
@@ -40,7 +40,7 @@ def test_upload_raise_Z_option(mocker):
     result = runner.invoke(cli, ["do-upload", "--fail-on-error"], obj={})
     upload_sender.assert_called()
     upload_collector.assert_called()
-    assert ("error", "Upload failed: Unauthorized") in parse_outstreams_into_log_lines(
+    assert ("error", "Upload queued for processing failed: Unauthorized") in parse_outstreams_into_log_lines(
         result.output
     )
     assert str(result) == "<Result SystemExit(1)>"
diff --git a/codecov-cli/tests/services/commit/test_commit_service.py b/codecov-cli/tests/services/commit/test_commit_service.py
@@ -31,7 +31,7 @@ def test_commit_command_with_warnings(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Commit creating complete"),
+        ("info", "Commit creating complete"),
         ("info", "Commit creating process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
@@ -81,7 +81,7 @@ def test_commit_command_with_error(mocker):
     assert out_bytes == [
         (
             "info",
-            "Process Commit creating complete",
+            "Commit creating complete",
         ),
         ("error", "Commit creating failed: Permission denied"),
     ]
diff --git a/codecov-cli/tests/services/empty_upload/test_empty_upload.py b/codecov-cli/tests/services/empty_upload/test_empty_upload.py
@@ -34,7 +34,7 @@ def test_empty_upload_with_warnings(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Empty Upload complete"),
+        ("info", "Empty Upload complete"),
         ("info", "Empty Upload process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
@@ -71,7 +71,7 @@ def test_empty_upload_with_error(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Empty Upload complete"),
+        ("info", "Empty Upload complete"),
         ("error", "Empty Upload failed: Permission denied"),
     ]
     assert res == mock_send_commit_data.return_value
@@ -97,7 +97,7 @@ def test_empty_upload_200(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Empty Upload complete"),
+        ("info", "Empty Upload complete"),
         ("info", "All changed files are ignored. Triggering passing notifications."),
         ("info", "Non ignored files []"),
     ]
@@ -142,7 +142,7 @@ def test_empty_upload_force(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Empty Upload complete"),
+        ("info", "Empty Upload complete"),
         ("info", "Force option was enabled. Triggering passing notifications."),
         ("info", "Non ignored files []"),
     ]
@@ -170,7 +170,7 @@ def test_empty_upload_no_token(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Empty Upload complete"),
+        ("info", "Empty Upload complete"),
         ("info", "All changed files are ignored. Triggering passing notifications."),
         ("info", "Non ignored files []"),
     ]
diff --git a/codecov-cli/tests/services/report/test_report_results.py b/codecov-cli/tests/services/report/test_report_results.py
@@ -36,7 +36,7 @@ def test_report_results_command_with_warnings(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Report results creating complete"),
+        ("info", "Report results creating complete"),
         ("info", "Report results creating process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
@@ -81,7 +81,7 @@ def test_report_results_command_with_error(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Report results creating complete"),
+        ("info", "Report results creating complete"),
         ("error", "Report results creating failed: Permission denied"),
     ]
     assert res == mock_send_reports_result_request.return_value
diff --git a/codecov-cli/tests/services/report/test_report_service.py b/codecov-cli/tests/services/report/test_report_service.py
@@ -87,7 +87,7 @@ def test_create_report_command_with_warnings(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Report creating complete"),
+        ("info", "Report creating complete"),
         ("info", "Report creating process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
@@ -130,7 +130,7 @@ def test_create_report_command_with_error(mocker):
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Report creating complete"),
+        ("info", "Report creating complete"),
         ("error", "Report creating failed: Permission denied"),
     ]
     assert res == RequestResult(
diff --git a/codecov-cli/tests/services/upload/test_upload_service.py b/codecov-cli/tests/services/upload/test_upload_service.py
@@ -80,8 +80,8 @@ def test_do_upload_logic_happy_path_legacy_uploader(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload complete"),
-        ("info", "Upload process had 1 warning"),
+        ("info", "Upload queued for processing complete"),
+        ("info", "Upload queued for processing process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
 
@@ -195,8 +195,8 @@ def test_do_upload_logic_happy_path(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload complete"),
-        ("info", "Upload process had 1 warning"),
+        ("info", "Upload queued for processing complete"),
+        ("info", "Upload queued for processing process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
 
@@ -333,7 +333,7 @@ def test_do_upload_logic_dry_run(mocker):
     )
     assert out_bytes == [
         ("info", "dry-run option activated. NOT sending data to Codecov."),
-        ("info", "Process Upload complete"),
+        ("info", "Upload queued for processing complete"),
     ]
     assert res == RequestResult(
         error=None,
@@ -399,10 +399,10 @@ def test_do_upload_logic_verbose(mocker, use_verbose_option):
             "Selected uploader to use: <class 'codecov_cli.services.upload.legacy_upload_sender.LegacyUploadSender'>",
         ),
         ("info", "dry-run option activated. NOT sending data to Codecov."),
-        ("info", "Process Upload complete"),
+        ("info", "Upload queued for processing complete"),
         (
             "debug",
-            'Upload result --- {"result": "RequestResult(error=None, warnings=None, status_code=200, text=\'Data NOT sent to Codecov because of dry-run option\')"}',
+            'Upload queued for processing result --- {"result": "RequestResult(error=None, warnings=None, status_code=200, text=\'Data NOT sent to Codecov because of dry-run option\')"}',
         ),
     ]
     assert res == RequestResult(
@@ -671,8 +671,8 @@ def test_do_upload_logic_happy_path_test_results(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload complete"),
-        ("info", "Upload process had 1 warning"),
+        ("info", "Upload queued for processing complete"),
+        ("info", "Upload queued for processing process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
 
diff --git a/codecov-cli/tests/services/upload_completion/test_upload_completion.py b/codecov-cli/tests/services/upload_completion/test_upload_completion.py
@@ -27,7 +27,7 @@ def test_upload_completion_with_warnings(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload Completion complete"),
+        ("info", "Upload Completion complete"),
         ("info", "Upload Completion process had 1 warning"),
         ("warning", "Warning 1: somewarningmessage"),
     ]
@@ -56,7 +56,7 @@ def test_upload_completion_with_error(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload Completion complete"),
+        ("info", "Upload Completion complete"),
         ("error", "Upload Completion failed: Permission denied"),
     ]
     assert res == mock_send_commit_data.return_value
@@ -84,7 +84,7 @@ def test_upload_completion_200(mocker):
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload Completion complete"),
+        ("info", "Upload Completion complete"),
         (
             "info",
             "{'uploads_total': 2, 'uploads_success': 2, 'uploads_processing': 0, 'uploads_error': 0}",
@@ -113,7 +113,7 @@ def test_upload_completion_no_token(mocker):
         res = upload_completion_logic("commit_sha", "owner/repo", None, "service", None)
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
-        ("info", "Process Upload Completion complete"),
+        ("info", "Upload Completion complete"),
         (
             "info",
             "{'uploads_total': 2, 'uploads_success': 2, 'uploads_processing': 0, 'uploads_error': 0}",

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ def test_upload_raise_Z_option(mocker):`
`40`	`40`	`result = runner.invoke(cli, ["do-upload", "--fail-on-error"], obj={})`
`41`	`41`	`upload_sender.assert_called()`
`42`	`42`	`upload_collector.assert_called()`
`43`		`- assert ("error", "Upload failed: Unauthorized") in parse_outstreams_into_log_lines(`
	`43`	`+ assert ("error", "Upload queued for processing failed: Unauthorized") in parse_outstreams_into_log_lines(`
`44`	`44`	`result.output`
`45`	`45`	`)`
`46`	`46`	`assert str(result) == "<Result SystemExit(1)>"`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ def test_commit_command_with_warnings(mocker):`
`31`	`31`
`32`	`32`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`33`	`33`	`assert out_bytes == [`
`34`		`- ("info", "Process Commit creating complete"),`
	`34`	`+ ("info", "Commit creating complete"),`
`35`	`35`	`("info", "Commit creating process had 1 warning"),`
`36`	`36`	`("warning", "Warning 1: somewarningmessage"),`
`37`	`37`	`]`
`@@ -81,7 +81,7 @@ def test_commit_command_with_error(mocker):`
`81`	`81`	`assert out_bytes == [`
`82`	`82`	`(`
`83`	`83`	`"info",`
`84`		`- "Process Commit creating complete",`
	`84`	`+ "Commit creating complete",`
`85`	`85`	`),`
`86`	`86`	`("error", "Commit creating failed: Permission denied"),`
`87`	`87`	`]`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ def test_empty_upload_with_warnings(mocker):`
`34`	`34`	`)`
`35`	`35`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`36`	`36`	`assert out_bytes == [`
`37`		`- ("info", "Process Empty Upload complete"),`
	`37`	`+ ("info", "Empty Upload complete"),`
`38`	`38`	`("info", "Empty Upload process had 1 warning"),`
`39`	`39`	`("warning", "Warning 1: somewarningmessage"),`
`40`	`40`	`]`
`@@ -71,7 +71,7 @@ def test_empty_upload_with_error(mocker):`
`71`	`71`
`72`	`72`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`73`	`73`	`assert out_bytes == [`
`74`		`- ("info", "Process Empty Upload complete"),`
	`74`	`+ ("info", "Empty Upload complete"),`
`75`	`75`	`("error", "Empty Upload failed: Permission denied"),`
`76`	`76`	`]`
`77`	`77`	`assert res == mock_send_commit_data.return_value`
`@@ -97,7 +97,7 @@ def test_empty_upload_200(mocker):`
`97`	`97`	`)`
`98`	`98`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`99`	`99`	`assert out_bytes == [`
`100`		`- ("info", "Process Empty Upload complete"),`
	`100`	`+ ("info", "Empty Upload complete"),`
`101`	`101`	`("info", "All changed files are ignored. Triggering passing notifications."),`
`102`	`102`	`("info", "Non ignored files []"),`
`103`	`103`	`]`
`@@ -142,7 +142,7 @@ def test_empty_upload_force(mocker):`
`142`	`142`	`)`
`143`	`143`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`144`	`144`	`assert out_bytes == [`
`145`		`- ("info", "Process Empty Upload complete"),`
	`145`	`+ ("info", "Empty Upload complete"),`
`146`	`146`	`("info", "Force option was enabled. Triggering passing notifications."),`
`147`	`147`	`("info", "Non ignored files []"),`
`148`	`148`	`]`
`@@ -170,7 +170,7 @@ def test_empty_upload_no_token(mocker):`
`170`	`170`
`171`	`171`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`172`	`172`	`assert out_bytes == [`
`173`		`- ("info", "Process Empty Upload complete"),`
	`173`	`+ ("info", "Empty Upload complete"),`
`174`	`174`	`("info", "All changed files are ignored. Triggering passing notifications."),`
`175`	`175`	`("info", "Non ignored files []"),`
`176`	`176`	`]`
Original file line number	Diff line number	Diff line change
`@@ -80,8 +80,8 @@ def test_do_upload_logic_happy_path_legacy_uploader(mocker):`
`80`	`80`	`)`
`81`	`81`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`82`	`82`	`assert out_bytes == [`
`83`		`- ("info", "Process Upload complete"),`
`84`		`- ("info", "Upload process had 1 warning"),`
	`83`	`+ ("info", "Upload queued for processing complete"),`
	`84`	`+ ("info", "Upload queued for processing process had 1 warning"),`
`85`	`85`	`("warning", "Warning 1: somewarningmessage"),`
`86`	`86`	`]`
`87`	`87`
`@@ -195,8 +195,8 @@ def test_do_upload_logic_happy_path(mocker):`
`195`	`195`	`)`
`196`	`196`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`197`	`197`	`assert out_bytes == [`
`198`		`- ("info", "Process Upload complete"),`
`199`		`- ("info", "Upload process had 1 warning"),`
	`198`	`+ ("info", "Upload queued for processing complete"),`
	`199`	`+ ("info", "Upload queued for processing process had 1 warning"),`
`200`	`200`	`("warning", "Warning 1: somewarningmessage"),`
`201`	`201`	`]`
`202`	`202`
`@@ -333,7 +333,7 @@ def test_do_upload_logic_dry_run(mocker):`
`333`	`333`	`)`
`334`	`334`	`assert out_bytes == [`
`335`	`335`	`("info", "dry-run option activated. NOT sending data to Codecov."),`
`336`		`- ("info", "Process Upload complete"),`
	`336`	`+ ("info", "Upload queued for processing complete"),`
`337`	`337`	`]`
`338`	`338`	`assert res == RequestResult(`
`339`	`339`	`error=None,`
`@@ -399,10 +399,10 @@ def test_do_upload_logic_verbose(mocker, use_verbose_option):`
`399`	`399`	`"Selected uploader to use: <class 'codecov_cli.services.upload.legacy_upload_sender.LegacyUploadSender'>",`
`400`	`400`	`),`
`401`	`401`	`("info", "dry-run option activated. NOT sending data to Codecov."),`
`402`		`- ("info", "Process Upload complete"),`
	`402`	`+ ("info", "Upload queued for processing complete"),`
`403`	`403`	`(`
`404`	`404`	`"debug",`
`405`		`- 'Upload result --- {"result": "RequestResult(error=None, warnings=None, status_code=200, text=\'Data NOT sent to Codecov because of dry-run option\')"}',`
	`405`	`+ 'Upload queued for processing result --- {"result": "RequestResult(error=None, warnings=None, status_code=200, text=\'Data NOT sent to Codecov because of dry-run option\')"}',`
`406`	`406`	`),`
`407`	`407`	`]`
`408`	`408`	`assert res == RequestResult(`
`@@ -671,8 +671,8 @@ def test_do_upload_logic_happy_path_test_results(mocker):`
`671`	`671`	`)`
`672`	`672`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`673`	`673`	`assert out_bytes == [`
`674`		`- ("info", "Process Upload complete"),`
`675`		`- ("info", "Upload process had 1 warning"),`
	`674`	`+ ("info", "Upload queued for processing complete"),`
	`675`	`+ ("info", "Upload queued for processing process had 1 warning"),`
`676`	`676`	`("warning", "Warning 1: somewarningmessage"),`
`677`	`677`	`]`
`678`	`678`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ def test_upload_completion_with_warnings(mocker):`
`27`	`27`	`)`
`28`	`28`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`29`	`29`	`assert out_bytes == [`
`30`		`- ("info", "Process Upload Completion complete"),`
	`30`	`+ ("info", "Upload Completion complete"),`
`31`	`31`	`("info", "Upload Completion process had 1 warning"),`
`32`	`32`	`("warning", "Warning 1: somewarningmessage"),`
`33`	`33`	`]`
`@@ -56,7 +56,7 @@ def test_upload_completion_with_error(mocker):`
`56`	`56`	`)`
`57`	`57`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`58`	`58`	`assert out_bytes == [`
`59`		`- ("info", "Process Upload Completion complete"),`
	`59`	`+ ("info", "Upload Completion complete"),`
`60`	`60`	`("error", "Upload Completion failed: Permission denied"),`
`61`	`61`	`]`
`62`	`62`	`assert res == mock_send_commit_data.return_value`
`@@ -84,7 +84,7 @@ def test_upload_completion_200(mocker):`
`84`	`84`	`)`
`85`	`85`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`86`	`86`	`assert out_bytes == [`
`87`		`- ("info", "Process Upload Completion complete"),`
	`87`	`+ ("info", "Upload Completion complete"),`
`88`	`88`	`(`
`89`	`89`	`"info",`
`90`	`90`	`"{'uploads_total': 2, 'uploads_success': 2, 'uploads_processing': 0, 'uploads_error': 0}",`
`@@ -113,7 +113,7 @@ def test_upload_completion_no_token(mocker):`
`113`	`113`	`res = upload_completion_logic("commit_sha", "owner/repo", None, "service", None)`
`114`	`114`	`out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())`
`115`	`115`	`assert out_bytes == [`
`116`		`- ("info", "Process Upload Completion complete"),`
	`116`	`+ ("info", "Upload Completion complete"),`
`117`	`117`	`(`
`118`	`118`	`"info",`
`119`	`119`	`"{'uploads_total': 2, 'uploads_success': 2, 'uploads_processing': 0, 'uploads_error': 0}",`