tests: add integration tests for get_region in grass.jupyter

Author: AkshadSonawane

pyproject.toml

@@ -533,6 +533,7 @@ addopts = """
 markers = [
     "slow: marks tests as slow (deselect with '-m \"not slow\"')",
     "needs_solo_run: marks tests that must be run without any other tests running in parallel",
+    "integration: marks tests requiring a live compiled GRASS environment",
 ]
 minversion = "8.2"
 python_files = """

python/grass/jupyter/tests/test_region.py

@@ -0,0 +1,65 @@
+"""Integration tests for grass.jupyter.utils.get_region()"""
+
+import pytest
+import grass.script as gs
+from grass.jupyter.utils import get_region
+
+REGION_KEYS = ["north", "south", "east", "west", "nsres", "ewres"]
+
+
+@pytest.mark.integration
+def test_get_region_returns_dict(session):
+    """get_region() must return a dict."""
+    result = get_region(env=session.env)
+    assert isinstance(result, dict), f"Expected dict, got {type(result).__name__}"
+
+
+@pytest.mark.integration
+def test_get_region_has_required_keys(session):
+    """get_region() must contain all six geographic keys."""
+    result = get_region(env=session.env)
+    missing = [k for k in REGION_KEYS if k not in result]
+    assert not missing, f"Missing keys: {missing}"
+
+
+@pytest.mark.integration
+def test_get_region_values_are_numeric(session):
+    """All geographic values in the region dict must be numeric."""
+    result = get_region(env=session.env)
+    non_numeric = {
+        k: result[k] for k in REGION_KEYS if not isinstance(result[k], (int, float))
+    }
+    assert not non_numeric, f"Non-numeric values: {non_numeric}"
+
+
+@pytest.mark.integration
+def test_get_region_geographic_sanity(session):
+    """North > South, East > West, resolutions positive."""
+    r = get_region(env=session.env)
+    assert r["north"] > r["south"], f"north={r['north']} is not > south={r['south']}"
+    assert r["east"] > r["west"], f"east={r['east']} is not > west={r['west']}"
+    assert r["nsres"] > 0, f"nsres={r['nsres']} is not positive"
+    assert r["ewres"] > 0, f"ewres={r['ewres']} is not positive"
+
+
+@pytest.mark.integration
+def test_get_region_reflects_live_state(session):
+    """get_region() must return current region, not a cached snapshot."""
+    gs.run_command("g.region", n=60, s=50, e=30, w=10, res=1, env=session.env)
+    r = get_region(env=session.env)
+    assert abs(r["north"] - 60) < 1e-6, f"Expected north=60, got {r['north']}"
+    assert abs(r["south"] - 50) < 1e-6, f"Expected south=50, got {r['south']}"
+    assert abs(r["east"] - 30) < 1e-6, f"Expected east=30, got {r['east']}"
+    assert abs(r["west"] - 10) < 1e-6, f"Expected west=10, got {r['west']}"
+
+
+@pytest.mark.integration
+def test_get_region_is_read_only(session):
+    """Calling get_region() must not modify the active GRASS region."""
+    before = gs.region(env=session.env)
+    get_region(env=session.env)
+    after = gs.region(env=session.env)
+    for key in ("n", "s", "e", "w", "nsres", "ewres"):
+        assert before[key] == after[key], (
+            f"get_region() changed '{key}': {before[key]} -> {after[key]}"
+        )