Skip to content
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

Tests assume installed version of SPAdes #40

Closed
satta opened this issue Dec 7, 2015 · 1 comment
Closed

Tests assume installed version of SPAdes #40

satta opened this issue Dec 7, 2015 · 1 comment
Assignees
@martinghunt
Labels
enhancement

Comments

@satta
Copy link
Collaborator

satta commented Dec 7, 2015

Currently the nose tests seem to assume that SPAdes is used and available, which breaks the test runs on systems without it. It would be nice to have conditional testing of the assembly function for SPAdes and/or Velvet.
I would like to run the tests as part of a Debian package build, and Debian does not have SPAdes available to depend on (yet).
@martinghunt martinghunt self-assigned this Jan 15, 2016
@satta
Copy link
Collaborator Author

satta commented Apr 7, 2016

SPAdes is now available in Debian, so this is no longer an issue.

@satta satta closed this as completed Apr 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martinghunt martinghunt

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@satta @martinghunt