Regression tests

[lewis6991]

Hi @CppCXY . As you know I've been raising lots of issues related to type analysis and diagnostics. You've been really good and responsive to fixing every single one of these, however I noticed that most (all?) of the commits to fix these issues are omitting regression tests. I noticed there are some tests, but I can't see many being added in any recent commits. Is this a temporary practice, and do you have any plans to introduce regression tests in the future.

I ask for two reasons:

• I'd like Neovim and my projects to eventually adopt this tool, and having a strong testsuite would give us good confidence.
• I'd like to eventually contribute to this tool, and having a strong testsuite would make that easier.

Thanks.

[CppCXY]

Most of my tests were written in the early stages of the project. I usually write tests for every change in the parser, but for the code analysis part, I have indeed written very few tests. This is mainly because my ideas often change, and the interfaces change frequently. If I wrote tests for every step, this project might not have been released yet.

However, you are right. The project now has many features, and without a significant amount of test code, it is difficult to ensure the quality of future development. Therefore, I plan to add some tests this weekend.

[lewis6991]

I can see commits are getting tests now so I think this can be closed. I'll let you close it in the case you are keeping this ticket open for any other reasons.