[tests]: are "rda file made when indicated" tests needed for every plot?
メンテナーはふだん 1 日以内に返信
評価
この issue はまだ評価されていません。
説明
For every plot, there's an associated "rda file made when indicated" test that runs a simple version of the plotting function with make_rda=TRUE specified, and then checks whether the associated ".rda" and ".csv" file are made, before deleting them. Given that all of these tests are identical (minus the exact plotting function call), would it make more sense to remove all of these tests, and instead just test the
export_kqs(...)
insert_kqs(...)
create_rda(...)
workflow, once? This eliminates a large number of largely redundant tests, and would alleviate many I/O issues associated with moving to a parallel testing framework.
The case for retaining all of the plot specific "rda file made when indicated" tests would be to verify that the feature was correctly implemented for the specific plot, but this check only really needs to be conducted once, rather than every time the test suite is executed. Maybe this is an automated check that could be done before a PR is approved?
- 主要言語
- R
- スター
- 20
- フォーク
- 9
- 平均マージ
- 1日 5時間
- マージ済み PR（30日）
- 12
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nmfs-ost/stockplotr のほかの issue
-
[tests]: Set modules in all plot_* calls within test functions対応中かも @Schiano-NOAA が 3 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
nmfs-ost/stockplotr#416 · コメント 5 件 · リアクション 1 件 · 担当者 2 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
nmfs-ost/stockplotr#422 ·
メンテナーはふだん 1 日以内に返信
-
[fix]: module name selection retained in environment when `make_rda = T`対応中かも @sbreitbart-NOAA が今日担当しました。 オープンbug
nmfs-ost/stockplotr#421 · コメント 5 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Make tests more efficient対応中かも @JZahner1 が 1 日前に担当しました。 オープン
nmfs-ost/stockplotr#420 · コメント 4 件 · 担当者 2 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 40/100
nmfs-ost/stockplotr#415 ·
メンテナーはふだん 1 日以内に返信
nmfs-ost/stockplotr の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
stan-dev/priorsense#150 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
bennettoxford/opencodecounts#170 ·
-
complexity::1-low effort::1-hours priority::3-could
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
jumpingrivers/audit.workbench#43 ·
メンテナーはふだん 1 日以内に返信
-
complexity::1-low effort::1-hours priority::3-could
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
jumpingrivers/audit.connect#71 ·
メンテナーはふだん 1 日以内に返信