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.

Sign up for GitHub

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

Jump to bottom

write command-line test for `lca summarize --query-from-file` #1451

Open

ctb opened this issue Apr 11, 2021 · 0 comments

Labels

good first issue python testing

Contributor

ctb commented Apr 11, 2021

per https://github.com/dib-lab/sourmash/pull/1423/files#r604406992 as well as attached screenshot, we do not currently test sourmash lca summarize --query-from-file

To address this issue:

make a copy of test_single_summarize() in tests/test_lca.py and name it something like test_single_summarize_query_from_file();
modify the new test to create a text file containing the path to input_sig
modify the utils.runscript(...) command to remove --query and instead pass in --query-from-file with the new text file

The text was updated successfully, but these errors were encountered:

ctb added good first issue python testing labels

himanshu007-creator mentioned this issue

done 2/3 changes #1452

Closed

bryshalm assigned bryshalm and unassigned bryshalm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment