[GRASS-git] [OSGeo/grass] d4e954: r.pack: Replace traceback by error message for out...

[Vaclav Petras]

  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: d4e9540c7dffb4e875fe6dedccf78227f9104de3
      https://github.com/OSGeo/grass/commit/d4e9540c7dffb4e875fe6dedccf78227f9104de3
  Author: Vaclav Petras <wenzeslaus at gmail.com>
  Date:   2025-09-24 (Wed, 24 Sep 2025)

  Changed paths:
    M scripts/conftest.py
    M scripts/r.pack/r.pack.py
    A scripts/r.pack/tests/r_pack_test.py

  Log Message:
  -----------
  r.pack: Replace traceback by error message for output issues (#6346)

Add check for issues with the output file parent directory, and report error messages. This avoids a traceback originating at the very end of the tool execution when calling shutil.move.

This adds tests for the newly handled error states as well as for other issues with the inputs.

Skip the read-only dir test for Windows. At least in GitHub Actions Windows CI, this does not cause r.pack to fail. Assuming here that the directory still allows writes, we can't fully set up the test, so we don't test that for Windows.

It adds the new pytest fixtures to scripts conftest.py file given there is at least some potential for reusability for similar simple tests.



To unsubscribe from these emails, change your notification settings at https://github.com/OSGeo/grass/settings/notifications