search

KMattis / COVID19Sim

Simulation of the spread of the novel coronavirus SARS-CoV-2 in a small city.
2 stars 0 forks source link

Add continuous integration unittests #41

Closed KMattis closed 4 years ago

KMattis commented 4 years ago

So first it was just a naming error. But now it says "permission denied" @juliusjh is this a linux/execution permission error?

Or maybe github does not allow execution of bash scripts.

Possible workaround: install_dependencies should call pip requirements.txt and requirements.txt should be a list of all required pip packages

Maybe we should also make an install_file for no-render mode, as one does not need to install OpenGL and pygame (and those installations might not work on some systems)

KMattis commented 4 years ago

Another thing: tests.py uses the python built-in unittests library. This library has text output. If one test fails, I don't know if this will be noticed by the continuous integration task. In their example, they are using the pyunit library, because it will return an error code to the continuous integration environment, if one test fails.

So maybe we should switch to this library.