Fix bounds check for negative states + Add tests

[pedromxavier]

This PR aims to address #157

Apparently, the offset computation was already correct but the bounds assertions were still not considering the relative first stage.

Tasks

• Fix bounds check, add as a separate function
• Add tests using the inflow files from Handle negative stages in OpenBinary reading and writing #157

[codecov-commenter]

Codecov Report

Patch coverage: 83.33% and project coverage change: +0.10% 🎉

Comparison is base (a990a4d) 79.85% compared to head (476b9de) 79.96%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   79.85%   79.96%   +0.10%     
==========================================
  Files          21       21              
  Lines        2810     2815       +5     
==========================================
+ Hits         2244     2251       +7     
+ Misses        566      564       -2     

Files Changed	Coverage Δ
src/tables/interface.jl	76.53% <57.14%> (+2.57%)	⬆️
src/OpenBinary/reader.jl	80.51% <100.00%> (+0.19%)	⬆️
src/utils.jl	55.55% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joaquimg]

The test case is too large 2Mb csv, 1Mb bin, it should be much much smaller.

[pedromxavier]

This PR should be closed. Handling variable names became infeasible at this point so I reimplemented the core ideas from scratch in #175, including test cases.