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.

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

Add testing files for the pipeline #14

Open
bhillmann opened this issue Nov 18, 2016 · 5 comments
Open

Add testing files for the pipeline #14

bhillmann opened this issue Nov 18, 2016 · 5 comments
Assignees

Comments

@bhillmann
Copy link
Collaborator

Currently, we have no defined way to test the software. This is bad practice.

@bhillmann bhillmann self-assigned this Nov 18, 2016
@GabeAl
Copy link
Collaborator

GabeAl commented Jan 11, 2017

We now have some test cases scattered around, but it's not in the repo (so yes, bad practice indeed).

@kaiweiang , would you mind uploading all the test files and commandlines you've accumulated over time?

@bhillmann
Copy link
Collaborator Author

I was thinking to eventually add unit test cases. Any suggestions for this software? I know there are several options in Python and we could emulate open source best practices. Any interest in learning a continuous integration suite?

For now, it might be simplest to include some small FASTQ files to the repository to check the pipeline.

@GabeAl
Copy link
Collaborator

GabeAl commented Jan 11, 2017

Good idea. I think this is fairly essential as we expand the number of possible run configurations. The more combinations of options possible, the more likely something subtle will go wrong, and unit testing becomes more relevant. Yes.

For now (1.0), some test files are probably enough, and we can verify they're working with the common commandlines.

@GabeAl
Copy link
Collaborator

GabeAl commented Jan 27, 2017

Added test files. Still need tests for gotta_split though!

@bhillmann
Copy link
Collaborator Author

Preliminary testing files have been added for other files. @GabeAl Can you make the gotta_split ones?

@bhillmann bhillmann reopened this Feb 23, 2017
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

No branches or pull requests

3 participants