Skip to content

Refactor tests - #395

Merged
ghtmtt merged 3 commits into
ghtmtt:masterfrom
dmarteau:migrate-to-pytest-qgis
Jan 9, 2026
Merged

Refactor tests#395
ghtmtt merged 3 commits into
ghtmtt:masterfrom
dmarteau:migrate-to-pytest-qgis

Conversation

@dmarteau

@dmarteau dmarteau commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

Refactor tests:

  • Use modified pytest-qgis package for leveraging Qgis tests
  • Refactor tests to use pytest fixtures
  • Add tests coverage

Handle dependency groups

  • Handle dependency groups in pyproject.toml
  • Use uv for locking dependency versions and export them as requirements.txt format

@dmarteau
dmarteau force-pushed the migrate-to-pytest-qgis branch 2 times, most recently from 443a393 to ec8d2eb Compare January 6, 2026 23:18
@dmarteau
dmarteau marked this pull request as draft January 6, 2026 23:19
@dmarteau
dmarteau force-pushed the migrate-to-pytest-qgis branch 3 times, most recently from ea4916b to cd3b473 Compare January 7, 2026 10:44
@dmarteau

dmarteau commented Jan 7, 2026

Copy link
Copy Markdown
Contributor Author

@ghtmtt This PR fix all tests

@dmarteau
dmarteau force-pushed the migrate-to-pytest-qgis branch 2 times, most recently from 6b5b140 to 8ae17a5 Compare January 7, 2026 11:22
@dmarteau
dmarteau force-pushed the migrate-to-pytest-qgis branch from 8ae17a5 to 412d136 Compare January 7, 2026 11:24
@ghtmtt

ghtmtt commented Jan 9, 2026

Copy link
Copy Markdown
Owner

@dmarteau AWESOME! Many many thanks!

@ghtmtt
ghtmtt marked this pull request as ready for review January 9, 2026 13:01
@ghtmtt
ghtmtt merged commit 767e0c7 into ghtmtt:master Jan 9, 2026
3 checks passed
@ghtmtt

ghtmtt commented Jan 9, 2026

Copy link
Copy Markdown
Owner

@dmarteau Transifex test is failing: https://github.com/ghtmtt/DataPlotly/actions/runs/20852748697. Is it difficult to fix?

@dmarteau

dmarteau commented Jan 13, 2026

Copy link
Copy Markdown
Contributor Author

@dmarteau Transifex test is failing: https://github.com/ghtmtt/DataPlotly/actions/runs/20852748697. Is it difficult to fix?

Frankly, I don't know, but it seems that it is using an old version of qgis-plugin-ci which use deprecated toml dependencies: see qgis/qgis-plugin-ci#339

@ghtmtt

ghtmtt commented Jan 20, 2026

Copy link
Copy Markdown
Owner

will have a look, thanks!

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