clockworklabs / clockworklabs/SpacetimeDB

Smoketests: remove interface for using a module source string; always require a precompiled module

オープン
#5,653 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Rust
スター
25.2k
フォーク
1.1k
平均マージ
2日 7時間
マージ済み PR(30日)
46

説明

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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、ソース文字列を受け取る Smoketest コンストラクターと、それを使用しているすべての smoketest を見つけます。既存のコンパイル失敗ケースを確認し、それらが明示的なビルドまたは別の抽象化を使用していること、そして残りのすべての Smoketest がプリコンパイル済みモジュールを使用していることを確認します。コンストラクターはソース文字列を受け取らないようにする必要があります。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
rust
領域
testing-qa
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
55/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。