Update to nf-test #436
Open
Update to nf-test #436
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changes: - Moves pipeline level test to the first section, in order to introduce the concept of testing early with a simple example. - Moves other tests lower, in order to increase the complexity of the tests as we go.
✅ Deploy Preview for nextflow-training ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
@adamrtalbot I like moving the pipeline tests to the beginning. Looks like this needs some updating based on the other changes, though. |
I'm thinking I'm just going to re-write the whole darned thing with a slightly different flow to make it clearer what the purpose of testing really is. We could include things like deliberately breaking a module to demonstrate an error. I've started a planning doc and shared it with you, we can dump a draft here once I've got he major points fleshed out. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates to nf-test which should help clarity
Changes:
Note I think we could go further, but these are quick and relatively simple changes.