tests: remove redundant asyncio markers

Author: pablogsal

backend/tests/conftest.py

@@ -1,6 +1,5 @@
 """Shared fixtures for backend tests."""
 
-import hashlib
 import secrets
 from datetime import datetime, timedelta
 

backend/tests/test_auth.py

@@ -1,9 +1,7 @@
 """Tests for token authentication."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_valid_bearer_token(client, auth_headers, sample_binary, sample_environment):
     """A valid Bearer token should authenticate successfully on a protected endpoint."""
     payload = {
@@ -24,7 +22,6 @@ async def test_valid_bearer_token(client, auth_headers, sample_binary, sample_en
     assert response.status_code == 200
 
 
-@pytest.mark.asyncio
 async def test_upload_with_invalid_token(client, sample_binary, sample_environment):
     """An invalid token should be rejected."""
     headers = {"Authorization": "Bearer invalid_token_value"}
@@ -36,7 +33,6 @@ async def test_upload_with_invalid_token(client, sample_binary, sample_environme
     assert response.status_code in (401, 403)
 
 
-@pytest.mark.asyncio
 async def test_upload_with_no_token(client):
     """Missing token should be rejected."""
     response = await client.post(
@@ -46,7 +42,6 @@ async def test_upload_with_no_token(client):
     assert response.status_code in (401, 403)
 
 
-@pytest.mark.asyncio
 async def test_token_format_bearer(client, auth_token, sample_binary, sample_environment):
     """'Bearer <token>' format should work."""
     raw_token, _ = auth_token
@@ -66,7 +61,6 @@ async def test_token_format_bearer(client, auth_token, sample_binary, sample_env
     assert response.status_code == 200
 
 
-@pytest.mark.asyncio
 async def test_token_format_token_prefix(client, auth_token, sample_binary, sample_environment):
     """'Token <token>' format should also work."""
     raw_token, _ = auth_token
@@ -85,7 +79,6 @@ async def test_token_format_token_prefix(client, auth_token, sample_binary, samp
     assert response.status_code == 200
 
 
-@pytest.mark.asyncio
 async def test_inactive_token_rejected(client, db_session, auth_token, sample_binary, sample_environment):
     """A deactivated token should be rejected."""
     raw_token, token_model = auth_token

backend/tests/test_benchmarks.py

@@ -1,24 +1,20 @@
 """Tests for the benchmarks API endpoints."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_all_benchmark_names_empty(client):
     response = await client.get("/api/benchmarks")
     assert response.status_code == 200
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_all_benchmark_names(client, sample_benchmark_result):
     response = await client.get("/api/benchmarks")
     assert response.status_code == 200
     data = response.json()
     assert "json_dumps" in data
 
 
-@pytest.mark.asyncio
 async def test_filtered_benchmark_names(client, sample_benchmark_result):
     response = await client.get(
         "/api/benchmark-names",
@@ -34,7 +30,6 @@ async def test_filtered_benchmark_names(client, sample_benchmark_result):
     assert "json_dumps" in data
 
 
-@pytest.mark.asyncio
 async def test_filtered_benchmark_names_no_match(client, sample_benchmark_result):
     response = await client.get(
         "/api/benchmark-names",
@@ -49,7 +44,6 @@ async def test_filtered_benchmark_names_no_match(client, sample_benchmark_result
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_diff_table(client, sample_benchmark_result):
     response = await client.get(
         "/api/diff",
@@ -69,7 +63,6 @@ async def test_diff_table(client, sample_benchmark_result):
     assert row["has_flamegraph"] is True
 
 
-@pytest.mark.asyncio
 async def test_diff_table_commit_not_found(client):
     response = await client.get(
         "/api/diff",
@@ -82,7 +75,6 @@ async def test_diff_table_commit_not_found(client):
     assert response.status_code == 404
 
 
-@pytest.mark.asyncio
 async def test_trends(client, sample_benchmark_result):
     response = await client.get(
         "/api/trends",
@@ -101,7 +93,6 @@ async def test_trends(client, sample_benchmark_result):
     assert data[0]["high_watermark_bytes"] == 1_000_000
 
 
-@pytest.mark.asyncio
 async def test_trends_batch(client, sample_benchmark_result):
     response = await client.post(
         "/api/trends-batch",
@@ -124,7 +115,6 @@ async def test_trends_batch(client, sample_benchmark_result):
     assert len(data["results"]) == 1
 
 
-@pytest.mark.asyncio
 async def test_flamegraph(client, sample_benchmark_result):
     result_id = sample_benchmark_result.id
     response = await client.get(f"/api/flamegraph/{result_id}")
@@ -134,7 +124,6 @@ async def test_flamegraph(client, sample_benchmark_result):
     assert data["benchmark_name"] == "json_dumps"
 
 
-@pytest.mark.asyncio
 async def test_flamegraph_not_found(client):
     response = await client.get("/api/flamegraph/nonexistent")
     assert response.status_code == 404

backend/tests/test_binaries.py

@@ -1,16 +1,13 @@
 """Tests for the binaries API endpoints."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_list_binaries_empty(client):
     response = await client.get("/api/binaries")
     assert response.status_code == 200
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_list_binaries(client, sample_binary):
     response = await client.get("/api/binaries")
     assert response.status_code == 200
@@ -21,7 +18,6 @@ async def test_list_binaries(client, sample_binary):
     assert "--enable-optimizations" in data[0]["flags"]
 
 
-@pytest.mark.asyncio
 async def test_get_binary_by_id(client, sample_binary):
     response = await client.get("/api/binaries/default")
     assert response.status_code == 200
@@ -30,13 +26,11 @@ async def test_get_binary_by_id(client, sample_binary):
     assert data["description"] == "Standard build"
 
 
-@pytest.mark.asyncio
 async def test_get_binary_not_found(client):
     response = await client.get("/api/binaries/nonexistent")
     assert response.status_code == 404
 
 
-@pytest.mark.asyncio
 async def test_environments_for_binary(client, sample_benchmark_result):
     response = await client.get("/api/binaries/default/environments")
     assert response.status_code == 200
@@ -46,13 +40,11 @@ async def test_environments_for_binary(client, sample_benchmark_result):
     assert data[0]["run_count"] >= 1
 
 
-@pytest.mark.asyncio
 async def test_environments_for_nonexistent_binary(client):
     response = await client.get("/api/binaries/nonexistent/environments")
     assert response.status_code == 404
 
 
-@pytest.mark.asyncio
 async def test_commits_for_binary_and_environment(client, sample_benchmark_result):
     response = await client.get(
         "/api/binaries/default/environments/linux-x86_64/commits"

backend/tests/test_commits.py

@@ -1,16 +1,13 @@
 """Tests for the commits API endpoints."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_list_commits_empty(client):
     response = await client.get("/api/commits")
     assert response.status_code == 200
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_list_commits(client, sample_commit):
     response = await client.get("/api/commits")
     assert response.status_code == 200
@@ -22,7 +19,6 @@ async def test_list_commits(client, sample_commit):
     assert data[0]["python_version"]["minor"] == 14
 
 
-@pytest.mark.asyncio
 async def test_list_commits_pagination(client, sample_commit):
     response = await client.get("/api/commits", params={"skip": 0, "limit": 1})
     assert response.status_code == 200
@@ -33,7 +29,6 @@ async def test_list_commits_pagination(client, sample_commit):
     assert len(response.json()) == 0
 
 
-@pytest.mark.asyncio
 async def test_get_commit_by_sha(client, sample_commit):
     response = await client.get(f"/api/commits/{sample_commit.sha}")
     assert response.status_code == 200
@@ -42,20 +37,17 @@ async def test_get_commit_by_sha(client, sample_commit):
     assert data["message"] == "Test commit"
 
 
-@pytest.mark.asyncio
 async def test_get_commit_not_found(client):
     response = await client.get("/api/commits/" + "f" * 40)
     assert response.status_code == 404
 
 
-@pytest.mark.asyncio
 async def test_python_versions_empty(client):
     response = await client.get("/api/python-versions")
     assert response.status_code == 200
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_python_versions(client, sample_benchmark_result):
     response = await client.get("/api/python-versions")
     assert response.status_code == 200

backend/tests/test_environments.py

@@ -1,16 +1,13 @@
 """Tests for the environments API endpoints."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_list_environments_empty(client):
     response = await client.get("/api/environments")
     assert response.status_code == 200
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_list_environments(client, sample_environment):
     response = await client.get("/api/environments")
     assert response.status_code == 200
@@ -20,15 +17,13 @@ async def test_list_environments(client, sample_environment):
     assert data[0]["name"] == "Linux x86_64"
 
 
-@pytest.mark.asyncio
 async def test_get_environment_by_id(client, sample_environment):
     response = await client.get("/api/environments/linux-x86_64")
     assert response.status_code == 200
     data = response.json()
     assert data["id"] == "linux-x86_64"
 
 
-@pytest.mark.asyncio
 async def test_get_environment_not_found(client):
     response = await client.get("/api/environments/nonexistent")
     assert response.status_code == 404

backend/tests/test_health.py

@@ -1,9 +1,7 @@
 """Tests for the health check endpoint."""
 
-import pytest
 
 
-@pytest.mark.asyncio
 async def test_health_check(client):
     response = await client.get("/health")
     assert response.status_code == 200
@@ -12,7 +10,6 @@ async def test_health_check(client):
     assert "timestamp" in data
 
 
-@pytest.mark.asyncio
 async def test_health_check_reports_db_status(client):
     """The health router uses a module-level settings object with
     enable_health_check_db=True (not overridable via test_settings).

backend/tests/test_production_data.py

@@ -46,7 +46,6 @@ async def prod_data(db_session):
     await db_session.commit()
 
 
-@pytest.mark.asyncio
 async def test_diff_detects_regression(client, prod_data):
     """The diff endpoint should show the deltablue_base regression."""
     response = await client.get(
@@ -74,7 +73,6 @@ async def test_diff_detects_regression(client, prod_data):
     assert nbody["metric_delta_percent"] == pytest.approx(0.0)
 
 
-@pytest.mark.asyncio
 async def test_diff_previous_commit_details(client, prod_data):
     """The diff should include correct previous commit metadata."""
     response = await client.get(
@@ -99,7 +97,6 @@ async def test_diff_previous_commit_details(client, prod_data):
     assert curr["author"] == "alm"
 
 
-@pytest.mark.asyncio
 async def test_diff_first_commit_has_no_previous(client, prod_data):
     """Diffing the earlier commit should show no previous data."""
     response = await client.get(
@@ -118,7 +115,6 @@ async def test_diff_first_commit_has_no_previous(client, prod_data):
         assert row["prev_commit_details"] is None
 
 
-@pytest.mark.asyncio
 async def test_diff_with_total_allocated_metric(client, prod_data):
     """Diff should work with total_allocated_bytes metric too."""
     response = await client.get(
@@ -138,9 +134,8 @@ async def test_diff_with_total_allocated_metric(client, prod_data):
     assert deltablue["prev_metric_value"] == BENCH_DELTABLUE_PREV["total_allocated_bytes"]
 
 
-@pytest.mark.asyncio
 async def test_trends_returns_chronological_data(client, prod_data):
-    """Trends should return data points in chronological order."""
+    """Trends should return data points in reverse chronological order (newest first)."""
     response = await client.get(
         "/api/trends",
         params={
@@ -162,7 +157,6 @@ async def test_trends_returns_chronological_data(client, prod_data):
     assert data[1]["high_watermark_bytes"] == 1_557_777
 
 
-@pytest.mark.asyncio
 async def test_batch_trends_multiple_benchmarks(client, prod_data):
     """Batch trends should return data for multiple benchmarks at once."""
     response = await client.post(
@@ -200,7 +194,6 @@ async def test_batch_trends_multiple_benchmarks(client, prod_data):
     assert len(results[json_key]) == 2
 
 
-@pytest.mark.asyncio
 async def test_benchmark_names_filtered_by_version(client, prod_data):
     """Benchmark names should filter correctly by Python version."""
     response = await client.get(
@@ -229,7 +222,6 @@ async def test_benchmark_names_filtered_by_version(client, prod_data):
     assert response.json() == []
 
 
-@pytest.mark.asyncio
 async def test_python_versions_from_production_data(client, prod_data):
     response = await client.get("/api/python-versions")
     assert response.status_code == 200
@@ -239,7 +231,6 @@ async def test_python_versions_from_production_data(client, prod_data):
     assert versions[0]["minor"] == 15
 
 
-@pytest.mark.asyncio
 async def test_environments_for_binary(client, prod_data):
     response = await client.get("/api/binaries/nogil/environments")
     assert response.status_code == 200
@@ -250,7 +241,6 @@ async def test_environments_for_binary(client, prod_data):
     assert envs[0]["commit_count"] == 2
 
 
-@pytest.mark.asyncio
 async def test_commits_for_binary_and_environment(client, prod_data):
     response = await client.get(
         "/api/binaries/nogil/environments/gh_actions/commits"