tests: only test changed packages

[defaude]

Relates to #1193

[defaude]

However, having this long-running tests clogs up everyone's workflow and not "just" the CI, therefore I recommend to not just uncommenting the test but rather explicitly skipping it. This way, it's still visible in the console. Finding a solution to the runtime behavior is step 2 - but meanwhile, everyone else can continue working without that delay.

What do you think?

[appgurueu]

We'll have to leave the issue open to find a proper fix eventually however.

[defaude]

Yup! 👍 I mereley mentioned the issue no. in the commit message but didn't close it.

[raklaptudirm]

Ideally, we would want to run the tests in the changed directories only.

[defaude]

Jest has an onlyChanged option that I've set up in the pre-commit hook. If that's what you like, we could also switch to that behavior for the CI, as well.

[raklaptudirm]

That could be great!

[defaude]

@raklaptudirm I've updated the CI config, as well 👍

[raklaptudirm]

The pr looks fine, but I had an idea. I think we would want to run all the tests in the master branch, but run tests on only the changed files in the prs. What do you all think?

[defaude]

I guess I could set that up with GitHub Actions, yeah. But as long as #1193 is not solved, the master build will take forever to complete.

[defaude]

I've updated the Workflow accordingly - please take a look 😀

[raklaptudirm]

I guess I could set that up with GitHub Actions, yeah. But as long as #1193 is not solved, the master build will take forever to complete.

The master build should not be an issue as no one will need to wait for that to complete.