Smoketests which use precompiled modules have better performance characteristics than those that use modules with source strings (mostly because our CI can be smarter about them), so we want to require that all smoketests use precompiled modules. Currently, we have a few smoketests which use non-precompiled modules specifically because they assert that the compilation will fail. We should rewrite those tests so that they either call spacetime build explicitly or use a separate, purpose-built abstraction, rather than calling a normal Smoketest constructor and asserting that it fails. Then, we should remove the Smoketest constructor that accepts a source string, so that all smoketests have to use precompiled modules.
Smoketests which use precompiled modules have better performance characteristics than those that use modules with source strings (mostly because our CI can be smarter about them), so we want to require that all smoketests use precompiled modules. Currently, we have a few smoketests which use non-precompiled modules specifically because they assert that the compilation will fail. We should rewrite those tests so that they either call
spacetime buildexplicitly or use a separate, purpose-built abstraction, rather than calling a normalSmoketestconstructor and asserting that it fails. Then, we should remove theSmoketestconstructor that accepts a source string, so that all smoketests have to use precompiled modules.