makhidkarun / traveller_pyroute

Traveller trade route generator
MIT License
14 stars 5 forks source link

Add TradeMPCalculation smoke test to CI job #106

Closed CyberiaResurrection closed 8 months ago

CyberiaResurrection commented 8 months ago

The multi-process version of trade routes, TradeMPCalculation, is rather difficult to test en suite, so almost inevitably it got bent out of shape without anyone noticing.

To at least make that bending-out-of-shape be visible, I've added a small smoke-test TradeMP run to the tail end of the CI job. Said run is over two sectors (Zdiedeiant and Stiatlchepr) with a just-low-enough min BTN (18) to run most of the TradeMP specific gubbins that are difficult to test en suite - the multiprocessing and the interprocess communication.

This is in no way shape or form a guarantee that future TradeMP breakage will always be noticed, but an attempt to constrain the unnoticed effects of such breakage.