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

[tests]: are "rda file made when indicated" tests needed for every plot?

Đã đóng
  • Có gì không ổn?
#418 10 bình luận 0 reaction 3 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@Schiano-NOAA đang làm issue này rồi.

Từ ngày 14/10/2026.

  • #425 của @JZahner1 — đang mở

Đánh giá

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

Mô tả

question

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?

Ngôn ngữ chính
R
Star
20
Fork
10
Merge trung bình
3 ngày 6 giờ
Pull request đã merge (30 ngày)
17

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

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

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 nmfs-ost/stockplotr

Tất cả issue của nmfs-ost/stockplotr

Issue tương tự

Thêm issue về R

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.