queryverse / queryverse/ReadStat.jl
read_por is untested and has no fixture
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 79
- Forks
- 18
- Avg merge
- 22h 31m
- Merged PRs (30d)
- 3
Description
Found while auditing the stack (see #95).
read_por is exported and documented, but the test item only covers read_dta, read_sav, read_sas7bdat and read_xport. There is no types.por fixture in test/, so the SPSS portable reader has no coverage at all.
StatFiles does ship a test/types.por, which could be reused here, though it likewise has no test exercising it (queryverse/StatFiles.jl#32).
Worth doing alongside that: SPSS portable files exercise a genuinely different code path in readstat, and .por is the one format where the row count comes back unknown (-1), the same case read_xport hits and which handle_variable! has a special branch for.
🤖 Generated with Claude Code
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the existing format test item in test/ and the read_por and read_xport entry points, then compare the available test/types.por fixture from StatFiles. Add the .por fixture and coverage for read_por, including the unknown row-count path handled by handle_variable!. Done means the SPSS portable reader has a passing test like the other supported formats.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 75/100