hap-cli 0.8.2__tar.gz → 0.8.4__tar.gz

{hap_cli-0.8.2 → hap_cli-0.8.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hap-cli
-Version: 0.8.2
+Version: 0.8.4
 Summary: CLI harness for MingDAO HAP - Enterprise no-code platform
 Author: hap-cli
 License: Apache-2.0
@@ -155,7 +155,7 @@ discover every subcommand.
 | `app create-section / edit-section / delete-section` | Section management |
 | `app sort-sections / sort-worksheets` | Reorder sections / a section's worksheets |
 | `app create-item` | Batch-create empty worksheets / custom pages |
-| `app backup / restore / export / exports / backup-logs` | Backup & export |
+| `app backup / export / exports / backup-logs` | Backup & export |
 | `app usage / logs` | Statistics and operation logs |
 | `app role …` | Role CRUD, permissions, members, leave-all, applications |
 | `app optionset …` | Option set CRUD and cross-app moves |

{hap_cli-0.8.2/hap_cli → hap_cli-0.8.4}/README.md

@@ -126,7 +126,7 @@ discover every subcommand.
 | `app create-section / edit-section / delete-section` | Section management |
 | `app sort-sections / sort-worksheets` | Reorder sections / a section's worksheets |
 | `app create-item` | Batch-create empty worksheets / custom pages |
-| `app backup / restore / export / exports / backup-logs` | Backup & export |
+| `app backup / export / exports / backup-logs` | Backup & export |
 | `app usage / logs` | Statistics and operation logs |
 | `app role …` | Role CRUD, permissions, members, leave-all, applications |
 | `app optionset …` | Option set CRUD and cross-app moves |

{hap_cli-0.8.2 → hap_cli-0.8.4/hap_cli}/README.md

@@ -126,7 +126,7 @@ discover every subcommand.
 | `app create-section / edit-section / delete-section` | Section management |
 | `app sort-sections / sort-worksheets` | Reorder sections / a section's worksheets |
 | `app create-item` | Batch-create empty worksheets / custom pages |
-| `app backup / restore / export / exports / backup-logs` | Backup & export |
+| `app backup / export / exports / backup-logs` | Backup & export |
 | `app usage / logs` | Statistics and operation logs |
 | `app role …` | Role CRUD, permissions, members, leave-all, applications |
 | `app optionset …` | Option set CRUD and cross-app moves |

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/README_CN.md

@@ -117,7 +117,7 @@ hap --json worksheet record list WORKSHEET_ID
 | `app create-section / edit-section / delete-section` | 分组管理 |
 | `app sort-sections / sort-worksheets` | 重排分组 / 分组内工作表顺序 |
 | `app create-item` | 批量创建空白工作表/自定义页面 |
-| `app backup / restore / export / exports / backup-logs` | 备份与导出 |
+| `app backup / export / exports / backup-logs` | 备份与导出 |
 | `app usage / logs` | 使用统计与操作日志 |
 | `app role …` | 角色 CRUD、权限、成员、leave-all、申请 |
 | `app optionset …` | 选项集 CRUD 与跨应用迁移 |

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/__init__.py

@@ -1,3 +1,3 @@
 """CLI harness for MingDAO HAP (hap-cli)."""
 
-__version__ = "0.8.2"
+__version__ = "0.8.4"

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/commands/app_cmd.py

@@ -410,37 +410,6 @@ def app_backup(ctx, app_id, org_id, no_data):
         ctx.handle_error(e)
 
 
-@app.command("restore")
-@click.argument("app_id")
-@org_id_option()
-@click.option("--backup-id", "-b", default="", help="Restore from an existing backup (its ID)")
-@click.option("--file-url", default="", help="Restore from an uploaded backup file (its URL)")
-@click.option("--file-name", default="", help="File name to show for an uploaded backup file")
-@click.option("--no-data", is_flag=True, help="Restore without data")
-@click.option("--as-new", is_flag=True, help="Restore as a new app")
-@pass_context
-def app_restore(ctx, app_id, org_id, backup_id, file_url, file_name, no_data, as_new):
-    """Restore an application, either from an existing backup or an uploaded file.
-
-    Provide exactly one source: --backup-id (an existing backup) or
-    --file-url (an uploaded backup file).
-    """
-    try:
-        if bool(backup_id) == bool(file_url):
-            raise click.UsageError(
-                "Provide exactly one of --backup-id or --file-url."
-            )
-        session = ctx.get_session()
-        data = app_mod.restore_app(
-            session, app_id, require_api_project_id(session, org_id),
-            backup_id=backup_id, file_url=file_url, file_name=file_name,
-            contain_data=not no_data, is_restore_new=as_new,
-        )
-        ctx.output(data, lambda d: click.echo(f"Restore started: {d}"))
-    except Exception as e:
-        ctx.handle_error(e)
-
-
 @app.command("export")
 @click.argument("app_ids", nargs=-1, required=True)
 @org_id_option()

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/commands/node_cmd.py

@@ -372,12 +372,16 @@ def node_form_property(ctx, process_id, node_id):
 
 @node.command("sub-processes")
 @click.argument("process_id")
+@click.argument("node_id")
 @pass_context
-def node_sub_processes(ctx, process_id):
-    """List available sub-processes."""
+def node_sub_processes(ctx, process_id, node_id):
+    """List sub-processes available to a sub-process node.
+
+    NODE_ID is the node being configured.
+    """
     try:
         session = ctx.get_session()
-        data = node_mod.get_sub_process_list(session, process_id)
+        data = node_mod.get_sub_process_list(session, process_id, select_node_id=node_id)
         ctx.output(data, lambda d: output_json(d))
     except Exception as e:
         ctx.handle_error(e)

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/core/app.py

@@ -620,55 +620,6 @@ def backup_app(
     )
 
 
-def restore_app(
-    session: Session,
-    app_id: str,
-    project_id: str,
-    backup_id: str = "",
-    file_url: str = "",
-    file_name: str = "",
-    contain_data: bool = True,
-    is_restore_new: bool = False,
-    auto_end_maintain: bool = False,
-    backup_current_version: bool = False,
-) -> dict[str, Any]:
-    """Restore an application from a backup log id or an uploaded file.
-
-    The HAP UI has two mutually exclusive restore modes; the API field
-    set differs between them, so we must send only one mode's fields:
-
-    - Backup-log restore (``src/pages/AppSettings/components/
-      appBackupRestore/components/BackupFiles.jsx``): sends ``id`` (the
-      backup-restore operation-log id) plus ``autoEndMaintain``,
-      ``backupCurrentVersion`` and ``isRestoreNew`` — no ``fileUrl``.
-    - Uploaded-file restore (``.../BackupFromFiles.jsx``): sends
-      ``fileUrl`` + ``fileName`` + ``containData`` plus the same three
-      flags — no ``id``.
-
-    Sending both at once (as an earlier wrapper did, with all three
-    forced ``required``) made the command impossible to invoke. Exactly
-    one of ``backup_id`` / ``file_url`` must be supplied.
-    """
-    if not backup_id and not file_url:
-        raise ValueError("restore requires either backup_id or file_url")
-
-    params: dict[str, Any] = {
-        "appId": app_id,
-        "projectId": project_id,
-        "autoEndMaintain": auto_end_maintain,
-        "backupCurrentVersion": backup_current_version,
-        "isRestoreNew": is_restore_new,
-    }
-    if backup_id:
-        # `id` is the backup-restore operation-log id (named `id` in the API).
-        params["id"] = backup_id
-    else:
-        params["fileUrl"] = file_url
-        params["fileName"] = file_name
-        params["containData"] = contain_data
-    return session.api_call("AppManagement", "Restore", params)
-
-
 def export_apps(
     session: Session,
     project_id: str,

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/core/flow_node.py

@@ -274,7 +274,9 @@ def get_node_detail(
         data["appId"] = app_id
     if instance_id:
         data["instanceId"] = instance_id
-    return session.workflow_call("flowNode/getNodeDetail", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getNodeDetail", data, method="GET")
 
 
 # ── Node Update ──────────────────────────────────────────────────────────
@@ -572,7 +574,9 @@ def get_webhook_data(
     data: dict[str, Any] = {"processId": process_id, "nodeId": node_id}
     if select_node_id:
         data["selectNodeId"] = select_node_id
-    return session.workflow_call("flowNode/getWebHookData", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getWebHookData", data, method="GET")
 
 
 # ── Utilities ────────────────────────────────────────────────────────────
@@ -617,7 +621,9 @@ def get_node_form_property(
     data: dict[str, Any] = {"processId": process_id, "nodeId": node_id}
     if select_node_id:
         data["selectNodeId"] = select_node_id
-    return session.workflow_call("flowNode/getNodeFormProperty", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getNodeFormProperty", data, method="GET")
 
 
 def get_sub_process_list(
@@ -630,7 +636,9 @@ def get_sub_process_list(
     Args:
         session: Active session
         process_id: Process ID
-        select_node_id: Context node
+        select_node_id: Context node. flowNode.js marks ``selectNodeId``
+            as required (``*选中节点ID``); omitting it makes the server
+            answer code 2000 "节点不存在".
 
     Returns:
         Sub-process list
@@ -638,7 +646,9 @@ def get_sub_process_list(
     data: dict[str, Any] = {"processId": process_id}
     if select_node_id:
         data["selectNodeId"] = select_node_id
-    return session.workflow_call("flowNode/getSubProcessList", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getSubProcessList", data, method="GET")
 
 
 # ── Code Templates ───────────────────────────────────────────────────────
@@ -672,7 +682,9 @@ def get_code_template_list(
         data["source"] = source
     if template_type:
         data["type"] = template_type
-    return session.workflow_call("flowNode/getCodeTemplateList", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getCodeTemplateList", data, method="GET")
 
 
 def create_code_template(
@@ -774,7 +786,9 @@ def get_flow_app_dtos(
         data["enumDefault"] = enum_default
     if field_type:
         data["type"] = field_type
-    return session.workflow_call("flowNode/getFlowAppDtos", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getFlowAppDtos", data, method="GET")
 
 
 def get_start_event_deploy(
@@ -809,7 +823,9 @@ def get_start_event_deploy(
         data["controlId"] = control_id
     if select_node_id:
         data["selectNodeId"] = select_node_id
-    return session.workflow_call("flowNode/getStartEventDeploy", data)
+    # Evidence: pd-openweb flowNode.js pins ``base.ajaxOptions.type = 'GET'``
+    # for this action; POSTing it returns 404 Not Found.
+    return session.workflow_call("flowNode/getStartEventDeploy", data, method="GET")
 
 
 # ── Data Processing Convenience Functions ────────────────────────────────

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/core/record.py

@@ -56,13 +56,26 @@ def _legacy_cell_value(ctrl: dict[str, Any], value: Any) -> str:
     if t == 10:  # MULTI_SELECT
         return json.dumps(_resolve_option_keys(ctrl, value), ensure_ascii=False)
 
-    if t == 29:  # RELATE_SHEET -> [{"sid": rowId}, ...]
+    if t in (29, 35):  # RELATE_SHEET / CASCADING_SELECT -> [{"sid": rowId}, ...]
+        # Cascade (35) is a relation to a row in another worksheet, persisted
+        # the same way as a plain relate (29): an array of {"sid": rowId}. A
+        # bare rowId makes the server answer "服务异常" (bug #642cc8).
         if isinstance(value, (list, tuple)):
             ids = [str(v) for v in value]
         else:
             ids = [s for s in str(value).split(",") if s != ""]
         return json.dumps([{"sid": rid} for rid in ids], ensure_ascii=False)
 
+    if t == 48:  # ORG_ROLE -> [{"organizeId": id}, ...]
+        # Mirrors DEPARTMENT (27): the legacy AddWorksheetRow takes an array
+        # of {"organizeId": id} objects; organizeName is display-only and the
+        # server resolves it from the id. A bare organizeId is rejected with
+        # "服务异常" (bug #642cc8). See pd-openweb CellControls/OrgRole.jsx.
+        ids = value if isinstance(value, (list, tuple)) else \
+            [s for s in str(value).split(",") if s != ""]
+        return json.dumps([{"organizeId": str(i)} for i in ids],
+                          ensure_ascii=False)
+
     if t == 26:  # USER_PICKER
         ids = value if isinstance(value, (list, tuple)) else \
             [s for s in str(value).split(",") if s != ""]

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/locale/messages.json

@@ -900,27 +900,6 @@
     "app optionset update --options-json": {
       "en": "Option list.  [JSON value]  [required]"
     },
-    "app restore": {
-      "en": "Restore an application from backup."
-    },
-    "app restore --as-new": {
-      "en": "Restore as a new app"
-    },
-    "app restore --backup-id": {
-      "en": "Backup ID"
-    },
-    "app restore --file-name": {
-      "en": "Backup file name"
-    },
-    "app restore --file-url": {
-      "en": "Backup file URL"
-    },
-    "app restore --no-data": {
-      "en": "Restore without data"
-    },
-    "app restore --org-id": {
-      "en": "Organization ID"
-    },
     "app role": {
       "en": "Role management."
     },

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/tests/test_core.py

@@ -486,6 +486,38 @@ class TestRecord:
         # endpoint is the legacy AddWorksheetRow
         assert mock_post.call_args[0][0].endswith("/Worksheet/AddWorksheetRow")
 
+    @patch("hap_cli.core.worksheet.get_worksheet_controls")
+    @patch("hap_cli.core.session.requests.post")
+    def test_legacy_batch_create_serializes_orgrole_and_cascade(
+        self, mock_post, mock_controls, mock_session
+    ):
+        # Org-role (48) and cascading-select (35) cells must be wrapped in
+        # the wire shapes the legacy AddWorksheetRow expects — a bare id
+        # makes the server answer "服务异常". OrgRole -> [{"organizeId": id}];
+        # Cascade is a relation-to-a-row -> [{"sid": rowId}] (like type 29).
+        mock_controls.return_value = [
+            {"controlId": "cOrg", "controlName": "组织角色", "type": 48,
+             "dot": 0},
+            {"controlId": "cCas", "controlName": "级联选择", "type": 35,
+             "dot": 0},
+        ]
+        mock_post.return_value = _mock_response({"rowid": "r_new"})
+
+        rec_mod.batch_create_by_name(
+            mock_session,
+            "ws1",
+            [{
+                "组织角色": "40d102b2-12bc-48d2-9364-f57070502609",
+                "级联选择": "97ebc5ba-4d57",
+            }],
+        )
+
+        rc = {c["controlId"]: c
+              for c in mock_post.call_args[1]["json"]["receiveControls"]}
+        assert json.loads(rc["cOrg"]["value"]) == [
+            {"organizeId": "40d102b2-12bc-48d2-9364-f57070502609"}]
+        assert json.loads(rc["cCas"]["value"]) == [{"sid": "97ebc5ba-4d57"}]
+
     @patch("hap_cli.core.worksheet.get_worksheet_controls")
     @patch("hap_cli.core.session.requests.post")
     def test_cli_batch_create_uses_main_api(
@@ -899,10 +931,18 @@ class TestFlowNode:
         assert not mock_post.called, "flowNode/get must not be POSTed"
         assert "startId" in result
 
+    @patch("hap_cli.core.session.requests.get")
     @patch("hap_cli.core.session.requests.post")
-    def test_get_node_detail(self, mock_post, mock_session):
-        mock_post.return_value = _mock_response({"nodeId": "n1", "flowNodeType": 4})
+    def test_get_node_detail(self, mock_post, mock_get, mock_session):
+        """``flowNode/getNodeDetail`` is GET per ``flowNode.js`` (pins
+        ``base.ajaxOptions.type = 'GET'``). POSTing returns 404."""
+        mock_get.return_value = _mock_response({"nodeId": "n1", "flowNodeType": 4})
         result = node_mod.get_node_detail(mock_session, "wf1", "n1")
+        assert mock_get.called, "flowNode/getNodeDetail must be issued as GET"
+        assert not mock_post.called
+        params = mock_get.call_args[1]["params"]
+        assert params["processId"] == "wf1"
+        assert params["nodeId"] == "n1"
         assert result["flowNodeType"] == 4
 
     @patch("hap_cli.core.session.requests.post")
@@ -972,12 +1012,18 @@ class TestFlowNode:
         body = mock_post.call_args[1]["json"]
         assert body["json"] == '{"name": "test"}'
 
+    @patch("hap_cli.core.session.requests.get")
     @patch("hap_cli.core.session.requests.post")
-    def test_get_code_template_list(self, mock_post, mock_session):
-        mock_post.return_value = _mock_response([{"id": "t1", "name": "Template1"}])
+    def test_get_code_template_list(self, mock_post, mock_get, mock_session):
+        """``flowNode/getCodeTemplateList`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response([{"id": "t1", "name": "Template1"}])
         result = node_mod.get_code_template_list(mock_session, keyword="test")
-        body = mock_post.call_args[1]["json"]
-        assert body["keyword"] == "test"
+        assert mock_get.called, "flowNode/getCodeTemplateList must be issued as GET"
+        assert not mock_post.called
+        params = mock_get.call_args[1]["params"]
+        assert params["keyword"] == "test"
+        assert result[0]["id"] == "t1"
 
     @patch("hap_cli.core.session.requests.post")
     def test_create_code_template(self, mock_post, mock_session):
@@ -987,12 +1033,64 @@ class TestFlowNode:
         assert body["name"] == "My Tmpl"
         assert body["code"] == "console.log('hi')"
 
+    @patch("hap_cli.core.session.requests.get")
     @patch("hap_cli.core.session.requests.post")
-    def test_get_sub_process_list(self, mock_post, mock_session):
-        mock_post.return_value = _mock_response([{"id": "sub1"}])
-        result = node_mod.get_sub_process_list(mock_session, "wf1")
-        body = mock_post.call_args[1]["json"]
-        assert body["processId"] == "wf1"
+    def test_get_sub_process_list(self, mock_post, mock_get, mock_session):
+        """``flowNode/getSubProcessList`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response([{"id": "sub1"}])
+        result = node_mod.get_sub_process_list(mock_session, "wf1", select_node_id="n1")
+        assert mock_get.called, "flowNode/getSubProcessList must be issued as GET"
+        assert not mock_post.called
+        params = mock_get.call_args[1]["params"]
+        assert params["processId"] == "wf1"
+        # selectNodeId is required by the server (code 2000 without it)
+        assert params["selectNodeId"] == "n1"
+        assert result[0]["id"] == "sub1"
+
+    @patch("hap_cli.core.session.requests.get")
+    @patch("hap_cli.core.session.requests.post")
+    def test_get_node_form_property(self, mock_post, mock_get, mock_session):
+        """``flowNode/getNodeFormProperty`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response({"controls": []})
+        result = node_mod.get_node_form_property(mock_session, "wf1", "n1")
+        assert mock_get.called, "flowNode/getNodeFormProperty must be issued as GET"
+        assert not mock_post.called
+        params = mock_get.call_args[1]["params"]
+        assert params["processId"] == "wf1"
+        assert params["nodeId"] == "n1"
+        assert "controls" in result
+
+    @patch("hap_cli.core.session.requests.get")
+    @patch("hap_cli.core.session.requests.post")
+    def test_get_webhook_data_uses_get(self, mock_post, mock_get, mock_session):
+        """``flowNode/getWebHookData`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response({"data": "{}"})
+        node_mod.get_webhook_data(mock_session, "wf1", "n1")
+        assert mock_get.called, "flowNode/getWebHookData must be issued as GET"
+        assert not mock_post.called
+
+    @patch("hap_cli.core.session.requests.get")
+    @patch("hap_cli.core.session.requests.post")
+    def test_get_flow_app_dtos_uses_get(self, mock_post, mock_get, mock_session):
+        """``flowNode/getFlowAppDtos`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response([{"id": "app1"}])
+        node_mod.get_flow_app_dtos(mock_session, "wf1", "n1")
+        assert mock_get.called, "flowNode/getFlowAppDtos must be issued as GET"
+        assert not mock_post.called
+
+    @patch("hap_cli.core.session.requests.get")
+    @patch("hap_cli.core.session.requests.post")
+    def test_get_start_event_deploy_uses_get(self, mock_post, mock_get, mock_session):
+        """``flowNode/getStartEventDeploy`` is GET per ``flowNode.js``.
+        POSTing returns 404."""
+        mock_get.return_value = _mock_response({"processId": "wf1"})
+        node_mod.get_start_event_deploy(mock_session, "wf1", "n1")
+        assert mock_get.called, "flowNode/getStartEventDeploy must be issued as GET"
+        assert not mock_post.called
 
     @patch("hap_cli.core.session.requests.post")
     def test_add_node_with_action_id(self, mock_post, mock_session):
@@ -6290,7 +6388,7 @@ class TestShellCompletion:
         comp = BashComplete(cli, {}, "hap", "_HAP_COMPLETE")
         items = comp.get_completions(["app"], "r")
         values = {item.value for item in items}
-        # app has a `role` subcommand; `restore` is fine too if it exists
+        # app has a `role` subcommand that completes from the "r" prefix
         assert "role" in values
 
     def test_completion_engine_completes_option_names(self):

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli/tests/test_org_id_cli.py

@@ -15,7 +15,6 @@ ORG_ID_HELP_CASES = [
     # as a body parameter (schema property ``org_id``).
     ["app", "create"],
     ["app", "backup"],
-    ["app", "restore"],
     ["app", "backup-logs"],
     ["app", "usage"],
     ["contact", "search"],

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hap-cli
-Version: 0.8.2
+Version: 0.8.4
 Summary: CLI harness for MingDAO HAP - Enterprise no-code platform
 Author: hap-cli
 License: Apache-2.0
@@ -155,7 +155,7 @@ discover every subcommand.
 | `app create-section / edit-section / delete-section` | Section management |
 | `app sort-sections / sort-worksheets` | Reorder sections / a section's worksheets |
 | `app create-item` | Batch-create empty worksheets / custom pages |
-| `app backup / restore / export / exports / backup-logs` | Backup & export |
+| `app backup / export / exports / backup-logs` | Backup & export |
 | `app usage / logs` | Statistics and operation logs |
 | `app role …` | Role CRUD, permissions, members, leave-all, applications |
 | `app optionset …` | Option set CRUD and cross-app moves |

{hap_cli-0.8.2 → hap_cli-0.8.4}/hap_cli.egg-info/SOURCES.txt

@@ -103,7 +103,6 @@ hap_cli/tests/__init__.py
 hap_cli/tests/conftest.py
 hap_cli/tests/test_app_editor.py
 hap_cli/tests/test_app_export_usage.py
-hap_cli/tests/test_app_restore.py
 hap_cli/tests/test_auth_prerelease.py
 hap_cli/tests/test_bug_036430.py
 hap_cli/tests/test_chart.py

hap_cli-0.8.2/hap_cli/tests/test_app_restore.py

@@ -1,123 +0,0 @@
-"""Bugfix coverage for `hap app restore` (bug #036418).
-
-The wrapper forced ``--backup-id``, ``--file-url`` and ``--file-name`` to
-all be ``required`` at once, but the HAP UI actually has two mutually
-exclusive restore modes (traced from pd-openweb):
-
-- Restore from an existing backup log
-  (``src/pages/AppSettings/components/appBackupRestore/components/BackupFiles.jsx``):
-  sends ``{appId, projectId, id, autoEndMaintain, backupCurrentVersion,
-  isRestoreNew}`` — NO fileUrl/fileName.
-- Restore from an uploaded file
-  (``.../BackupFromFiles.jsx``): sends ``{appId, projectId, fileUrl,
-  fileName, autoEndMaintain, backupCurrentVersion, isRestoreNew,
-  containData}`` — NO id.
-
-So the user only ever has one of (backup-id) or (file-url) on hand;
-requiring both made the command impossible to run.
-"""
-
-from unittest.mock import MagicMock, patch
-
-import pytest
-from click.testing import CliRunner
-
-from hap_cli.core import app as app_mod
-from hap_cli.core.session import Session
-from hap_cli.hap_cli import cli
-
-
-def _mock_response(data, state=1):
-    resp = MagicMock()
-    resp.status_code = 200
-    resp.json.return_value = {"data": data, "state": state}
-    resp.raise_for_status = MagicMock()
-    return resp
-
-
-@pytest.fixture
-def mock_session():
-    return Session(server_url="https://test.mingdao.com", auth_token="tok123")
-
-
-@pytest.fixture
-def logged_in(monkeypatch):
-    sess = Session(server_url="https://test.mingdao.com", auth_token="tok123")
-    sess.default_project_id = "org-default"
-    monkeypatch.setattr("hap_cli.context.Session.load", lambda *a, **k: sess)
-    return sess
-
-
-def _posted_body(mock_post):
-    return mock_post.call_args[1]["json"]
-
-
-class TestRestoreCore:
-    @patch("hap_cli.core.session.requests.post")
-    def test_backup_mode_sends_id_not_file(self, mock_post, mock_session):
-        mock_post.return_value = _mock_response(True)
-        app_mod.restore_app(
-            mock_session, "app1", "org1", backup_id="bk1", is_restore_new=True
-        )
-        body = _posted_body(mock_post)
-        assert body["id"] == "bk1"
-        assert "fileUrl" not in body
-        assert "fileName" not in body
-        assert body["appId"] == "app1"
-        assert body["projectId"] == "org1"
-        assert body["isRestoreNew"] is True
-        assert body["autoEndMaintain"] is False
-        assert body["backupCurrentVersion"] is False
-
-    @patch("hap_cli.core.session.requests.post")
-    def test_file_mode_sends_file_not_id(self, mock_post, mock_session):
-        mock_post.return_value = _mock_response(True)
-        app_mod.restore_app(
-            mock_session,
-            "app1",
-            "org1",
-            file_url="https://x/y.mdy",
-            file_name="y.mdy",
-            contain_data=True,
-        )
-        body = _posted_body(mock_post)
-        assert body["fileUrl"] == "https://x/y.mdy"
-        assert body["fileName"] == "y.mdy"
-        assert body["containData"] is True
-        assert "id" not in body
-        assert body["autoEndMaintain"] is False
-        assert body["backupCurrentVersion"] is False
-
-    def test_requires_one_mode(self, mock_session):
-        with pytest.raises(ValueError):
-            app_mod.restore_app(mock_session, "app1", "org1")
-
-
-class TestRestoreCli:
-    def test_backup_id_only_is_accepted(self, logged_in):
-        with patch("hap_cli.core.session.requests.post") as mock_post:
-            mock_post.return_value = _mock_response(True)
-            res = CliRunner().invoke(
-                cli, ["app", "restore", "app1", "--backup-id", "bk1"]
-            )
-        assert res.exit_code == 0, res.output
-        assert _posted_body(mock_post)["id"] == "bk1"
-
-    def test_file_url_only_is_accepted(self, logged_in):
-        with patch("hap_cli.core.session.requests.post") as mock_post:
-            mock_post.return_value = _mock_response(True)
-            res = CliRunner().invoke(
-                cli,
-                [
-                    "app", "restore", "app1",
-                    "--file-url", "https://x/y.mdy",
-                    "--file-name", "y.mdy",
-                ],
-            )
-        assert res.exit_code == 0, res.output
-        assert _posted_body(mock_post)["fileUrl"] == "https://x/y.mdy"
-
-    def test_neither_mode_errors(self, logged_in):
-        res = CliRunner().invoke(cli, ["app", "restore", "app1"])
-        assert res.exit_code != 0
-        assert "backup-id" in res.output or "file-url" in res.output