Skip to content

Skip test if radar has not recently been updated - #113

Merged
bart1 merged 2 commits into
mainfrom
skip_test_se_404
Aug 20, 2025
Merged

bart1 merged 2 commits into
mainfrom
skip_test_se_404

Conversation

@bart1

@bart1 bart1 commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator

Fixes #112 for se

@bart1
bart1 requested review from PietrH and peterdesmet August 20, 2025 09:26
@codecov

codecov Bot commented Aug 20, 2025

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@bart1 bart1 mentioned this pull request Aug 20, 2025
@bart1

bart1 commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator Author

@PietrH

PietrH commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

Is there any way we could wrap this into a custom skip function? skip_if_radar_not_updated() ? Or if not, skip_if_se_not_updated() ?

Comment thread tests/testthat/test-get_pvol_se.R Resolved
Comment thread tests/testthat/test-get_pvol_se.R Resolved
Comment thread tests/testthat/test-get_pvol_se.R Resolved
@bart1
bart1 requested a review from PietrH August 20, 2025 12:12
@bart1
bart1 merged commit 1262edd into main Aug 20, 2025
@bart1
bart1 deleted the skip_test_se_404 branch August 20, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sometimes test fail for a specific radar not being available

2 participants