refactor the tests

[amtoine]

cc/ @AucaCoyan

i was having a look at the tests and wanted to give a go at

• understanding them
• refactoring them a bit for later ease of maintainance hopefully :relieved:

in this PR, i've

• moved the "ignore" comment to the "reason" of the ignore clause in the ignore_comments test
• removed whitespaces from handle_escaped_strings because ignore_whitespace_in_string already tests this
• made the expected part of the asserts explicit
• put expected always after nu, for consistency

i hope you'll find that somewhat better :crossed_fingers:

:question: @AucaCoyan, i've got a question :smirk: in all the tests, the input string does not have a newline at the end, whereas the output of nufmt does. it is by design?

[AucaCoyan]

in all the tests, the input string does not have a newline at the end, whereas the output of nufmt does. it is by design?

Yes, this is intended. The nufmt should add a new line at the end of the script.

Thanks! It is clearer this way 😄 You can merge @fdncred 🚢