ElectricalAge icon indicating copy to clipboard operation
ElectricalAge copied to clipboard

Setup automated testing

Open Baughn opened this issue 9 years ago • 7 comments

At a minimum, PRs need to build. :X

I've been thinking about automated testing of the game mechanics, as well. Looks like it'd come in handy.

Baughn avatar Dec 08 '15 21:12 Baughn

Hmm. Looks like only @Dolu1990 can set this up? @metc, you've used this before.

Baughn avatar Dec 08 '15 21:12 Baughn

The repo already contains a travis config file, but we need to enable travis for the repo. As you stated correctly only @Dolu1990 can do this as he is the owner of the repo. So we have basically two possibilities:

  • @Dolu1990 signs up for travis and enables the repo.
  • @Dolu1990 transfers the repo to the Electrical Age organisation and then anyone of us can use his travis account to do the job.

What do you think @Dolu1990 ?

cm0x4D avatar Dec 08 '15 22:12 cm0x4D

Any PR must be compiled before merging, we cannot have noncompilable code in the master branch.

We already discussed long time ago about transferring the repo to the Electrical Age organisation, IMHO this would be great but many links (forum, mod versions, analytics, movies) will be broken and some of them can be critical...

If we need Travis, this is a one click job for @Dolu1990 to setup it on the Travis online interface.

metc avatar Dec 08 '15 22:12 metc

Transitioning EA to an organization is only going to get more difficult over time. It's ultimately up to you and Dolu, as you're the ones who'd have to do most of the work, but it it's going to be transitioned at all it'd probably make sense to do so sooner rather than later.

Baughn avatar Dec 09 '15 16:12 Baughn

Of course if we had transitioning sooner we would have had less pain... This is a decision @Dolu1990 must take, and it would be good to answer this before installing Travis or any CI system.

metc avatar Dec 09 '15 19:12 metc

Reopening this, because I still have tests I want to add~

For starters, just "Does the server start up".

Baughn avatar Dec 09 '15 21:12 Baughn

This would be trivial to complete? I think @Baughn has access to set this up now if it isn't already. :)

jrddunbr avatar Apr 06 '19 20:04 jrddunbr