clockworklabs / clockworklabs/SpacetimeDB
Smoketests: remove interface for using a module source string; always require a precompiled module
Personne n'a encore pris cette issue.
- Langage dominant
- Rust
- Étoiles
- 25.2k
- Forks
- 1.1k
- Merge moyen
- 2 j 7 h
- PR mergées (30 j)
- 46
Description
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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser le constructeur de Smoketest qui accepte une chaîne source ainsi que tous les smoketests qui l’utilisent. Examinez les cas existants d’échec de compilation, puis confirmez qu’ils utilisent un build explicite ou une abstraction distincte et que chaque Smoketest restant utilise un module précompilé ; le constructeur ne devrait plus accepter de chaînes source.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- rust
- Domaine
- testing-qa
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 55/100