Skip to content

t.vect.list, t.rast.list: Add CSV support with pipe + skip pytest#7106

Merged
petrasovaa merged 5 commits intoOSGeo:mainfrom
gulshan-123:t_list_rast
Mar 18, 2026
Merged

t.vect.list, t.rast.list: Add CSV support with pipe + skip pytest#7106
petrasovaa merged 5 commits intoOSGeo:mainfrom
gulshan-123:t_list_rast

Conversation

@gulshan-123
Copy link
Copy Markdown
Contributor

@gulshan-123 gulshan-123 commented Feb 19, 2026

This is a followup from #7051

Adds:

  1. csv and line format can now use pipe as separator
  2. with format=json and yaml if separator is provided, give user error that separator is part of the format
  3. Default separator in case of "plain" will be pipe, in case of "line" and "csv" will be comma.
  4. skips only tests related to yaml, if yaml is not imported. (Since this is a small change, I included this in the current PR only)

Both t.vect.list and t.rast.list have very similar changes.


Skipping yaml test logic in case of parameter is taken from Gemini:

@pytest.mark.parametrize(
    "output_format",
    [
        "json",
        pytest.param(
            "yaml",
            marks=pytest.mark.skipif(
                yaml is None, reason="PyYAML package not available"
            ),
        ),
    ],
)

@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 Feb 19, 2026
Comment thread temporal/t.rast.list/t.rast.list.py
Copy link
Copy Markdown
Contributor

@petrasovaa petrasovaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, thank you!

@petrasovaa petrasovaa merged commit 30642ab into OSGeo:main Mar 18, 2026
29 checks passed
@github-actions github-actions bot added this to the 8.6.0 milestone Mar 18, 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