hap-cli 0.9.2__tar.gz → 0.9.3__tar.gz

{hap_cli-0.9.2 → hap_cli-0.9.3}/PKG-INFO

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

{hap_cli-0.9.2 → hap_cli-0.9.3}/app_live_tests/smoke.py

@@ -3602,6 +3602,89 @@ def case_editor_inline_subtable(app_id: str, section_id: str) -> None:
         hap(["worksheet", "delete", parent, "-a", app_id, "-y"], check=False)
 
 
+def case_create_partial(app_id: str, section_id: str) -> None:
+    """Bug #455775: a failed `worksheet create --fields` leaves nothing."""
+    print("\nscenario: worksheet create --fields failure (#455775)")
+    import json as _json
+    from hap_cli.core import app as app_mod
+    from hap_cli.core.session import Session
+    session = Session.load()
+    project_id = app_mod.app_project_id(session, app_id)
+    prefix = "QA455775"
+    for leftover in [n for n in _worksheets(app_id) if n.startswith(prefix)]:
+        _ensure_absent(app_id, leftover)
+
+    def _ours() -> list[str]:
+        return [n for n in _worksheets(app_id) if n.startswith(prefix)]
+
+    def _purge(ws_id: str) -> None:
+        try:
+            app_mod.purge_app_item(session, app_id=app_id,
+                                   project_id=project_id,
+                                   section_id=section_id, item_id=ws_id,
+                                   item_type=0)
+        except Exception as exc:  # noqa: BLE001 — reported, not fatal
+            print(f"  note: could not purge {ws_id}: {exc}")
+
+    single = {"source": "static", "value": "DEFAULT-QA"}
+    ok, res = _try(["worksheet", "create", app_id, f"{prefix} Invalid",
+                    "--section-id", section_id, "--fields", _json.dumps([
+                        {"type": "Text", "name": "名称"},
+                        {"type": "Text", "name": "备注",
+                         "defaultValue": single}], ensure_ascii=False)])
+    check("create-partial: a single-object default is refused", not ok,
+          str(res)[:200])
+    check("create-partial: the refusal names the field and the shape",
+          "备注" in str(res) and "list of objects" in str(res),
+          str(res)[:300])
+    check("create-partial: nothing was created", _ours() == [], str(_ours()))
+
+    # Passes every offline check, refused by the server at save time:
+    # an alias the worksheet's own record-id column already holds.
+    ok, res = _try(["worksheet", "create", app_id, f"{prefix} PostFail",
+                    "--section-id", section_id, "--fields", _json.dumps([
+                        {"type": "Text", "name": "名称"},
+                        {"type": "Text", "name": "冲突",
+                         "extra": {"alias": "rowid"}}],
+                        ensure_ascii=False)])
+    check("create-partial: a save rejected after the create fails", not ok,
+          str(res)[:200])
+    check("create-partial: the error says the create was undone",
+          "undone" in str(res), str(res)[:300])
+    check("create-partial: no half-built worksheet is left",
+          _ours() == [], str(_ours()))
+
+    ws_id = ""
+    try:
+        made = hap(["worksheet", "create", app_id, f"{prefix} Array",
+                    "--section-id", section_id, "--fields", _json.dumps([
+                        {"type": "Text", "name": "名称"},
+                        {"type": "Text", "name": "备注",
+                         "defaultValue": [single]}], ensure_ascii=False)],
+                   check=False)
+        ws_id = made.get("worksheetId", "") if isinstance(made, dict) else ""
+        check("create-partial: the list form still creates", bool(ws_id),
+              str(made)[:200])
+        if not ws_id:
+            return
+        note = next((c for c in _fields(ws_id)
+                     if c.get("controlName") == "备注"), {})
+        check("create-partial: its default is stored",
+              "DEFAULT-QA" in str((note.get("advancedSetting") or {})
+                                  .get("defsource")), str(note)[:200])
+        ok, res = _try(["worksheet", "update-fields", ws_id, "--fields",
+                        _json.dumps([{"type": "Text", "name": "名称"},
+                                     {"type": "Text", "name": "备注",
+                                      "defaultValue": single}],
+                                    ensure_ascii=False)])
+        check("create-partial: update-fields refuses it clearly too",
+              not ok and "defaultValue" in str(res)
+              and "AttributeError" not in str(res), str(res)[:300])
+    finally:
+        if ws_id:
+            _purge(ws_id)
+
+
 def case_fb_subgroups(app_id: str, section_id: str) -> None:
     """WO-FB 3.1: the editor can see worksheets in second-level groups."""
     print("\nscenario: sub-group visibility (WO-FB 3.1)")
@@ -4792,6 +4875,7 @@ _CASES = {
     "fb-layout-check": case_fb_layout_check,
     "fb-pair-relation": case_fb_pair_relation,
     "fb-subtable-read": case_fb_subtable_read,
+    "create-partial": case_create_partial,
     "editor-inline-subtable": case_editor_inline_subtable,
     "fb-subgroups": case_fb_subgroups,
     "fb-button-target": case_fb_button_target,

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli/__init__.py

@@ -3,7 +3,7 @@
 import sys as _sys
 import types as _types
 
-__version__ = "0.9.2"
+__version__ = "0.9.3"
 
 # The command tree used to live in hap_cli/hap_cli.py. A module named
 # after the package it sits in shadows that package: run python with the

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli/context.py

@@ -84,6 +84,11 @@ class Context:
         is_auth = isinstance(e, (AuthenticationError, SessionError))
         if self.json_mode:
             payload = {"error": msg, "type": type(e).__name__}
+            # An error may carry state the caller needs to act on, e.g.
+            # the id of a worksheet a failed create left behind.
+            details = getattr(e, "details", None)
+            if isinstance(details, dict):
+                payload.update(details)
             if is_auth:
                 payload["hint"] = NOT_LOGGED_IN_HINT
                 payload["not_logged_in"] = True

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli/core/field_spec_adapter.py

@@ -156,6 +156,12 @@ def adapt_relation_config(spec: dict[str, Any]) -> None:
         spec["advanced_setting"] = adv
 
 
+# The ``source`` values build_defsource below can encode. Anything else
+# was skipped there without a word; the field-spec check refuses it
+# up front instead (bug #455775).
+DEFAULT_VALUE_SOURCES = ("system", "static", "field", "relation")
+
+
 def build_defsource(
     default_value: list[dict[str, Any]],
     *,

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli/core/worksheet.py

@@ -490,9 +490,24 @@ def create_worksheet_with_fields(
         ``{"worksheetId": "...", "saveResult": <SaveWorksheetControls
            response>}``
     """
-    # Checked here and not only inside update_fields below: that runs
-    # after the worksheet exists, so a bad spec used to leave one behind.
-    check_field_specs(fields)
+    # Shape the whole field payload before the worksheet exists. Every
+    # step of it is offline here (no default columns are kept, so nothing
+    # is read back), and it is the same shaping update_fields and
+    # save_controls run below — so any spec they would refuse is refused
+    # now, instead of after AddWorkSheet left a worksheet behind with
+    # only its auto-created columns (bug #455775). The host id only
+    # feeds a two-way relation's reverse half; a stand-in builds it the
+    # same way the real one will.
+    preview = build_fields_payload(
+        fields, worksheet_id=_PREFLIGHT_WORKSHEET_ID, title_name=title_name,
+        keep_default_controls=False)
+    _normalize_defsource(_checked_values(preview))
+    # Resolved here rather than left to create_worksheet (which would do
+    # the same lookup): undoing a failed create needs the group too, and
+    # without it the removal does not reach the recycle bin (seen live
+    # 2026-10-08).
+    if not section_id:
+        section_id = _first_app_section_id(session, app_id)
     created = create_worksheet(
         session, app_id, name,
         project_id=project_id, section_id=section_id,
@@ -505,18 +520,86 @@ def create_worksheet_with_fields(
         raise ValueError(
             f"AddWorkSheet did not return a worksheetId; got {created!r}"
         )
-    save_result = update_fields(
-        session, worksheet_id, fields,
-        title_name=title_name,
-        # Match `worksheet update-fields` CLI: drop the auto-created
-        # 名称/描述/附件 columns so the caller's `fields[]` is the full
-        # layout. Callers wanting to preserve them can issue an extra
-        # update-fields call with keep_default_controls=True.
-        keep_default_controls=False,
-    )
+    try:
+        save_result = update_fields(
+            session, worksheet_id, fields,
+            title_name=title_name,
+            # Match `worksheet update-fields` CLI: drop the auto-created
+            # 名称/描述/附件 columns so the caller's `fields[]` is the full
+            # layout. Callers wanting to preserve them can issue an extra
+            # update-fields call with keep_default_controls=True.
+            keep_default_controls=False,
+        )
+        # The save reports a rejection in its body rather than raising.
+        if isinstance(save_result, dict) and "code" in save_result \
+                and save_result.get("code") != 1:
+            raise APIError(
+                f"The fields were not saved: "
+                f"{save_result.get('msg') or save_result.get('code')}",
+                code=save_result.get("code"))
+    except Exception as exc:
+        raise _undo_partial_create(
+            session, exc, app_id=app_id, project_id=project_id,
+            section_id=section_id, worksheet_id=worksheet_id) from exc
     return {"worksheetId": worksheet_id, "saveResult": save_result}
 
 
+_PREFLIGHT_WORKSHEET_ID = "0" * 24
+
+
+class WorksheetCreateError(APIError):
+    """A create that failed after the worksheet already existed.
+
+    ``details`` is added to the CLI's JSON error output, so a caller
+    reading it can tell whether the worksheet is gone or still there.
+    """
+
+    def __init__(self, message: str, *, worksheet_id: str,
+                 rolled_back: bool, code: Optional[int] = None):
+        super().__init__(message, code=code)
+        self.worksheet_id = worksheet_id
+        self.rolled_back = rolled_back
+        self.details = {"worksheetId": worksheet_id,
+                        "rolledBack": rolled_back}
+
+
+def _undo_partial_create(session: Session, exc: Exception, *, app_id: str,
+                         project_id: str, section_id: str,
+                         worksheet_id: str) -> WorksheetCreateError:
+    """Remove the worksheet a failed create left behind.
+
+    It is removed the way the product removes one — to the recycle bin,
+    then out of it (``app.purge_app_item``) — since it holds nothing but
+    the auto-created columns and keeping it in the bin would only leave
+    a stray to clean up. When that fails as well, the error names the
+    worksheet so the caller can finish the job. Either way the original
+    reason leads the message.
+    """
+    from hap_cli.core import app as app_mod
+    reason = str(exc).rstrip(".") or type(exc).__name__
+    code = getattr(exc, "code", None)
+    logger.warning(
+        "worksheet create: fields failed on new worksheet %s (%s); "
+        "removing it", worksheet_id, reason)
+    try:
+        app_mod.purge_app_item(
+            session, app_id=app_id, project_id=project_id,
+            section_id=section_id, item_id=worksheet_id, item_type=0)
+    except Exception as undo_exc:  # noqa: BLE001 — reported below
+        logger.warning("worksheet create: could not remove %s: %s",
+                       worksheet_id, undo_exc, exc_info=True)
+        return WorksheetCreateError(
+            f"{reason}. Worksheet {worksheet_id} was created but its "
+            f"fields were not written — delete it, or retry with "
+            f"'hap worksheet update-fields {worksheet_id}'.",
+            worksheet_id=worksheet_id, rolled_back=False, code=code)
+    logger.warning("worksheet create: removed %s", worksheet_id)
+    return WorksheetCreateError(
+        f"{reason}. The worksheet it had just created was undone; "
+        f"nothing was left in the app.",
+        worksheet_id=worksheet_id, rolled_back=True, code=code)
+
+
 def copy_worksheet(
     session: Session,
     worksheet_id: str,
@@ -1026,6 +1109,7 @@ def check_field_specs(fields: Any) -> None:
                 f'"Text" / "Number" / "Date" / "Dropdown".')
         name = spec.get("name")
         if isinstance(name, str) and name.strip():
+            _check_default_value(spec, f'{where} ("{name}")')
             continue
         alias = next((k for k in _FIELD_SPEC_NAME_ALIASES if k in spec), "")
         if alias:
@@ -1041,6 +1125,41 @@ def check_field_specs(fields: Any) -> None:
             f'name it will show under.')
 
 
+_DEFAULT_VALUE_EXAMPLE = '"defaultValue": [{"source": "static", "value": "..."}]'
+
+
+def _check_default_value(spec: dict[str, Any], where: str) -> None:
+    """Refuse a ``defaultValue`` the field shaping cannot read.
+
+    It is a list of objects, each naming a ``source`` that
+    ``field_spec_adapter.build_defsource`` encodes. A single object or a
+    plain string used to pass this check and crash the shaping halfway —
+    on a create, after the worksheet already existed (bug #455775). An
+    unknown ``source`` was dropped there silently.
+    """
+    from hap_cli.core.field_spec_adapter import DEFAULT_VALUE_SOURCES
+    value = spec.get("defaultValue")
+    if value in (None, []):
+        return
+    sources = ", ".join(DEFAULT_VALUE_SOURCES)
+    rule = (f"Write it as a list of objects, e.g. {_DEFAULT_VALUE_EXAMPLE}; "
+            f'"source" is one of {sources}.')
+    if not isinstance(value, list):
+        kind = "a single object" if isinstance(value, dict) \
+            else type(value).__name__
+        raise ValueError(f'{where} has "defaultValue" as {kind}. {rule}')
+    for n, entry in enumerate(value, start=1):
+        if not isinstance(entry, dict):
+            raise ValueError(
+                f'{where}: "defaultValue" entry {n} is '
+                f"{type(entry).__name__}, not an object. {rule}")
+        if entry.get("source") not in DEFAULT_VALUE_SOURCES:
+            got = entry.get("source")
+            said = f'"source" {got!r}' if got is not None else 'no "source"'
+            raise ValueError(
+                f'{where}: "defaultValue" entry {n} has {said}. {rule}')
+
+
 def build_fields_payload(
     fields: list[dict[str, Any]],
     *,

hap_cli-0.9.3/hap_cli/tests/test_create_worksheet_partial.py

@@ -0,0 +1,231 @@
+"""Bug #455775: `worksheet create --fields` left a half-built worksheet.
+
+A Text field with ``"defaultValue": {"source": "static", ...}`` (one
+object, not a list) passed the pre-create check, the worksheet was
+created, and shaping the fields then crashed with ``AttributeError`` —
+the app kept a worksheet holding only the auto-created columns and the
+caller never learned its id.
+
+Two layers: every field-shaping error now surfaces before the worksheet
+exists, and a failure after it exists undoes the create (or, when that
+fails too, names the worksheet that was left behind).
+"""
+import json
+
+import pytest
+from click.testing import CliRunner
+
+from hap_cli.core import app as app_mod
+from hap_cli.core import worksheet as ws_mod
+from hap_cli.core.session import APIError, Session
+
+
+GOOD = {"type": "Text", "name": "名称"}
+
+
+@pytest.fixture
+def tmp_config(tmp_path, monkeypatch):
+    cfg = tmp_path / ".hap-cli"
+    monkeypatch.setattr("hap_cli.core.session.CONFIG_DIR", cfg)
+    monkeypatch.setattr("hap_cli.core.session.CONFIG_FILE",
+                        cfg / "config.json")
+    monkeypatch.setattr("hap_cli.core.token_crypto._KEY_FILE", cfg / ".key")
+    return cfg
+
+
+def _with_default(value):
+    return [GOOD, {"type": "Text", "name": "备注", "defaultValue": value}]
+
+
+# ── defaultValue shape is checked before anything is sent ────────────────
+class TestDefaultValueShape:
+    @pytest.mark.parametrize("value", [
+        {"source": "static", "value": "DEFAULT-QA"},   # the report
+        "DEFAULT-QA",
+        ["x"],
+        [{"value": "x"}],                              # no source
+        [{"source": "constant", "value": "x"}],        # unknown source
+        7,
+    ])
+    def test_refused_with_field_and_right_shape(self, value):
+        with pytest.raises(ValueError) as exc:
+            ws_mod.check_field_specs(_with_default(value))
+        msg = str(exc.value)
+        assert "Field 2" in msg and "备注" in msg
+        assert '"defaultValue": [{"source": "static"' in msg
+        for name in ("system", "static", "field", "relation"):
+            assert name in msg
+
+    @pytest.mark.parametrize("value", [
+        None, [],
+        [{"source": "static", "value": "x"}],
+        [{"source": "system", "value": "currentUser"}],
+        [{"source": "field", "field": "abc"}],
+        [{"source": "relation", "relationField": "r", "field": "f"}],
+    ])
+    def test_accepted(self, value):
+        ws_mod.check_field_specs(_with_default(value))
+
+    def test_update_fields_refuses_cleanly_not_attribute_error(self):
+        with pytest.raises(ValueError, match="defaultValue"):
+            ws_mod.build_fields_payload(
+                _with_default({"source": "static", "value": "x"}),
+                keep_default_controls=False)
+
+
+# ── create: preflight builds the whole payload offline ───────────────────
+class _Recorder:
+    def __init__(self, monkeypatch, *, save=None, save_raises=None,
+                 purge_raises=None):
+        self.calls = []
+        rec = self
+
+        def fake_create(session, app_id, name, **kw):
+            rec.calls.append(("create", app_id, name, kw.get("section_id")))
+            return {"worksheetId": "ws_new"}
+
+        def fake_update(session, worksheet_id, fields, **kw):
+            rec.calls.append(("update", worksheet_id))
+            if save_raises:
+                raise save_raises
+            return save if save is not None else {"code": 1, "data": {}}
+
+        def fake_purge(session, **kw):
+            rec.calls.append(("purge", kw.get("item_id"), kw.get("item_type"),
+                              kw.get("app_id")))
+            if purge_raises:
+                raise purge_raises
+            return {"deleted": kw.get("item_id"), "permanently": True}
+
+        monkeypatch.setattr(ws_mod, "create_worksheet", fake_create)
+        monkeypatch.setattr(ws_mod, "update_fields", fake_update)
+        monkeypatch.setattr(app_mod, "purge_app_item", fake_purge)
+
+    def kinds(self):
+        return [c[0] for c in self.calls]
+
+
+def _create(fields, **kw):
+    return ws_mod.create_worksheet_with_fields(
+        object(), "app1", "QA", fields, project_id="org1",
+        section_id="sec1", **kw)
+
+
+class TestCreatePreflight:
+    def test_bad_default_never_creates(self, monkeypatch):
+        rec = _Recorder(monkeypatch)
+        with pytest.raises(ValueError, match="defaultValue"):
+            _create(_with_default({"source": "static", "value": "x"}))
+        assert rec.calls == []
+
+    def test_any_shaping_error_surfaces_before_create(self, monkeypatch):
+        # A field-settings value the save path refuses (an object where
+        # text belongs) used to be found only after the create.
+        rec = _Recorder(monkeypatch)
+        bad = [GOOD, {"type": "Text", "name": "x",
+                      "advancedSetting": {"sorttype": {"a": 1}}}]
+        with pytest.raises(ValueError):
+            _create(bad)
+        assert rec.calls == []
+
+    def test_good_spec_creates_and_saves(self, monkeypatch):
+        rec = _Recorder(monkeypatch)
+        out = _create([GOOD, {"type": "Text", "name": "备注", "defaultValue":
+                              [{"source": "static", "value": "D"}]}])
+        assert out["worksheetId"] == "ws_new"
+        assert rec.kinds() == ["create", "update"]
+
+
+# ── create: a failure after the worksheet exists ─────────────────────────
+class TestCreateRollback:
+    def test_rollback_knows_the_group_when_none_was_given(self, monkeypatch):
+        rec = _Recorder(monkeypatch, save_raises=APIError("x"))
+        monkeypatch.setattr(ws_mod, "_first_app_section_id",
+                            lambda session, app_id: "sec-first")
+        seen = {}
+        orig = app_mod.purge_app_item
+
+        def spy(session, **kw):
+            seen.update(kw)
+            return orig(session, **kw)
+
+        monkeypatch.setattr(app_mod, "purge_app_item", spy)
+        with pytest.raises(ws_mod.WorksheetCreateError):
+            ws_mod.create_worksheet_with_fields(
+                object(), "app1", "QA", [GOOD], project_id="org1")
+        assert seen["section_id"] == "sec-first"
+        assert rec.calls[0][3] == "sec-first"
+
+    def test_save_error_rolls_back_and_keeps_original_reason(
+            self, monkeypatch):
+        rec = _Recorder(monkeypatch, save_raises=APIError("字段保存失败"))
+        with pytest.raises(ws_mod.WorksheetCreateError) as exc:
+            _create([GOOD])
+        assert rec.kinds() == ["create", "update", "purge"]
+        assert rec.calls[2][1:] == ("ws_new", 0, "app1")
+        msg = str(exc.value)
+        assert "字段保存失败" in msg
+        assert "undone" in msg
+        assert exc.value.details == {"worksheetId": "ws_new",
+                                     "rolledBack": True}
+
+    def test_rejected_save_code_rolls_back(self, monkeypatch):
+        rec = _Recorder(monkeypatch, save={"code": 16, "msg": "别名重复"})
+        with pytest.raises(ws_mod.WorksheetCreateError, match="别名重复"):
+            _create([GOOD])
+        assert "purge" in rec.kinds()
+
+    def test_failed_rollback_names_the_worksheet(self, monkeypatch):
+        rec = _Recorder(monkeypatch, save_raises=RuntimeError("boom"),
+                        purge_raises=APIError("no"))
+        with pytest.raises(ws_mod.WorksheetCreateError) as exc:
+            _create([GOOD])
+        msg = str(exc.value)
+        assert "boom" in msg and "ws_new" in msg
+        assert "update-fields" in msg
+        assert exc.value.details == {"worksheetId": "ws_new",
+                                     "rolledBack": False}
+        assert rec.kinds() == ["create", "update", "purge"]
+
+    def test_rollback_is_logged(self, monkeypatch):
+        import logging
+        seen = []
+
+        class Grab(logging.Handler):
+            def emit(self, record):
+                seen.append(record)
+
+        log = logging.getLogger("hap_cli.worksheet")
+        handler = Grab(level=logging.WARNING)
+        old_level = log.level
+        log.addHandler(handler)
+        log.setLevel(logging.WARNING)
+        try:
+            _Recorder(monkeypatch, save_raises=APIError("x"))
+            with pytest.raises(ws_mod.WorksheetCreateError):
+                _create([GOOD])
+        finally:
+            log.removeHandler(handler)
+            log.setLevel(old_level)
+        assert any("ws_new" in r.getMessage() for r in seen
+                   if r.levelno == logging.WARNING)
+
+
+def test_cli_json_error_carries_worksheet_id(tmp_config, monkeypatch):
+    from hap_cli._entry import cli
+    sess = Session(server_url="https://t.com", auth_token="tok")
+    sess.default_project_id = "proj1"
+    sess.save()
+
+    def boom(*a, **kw):
+        raise ws_mod.WorksheetCreateError(
+            "x. Worksheet ws_new was created but its fields were not "
+            "written", worksheet_id="ws_new", rolled_back=False)
+
+    monkeypatch.setattr(ws_mod, "create_worksheet_with_fields", boom)
+    res = CliRunner().invoke(cli, ["--json", "worksheet", "create", "app1",
+                                   "QA", "--fields", json.dumps([GOOD])])
+    assert res.exit_code == 1
+    payload = json.loads(res.output)
+    assert payload["worksheetId"] == "ws_new"
+    assert payload["rolledBack"] is False

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli.egg-info/PKG-INFO

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

{hap_cli-0.9.2 → hap_cli-0.9.3}/hap_cli.egg-info/SOURCES.txt

@@ -216,6 +216,7 @@ hap_cli/tests/test_config_cmd.py
 hap_cli/tests/test_content_language.py
 hap_cli/tests/test_control_lookup_guard.py
 hap_cli/tests/test_core.py
+hap_cli/tests/test_create_worksheet_partial.py
 hap_cli/tests/test_destructive_confirm_cli.py
 hap_cli/tests/test_dsl_existing_node_refs.py
 hap_cli/tests/test_dsl_multi_record_guard.py