jaraco / inflect

Correctly generate plurals, ordinals, indefinite articles; convert numbers to words
https://pypi.org/project/inflect
MIT License
957 stars 107 forks source link

Test coverage: ordinals for real numbers #181

Closed jayaddison closed 1 year ago

jayaddison commented 1 year ago

Follows-on from some code inspection in #174 (and this related discussion).

The inflect library does support production of ordinals for non-integer numbers currently, so for refactor-safety, that'd seem worthwhile to add coverage for.

However, at the same time: it seems unclear to me whether ordinals make sense for non-integers. So the test coverage is marked as xfail using pytest (it currently passes and produces an xpass result. it would not fail the build but would still be noticeable in report output if the test began to fail).