map-services / next-departures-api / 25998499927

29%

main: 29%

Build Type

Pull #21

github

Committed by rm-hull

Commit Message

refactor: add context to repository methods

Propagated `context.Context` through the `NaptanRepository` interface
and its SQL implementations to enable proper cancellation and timeout
handling during database operations.

- Updated `sqliteRepository` methods to use context-aware SQL execution
  (`BeginTx`, `PrepareContext`, `ExecContext`, `QueryRowContext`).
- Updated all call sites (API routes, cron jobs, and CLI commands) to
  pass appropriate context.
- Improved transaction rollback logic in `ImportCSV` for better
  consistency.

```mermaid
sequenceDiagram
    participant User
    participant Route
    participant Repo
    participant DB

    User->>Route: Search Request
    Route->>Repo: Search(ctx, bbox)
    Repo->>DB: QueryContext(ctx, sql)
    DB-->>Repo: Results
    Repo-->>Route: Results
    Route-->>User: JSON Response
```

Pull Request Pull Request #21: fix: Conduct a code review to test automated PR generation

Coverage Stats

2 of 25 new or added lines in 5 files covered. (8.0%)

1 existing line in 1 file now uncovered.

341 of 1171 relevant lines covered (29.12%)

0.32 hits per line