Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use paths rather than config map for test data #2858

Open
adamrtalbot opened this issue Mar 18, 2024 · 0 comments · May be fixed by #2877
Open

Use paths rather than config map for test data #2858

adamrtalbot opened this issue Mar 18, 2024 · 0 comments · May be fixed by #2877
Labels
enhancement

Comments

@adamrtalbot
Copy link
Contributor

Description of feature

If we use a path instead of a config map for test data, our tests are more portable and less dependent on many external files. Simpler = more robust. It's already in action in many places, e.g. UNTAR
adamrtalbot added a commit to adamrtalbot/tools that referenced this issue Mar 19, 2024
@adamrtalbot
Module test data uses paths rather than config map
0e9640f 
This makes tests reusable in pipeline repos

Closes nf-core#2858
@adamrtalbot adamrtalbot linked a pull request Mar 19, 2024 that will close this issue
Test data uses paths instead of config map #2877
Open
4 tasks
@adamrtalbot adamrtalbot linked a pull request Mar 19, 2024 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Test data uses paths instead of config map adamrtalbot/tools
1 participant
@adamrtalbot