-
Notifications
You must be signed in to change notification settings - Fork 333
Configure PyTest and SonarQube Scan #84
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
Merged
Conversation
This file contains hidden or 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
|
Marti2203
approved these changes
Mar 11, 2025
Collaborator
Marti2203
left a comment
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.
Looks good to me. I have some suggestions about additional tests but will send separately
Yocelinn
pushed a commit
to Yocelinn/auto-code-rover
that referenced
this pull request
Apr 14, 2025
* add PyTest framework * update pytest yml * update CI * activate workflow * fix branch name * add missing checkout * add correct docker command * add sonar source files * output converage to both term and xml * use acr-pytest for container name * Update pytest.yml * Update sonar-project.properties * sonar fix and update sonar exclusions * add coverage file path * reduce code duplication * add temp dir in workflow for coverage report * update sonar config * use tox and pytest outside container * add missing tox.ini file * separate pytest and build workflow * fix conda env name in workflow * update shell to use login mode for pytest workflow * update conda config * set PYTHONPATH inside tox.ini * update tox.ini * include Coverage.py in tox * add passenv to tox * change build yaml to only run on PR merge * remove redundant deps in tox.ini * abort Sonarqube Scan if coverage.xml is missing * specify source and test files for sonar * add test_search_utils --------- Co-authored-by: Martin Mirchev <martin.mirchev@sonarsource.com>
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.



Uh oh!
There was an error while loading. Please reload this page.