Skip to content

Add resume option#339

Open
LouisLeNezet wants to merge 7 commits intoaskimed:mainfrom
LouisLeNezet:test_resume
Open

Add resume option#339
LouisLeNezet wants to merge 7 commits intoaskimed:mainfrom
LouisLeNezet:test_resume

Conversation

@LouisLeNezet
Copy link
Copy Markdown

This is a experimental PR to answer the following problematic:

When running test we often want to rerun the test without having to download back all the input data.
Sometimes we also just want to rerun the nf-test logic and not all the nextflow processing.

This PR introduce a new flag, that would have the same concept as the --resume of nextflow.
The aim is to not reset the test folder, and let nextflow do it's resume job by adding this flag to the execution.

I would like to go even further by putting all the input into a separate folder to allow it to be shared across test.
It is often the case that different test case use the same input file but with different configuration. This would then allow to reduce the memory used.

What do you think ?

@nttg8100
Copy link
Copy Markdown

Hi @LouisLeNezet , it is what I also think about. Can you add the workflow.nf.test with the new compiled nf-test so it can be verify directly instead of testing only on the java abstraction ?

@LouisLeNezet
Copy link
Copy Markdown
Author

Hi @nttg8100,

I've added a test and made sure that the workflow was indeed resumed.
This does seems to work.

@nttg8100
Copy link
Copy Markdown

Hi @nttg8100,

I've added a test and made sure that the workflow was indeed resumed. This does seems to work.

I tested it locally and found no error. I think the error may come from the specific version.

@LouisLeNezet
Copy link
Copy Markdown
Author

I've added the missing params.file initialisation for the NextflowCommandTest.

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