atlanhq / camelot

Camelot: PDF Table Extraction for Humans
https://camelot-py.readthedocs.io
Other
3.65k stars 357 forks source link

[MRG] Fixed #350: make sure strip_text= argument works #351

Closed dimitern closed 5 years ago

dimitern commented 5 years ago

Fixes #350

Added a helper to do the stripping, and updated a test to verify it works.

As tests were failing due to pytest-5.0.0 changes with pytest.raises, I've updated those as needed to pass them, but those changes are not related to fixing the issue.

dimitern commented 5 years ago

I've removed the unrelated changes to test_errors.py in favour of the separate PR I pushed #354

dimitern commented 5 years ago

Closing in favour of https://github.com/camelot-dev/camelot/pull/4