clockworklabs / clockworklabs/SpacetimeDB
Smoketests: remove interface for using a module source string; always require a precompiled module
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Rust
- Estrelas
- 25.2k
- Forks
- 1.1k
- Merge médio
- 2d 7h
- PRs com merge (30d)
- 46
Descrição
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.
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece localizando o construtor de Smoketest que aceita uma string de origem e todos os smoketests que o utilizam. Revise os casos existentes de falha de compilação e, em seguida, confirme que eles usam um build explícito ou uma abstração separada e que todos os Smoketest restantes usam um módulo pré-compilado; o construtor não deve mais aceitar strings de origem.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- rust
- Domínio
- testing-qa
- Tipo de issue
- Refatoração
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Status de atividade
- Pouca atividade
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 55/100