EMAworkbench icon indicating copy to clipboard operation
EMAworkbench copied to clipboard

Enable connector tests in CI

Open EwoutH opened this issue 2 years ago • 1 comments

Currently the connector tests (in test/test_connectors/) are disabled in the CI. I think it would be useful to at least test them periodically, to see if the upstream changes don't break anything.

Currently all the connector tests are disabled, it's also possible to enable then one-by-one instead of all at the same time.

Enable connector tests:

EwoutH avatar Jun 13 '22 08:06 EwoutH

You can add test_simio to this list, even if no test for it yet exist.

The bigger question is how we want to approach it. It seems some form of mocking will be required, at least for Vensim, NetLogo and Simio. We could test against pysd directly by including it as a dependency, and likewise with pynetlogy and jpype, but I think that having a standard test setup for connectors is more sensible going forward.

quaquel avatar Jun 13 '22 18:06 quaquel