Comment clarification in test class
authoreileen <emcnaughton@wikimedia.org>
Tue, 21 Apr 2020 11:25:16 +0000 (23:25 +1200)
committereileen <emcnaughton@wikimedia.org>
Tue, 21 Apr 2020 11:25:16 +0000 (23:25 +1200)
commit34e322db01f7bedc1a801a8c3049f6afd5805fb5
treed2aa4402459f6e68dbfb1c803ba6b7518ba422e9
parent53ad44ba272a547eafae850134d0b5408e176a69
Comment clarification in test class

I just updated the comments on this helper to clarify the limitations of the function & the
fact that it should not be our only way to test thousand separators.

I was noticing perfect was becoming the enemy of the good here. The function was marked as deprecated
because it doesn't cover all scenarios - but the upshot was that we stopped increasing out
thousand separator testing. In fact we need lots of form tests to do some testing of
the separators and a very small number to test more variants - this latter has been added
& the comments point to the need for more without going as far as deprecating
tests/phpunit/CiviTest/CiviUnitTestCase.php