search

openmethane / setup-wrf

Setup the WRF model
0 stars 0 forks source link

Testing ideas #10

Open lewisjared opened 1 month ago

lewisjared commented 1 month ago

The problem

We want to add some additional tests to verify that the system is operating as expected. This will have to happen after #6 as it is already complex enough.

Add to the list below as more things are identified

Things to test

Additional context

Currently we execute the docker container multiple times which makes it hard to test what happens. We should instead run pytest inside the container to control what is executed.

We should also split the testing into 2 separate checks. 1st one runs unit tests and the 2nd runs a full integration test which might take a bit longer

lewisjared commented 3 weeks ago
lewisjared commented 3 weeks ago

@crdanielbusch The tests aren't run in the CI #19 #17.

Can you figure out how to add them in? It might require some reshuffling or running as a second job ideally using the docker container because it already has the required dependencies installed.

In a perfect world it would go: build docker container > run tests (unit and integration in parallel) > publish image. That might be a bit complex for now so if there is a second test job which is dependent on the build step that is fine