feat: add force flag to empty upload (#362)

* feat: add force flag to empty upload

* fix: change format of option for empty upload force

Signed-off-by: joseph-sentry <[email protected]>

Author: joseph-sentry

codecov_cli/commands/empty_upload.py

@@ -12,11 +12,13 @@
 
 
 @click.command()
+@click.option("--force", is_flag=True, default=False)
 @global_options
 @click.pass_context
 def empty_upload(
     ctx,
     commit_sha: str,
+    force: bool,
     slug: typing.Optional[str],
     token: typing.Optional[str],
     git_service: typing.Optional[str],
@@ -37,5 +39,5 @@ def empty_upload(
         ),
     )
     return empty_upload_logic(
-        commit_sha, slug, token, git_service, enterprise_url, fail_on_error
+        commit_sha, slug, token, git_service, enterprise_url, fail_on_error, force
     )

codecov_cli/services/empty_upload/__init__.py

@@ -13,13 +13,15 @@
 
 
 def empty_upload_logic(
-    commit_sha, slug, token, git_service, enterprise_url, fail_on_error
+    commit_sha, slug, token, git_service, enterprise_url, fail_on_error, should_force
 ):
     encoded_slug = encode_slug(slug)
     headers = get_token_header_or_fail(token)
     upload_url = enterprise_url or CODECOV_API_URL
     url = f"{upload_url}/upload/{git_service}/{encoded_slug}/commits/{commit_sha}/empty-upload"
-    sending_result = send_post_request(url=url, headers=headers)
+    sending_result = send_post_request(
+        url=url, headers=headers, data={"should_force": should_force}
+    )
     log_warnings_and_errors_if_any(sending_result, "Empty Upload", fail_on_error)
     if sending_result.status_code == 200:
         response_json = json.loads(sending_result.text)

tests/services/empty_upload/test_empty_upload.py

@@ -21,7 +21,7 @@ def test_empty_upload_with_warnings(mocker):
     runner = CliRunner()
     with runner.isolation() as outstreams:
         res = empty_upload_logic(
-            "commit_sha", "owner/repo", uuid.uuid4(), "service", None, False
+            "commit_sha", "owner/repo", uuid.uuid4(), "service", None, False, False
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
@@ -50,7 +50,7 @@ def test_empty_upload_with_error(mocker):
     runner = CliRunner()
     with runner.isolation() as outstreams:
         res = empty_upload_logic(
-            "commit_sha", "owner/repo", uuid.uuid4(), "service", None, False
+            "commit_sha", "owner/repo", uuid.uuid4(), "service", None, False, False
         )
 
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
@@ -77,7 +77,7 @@ def test_empty_upload_200(mocker):
     runner = CliRunner()
     with runner.isolation() as outstreams:
         res = empty_upload_logic(
-            "commit_sha", "owner/repo", token, "service", None, False
+            "commit_sha", "owner/repo", token, "service", None, False, False
         )
     out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
     assert out_bytes == [
@@ -96,10 +96,40 @@ def test_empty_upload_403(mocker):
         return_value=mocker.MagicMock(status_code=403, text="Permission denied"),
     )
     token = uuid.uuid4()
-    res = empty_upload_logic("commit_sha", "owner/repo", token, "service", None, False)
+    res = empty_upload_logic(
+        "commit_sha", "owner/repo", token, "service", None, False, False
+    )
     assert res.error == RequestError(
         code="HTTP Error 403",
         description="Permission denied",
         params={},
     )
     mocked_response.assert_called_once()
+
+
+def test_empty_upload_force(mocker):
+    res = {
+        "result": "Force option was enabled. Triggering passing notifications.",
+        "non_ignored_files": [],
+    }
+    mocked_response = mocker.patch(
+        "codecov_cli.helpers.request.requests.post",
+        return_value=RequestResult(
+            status_code=200, error=None, warnings=[], text=json.dumps(res)
+        ),
+    )
+    token = uuid.uuid4()
+    runner = CliRunner()
+    with runner.isolation() as outstreams:
+        res = empty_upload_logic(
+            "commit_sha", "owner/repo", token, "service", None, False, True
+        )
+    out_bytes = parse_outstreams_into_log_lines(outstreams[0].getvalue())
+    assert out_bytes == [
+        ("info", "Process Empty Upload complete"),
+        ("info", "Force option was enabled. Triggering passing notifications."),
+        ("info", "Non ignored files []"),
+    ]
+    assert res.error is None
+    assert res.warnings == []
+    mocked_response.assert_called_once()