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

Missing unit tests for specifiers: a, A, E, F #189

Open
ashwalk33r opened this issue Apr 4, 2020 · 0 comments
Open

Missing unit tests for specifiers: a, A, E, F #189

ashwalk33r opened this issue Apr 4, 2020 · 0 comments

Comments

@ashwalk33r
Copy link

ashwalk33r commented Apr 4, 2020

https://github.com/alexei/sprintf.js/blob/master/test/test.js

Does not contain tests for specifiers:

  • a
  • A
  • E
  • F

All other specifiers(as in http://www.cplusplus.com/reference/cstdio/printf/) are at least once occuring in tests. It does not mean those have good coverage. This issue simply reports that while some specifiershave some tests, the reported have none.

What about creating validation https://github.com/alexei/sprintf.js/blob/master/test/test_validation.js to determine if all specifiers are tested?

@ashwalk33r ashwalk33r changed the title Missing unit tests for modifiers: a, A, E, F Missing unit tests for specifiers: a, A, E, F Apr 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant