CI idea: Check PRs and the main branch for successfully running about capturetheflag HOT 8 OPEN

I'd prefer to have code documentation before we start doing unit tests https://github.com/MT-CTF/capturetheflag/blob/master/docs/ctf-api.md

from capturetheflag.

If we really do that, we might need a way to interact with the Minetest client engine. I am thinking a hacked client, combined with a headless Xorg server, but if Minetest Engine itself can make some shared libraries for building a test client, it would be better.

from capturetheflag.

headless clients exist, but it's not really possible for testing code to do any more than the developer just testing their code locally

from capturetheflag.

headless clients exist, but it's not really possible for testing code to do any more than the developer just testing their code locally

That's exactly the point. We make some common tests automatic. Therefore we can know about the PRs and the master/main branch that certain things work for sure without having to test the PRs ourselves.

from capturetheflag.

If you make a PR you should test it. I don't want to allow bad habits to form

from capturetheflag.

If you make a PR you should test it. I don't want to allow bad habits to form

Yeah and if I don't test it, the merging will be blocked. As a usage example.

from capturetheflag.

Yeah and if I don't test it, the merging will be blocked. As a usage example.

There's no point in the suggested automated testing in that case then

from capturetheflag.

Yeah and if I don't test it, the merging will be blocked. As a usage example.

There's no point in the suggested automated testing in that case then

I don't understand. Can you elaborate on your point? My point is that, like any other program, this game could have unit tests and use Actions+Headless client to enforce them.

from capturetheflag.