rm-hull / next-departures-api / 25073621163

18%

main: 28%

Build Type

Pull #4

github

Committed by rm-hull

Commit Message

feat: support name-based CSV column mapping

Refactor the CSV parsing logic to use a `headerMap` for looking up
column indices by name. This replaces fragile hardcoded index
references in the `FromTuple` function, making the parser resilient to
changes in column order.

*   `ParseCSV` now generates a `map[string]int` from the header row.
*   `FromTuple` utilizes a `val` helper to retrieve data by header name.
*   Added unit tests to verify mapping with reordered columns.

```mermaid
sequenceDiagram
    participant R as CSV Reader
    participant P as ParseCSV
    participant M as FromTuple

    R->>P: Read Header Row
    P->>P: Build map[name]index
    loop Each Record
        R->>P: Read Data Row
        P->>M: FromTuple(record, map)
        M->>M: Map lookup (e.g. "Easting")
        M-->>P: Model Instance
    end
```

Pull Request Pull Request #4: refactor: support name-based CSV column mapping

Coverage Stats

49 of 56 new or added lines in 2 files covered. (87.5%)

4 existing lines in 1 file now uncovered.

156 of 815 relevant lines covered (19.14%)

0.2 hits per line