clockworklabs / clockworklabs/SpacetimeDB
Smoketests: remove interface for using a module source string; always require a precompiled module
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Rust
- Star
- 25.2k
- Fork
- 1.1k
- Merge trung bình
- 2 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 46
Mô tả
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.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách xác định constructor của Smoketest nhận một chuỗi nguồn và tất cả các smoketest sử dụng nó. Xem xét các trường hợp lỗi biên dịch hiện có, sau đó xác nhận rằng chúng sử dụng một build rõ ràng hoặc một abstraction riêng biệt, và rằng mọi Smoketest còn lại đều sử dụng một module được biên dịch trước; constructor không còn nên chấp nhận các chuỗi nguồn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- rust
- Lĩnh vực
- testing-qa
- Loại issue
- Tái cấu trúc
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 55/100