mriqc icon indicating copy to clipboard operation
mriqc copied to clipboard

ENH: Add test for the CLI parser

Open jhlegarreta opened this issue 2 months ago • 1 comments

Add test for the CLI parser.

jhlegarreta avatar Apr 27 '24 16:04 jhlegarreta

Leaving the groups for a separate PR: https://github.com/nipreps/mriqc/blob/master/mriqc/cli/parser.py#L166

jhlegarreta avatar Apr 27 '24 16:04 jhlegarreta

Very much needed -- thanks for initiating this. Let me know how I can help.

oesteban avatar Apr 30 '24 10:04 oesteban

Very much needed -- thanks for initiating this. Let me know how I can help.

Thanks Óscar. The CircleCI test errors seem unrelated: https://app.circleci.com/pipelines/github/nipreps/mriqc/1353/workflows/ae93f4b7-94c6-492c-9204-455abfcdd103/jobs/8173?invite=true#step-103-264_18 https://app.circleci.com/pipelines/github/nipreps/mriqc/1353/workflows/ae93f4b7-94c6-492c-9204-455abfcdd103/jobs/8174?invite=true#step-103-263_18

I think I had tried locally with success.

But is there a mechanism in place to fetch some testing data from Python? I see that a step in CircleCI retrieves some data from a URL

jhlegarreta avatar Apr 30 '24 13:04 jhlegarreta

Thanks for the review and suggestions Óscar.

Created a helper _create_set_bids_dir as the pattern was being repeated.

Squashed commits.

Push forced. Not sure about the verbosity one. Will see in the CI.

jhlegarreta avatar May 01 '24 22:05 jhlegarreta