idris-hackers / idris-mode

Idris syntax highlighting, compiler-supported editing, interactive REPL and more things for Emacs.
GNU General Public License v3.0
267 stars 70 forks source link

Move test files to test/ directory #606

Closed keram closed 1 year ago

keram commented 1 year ago

Why: To better separate production code and tests and improve maintainability. h Pros:

Cons:

jfdm commented 1 year ago

Looks like one of the builds aborted for reasons not connected to your PR. I have restarted said build, and one if successfully finishes we can merge.

keram commented 1 year ago

Looks like one of the builds aborted for reasons not connected to your PR. I have restarted said build, and one if successfully finishes we can merge.

One of my ideas for next weeks is to separate API contract tests between Idris and idris-mode out of the actual tests of the idris-mode feature(s). On high level it may consist of defining two separate steps in CI

This will lead to reduced CI but also local "random" failures and also improve speed of running tests. (some of the tests take more than 5 seconds which is quite painful too)

jfdm commented 1 year ago

It has aborted again. I shall merge this, as the issue is not the PRs fault.