Commit c0d0e6b
fix: Support HIAC Run Counter format in .xlsx files for Beckman PharmSpec parser (#1238)
## Summary
- HIAC instruments can export the "Run Counter Test" format as `.xlsx`,
not just `.xls`
- The `_read_xlsx` method assumed all `.xlsx` files use the PharmSpec
format (which has an `Approver_` footer marker), causing
`AllotropeConversionError` on HIAC Run Counter `.xlsx` files
- Added format detection: PharmSpec uses a 3-column header layout (`key`
| `:` | `value`), while HIAC Run Counter uses a 2-column layout (`key` |
`: value`). Routes to the existing `_parse_hiac_run_counter_format`
handler when the file isn't PharmSpec.
## Test plan
- [x] All 17 existing beckman_pharmspec tests pass (including the
`missing_approver` error test)
- [x] Verified the failing `.xlsx` HIAC file now parses successfully
- [x] Verified existing `.xls` HIAC files still parse correctly
- [x] Lint clean (ruff, black, mypy)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent 7b6f6fc commit c0d0e6b
1 file changed
Lines changed: 7 additions & 0 deletions
Lines changed: 7 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
24 | 31 | | |
25 | 32 | | |
26 | 33 | | |
| |||
0 commit comments