Define policy for tests
@folkyatina がすでに取り組んでいます。
2022年6月16日 から。
評価
この issue はまだ評価されていません。
説明
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
- 主要言語
- Python
- スター
- 9
- フォーク
- 3
- 平均マージ
- 5日 53分
- マージ済み PR（30日）
- 4
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
lidofinance/scripts のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
lidofinance/scripts#265 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
lidofinance/scripts#96 ·
lidofinance/scripts の issue をすべて見る
似ている issue
-
[oblt-aw][autodoc] Update docs for security org scaffolding and Docs AI menu permission changes対応中かも @github-actions が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
external integration integration-submission
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
langchain-ai/docs#7077 ·
メンテナーはふだん 1 日以内に返信
-
author-page correction
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
acl-org/acl-anthology#10623 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Use context to recognize personal initials and lowercase names対応中かも @lipikaramaswamy が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
NVIDIA-NeMo/Anonymizer#340 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
btclib-org/btclib-node#2083 ·
メンテナーはふだん 1 日以内に返信