Closed
Description
@MarcoGorelli suggested in #51459 (review) to add more tests for Period repr before #51459 can be merged.
What I'm most concerned about is that not all these reprs seems to be explicitly tested for. Some of them are, e.g.
pandas/pandas/tests/scalar/period/test_period.py
Lines 711 to 719 in 044227f
but thenFR_SEC
isn't (as far as I can tell).Reckon we should add such unit tests for each period group (perhaps in a separate PR, which we could get in very quickly)? That would give me much more confidence about merging this