Race Conditions when Tests run in Parallel for Functions with File I/O Capabailities
メンテナーはふだん 2 日以内に返信
@JZahner1 がすでに取り組んでいます。
2026年11月2日 から。
評価
この issue はまだ評価されていません。
説明
the addition of the "extract_sis_data" tests results in a race condition with the "save_all_plots" test, due to coincident file writing and deletion to the same directory. How would you like to handle this? One way is to merge the two sets of tests, since testthat/parallel runs test files in parallel rather than functions. This feels like a bad way to fix this issue. Another way would be to this would be have each test write figures and temp file to a custom directory (e.g., test-save_all_plots.R writes to tests/testthat/test-save_all_plots/figures). This is more extensible, but requires adding functionality to export_kqs that allows for writing those files to custom directories rather than getwd().
I have a tested/verified version of option 2 here: https://github.com/JZahner1/stockplotr/tree/test-race-conditions
Originally posted by @JZahner1 in https://github.com/nmfs-ost/stockplotr/issues/433#issuecomment-6369873897
- 主要言語
- R
- スター
- 20
- フォーク
- 10
- 平均マージ
- 4日 48分
- マージ済み PR（30日）
- 25
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nmfs-ost/stockplotr のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
nmfs-ost/stockplotr#457 ·
メンテナーはふだん 2 日以内に返信
-
Check for obs and exp. for each example data in the package対応中かも @Schiano-NOAA が 1 日前に担当しました。 オープンquestion
nmfs-ost/stockplotr#452 · 担当者 1 名 ·
メンテナーはふだん 2 日以内に返信
-
Add make_rda and rda bundle to composition plots対応中かも @Schiano-NOAA が 1 日前に担当しました。 オープンdevelopment enhancement
nmfs-ost/stockplotr#451 · 担当者 1 名 ·
メンテナーはふだん 2 日以内に返信
-
Release Checklist - Monthly Update対応中かも @Schiano-NOAA が 6 日前に担当しました。 オープン
nmfs-ost/stockplotr#442 · 担当者 2 名 ·
メンテナーはふだん 2 日以内に返信
-
Brainstorm ways to automatically update `save_all_plots()`対応中かも @sbreitbart-NOAA が 14 日前に担当しました。 オープン
nmfs-ost/stockplotr#432 · コメント 4 件 · 担当者 2 名 ·
メンテナーはふだん 2 日以内に返信
nmfs-ost/stockplotr の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
epiverse-trace/epiparameter#526 ·
-
難易度 1/5 1〜3時間 初心者へのやさしさ 68/100
eirini-zormpa/hpc-intro#30 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
REditorSupport/languageserver#833 ·
メンテナーはふだん 1 日以内に返信