search

d3v-null / wp-api-python

A Python wrapper for the WooCommerce API.
MIT License
145 stars 33 forks source link

Not enough test coverage #10

Closed d3v-null closed 6 years ago

d3v-null commented 6 years ago

The automated test cases have not been doing a good enough job at detecting when I accidentally slip Python2 specific language (e.g. unicode instead of six.text_tpe) and since I mostly use this repo myself in a Python2 environment, I miss the errors that happen for Python3 users.

If the repo had better coverage, I would get emails from Travis about the build failing.

Will try and fix today but I am open to PRs if anyone would like to write test cases.

d3v-null commented 6 years ago

Coverage is at 86% now, but Feel free to add more. Thanks.