Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(NFC) Comment clarification in test class #17133

Merged
merged 1 commit into from
Apr 22, 2020
Merged

Commits on Apr 21, 2020

  1. 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
    eileenmcnaughton committed Apr 21, 2020
    Configuration menu
    Copy the full SHA
    34e322d View commit details
    Browse the repository at this point in the history