Skip to content

Conversation

@WangGLJoseph
Copy link
Collaborator

@WangGLJoseph WangGLJoseph commented Mar 11, 2025

  • Configured PyTest Testing Framework for Unit Testing
  • Added Tox + PyTest + Coverage.py for Test Coverage Report generation (required by SonarQube Scan for Coverage Report)
  • Configured SonarQube Project settings and SonarQube Scan workflow
  • Configured Docker Build + Run container workflow

@sonarqubecloud
Copy link

Copy link
Collaborator

@Marti2203 Marti2203 left a 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

@Marti2203 Marti2203 merged commit c1e99be into main Mar 11, 2025
6 checks passed
@Marti2203 Marti2203 deleted the pytest-ci branch March 11, 2025 13:56
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants