censoredplanet / censoredplanet-analysis

Analysis of the CensoredPlanet data.
Apache License 2.0
14 stars 5 forks source link

merge satellite e2e tests together #175

Closed ohnorobo closed 2 years ago

ohnorobo commented 2 years ago

Testing improvement

Historically some of the satellite e2e tests were split up into v1, v2p1 and v2p2 runs. This was to help me when I was making changes to the parsing for the different satellite versions. That code is now pretty stable and we're not likely to break parsing for just one version at a time.

This converts those tests to merge the v2p1 and v2p2 runs together as well as merging some individual tests. This will help us test tricky potential changes involving per-source beam runs since now the tests will involve a single run containing data from multiple source dates (v2p1 and v2p2).

This should also make the e2e tests run somewhat faster since we're merging some startup costs.

Tested: e2e tests passed