diff --git a/PanACoTA/subcommands/prepare.py b/PanACoTA/subcommands/prepare.py index 9f4d221b3115bf42cec79b2e2ff849fd94636939..c791c6ccd5cd5b18843a306132f927c178eb1c0c 100644 --- a/PanACoTA/subcommands/prepare.py +++ b/PanACoTA/subcommands/prepare.py @@ -476,8 +476,9 @@ def check_args(parser, args): # WARNINGS # User did not specify a species name - if not args.NCBI_species: - print(colored("WARNING: you did not provide a species name ('-s species' option'). " + if not args.NCBI_species and not args.outdir: + print(colored("WARNING: you did not provide a species name ('-s species' option') " + "nor an output directory ('-o outdir'). " "All files will be downloaded in a folder called with the NCBI species " f"taxid {args.NCBI_species_taxid} instead of the species name.", "yellow")) # If user wants to cut genomes, warn him to check that it is on purpose (because default is cut at each 5'N') diff --git a/test/test_functional/test_prepare-parser.py b/test/test_functional/test_prepare-parser.py index b05717101897e61ef17b9d980e29b6751b417bf3..58d11329952592862955b5e3a6fdd34fe91e395b 100644 --- a/test/test_functional/test_prepare-parser.py +++ b/test/test_functional/test_prepare-parser.py @@ -336,13 +336,14 @@ def test_parser_nospecies(capsys): """ parser = argparse.ArgumentParser(description="Prepare", add_help=False) prepare.build_parser(parser) - options = prepare.parse(parser, "-t 1234 -o outdir".split()) + options = prepare.parse(parser, "-t 1234".split()) assert not options.no_refseq assert not options.only_mash assert options.NCBI_species_taxid == "1234" assert options.NCBI_species == "" out, err = capsys.readouterr() - assert ("WARNING: you did not provide a species name ('-s species' option'). " + assert ("WARNING: you did not provide a species name ('-s species' option') " + "nor an output directory ('-o outdir'). " "All files will be downloaded in a folder called with the NCBI species " "taxid 1234 instead of the species name.") in out