Style & Tests

See merge request !1

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

This source should be more reproducable, than the deb package
repositories from nodesource.com.
In addition we avoid sourcing a shell script from an untrusted location.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

ci: avoid hard-coded references to "master" branch

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

The import path changed from "." to "src" at the same time.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

Previously "PlayLog()" and "HealthLog()" were used as input for the POST
request's body.
But the JSON serialization of both instances is empty (for unknown
reasons).  Thus the POST requests failed, due to missing required fields.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

The previous code intended to silence the most relevant source of log noise
("connexion.operations"), but due to the missing trailing "s", it failed
to do so.
Now all debug messages emitted during a test are silenced.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

Somewhen two years ago the tests were disabled in the CI.
Since then the test were failing.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

The environment should be configured outside of the application.

Lars Kruse authored 3 years ago and David Trattnig committed 2 years ago

* `/usr/bin/env` is not necessary: `python3` is just the same
  (`/usr/bin/env python3` is helpful in a shebang, since an absolute
  path is required there)
* use the variable `$PYTHON_EXEC`: it was introduced recently, but not
  used for now

- When using the docker setup, the image now expects to get all config variables as environment variables