clockworklabs / clockworklabs/SpacetimeDB
Smoketests: remove interface for using a module source string; always require a precompiled module
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Rust
- Estrellas
- 25.2k
- Forks
- 1.1k
- Merge medio
- 2 d 7 h
- PR fusionados (30 d)
- 46
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza localizando el constructor de Smoketest que acepta una cadena de origen y todos los smoketests que lo utilizan. Revisa los casos existentes de fallos de compilación y confirma después que utilizan una compilación explícita o una abstracción separada, y que todos los Smoketest restantes utilizan un módulo precompilado; el constructor ya no debería aceptar cadenas de origen.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust
- Área
- testing-qa
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 55/100