Adjust Test Case for Very Long Sequences #128
Merged
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.
Pull Request: Adjust Test Case for Very Long Sequences
Description
This pull request addresses the issue with the test case for very long sequences in the
test_alphamissense_integration
function. The test was failing because it did not correctly expect aValueError
with the specific message for sequences longer than 1000 amino acids.Changes Made
test_alphafold_integration.py
to expect theValueError
with the message "Sequence is too long. Please provide a sequence with at most 1000 amino acids."Impact
These changes ensure that the test suite accurately reflects the validation logic in the
run_alphamissense_analysis
function, improving the robustness of the test suite.Next Steps
Please review the changes and provide feedback. Once approved, the changes can be merged into the main branch.
Related Issues
test_alphamissense_integration
.This Devin run was requested by kasinadhsarma.
If you have any feedback, you can leave comments in the PR and I'll address them in the app!