-
Notifications
You must be signed in to change notification settings - Fork 0
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
test: Creation of a test suite #85
Open
raaijmag
wants to merge
12
commits into
2.0.0
Choose a base branch
from
test-suite
base: 2.0.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
style: formatting with isort & black
…hes--main--components--AmpliGone chore(main): release 1.3.1
chore: sync main to dev
test: tests/unit/ - added unit test for argument parsing, cutlery and fasta2bed test: tests/data/ - added files to GitHub LFS for testing purposes fix: .gitignore - added *:Zone.Identifier fix: improved static type checking, linting, and formatting for all files
feat: test-requirements- added seperate test reqs feat: .github.workflows.tests - created automatic testing workflow chore: __main__ - bandit B101 says that assert should not be used directly, because they are removed when compiling. chore: alignmentpreset - removed asserts (bandit B101) chore: args.py - removed asserts (bandit B101), and trailing whitespace chore: cut_reads - mypy didnt get the ignore comment chore: fasta2bed - no need to return chore: reports/ - linting reports for sonarcloud chore: test_e2e - bandit B101 chore: test_args - bandit B101 chore: test_cutlery - bandit B101 chore: test_fasta2bed - bandit B101 chore: .gitignore - removed coverage.xml so it goed in the reports folder
…a2bed. test: finished e2e tests
There are some e2e tests that fail. While I did not find a solution for them yet, I am quite sure that they are not faulty tests but actual issues. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Major Update: Complete Test Suite Integration
This pull request introduces a comprehensive test suite, alongside minor improvements in the AmpliGone package to enhance static analysis and ease of testing. Below is a summary of the major new features included:
New GitHub Actions Workflow for Static Analysis and Testing
This workflow automates code quality checks and testing through the following steps:
Unit Tests for Core Modules
Detailed unit tests have been implemented for the following modules:
args.py
cut_reads
cutlery
fasta2bed
End-to-End (E2E) Testing Framework
A modular and easily extendable framework to simulate real-world scenarios, currently covering:
.fasta
and.bed
primer files.