Skip to content

temporal: Remove needs_solo_run from t.vect.list and fix typos in t.vect.list & t.rast.list tests - #7636

Merged
echoix merged 4 commits into
OSGeo:mainfrom
saket0187:Fix-typo-and-remove-markers
Jul 9, 2026
Merged

echoix merged 4 commits into
OSGeo:mainfrom
saket0187:Fix-typo-and-remove-markers

Conversation

@saket0187

@saket0187 saket0187 commented Jul 4, 2026 •

Copy link
Copy Markdown
Contributor

Removed the @pytest.mark.needs_solo_run markers from the t.vect.list tests. I'm not sure why these were added before, but it seems to work perfectly fine without the restriction.

I also caught and fixed one typo in the test module:
Fixed the test_separator_rejected test mistakenly calling t_rast_list instead of t_vect_list.

@echoix can you check this?

@saket0187 saket0187 changed the title t.vect.list: Remove needs_solo_run and fix parsing typos in t.vect.list tests temporal: Remove needs_solo_run and fix parsing typos in t.vect.list & t.rast.list tests Jul 4, 2026
@github-actions github-actions Bot added temporal Related to temporal data processing Python Related code is in Python module tests Related to Test Suite labels Jul 4, 2026
@echoix

echoix commented Jul 4, 2026

Copy link
Copy Markdown
Member

The typos fixed are fine. But there's some timeouts. Is it because too much temporal tests are running at once, and they can't really be ran in parallel (the Python code backing it up must have something either slowing it down, using too much resources, or serializing operations/comms). Maybe that's the reason why it was still separated

@saket0187

Copy link
Copy Markdown
Contributor Author

I think we can keep the t.vect.list test file changes and the typo fixes, so I have only reverted the t.rast.list markers. I'm running the CI again just to double-check that everything is working properly.

@echoix
echoix merged commit cc6382c into OSGeo:main Jul 9, 2026
26 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Jul 9, 2026
@wenzeslaus

Copy link
Copy Markdown
Member

Please, keep at least the PR title up to date, ideally also the description, so that it fits with the changes in the PR and it is easy to use for a matching commit message.

@saket0187 saket0187 changed the title temporal: Remove needs_solo_run and fix parsing typos in t.vect.list & t.rast.list tests temporal: Remove needs_solo_run from t.vect.list and fix typos in t.vect.list & t.rast.list tests Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module Python Related code is in Python temporal Related to temporal data processing tests Related to Test Suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants