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

Set up end-to-end / integration testing via GitHub actions #2

Closed
oneillkza opened this issue Jan 18, 2023 · 3 comments · Fixed by #3
Closed

Set up end-to-end / integration testing via GitHub actions #2

oneillkza opened this issue Jan 18, 2023 · 3 comments · Fixed by #3
Assignees

Comments

@oneillkza
Copy link
Contributor

We should set up automated testing by running the complete workflow on a mini-example (including adding some small example data to the repo).

@Glenn032787
Copy link
Collaborator

I included a small test file in github_action branch which is generated from chr22 ASE simulated data and also randomly generated CNV and DMR data. I also included the github action yaml file which runs the workflow, however, github actions prevents it from using the --use-singularity command with snakemake when using premade snakemake github action (mentioned in this issue). I will try to either manually install snakemake into github runner or run unit tests.

@oneillkza
Copy link
Contributor Author

Great!

And yeah, manually installing, either directly through apt, or via conda as per this comment, will be the way to go for running snakemake.

When I've run Travis in the past for this kind of testing, it allowed me to just run a few install commands before the test itself.

@Glenn032787
Copy link
Collaborator

Finally got github actions to run through with no issues! I have created a pull request #3 for adding the small example and adding github actions workflow to the repository. If there is no issues I will merge it to main.

@oneillkza oneillkza linked a pull request Jan 25, 2023 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants