[echoix]

Description

Closes the sidx file handle that Vect_open_sidx() left open when a map is opened in update mode, and compares normalized paths in the get_path doctest.

Motivation and context

On Windows test_pygrass_vector_doctests.py failed in two doctests: removing a map after it was opened with rw failed because the sidx file was still open (on other platforms this leaked a file descriptor), and get_path compared a path with slashes to one built by os.path.join().

How has this been tested?

New pytest test that fails without the fix, test_pygrass_vector_doctests.py with the CI dataset (99 tests pass), and the pygrass vector pytest tests.

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist

• PR title provides summary of the changes and starts with one of the
  pre-defined prefixes
• My code follows the code style
  of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

AI assistance was used to diagnose the failure and to write the fix and the test.

---

Generated by Claude Code

[echoix]

Can you verify that the pytest and the doctest actually run on windows, and that it passes?

[echoix]

Checked in the Windows job log for 27aba9e:

• pytest: grass_pygrass_vector_sidx_test.py::test_update_mode_releases_sidx ran and PASSED.
• Doctests: test_pygrass_vector_doctests.py ran all 99 tests. rewrite and get_path, the two this PR targets, pass. The file still fails because of areas_to_wkb_list and features_to_wkb_list (access violation in G_free of GEOS memory), which lib/vector: Return WKB buffers allocated by G_malloc #748 fixes, and this branch does not include that.
• Not yet verified: that the new test fails on Windows without the fix. The earlier lib/vector: Return WKB buffers allocated by G_malloc #748 run showed Unable to delete file .../sidx in rewrite, but I haven't run the test alone against the unfixed code. Also, only a combined lib/vector: Return WKB buffers allocated by G_malloc #748 + grass.pygrass: Fix the vector doctests on Windows #749 run can show the whole vector doctest file passing.

---

Generated by Claude Code

[echoix]

Verified on Windows with #750, which has only the new test and no fix. test_update_mode_releases_sidx FAILED there (The file ...\sidx_map\sidx was left open after closing the map), and it was the only pytest failure (1 failed, 1896 passed). It passes on this PR (27aba9e). It also fails without the fix on Linux (1 failed, 1940 passed).

---

Generated by Claude Code