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.

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

update pipeline level nf-test documentation #3115

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

maxulysse
Copy link
Member

@maxulysse maxulysse commented Feb 26, 2025

@netlify /docs/guidelines/pipelines/recommendations/testing

Copy link

netlify bot commented Feb 26, 2025

Deploy Preview for nf-core-main-site ready!

Name Link
🔨 Latest commit a6c6f86
🔍 Latest deploy log https://app.netlify.com/sites/nf-core-main-site/deploys/67c03ea5fd03b800081ebbd0
😎 Deploy Preview https://deploy-preview-3115--nf-core-main-site.netlify.app/docs/guidelines/pipelines/recommendations/testing
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Feb 26, 2025

Deploy Preview for nf-core-docs ready!

Name Link
🔨 Latest commit a6c6f86
🔍 Latest deploy log https://app.netlify.com/sites/nf-core-docs/deploys/67c03ea55c9b3b000822f32e
😎 Deploy Preview https://deploy-preview-3115--nf-core-docs.netlify.app/docs/guidelines/pipelines/recommendations/testing
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.


- To ensure all output files are caught, the `params.outdir` should be set the the nf-test variable `outputDir`
- The tag `PIPELINE` and the pipeline name should be added to all tests
- To ensure all output files are caught, it is highly recommended to use the `nft-utils` plugin to parse the output of the pipeline.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide example code.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's the example that follows

`stable_path` capture all of the stable files in the output directory, ignoring the files listed in the `.nftignore` file.

Additional plugins can be used if files with unstable content can be read using a nf-test plugin.
For example for BAM and CRAM files the `nft-bam` plugin is used, while the `nft-vcf` plugin is also used, both in conjonction with `nft-utils`:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we move this section into the nf-test assertion examples (and split it up into component tests and pipeline tests?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that idea

Co-authored-by: Matthias Hörtenhuber <[email protected]>
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.

2 participants