hap-cli 0.8.28__tar.gz → 0.8.29__tar.gz

{hap_cli-0.8.28 → hap_cli-0.8.29}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hap-cli
-Version: 0.8.28
+Version: 0.8.29
 Summary: CLI harness for Mingdao HAP - Enterprise no-code platform
 Author: hap-cli
 License: Apache-2.0

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli/__init__.py

@@ -1,3 +1,3 @@
 """CLI harness for Mingdao HAP (hap-cli)."""
 
-__version__ = "0.8.28"
+__version__ = "0.8.29"

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli/commands/worksheet_cmd.py

@@ -919,24 +919,143 @@ def worksheet_delete_button(ctx, worksheet_id, btn_id, app_id, view_id, yes):
 
 @worksheet.command("switches")
 @click.argument("worksheet_id")
+@click.option("--raw", is_flag=True,
+              help="Return the server's switch list unannotated.")
 @pass_context
-def worksheet_switches(ctx, worksheet_id):
-    """Get feature switches for a worksheet."""
+def worksheet_switches(ctx, worksheet_id, raw):
+    """Get feature switches for a worksheet.
+
+    Each switch is listed with the name and group it has on the feature
+    switch page, because several numbers share a name and only the group
+    tells them apart.
+    """
     try:
         session = ctx.get_session()
         data = ws_mod.get_switches(session, worksheet_id)
+        if not raw:
+            data = ws_mod.annotate_switches(data)
         ctx.output(data, lambda d: output_json(d))
     except Exception as e:
         ctx.handle_error(e)
 
 
+@worksheet.command("switch-types")
+@pass_context
+def worksheet_switch_types(ctx):
+    """Show the feature switch numbers with their names and groups."""
+    rows = [ws_mod.describe_switch(t) for t in sorted(ws_mod.SWITCH_TYPES)]
+    ctx.output(
+        rows,
+        lambda d: output_table(
+            d, ["type", "group", "name", "path"],
+            ["Type", "Group", "Name", "Path"],
+        ),
+    )
+
+
+def _parse_switch_changes(pairs):
+    changes = {}
+    for raw in pairs:
+        if "=" not in raw:
+            raise click.UsageError(
+                f"Invalid --set value {raw!r}. Use --set TYPE=true|false.")
+        key, value = raw.split("=", 1)
+        try:
+            stype = int(key.strip())
+        except ValueError:
+            raise click.UsageError(
+                f"Invalid switch number {key!r}. Run `hap worksheet "
+                "switch-types` for the list.")
+        val = value.strip().lower()
+        if val not in ("true", "false", "1", "0", "on", "off"):
+            raise click.UsageError(
+                f"Invalid state {value!r} for switch {stype}. Use true or false.")
+        changes[stype] = val in ("true", "1", "on")
+    return changes
+
+
+@worksheet.command("batch-edit-switch")
+@click.argument("worksheet_ids", nargs=-1, required=True)
+@click.option("--set", "pairs", multiple=True, required=True,
+              help="Switch to change as TYPE=true|false (repeatable). "
+                   "Numbers come from `hap worksheet switch-types`.")
+@click.option("--dry-run", is_flag=True,
+              help="Show what would change on each worksheet, without saving.")
+@pass_context
+def worksheet_batch_edit_switch(ctx, worksheet_ids, pairs, dry_run):
+    """Change several feature switches on one or more worksheets.
+
+    The switches you do not name keep their current values. Turning the
+    批量操作 or 统计 parent on or off carries its children with it. Each
+    worksheet is read back after saving, and any switch that did not take
+    is reported rather than counted as success.
+    """
+    try:
+        changes = _parse_switch_changes(pairs)
+        session = ctx.get_session()
+        if dry_run:
+            preview = [
+                {**ws_mod.describe_switch(t), "state": v}
+                for t, v in changes.items()
+            ]
+            ctx.output(
+                {"worksheetIds": list(worksheet_ids), "changes": preview,
+                 "saved": False},
+                lambda d: click.echo(
+                    f"Would set {len(preview)} switch(es) on "
+                    f"{len(worksheet_ids)} worksheet(s): "
+                    + ", ".join(f"{c['path'] or c['type']}={c['state']}"
+                                for c in preview)),
+            )
+            return
+        results, failed_sheets = [], []
+        for wid in worksheet_ids:
+            try:
+                res = ws_mod.batch_edit_switch(session, wid, changes)
+            except Exception as exc:          # keep going across worksheets
+                results.append({"worksheetId": wid, "error": str(exc)})
+                failed_sheets.append(wid)
+                continue
+            results.append(res)
+            if res.get("failed"):
+                failed_sheets.append(wid)
+        payload = {"results": results,
+                   "worksheetsChanged": len(worksheet_ids) - len(failed_sheets),
+                   "worksheetsFailed": failed_sheets}
+
+        def _render(d):
+            click.echo(f"Changed {d['worksheetsChanged']} of "
+                       f"{len(worksheet_ids)} worksheet(s).")
+            for r in d["results"]:
+                if r.get("error"):
+                    click.echo(f"  {r['worksheetId']}: {r['error']}")
+                elif r.get("failed"):
+                    for t, info in r["failed"].items():
+                        click.echo(f"  {r['worksheetId']}: switch {t} is "
+                                   f"{info['actual']!r}, wanted "
+                                   f"{info['expected']!r}")
+        ctx.output(payload, _render)
+        if failed_sheets:
+            raise click.ClickException(
+                f"{len(failed_sheets)} worksheet(s) did not end up in the "
+                "requested state: " + ", ".join(failed_sheets))
+    except Exception as e:
+        ctx.handle_error(e)
+
+
 @worksheet.command("edit-switch")
 @click.argument("worksheet_id")
 @click.argument("switch_id", type=int)
 @click.argument("value", type=bool)
 @pass_context
 def worksheet_edit_switch(ctx, worksheet_id, switch_id, value):
-    """Edit a single feature switch."""
+    """Turn one feature switch on or off.
+
+    SWITCH_ID is a switch number from `hap worksheet switch-types`. The
+    new state is read back, so a change that did not take is reported as
+    an error instead of success. To change several switches, or the same
+    ones across many worksheets, use `worksheet batch-edit-switch`.
+    """
     try:
         session = ctx.get_session()
         data = ws_mod.edit_switch(session, worksheet_id, switch_id, value)

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli/core/worksheet.py

@@ -4,7 +4,7 @@ import json
 import logging
 from typing import Any, Optional
 
-from hap_cli.core.session import Session
+from hap_cli.core.session import Session, APIError
 from hap_cli.core import worksheet_templates as wt
 
 logger = logging.getLogger("hap_cli.worksheet")
@@ -1681,14 +1681,237 @@ def get_switches(
     )
 
 
+# ── Feature switch catalog ───────────────────────────────────────────────
+#
+# GetSwitch answers bare numbers, which say nothing about what they
+# control, and several numbers share a label: 26 and 36 are both "复制",
+# 28 and 39 are both "删除", 23 and 32 are both "系统默认打印". Only the
+# group tells them apart, so the group travels with the name.
+#
+# Names, grouping and parent/child structure come from pd-openweb
+# ``src/pages/FormSet/containers/FunctionalSwitch/config.js``
+# (``listConfigStr``, ``worksheetSwitch`` / ``viewSwitch`` /
+# ``recordSwitch`` / ``approveSwitch``, ``batch``, ``statistics``).
+
+SWITCH_PARENT_BATCH = 25        # 视图 > 批量操作
+SWITCH_PARENT_STATISTICS = 500  # 工作表 > 统计
+
+_SWITCH_BATCH_CHILDREN = (24, 26, 23, 27, 28, 29)
+_SWITCH_STATISTICS_CHILDREN = (50, 51)
+
+SWITCH_TYPES: dict[int, dict[str, Any]] = {
+    # 工作表
+    10: {"name": "显示创建按钮", "group": "工作表"},
+    11: {"name": "讨论", "group": "工作表"},
+    12: {"name": "日志", "group": "工作表"},
+    13: {"name": "导入", "group": "工作表"},
+    14: {"name": "回收站", "group": "工作表"},
+    500: {"name": "统计", "group": "工作表"},
+    50: {"name": "公共", "group": "工作表", "parent": SWITCH_PARENT_STATISTICS},
+    51: {"name": "个人", "group": "工作表", "parent": SWITCH_PARENT_STATISTICS},
+    # 视图
+    20: {"name": "公开分享", "group": "视图"},
+    38: {"name": "嵌入链接", "group": "视图"},
+    21: {"name": "导出", "group": "视图"},
+    22: {"name": "快捷编辑", "group": "视图"},
+    25: {"name": "批量操作", "group": "视图"},
+    24: {"name": "编辑", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    26: {"name": "复制", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    23: {"name": "系统默认打印", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    27: {"name": "导出", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    28: {"name": "删除", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    29: {"name": "执行自定义动作", "group": "视图", "parent": SWITCH_PARENT_BATCH},
+    # 记录
+    30: {"name": "公开分享", "group": "记录"},
+    52: {"name": "内部访问链接", "group": "记录"},
+    32: {"name": "系统默认打印", "group": "记录"},
+    33: {"name": "附件下载", "group": "记录"},
+    34: {"name": "日志", "group": "记录"},
+    35: {"name": "讨论", "group": "记录"},
+    36: {"name": "复制", "group": "记录"},
+    39: {"name": "删除", "group": "记录"},
+    37: {"name": "重新创建", "group": "记录"},
+    # 审批
+    40: {"name": "系统字段", "group": "审批"},
+    41: {"name": "审批流转详情", "group": "审批"},
+}
+
+
+def describe_switch(switch_type: int) -> dict[str, Any]:
+    """Name, group and parent of a switch number; empty-ish when unknown."""
+    meta = SWITCH_TYPES.get(switch_type)
+    if not meta:
+        return {"type": switch_type, "name": "", "group": "", "path": ""}
+    parent = meta.get("parent")
+    path = " > ".join(
+        [meta["group"]]
+        + ([SWITCH_TYPES[parent]["name"]] if parent in SWITCH_TYPES else [])
+        + [meta["name"]]
+    )
+    out = {"type": switch_type, "name": meta["name"], "group": meta["group"],
+           "path": path}
+    if parent:
+        out["parent"] = parent
+    return out
+
+
+def annotate_switches(switches: Any) -> list[dict[str, Any]]:
+    """Return the switch list with name/group/path added to each entry."""
+    out = []
+    for entry in switches or []:
+        if not isinstance(entry, dict):
+            continue
+        merged = dict(entry)
+        merged.update(describe_switch(entry.get("type")))
+        out.append(merged)
+    return out
+
+
+def _switch_wire_fields(entry: dict[str, Any]) -> dict[str, Any]:
+    """The subset of a switch the page sends back on save.
+
+    ``FunctionalSwitch/index.jsx`` picks ``roleType/type/state/viewIds``;
+    ``displayFlowChart`` rides along where the switch carries one.
+    """
+    out = {k: entry[k] for k in ("roleType", "type", "state", "viewIds")
+           if k in entry}
+    if "displayFlowChart" in entry:
+        out["displayFlowChart"] = entry["displayFlowChart"]
+    out.setdefault("type", entry.get("type"))
+    out.setdefault("roleType", 0)
+    return out
+
+
+def _apply_switch_linkage(
+    current: list[dict[str, Any]], changes: dict[int, bool],
+) -> dict[int, bool]:
+    """Expand ``changes`` with the parent/child rules the page applies.
+
+    * Turning the 批量操作 (25) or 统计 (500) parent on or off carries all
+      of its children with it.
+    * When every 统计 child ends up off, the parent goes off too. The
+      批量操作 children deliberately do **not** close their parent — the
+      page only does this for 统计 (``index.jsx:108-123``; its comment
+      says 批量操作 but the code targets the statistics parent).
+    """
+    resolved = dict(changes)
+    for parent, children in (
+        (SWITCH_PARENT_BATCH, _SWITCH_BATCH_CHILDREN),
+        (SWITCH_PARENT_STATISTICS, _SWITCH_STATISTICS_CHILDREN),
+    ):
+        if parent in resolved:
+            for child in children:
+                resolved.setdefault(child, resolved[parent])
+    states = {e.get("type"): e.get("state") for e in current
+              if isinstance(e, dict)}
+    states.update(resolved)
+    if any(c in resolved for c in _SWITCH_STATISTICS_CHILDREN):
+        if not any(states.get(c) for c in _SWITCH_STATISTICS_CHILDREN):
+            resolved.setdefault(SWITCH_PARENT_STATISTICS, False)
+    return resolved
+
+
+def batch_edit_switch(
+    session: Session,
+    worksheet_id: str,
+    changes: dict[int, bool],
+    *,
+    verify: bool = True,
+) -> dict[str, Any]:
+    """Apply several switch changes to one worksheet in a single call.
+
+    This is what the feature-switch page itself uses
+    (``Worksheet/BatchEditSwitch``, ``src/api/worksheet.js:2346-2356``):
+    it submits the **whole** ``switchList``, so the untouched switches
+    have to be read first and sent back unchanged or they are lost.
+
+    Parent/child linkage is applied for you (see
+    :func:`_apply_switch_linkage`). With ``verify`` the new state is read
+    back and any switch that did not take is reported, because the
+    endpoint answers ``true`` regardless.
+
+    Returns ``{"worksheetId", "applied": {type: bool}, "failed": {type:
+    {"expected", "actual"}}}``.
+    """
+    current = get_switches(session, worksheet_id) or []
+    if not isinstance(current, list):
+        raise APIError(
+            f"could not read the switches of worksheet {worksheet_id}")
+    resolved = _apply_switch_linkage(current, {int(k): bool(v)
+                                               for k, v in changes.items()})
+    switch_list = []
+    for entry in current:
+        if not isinstance(entry, dict):
+            continue
+        wire = _switch_wire_fields(entry)
+        if entry.get("type") in resolved:
+            wire["state"] = resolved[entry["type"]]
+        switch_list.append(wire)
+    known = {e.get("type") for e in current if isinstance(e, dict)}
+    for stype, state in resolved.items():
+        if stype not in known:
+            switch_list.append({"type": stype, "state": state, "roleType": 0})
+    session.api_call(
+        "Worksheet", "BatchEditSwitch",
+        {"worksheetId": worksheet_id, "switchList": switch_list},
+    )
+    result: dict[str, Any] = {"worksheetId": worksheet_id,
+                              "applied": dict(resolved), "failed": {}}
+    if verify:
+        after = {e.get("type"): e.get("state")
+                 for e in (get_switches(session, worksheet_id) or [])
+                 if isinstance(e, dict)}
+        for stype, want in resolved.items():
+            if bool(after.get(stype)) != bool(want):
+                result["failed"][stype] = {"expected": want,
+                                           "actual": after.get(stype)}
+    return result
+
+
 def edit_switch(
     session: Session, worksheet_id: str, switch_id: int, value: bool,
 ) -> dict[str, Any]:
-    """Edit a single feature switch."""
-    return session.api_call(
-        "Worksheet", "EditSwitch",
-        {"worksheetId": worksheet_id, "state": switch_id, "value": value},
-    )
+    """Turn one feature switch on or off, and confirm it stuck.
+
+    ``EditSwitch`` names the boolean ``state`` and the switch number
+    ``type`` (pd-openweb ``src/api/worksheet.js:2330-2343``). This wrapper
+    used to send ``{state: <switch number>, value: <bool>}`` — the two
+    swapped, plus a ``value`` field the API does not define. The server
+    accepts that quietly and answers ``{"data": true}`` without changing
+    anything, so every edit looked like it worked and none of them did.
+
+    The switch's remaining fields are sent back as they are, the way the
+    page does (``_.pick(o, ['roleType', 'type', 'state', 'viewIds'])`` in
+    ``FormSet/containers/FunctionalSwitch/index.jsx``); dropping
+    ``viewIds`` would reset a view-scoped switch.
+
+    Because a stale payload fails silently rather than loudly, the new
+    state is read back and a write that did not land raises instead of
+    reporting success.
+    """
+    current = {}
+    for entry in get_switches(session, worksheet_id) or []:
+        if isinstance(entry, dict) and entry.get("type") == switch_id:
+            current = entry
+            break
+    payload: dict[str, Any] = {"worksheetId": worksheet_id,
+                               "type": switch_id, "state": value}
+    for field in ("roleType", "viewIds", "displayFlowChart"):
+        if field in current:
+            payload[field] = current[field]
+    payload.setdefault("roleType", 0)
+    result = session.api_call("Worksheet", "EditSwitch", payload)
+    applied = None
+    for entry in get_switches(session, worksheet_id) or []:
+        if isinstance(entry, dict) and entry.get("type") == switch_id:
+            applied = entry.get("state")
+            break
+    if bool(applied) != bool(value):
+        raise APIError(
+            f"switch {switch_id} is still {applied!r} after asking for "
+            f"{value!r} — the server reported success but did not apply it"
+        )
+    return result
 
 
 # ── Logs ─────────────────────────────────────────────────────────────────

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli/locale/messages.json

@@ -3969,6 +3969,18 @@
       "en": "Raw JSON array of new controls to append. Each item matches the AddWorksheetControls `controls` schema (same shape as `worksheet fields --raw` output).",
       "zh-Hans": "要追加的新字段原始 JSON 数组。每一项遵循 AddWorksheetControls 的 `controls` 结构（与 `worksheet fields --raw` 输出形态相同）"
     },
+    "worksheet batch-edit-switch": {
+      "en": "Change several feature switches on one or more worksheets.\n\nThe switches you do not name keep their current values. Turning the\n批量操作 or 统计 parent on or off carries its children with it. Each\nworksheet is read back after saving, and any switch that did not take\nis reported rather than counted as success.",
+      "zh-Hans": "批量修改一张或多张工作表的功能开关。\n\n未指定的开关保持原值。开启或关闭「批量操作」「统计」父开关时，其子项会随之联动。每张表保存后都会回读校验，未生效的开关会被报出而不是计为成功。"
+    },
+    "worksheet batch-edit-switch --dry-run": {
+      "en": "Show what would change on each worksheet, without saving.",
+      "zh-Hans": "仅显示每张工作表将要修改的内容，不实际保存。"
+    },
+    "worksheet batch-edit-switch --set": {
+      "en": "Switch to change as TYPE=true|false (repeatable). Numbers come from `hap worksheet switch-types`.",
+      "zh-Hans": "要修改的开关，格式为 TYPE=true|false（可重复指定）。开关号见 `hap worksheet switch-types`。"
+    },
     "worksheet chart": {
       "en": "Worksheet chart (统计图) management.",
       "zh-Hans": "工作表统计图管理"
@@ -4198,8 +4210,8 @@
       "zh-Hans": "跳过确认"
     },
     "worksheet edit-switch": {
-      "en": "Edit a single feature switch.",
-      "zh-Hans": "编辑单个功能开关"
+      "en": "Turn one feature switch on or off.\n\nSWITCH_ID is a switch number from `hap worksheet switch-types`. The\nnew state is read back, so a change that did not take is reported as\nan error instead of success. To change several switches, or the same\nones across many worksheets, use `worksheet batch-edit-switch`.",
+      "zh-Hans": "开启或关闭单个功能开关。\n\nSWITCH_ID 为开关号，可用 `hap worksheet switch-types` 查看。修改后会回读校验，未生效将报错而不是报成功。要一次改多个开关、或对多张工作表改同一批开关，请用 `worksheet batch-edit-switch`。"
     },
     "worksheet field-types": {
       "en": "Show field type constants as a reference table.\n\nLists the builder-style CODE names (Text/Number/Relation/...) you\npass via 'update-fields --fields' alongside the wire ControlTypeId\nand the legacy SCREAMING_SNAKE alias still accepted for backward\ncompatibility. Region maps to three ControlTypeIds — pass\n'regionLevel' in the field spec's `extra` to pick one.",
@@ -4705,9 +4717,17 @@
       "en": "Rule type: 0=interact 1=validate 2=lock",
       "zh-Hans": "规则类型：0=交互 1=校验 2=锁定"
     },
+    "worksheet switch-types": {
+      "en": "Show the feature switch numbers with their names and groups.",
+      "zh-Hans": "显示功能开关号及其对应的名称和分组。"
+    },
     "worksheet switches": {
-      "en": "Get feature switches for a worksheet.",
-      "zh-Hans": "获取工作表的功能开关"
+      "en": "Get feature switches for a worksheet.\n\nEach switch is listed with the name and group it has on the feature\nswitch page, because several numbers share a name and only the group\ntells them apart.",
+      "zh-Hans": "获取工作表的功能开关。每个开关会附带它在功能开关页面上的名称和分组——因为多个开关号共用同一名称，只有分组能区分它们。"
+    },
+    "worksheet switches --raw": {
+      "en": "Return the server's switch list unannotated.",
+      "zh-Hans": "返回服务端原始开关列表，不附加名称和分组。"
     },
     "worksheet update": {
       "en": "Update worksheet alias, description, sidebar name, or sidebar icon.\n\n`--name` and `--icon` change how the worksheet appears in the app\nsidebar; both require --app-id so the section can be resolved.",

hap_cli-0.8.29/hap_cli/tests/test_worksheet_switch.py

@@ -0,0 +1,174 @@
+"""Worksheet feature switches (功能开关).
+
+`Worksheet/EditSwitch` takes `state` as the **boolean** and `type` as the
+switch number (pd-openweb src/api/worksheet.js:2330-2343). The wrapper used
+to send `{state: <switch number>, value: <bool>}` — the two swapped, plus a
+`value` key the API does not define. The server accepted that and answered
+`{"data": true}` while changing nothing, so every switch edit reported
+success and silently did nothing (reported by a customer configuring 105
+worksheets, reproduced live 2026-09-05).
+
+The page sends the switch's other fields back untouched
+(`_.pick(o, ['roleType', 'type', 'state', 'viewIds'])` in
+FormSet/containers/FunctionalSwitch/index.jsx), so a toggle must preserve
+them rather than reset a view-scoped switch to defaults.
+"""
+
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+from hap_cli.core import worksheet as ws_mod
+
+
+def _switches(**over):
+    base = {"view": [], "state": True, "type": 24, "roleType": 0,
+            "viewIds": [], "displayFlowChart": 0}
+    base.update(over)
+    return [base, {"view": [], "state": True, "type": 26, "roleType": 0,
+                   "viewIds": [], "displayFlowChart": 0}]
+
+
+class TestEditSwitch:
+    def _session(self, switches=None, after=None):
+        session = MagicMock()
+        reads = [switches or _switches(), after if after is not None else
+                 _switches(state=False)]
+        session.api_call.side_effect = lambda c, a, p=None, **k: (
+            reads.pop(0) if a == "GetSwitch" else {"data": True}
+        )
+        return session
+
+    def _edit_payload(self, session):
+        for call in session.api_call.call_args_list:
+            if call[0][1] == "EditSwitch":
+                return call[0][2]
+        raise AssertionError("EditSwitch was never called")
+
+    def test_state_carries_the_boolean_and_type_the_switch_number(self):
+        session = self._session()
+        ws_mod.edit_switch(session, "ws1", 24, False)
+        payload = self._edit_payload(session)
+        assert payload["type"] == 24        # the switch, not the state
+        assert payload["state"] is False    # the boolean, not the switch
+        assert "value" not in payload       # no such field in the API
+
+    def test_other_fields_of_the_switch_are_preserved(self):
+        session = self._session(
+            switches=[{"state": True, "type": 24, "roleType": 2,
+                       "viewIds": ["v1"], "displayFlowChart": 1}],
+            after=[{"state": False, "type": 24, "roleType": 2,
+                    "viewIds": ["v1"], "displayFlowChart": 1}],
+        )
+        ws_mod.edit_switch(session, "ws1", 24, False)
+        payload = self._edit_payload(session)
+        assert payload["roleType"] == 2
+        assert payload["viewIds"] == ["v1"]
+        assert payload["displayFlowChart"] == 1
+
+    def test_unknown_switch_still_sends_a_well_formed_payload(self):
+        session = self._session(
+            switches=[{"state": True, "type": 99}],
+            after=[{"state": False, "type": 24}],
+        )
+        ws_mod.edit_switch(session, "ws1", 24, False)
+        payload = self._edit_payload(session)
+        assert payload["type"] == 24
+        assert payload["state"] is False
+
+    def test_a_write_that_did_not_stick_is_an_error(self):
+        """The whole point: `true` from the server is not proof."""
+        session = self._session(after=_switches(state=True))  # unchanged
+        with pytest.raises(Exception) as exc:
+            ws_mod.edit_switch(session, "ws1", 24, False)
+        assert "24" in str(exc.value)
+
+    def test_successful_write_is_confirmed_by_reading_back(self):
+        session = self._session()               # after == state False
+        result = ws_mod.edit_switch(session, "ws1", 24, False)
+        assert result is True or result == {"data": True} or result
+
+
+class TestSwitchCatalog:
+    def test_same_label_is_disambiguated_by_group(self):
+        """26/36 are both 复制 and 28/39 both 删除 — the group separates them."""
+        assert ws_mod.describe_switch(26)["group"] == "视图"
+        assert ws_mod.describe_switch(36)["group"] == "记录"
+        assert ws_mod.describe_switch(26)["name"] == ws_mod.describe_switch(36)["name"]
+        assert ws_mod.describe_switch(28)["group"] != ws_mod.describe_switch(39)["group"]
+
+    def test_batch_children_carry_their_parent_in_the_path(self):
+        assert ws_mod.describe_switch(24)["path"] == "视图 > 批量操作 > 编辑"
+        assert ws_mod.describe_switch(24)["parent"] == ws_mod.SWITCH_PARENT_BATCH
+
+    def test_unknown_type_degrades_without_raising(self):
+        d = ws_mod.describe_switch(9999)
+        assert d["type"] == 9999 and d["name"] == ""
+
+    def test_annotate_adds_names_to_the_server_list(self):
+        out = ws_mod.annotate_switches([{"type": 36, "state": True}])
+        assert out[0]["state"] is True
+        assert out[0]["group"] == "记录"
+
+
+class TestBatchEditSwitch:
+    def _session(self, current, after=None):
+        session = MagicMock()
+        reads = [current, after if after is not None else current]
+        session.api_call.side_effect = lambda c, a, p=None, **k: (
+            reads.pop(0) if a == "GetSwitch" else {"data": True}
+        )
+        return session
+
+    def _sent(self, session):
+        for call in session.api_call.call_args_list:
+            if call[0][1] == "BatchEditSwitch":
+                return call[0][2]["switchList"]
+        raise AssertionError("BatchEditSwitch was never called")
+
+    def test_untouched_switches_are_sent_back_unchanged(self):
+        current = [{"type": 24, "state": True, "roleType": 0, "viewIds": []},
+                   {"type": 30, "state": True, "roleType": 2, "viewIds": ["v"]}]
+        after = [{"type": 24, "state": False}, {"type": 30, "state": True}]
+        session = self._session(current, after)
+        ws_mod.batch_edit_switch(session, "ws1", {24: False})
+        sent = {s["type"]: s for s in self._sent(session)}
+        assert sent[24]["state"] is False
+        assert sent[30]["state"] is True          # not dropped
+        assert sent[30]["viewIds"] == ["v"]       # and not reset
+
+    def test_turning_off_the_batch_parent_closes_its_children(self):
+        current = [{"type": 25, "state": True}, {"type": 24, "state": True},
+                   {"type": 26, "state": True}]
+        after = [{"type": 25, "state": False}, {"type": 24, "state": False},
+                 {"type": 26, "state": False}]
+        session = self._session(current, after)
+        res = ws_mod.batch_edit_switch(session, "ws1", {25: False})
+        assert res["applied"][24] is False and res["applied"][26] is False
+
+    def test_batch_children_do_not_close_their_parent(self):
+        """Only 统计 closes its parent; 批量操作 does not (index.jsx:108-123)."""
+        current = [{"type": 25, "state": True}] + [
+            {"type": t, "state": True} for t in (24, 26, 23, 27, 28, 29)]
+        after = [{"type": 25, "state": True}] + [
+            {"type": t, "state": False} for t in (24, 26, 23, 27, 28, 29)]
+        session = self._session(current, after)
+        res = ws_mod.batch_edit_switch(
+            session, "ws1", {t: False for t in (24, 26, 23, 27, 28, 29)})
+        assert 25 not in res["applied"]
+
+    def test_last_statistics_child_off_also_closes_the_parent(self):
+        current = [{"type": 500, "state": True}, {"type": 50, "state": True},
+                   {"type": 51, "state": False}]
+        after = [{"type": 500, "state": False}, {"type": 50, "state": False},
+                 {"type": 51, "state": False}]
+        session = self._session(current, after)
+        res = ws_mod.batch_edit_switch(session, "ws1", {50: False})
+        assert res["applied"][500] is False
+
+    def test_a_switch_that_did_not_take_is_reported_not_swallowed(self):
+        current = [{"type": 24, "state": True}]
+        after = [{"type": 24, "state": True}]      # server ignored it
+        session = self._session(current, after)
+        res = ws_mod.batch_edit_switch(session, "ws1", {24: False})
+        assert res["failed"][24] == {"expected": False, "actual": True}

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hap-cli
-Version: 0.8.28
+Version: 0.8.29
 Summary: CLI harness for Mingdao HAP - Enterprise no-code platform
 Author: hap-cli
 License: Apache-2.0

{hap_cli-0.8.28 → hap_cli-0.8.29}/hap_cli.egg-info/SOURCES.txt

@@ -230,6 +230,7 @@ hap_cli/tests/test_workflow_read_write_keys.py
 hap_cli/tests/test_workflow_v3_semantic_cli.py
 hap_cli/tests/test_worksheet_create_section.py
 hap_cli/tests/test_worksheet_crud_cli.py
+hap_cli/tests/test_worksheet_switch.py
 hap_cli/tests/test_worksheet_v3_semantic_cli.py
 hap_cli/utils/__init__.py
 hap_cli/utils/formatting.py