-
Notifications
You must be signed in to change notification settings - Fork 30
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
Analysisd - add new test to check the pre-decoding stage of analysisd #2406
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@danisan90 Please, modify the code following the PEP8 guidelines. You can follow this guide to doing it, take into account that you must change the max-line-length
argument following this guide
Description
Closes #1498
This PR adds a new test for Analysisd requested on the issue #1498 . To increase the integration test coverage, the test
test_predecoder_stage.py
was added for the pre-decoding stage of analysisd.This test check:
Tests
pycodestyle --max-line-length=120 --show-source --show-pep8 file.py
.provision_documentation.sh
generate the docs without errors.