Skip to content

Commit

Permalink
chore: make lint and rm leftover test
Browse files Browse the repository at this point in the history
  • Loading branch information
joseph-sentry committed Feb 22, 2024
1 parent 5657665 commit 74d3a95
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 30 deletions.
29 changes: 0 additions & 29 deletions upload/tests/views/test_test_results.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,32 +150,3 @@ def test_upload_test_results_missing_args(db, client, mocker, mock_redis):
assert res.status_code == 400
assert res.json() == {"commit": ["This field is required."]}
assert not upload.called


def test_upload_test_results_rollout_fails(db, client, mocker, mock_redis):
upload = mocker.patch.object(TaskService, "upload")
create_presigned_put = mocker.patch(
"services.archive.StorageService.create_presigned_put",
return_value="test-presigned-put",
)

owner = OwnerFactory(service="github", username="not-codecov")
repository = RepositoryFactory.create(author=owner)
commit_sha = "6fd5b89357fc8cdf34d6197549ac7c6d7e5977ef"

client = APIClient()
client.credentials(HTTP_AUTHORIZATION=f"token {repository.upload_token}")

res = client.post(
reverse("upload-test-results"),
{
"commit": commit_sha,
"slug": f"{repository.author.username}::::{repository.name}",
"build": "test-build",
"buildURL": "test-build-url",
"job": "test-job",
"service": "test-service",
},
format="json",
)
assert res.status_code == 403
1 change: 0 additions & 1 deletion upload/views/test_results.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
from upload.views.base import ShelterMixin
from upload.views.helpers import get_repository_from_string


log = logging.getLogger(__name__)


Expand Down

0 comments on commit 74d3a95

Please sign in to comment.