Update pytest assertions for SUMMARY mnemonic array reshape

Six pytest cases still asserted the old 'fixed/size_count=1' shape
for non-F SUMMARY mnemonics. Update them to the new 'array' +
optional_body=True shape introduced when WOPR/GMWIN multi-line and
bare-stack bodies were fixed, and import the new _summary_optional_body
helper.

Author: magnesj

scripts/test_build_keyword_index.py

@@ -39,6 +39,7 @@
     _opm_item_for_param,
     _classify_size,
     _summary_size_shape,
+    _summary_optional_body,
     NS,
     SECTION_MAP,
 )
@@ -1146,19 +1147,22 @@ def test_field_scope_gets_size_kind_none(self, tmp_path):
         assert entry["size_kind"] == "none"
         assert "size_count" not in entry
 
-    def test_well_and_group_scope_get_size_kind_fixed_one(self, tmp_path):
-        # Group/well/region/etc. mnemonics take a single record (a list of
-        # names or just '/'); modelling them as 'list' would wrongly demand
-        # a closing standalone '/'. They're 'fixed' with size_count=1.
+    def test_well_and_group_scope_get_size_kind_array(self, tmp_path):
+        # Group/well/region/etc. mnemonics take an optional list of names
+        # spread across one or more lines and closed by a single '/'. That's
+        # ``size_kind: 'array'`` plus ``optional_body: True`` so a bare
+        # ``WOPR`` stacked back-to-back with another mnemonic is accepted
+        # but a forgotten closing '/' after listed names is still flagged.
         body = self._fgwcl_table(
             _row("Flow", "Gas-Oil Ratio", "GOR",
                  "", "GGOR", "WGOR", "", "", ""),
         )
         fodt = self._write_section_fodt(tmp_path, body)
         out = parse_summary_mnemonics(fodt)
         for kw in ("GGOR", "WGOR"):
-            assert out[kw]["size_kind"] == "fixed"
-            assert out[kw]["size_count"] == 1
+            assert out[kw]["size_kind"] == "array"
+            assert "size_count" not in out[kw]
+            assert out[kw]["optional_body"] is True
 
     def test_skips_empty_scope_cells(self, tmp_path):
         # Only WOPT exists for this row; the empty Field/Group cells must
@@ -1218,8 +1222,9 @@ def test_picks_up_network_model_gpr(self, tmp_path):
         fodt = self._write_section_fodt(tmp_path, body)
         out = parse_summary_mnemonics(fodt)
         assert "GPR" in out
-        assert out["GPR"]["size_kind"] == "fixed"
-        assert out["GPR"]["size_count"] == 1
+        assert out["GPR"]["size_kind"] == "array"
+        assert "size_count" not in out["GPR"]
+        assert out["GPR"]["optional_body"] is True
 
     def test_tags_tracer_rows_as_templated(self, tmp_path):
         # Tracer mnemonics (FTPR, WTPC, …) are templates — the user appends
@@ -1256,10 +1261,11 @@ def test_picks_up_field_group_control_mode_table(self, tmp_path):
         out = parse_summary_mnemonics(fodt)
         for kw in ("FMCTP", "GMCTP", "FMCTW", "GMCTW", "FMCTG", "GMCTG"):
             assert kw in out
-        # Field-scope stays bare; group-scope takes a single record.
+        # Field-scope stays bare; group-scope takes an optional list of names.
         assert out["FMCTP"]["size_kind"] == "none"
-        assert out["GMCTP"]["size_kind"] == "fixed"
-        assert out["GMCTP"]["size_count"] == 1
+        assert out["GMCTP"]["size_kind"] == "array"
+        assert "size_count" not in out["GMCTP"]
+        assert out["GMCTP"]["optional_body"] is True
         # Description spans pair Field/Group correctly.
         assert "Production Group" in out["FMCTP"]["summary"]
         assert "Production Group" in out["GMCTP"]["summary"]
@@ -1278,11 +1284,13 @@ def test_picks_up_well_control_mode_table(self, tmp_path):
         body = _table(title, groups, mnem, desc)
         fodt = self._write_section_fodt(tmp_path, body)
         out = parse_summary_mnemonics(fodt)
-        assert out["WSTAT"]["size_kind"] == "fixed"
-        assert out["WSTAT"]["size_count"] == 1
+        assert out["WSTAT"]["size_kind"] == "array"
+        assert "size_count" not in out["WSTAT"]
+        assert out["WSTAT"]["optional_body"] is True
         assert "Well Status" in out["WSTAT"]["summary"]
         assert "Well Mode of Control" in out["WMCTL"]["summary"]
-        assert out["WMCTL"]["size_kind"] == "fixed"
+        assert out["WMCTL"]["size_kind"] == "array"
+        assert out["WMCTL"]["optional_body"] is True
 
     def test_picks_up_performance_table(self, tmp_path):
         # The "OPM Flow Simulation Performance" table has a different
@@ -1328,19 +1336,22 @@ def test_handles_aquifer_and_recovery_table_titles(self, tmp_path):
         for kw in ("FAQR", "AAQR", "ALQR", "ANQR", "FOE", "ROE"):
             assert kw in out
         assert out["FAQR"]["size_kind"] == "none"
-        assert out["AAQR"]["size_kind"] == "fixed" and out["AAQR"]["size_count"] == 1
+        assert out["AAQR"]["size_kind"] == "array" and out["AAQR"]["optional_body"] is True
         assert out["FOE"]["size_kind"] == "none"
-        assert out["ROE"]["size_kind"] == "fixed" and out["ROE"]["size_count"] == 1
+        assert out["ROE"]["size_kind"] == "array" and out["ROE"]["optional_body"] is True
 
 
 class TestSummarySizeShape:
     def test_field_scope_none(self):
         assert _summary_size_shape("FOPR") == ("none", None)
         assert _summary_size_shape("FWPR") == ("none", None)
 
-    def test_other_scopes_fixed_one(self):
+    def test_other_scopes_array_optional(self):
+        # W/G/R/B/A-prefixed mnemonics take an optional list of names
+        # spread across one or more lines and closed by a single '/'.
         for kw in ("WOPR", "WWIR", "GGOR", "ROE", "BPR", "AAQR"):
-            assert _summary_size_shape(kw) == ("fixed", 1)
+            assert _summary_size_shape(kw) == ("array", None)
+            assert _summary_optional_body(kw) is True
 
 
 class TestAttachStringOptions: