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
Found while auditing the stack (see #95).
read_poris exported and documented, but the test item only coversread_dta,read_sav,read_sas7bdatandread_xport. There is notypes.porfixture intest/, 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
.poris the one format where the row count comes back unknown (-1), the same caseread_xporthits and whichhandle_variable!has a special branch for.🤖 Generated with Claude Code