Fixed small issue with ordering of statements in extract_text

[alexreg]

chars = to_list(chars) and len(chars) need to be swapped to avoid error sometimes.

Fixes issue #386.

[codecov]

Codecov Report

Merging #385 (47fb9d5) into stable (3b6fee4) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 47fb9d5 differs from pull request most recent head cbfb606. Consider uploading reports for the commit cbfb606 to get more accurate results

@@           Coverage Diff           @@
##           stable     #385   +/-   ##
=======================================
  Coverage   98.25%   98.25%           
=======================================
  Files          10       10           
  Lines        1205     1205           
=======================================
  Hits         1184     1184           
  Misses         21       21           

Impacted Files	Coverage Δ
pdfplumber/table.py	100.00% <ø> (ø)
pdfplumber/utils.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b6fee4...cbfb606. Read the comment docs.

[samkit-jain]

Appreciate you opening a PR @alexreg Request you to add a test case that passes on this commit and fails otherwise.

[alexreg]

@samkit-jain Sure, will do. How do I run tests though? Ideally, this should be documented in the README.

[jsvine]

Thanks, @alexreg and @samkit-jain. I really appreciate the test addition — generally ... but also specifically in this case, because it clarifies what's going on here: The previous code didn't work when the chars argument is a generator. It might be a fairly edge-case situation, but since the fix is very straightforward and doesn't affect performance, I'll merge it.

[jsvine]

And, yes, @alexreg — it's definitely worth us adding PR guidelines (including re. tests). Will open an issue on that and self-assign.

[alexreg]

@jsvine That was my thought too, basically. Thanks for merging.