[TASK] Move some tests for Comment to functional tests

[oliverklee]

Move the tests that rely on another class (OutputFormat) (or might in the future).

Part of #757.

[JakeQZ]

The tests for 'pretty' and 'compact' should actually use the applicable creation methods.

Perhaps there should also be a test for the create method, as well as construction with new.

[coveralls]

coverage: 49.156%. remained the same
when pulling f86244e on task/comment-test
into 537f5e8 on main.

[oliverklee]

Done and repushed.

[JakeQZ]

Virgin was an interesting choice for the test method name, and slightly amusing, but works :)

[oliverklee]

Virgin was an interesting choice for the test method name, and slightly amusing, but works :)

I actually was struggling with the name a bit. A close contender was "newly minted". Any other suggestions (from a native BE speaker)?

[JakeQZ]

Virgin was an interesting choice for the test method name, and slightly amusing, but works :)

I actually was struggling with the name a bit. A close contender was "newly minted". Any other suggestions (from a native BE speaker)?

Coincidentally, I just listened to a programme about cooking oils. You might not be able to access it outside the UK. 14 minutes in the 'expert' talks about olive oil, and says: "Extra virgin overall means it's as close to the natural state as possible."

So I think you've hit the nomeclature nail directly on the head here, and the naming cannot be improvied upon. Well done. (Virgin does not always imply lack of promiscuity, particularly when applied to inanimate objects.)

Freshly-minted has a strong association with the bank prressing coins. It would not have worked as well, but would have been acceptably whimsical :))