Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Define policy for tests

Đang mở
  • Có gì không ổn?
#77 2 bình luận 0 reaction 2 người được giao Xem trên GitHub

@folkyatina đang làm issue này rồi.

Từ ngày 16/6/2022.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

Simple summary

We should discriminate omnibus votes into two categories: sensitive protocol upgrades & others

  • The First category requires unit, regression, snapshot, and integration test suites.
  • For the second category, it's possible to skip the snapshot tests for the sake of reducing process overheads

Motivation

We have adopted GitHub CI process recently for the repo. While it is always good to achieve a bright green mark for every pushed head by running as many tests as possible, it also requires additional effort to maintain the repo consistency.

Snapshot tests usually represent the following invariant: 'all states should be left as is except only X', thus you have to define 'X' properly for each voting script. If you have multiple voting scripts then complexity rises drastically.
At the same time, other types of tests usually check only the new state and have no assumptions for the diff items over all state slots and their values.

Proposal

Option 1. Mark protocol upgrade votings differently (naive and dirty)

For e.g., we may use upvote_xx_xx_xx.py filename pattern which should be followed by the tests runner as a flag to run additional tests.

Option 2. Bake a framework for the snapshot tests

Following the approach, one will need to import the desired snapshot tests and define the expected state diffs for them.

Decision (TBA)

Personally, I believe that option 2 should be implemented.
Other opinions and options are highly appreciated here.

FYI: @folkyatina, @ujenjt

Ngôn ngữ chính
Python
Star
8
Fork
3
Merge trung bình
2 ngày 18 giờ
Pull request đã merge (30 ngày)
6

Chuẩn bị môi trường

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của lidofinance/scripts

Tất cả issue của lidofinance/scripts

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.