Skip to content

Commit b78b0c3

Browse files
style: fix whitespace in test files
Co-Authored-By: Rushil Srivastava <[email protected]>
1 parent ed105d2 commit b78b0c3

File tree

3 files changed

+36
-42
lines changed

3 files changed

+36
-42
lines changed

tests/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ def scheduled_route():
3131
"""Create a ScheduledRouteBuilder configured for testing."""
3232
return ScheduledRouteBuilder(
3333
base_url="http://localhost:8000",
34-
location_path=f"projects/test-project/locations/us-central1",
34+
location_path="projects/test-project/locations/us-central1",
3535
)
3636

3737

tests/test_delayed_route.py

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,76 +1,75 @@
11
"""Tests for DelayedRouteBuilder functionality."""
2-
import pytest
2+
33
from fastapi import APIRouter, Depends
4+
from google.protobuf import duration_pb2
45
from pydantic import BaseModel
56

67
from fastapi_gcp_tasks.dependencies import max_retries
78
from fastapi_gcp_tasks.hooks import deadline_delayed_hook
8-
from google.protobuf import duration_pb2
99

1010

1111
class TestPayload(BaseModel):
1212
"""Test payload for task endpoints."""
13+
1314
message: str
1415

1516

1617
def test_delayed_task_creation(app, delayed_route, test_client):
1718
"""Test basic task creation and execution."""
1819
router = APIRouter(route_class=delayed_route)
19-
20+
2021
@router.post("/test")
2122
async def test_task(payload: TestPayload):
2223
return {"received": payload.message}
23-
24+
2425
app.include_router(router)
25-
26+
2627
response = test_client.post("/test", json={"message": "test"})
2728
assert response.status_code == 200
2829
assert response.json() == {"received": "test"}
2930

3031

3132
def test_delayed_task_with_hook(app, delayed_route, test_client):
3233
"""Test task creation with deadline hook."""
33-
delayed_route.pre_create_hook = deadline_delayed_hook(
34-
duration=duration_pb2.Duration(seconds=300)
35-
)
34+
delayed_route.pre_create_hook = deadline_delayed_hook(duration=duration_pb2.Duration(seconds=300))
3635
router = APIRouter(route_class=delayed_route)
37-
36+
3837
@router.post("/test-hook")
3938
async def test_task(payload: TestPayload):
4039
return {"received": payload.message}
41-
40+
4241
app.include_router(router)
43-
42+
4443
response = test_client.post("/test-hook", json={"message": "test"})
4544
assert response.status_code == 200
4645

4746

4847
def test_delayed_task_with_retries(app, delayed_route, test_client):
4948
"""Test task with max retries dependency."""
5049
router = APIRouter(route_class=delayed_route)
51-
50+
5251
@router.post("/test-retries", dependencies=[Depends(max_retries(2))])
5352
async def test_task():
5453
raise Exception("Test failure")
55-
54+
5655
app.include_router(router)
57-
56+
5857
response = test_client.post("/test-retries")
5958
assert response.status_code == 500 # Should fail after 2 retries
6059

6160

6261
def test_delayed_task_with_countdown(app, delayed_route, test_client):
6362
"""Test task creation with countdown."""
6463
router = APIRouter(route_class=delayed_route)
65-
64+
6665
@router.post("/test-countdown")
6766
async def test_task(payload: TestPayload):
6867
return {"received": payload.message}
69-
68+
7069
app.include_router(router)
71-
70+
7271
# Test with countdown option
7372
test_task.options(countdown=60).delay(payload=TestPayload(message="delayed"))
74-
73+
7574
response = test_client.post("/test-countdown", json={"message": "test"})
7675
assert response.status_code == 200

tests/test_scheduled_route.py

Lines changed: 18 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,57 +1,52 @@
11
"""Tests for ScheduledRouteBuilder functionality."""
2-
import pytest
2+
33
from fastapi import APIRouter
4+
from google.protobuf import duration_pb2
45
from pydantic import BaseModel
56

67
from fastapi_gcp_tasks.hooks import deadline_scheduled_hook
7-
from google.protobuf import duration_pb2
88

99

1010
class TestPayload(BaseModel):
1111
"""Test payload for scheduled task endpoints."""
12+
1213
message: str
1314

1415

1516
def test_scheduled_task_creation(app, scheduled_route, test_client):
1617
"""Test basic scheduled task creation."""
1718
router = APIRouter(route_class=scheduled_route)
18-
19+
1920
@router.post("/test-scheduled")
2021
async def test_task(payload: TestPayload):
2122
return {"received": payload.message}
22-
23+
2324
app.include_router(router)
24-
25+
2526
# Schedule a task
26-
test_task.scheduler(
27-
name="test-scheduled-task",
28-
schedule="*/5 * * * *",
29-
time_zone="UTC"
30-
).schedule(payload=TestPayload(message="scheduled"))
31-
27+
test_task.scheduler(name="test-scheduled-task", schedule="*/5 * * * *", time_zone="UTC").schedule(
28+
payload=TestPayload(message="scheduled")
29+
)
30+
3231
response = test_client.post("/test-scheduled", json={"message": "test"})
3332
assert response.status_code == 200
3433

3534

3635
def test_scheduled_task_with_hook(app, scheduled_route, test_client):
3736
"""Test scheduled task with deadline hook."""
38-
scheduled_route.pre_create_hook = deadline_scheduled_hook(
39-
duration=duration_pb2.Duration(seconds=300)
40-
)
37+
scheduled_route.pre_create_hook = deadline_scheduled_hook(duration=duration_pb2.Duration(seconds=300))
4138
router = APIRouter(route_class=scheduled_route)
42-
39+
4340
@router.post("/test-scheduled-hook")
4441
async def test_task(payload: TestPayload):
4542
return {"received": payload.message}
46-
43+
4744
app.include_router(router)
48-
45+
4946
# Schedule a task with hook
50-
test_task.scheduler(
51-
name="test-scheduled-task-hook",
52-
schedule="0 * * * *",
53-
time_zone="UTC"
54-
).schedule(payload=TestPayload(message="scheduled with hook"))
55-
47+
test_task.scheduler(name="test-scheduled-task-hook", schedule="0 * * * *", time_zone="UTC").schedule(
48+
payload=TestPayload(message="scheduled with hook")
49+
)
50+
5651
response = test_client.post("/test-scheduled-hook", json={"message": "test"})
5752
assert response.status_code == 200

0 commit comments

Comments
 (0)