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.

Sign up for GitHub

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

Jump to bottom

SplitWithQuotation full code coverage #12

Open
13 of 15 tasks
Tracked by #1
jjvanzon opened this issue Jul 31, 2021 · 0 comments
Open
13 of 15 tasks
Tracked by #1

SplitWithQuotation full code coverage #12

jjvanzon opened this issue Jul 31, 2021 · 0 comments
Assignees
@jjvanzon
Labels
JJ.Framework.Text unit-test
Milestone
Full code coverag...

Comments

@jjvanzon
Copy link
Owner

jjvanzon commented Jul 31, 2021 •
edited
Loading

Retroactively adding missing automated tests aiming for full code coverage.

  • Had bugs
  • Perhaps remember previous char instead of looking ahead 1 char?
  • Default quote-less to split without quote.
  • Perhaps handling escaped quote with SubstringOrLess too or a variation thereof.
  • Separate overloads with char and string separator and quotes.
  • More tests.
    • Programming.
    • Making them run.
  • Separate test class with char and string separator and quotes.
  • Preventing false positives:
    • Varying values.
    • Varying quote characters and separators more.
    • Making separator and quote strings different lengths.
  • Debuging tests.
  • Testing members made obsolete.
@jjvanzon jjvanzon mentioned this issue Jul 31, 2021
Open
9 tasks
@jjvanzon jjvanzon changed the title Split method full code coverage SplitWithQuotation full code coverage Jul 31, 2021
@jjvanzon jjvanzon self-assigned this Jul 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jjvanzon jjvanzon

Labels
JJ.Framework.Text unit-test
Projects
None yet
Milestone
Full code coverage public JJ.Framework
Development

No branches or pull requests
1 participant
@jjvanzon