read_por is untested and has no fixture
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 75/100
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.
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
- Dominant language
- Julia
- Stars
- 79
- Forks
- 18
- Avg merge
- 22h 31m
- Merged PRs (30d)
- 3
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.
More from queryverse/ReadStat.jl
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
queryverse/ReadStat.jl#82 · 1 reaction ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
queryverse/ReadStat.jl#74 · 2 comments ·
-
Difficulty 3/5 1-2 days Newbie friendliness 42/100
queryverse/ReadStat.jl#67 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 25/100
queryverse/ReadStat.jl#47 · 1 comment ·
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
queryverse/ReadStat.jl#39 · 1 comment ·
All issues in queryverse/ReadStat.jl
Similar issues
-
bug 🐞 distributed 🕸️
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
CliMA/Oceananigans.jl#6049 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
-
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
JuliaGPU/GPUArrays.jl#780 ·
-
Difficulty 2/5 Half a day Newbie friendliness 76/100
epiforecasts/EpiBranch.jl#304 ·