[GRASS-git] [OSGeo/grass] aa1c01: lib/raster3d/tilewrite: Use `>=` for tile index bo...

[Edouard Choinière]

  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: aa1c015fa39f758c48f2267584afc75174e2ba24
      https://github.com/OSGeo/grass/commit/aa1c015fa39f758c48f2267584afc75174e2ba24
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2026-09-06 (Sun, 06 Sep 2026)

  Changed paths:
    A lib/raster3d/tests/lib_raster3d_tilewrite_test.py
    M lib/raster3d/tilewrite.c

  Log Message:
  -----------
  lib/raster3d/tilewrite: Use `>=` for tile index bounds check (#7723)

* Apply suggested fix to lib/raster3d/tilewrite.c from Copilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>

* lib/raster3d: Add test for Rast3d_write_tile tile index bounds check

Add a pytest test covering the tileIndex bounds check in
Rast3d_write_tile: an index equal to nTiles must be rejected as out of
range, and the last valid index must be accepted. The invalid call runs
in a subprocess using the grass.lib.raster3d ctypes bindings because the
rejection is a fatal error which exits the calling process.

The out-of-range test fails without the bounds check change in this
branch and passes with it.

The test was written by Claude Code using the Claude Fable 5 model.

---------

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: Vaclav Petras <wenzeslaus at gmail.com>



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