-
Notifications
You must be signed in to change notification settings - Fork 16
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
Additional test cases #33
Comments
@jonasfj , I would love to take on this task! I am confident that I can make a contribution here. Would you mind assigning this issue to me? Thank you! |
Sure, no problem. |
Honestly, anyone is welcome to help write a few test cases. Especially, if we can make some test cases that finds new bugs, then we can file issues to fix those bugs :D |
In particular, we need test cases for strings that contain special characters, line breaks and the lines. I think we have issues when such strings are formatted as folded strings, etc. |
This package could use more test cases.
Both of the kind in here:
https://github.com/dart-lang/yaml_edit/tree/main/test/testdata
And of the kind written in
test/*_test.dart
.Issues like broken string escaping noticed in #28 indicate that we could probably use more test cases.
An approach to systematically test more of the code, or perhaps some code-coverage could be useful to.
The text was updated successfully, but these errors were encountered: