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

[Feature] Add comprehensive sanity tests at the parser level #3

Closed
3 of 4 tasks
MaelLefeuvre opened this issue Mar 10, 2023 · 0 comments · Fixed by #14
Closed
3 of 4 tasks

[Feature] Add comprehensive sanity tests at the parser level #3

MaelLefeuvre opened this issue Mar 10, 2023 · 0 comments · Fixed by #14
Assignees
@MaelLefeuvre
Labels
enhancement New feature or request

Comments

@MaelLefeuvre
Copy link
Owner

MaelLefeuvre commented Mar 10, 2023 •
edited
Loading

Add a set of basic I/O sanity checks for inputs and outputs (i.e.: --bam, --misincorporation, --reference --output)

  • Check OS read permissions are set for all input files
  • Check OS write permissions are set for the directory of --output
  • Check the existence of input files
  • Check the existence of companion files (i.e. .fai files)
@MaelLefeuvre MaelLefeuvre self-assigned this Mar 10, 2023
@MaelLefeuvre MaelLefeuvre added the enhancement New feature or request label Mar 10, 2023
@MaelLefeuvre MaelLefeuvre linked a pull request Mar 12, 2023 that will close this issue
Add basic CI #5
Merged
@MaelLefeuvre MaelLefeuvre removed a link to a pull request Mar 12, 2023
Add basic CI #5
Merged
@MaelLefeuvre MaelLefeuvre linked a pull request Aug 8, 2023 that will close this issue
Bump to v0.3.1 - various bugfixes #14
Merged
@MaelLefeuvre MaelLefeuvre mentioned this issue Aug 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MaelLefeuvre MaelLefeuvre

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump to v0.3.1 - various bugfixes MaelLefeuvre/pmd-mask
1 participant
@MaelLefeuvre