Upgrade to testthat 3e
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- r
- 領域
- testing-qa
調査の方向性
Start with tests/testthat/test-utils.R and tests/testthat/test-vendor.R, then inspect the other tests using describe(..., it(...)) style. Migrate the tests to testthat 3e and remove the {mockery} dependency; done means the test suite passes without that dependency.
索引モデルが issue の本文から書いたものです。
説明
I was aiming to send along a PR dropping the {mockery} dependency but it's a bit of a bigger project than at first glance despite the small footprint of {mockery}:
https://github.com/r-lib/cpp11/blob/dc93fb84f482c2ac93b5578fee955c75a40b093f/tests/testthat/test-utils.R#L28
https://github.com/r-lib/cpp11/blob/dc93fb84f482c2ac93b5578fee955c75a40b093f/tests/testthat/test-vendor.R#L4
Because currently {cpp11} uses a lot of describe(..., it(...)) style tests that require more lift to move to 3e.
Filing this issue as an earmark that it would be something nice to have.
- 主要言語
- C++
- スター
- 225
- フォーク
- 52
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
r-lib/cpp11 のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
似ている issue
-
Feature
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
Narezzurri/OpenVPN-Config-Manager#95 ·
メンテナーはふだん 1 日以内に返信
-
bot-found bug priority: P1
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
madenvel/KalinkaPlayer#251 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sqlitebrowser/sqlitebrowser#4208 ·
-
ROSES ROSES - Student Review
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
area/ysql kind/bug priority/medium
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
yugabyte/yugabyte-db#34552 ·
メンテナーはふだん 1 日以内に返信