tools icon indicating copy to clipboard operation
tools copied to clipboard
verified publisher icon dart-lang

Additional test cases

Open jonasfj opened this issue 3 years ago • 5 comments

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 https://github.com/dart-lang/yaml_edit/pull/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.

jonasfj avatar Feb 10 '23 12:02 jonasfj

@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!

MikiPAUL avatar Feb 16 '23 15:02 MikiPAUL

Sure, no problem.

jonasfj avatar Feb 16 '23 16:02 jonasfj

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

jonasfj avatar Feb 20 '23 13:02 jonasfj

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.

jonasfj avatar Mar 01 '23 15:03 jonasfj

@jonasfj I will add some tests before I try to add for https://github.com/dart-lang/yaml_edit/issues/26

seifibrahim32 avatar Aug 03 '24 20:08 seifibrahim32