test(agent): align API tests with drive manifest defaults

api/tests/unit_tests/core/app/apps/agent_app/test_app_runner.py

@@ -46,6 +46,13 @@ class _FakeCredentialsProvider:
         return {"openai_api_key": "sk-test"}
 
 
+@pytest.fixture(autouse=True)
+def _disable_drive_manifest_by_default(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(
+        "core.app.apps.agent_app.runtime_request_builder.dify_config.AGENT_DRIVE_MANIFEST_ENABLED", False
+    )
+
+
 class _NoToolsBuilder:
     def build(self, **kwargs):
         del kwargs

api/tests/unit_tests/core/app/apps/agent_app/test_resolve_agent.py

@@ -14,6 +14,7 @@ import pytest
 
 from core.app.apps.agent_app import app_generator as gen_mod
 from core.app.apps.agent_app.app_generator import AgentAppGenerator, AgentAppGeneratorError
+from core.app.entities.app_invoke_entities import InvokeFrom
 
 _SOUL_DICT = {
     "model": {
@@ -84,14 +85,24 @@ class TestResolveAgent:
         _patch_session(monkeypatch, [bound_agent, inner_agent, snapshot])
         app_model = SimpleNamespace(id="app-1", tenant_id="t1")
 
-        agent, snap, soul = AgentAppGenerator()._resolve_agent(app_model)  # type: ignore[arg-type]
+        agent, snap, soul = AgentAppGenerator()._resolve_agent(
+            app_model,
+            invoke_from=InvokeFrom.WEB_APP,
+            draft_type=None,
+            user=SimpleNamespace(id="user-1"),
+        )  # type: ignore[arg-type]
 
-        assert agent is inner_agent
-        assert snap is snapshot
+        assert agent is bound_agent
+        assert snap == snapshot.id
         assert soul.model is not None
 
     def test_unbound_app_raises(self, monkeypatch: pytest.MonkeyPatch):
         _patch_session(monkeypatch, [None])
         app_model = SimpleNamespace(id="app-1", tenant_id="t1")
         with pytest.raises(AgentAppGeneratorError, match="has no bound Agent"):
-            AgentAppGenerator()._resolve_agent(app_model)  # type: ignore[arg-type]
+            AgentAppGenerator()._resolve_agent(
+                app_model,
+                invoke_from=InvokeFrom.WEB_APP,
+                draft_type=None,
+                user=SimpleNamespace(id="user-1"),
+            )  # type: ignore[arg-type]

api/tests/unit_tests/core/workflow/nodes/agent_v2/test_agent_node.py

@@ -2,6 +2,7 @@ from types import SimpleNamespace
 from typing import cast
 from unittest.mock import MagicMock, patch
 
+import pytest
 from agenton.compositor import CompositorSessionSnapshot
 from dify_agent.layers.ask_human import AskHumanToolResult
 from dify_agent.protocol import RunStartedEvent, RunSucceededEvent, RunSucceededEventData
@@ -50,6 +51,13 @@ class FakeCredentialsProvider:
         return {"api_key": "secret-key"}
 
 
+@pytest.fixture(autouse=True)
+def _disable_drive_manifest_by_default(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(
+        "core.workflow.nodes.agent_v2.runtime_request_builder.dify_config.AGENT_DRIVE_MANIFEST_ENABLED", False
+    )
+
+
 def _restored_file(*, transfer_method: FileTransferMethod, reference: str) -> File:
     return File(
         type=FileType.DOCUMENT,

api/tests/unit_tests/core/workflow/nodes/agent_v2/test_runtime_request_builder.py

@@ -36,6 +36,13 @@ class FakeCredentialsProvider:
         return {"api_key": "secret-key"}
 
 
+@pytest.fixture(autouse=True)
+def _disable_drive_manifest_by_default(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(
+        "core.workflow.nodes.agent_v2.runtime_request_builder.dify_config.AGENT_DRIVE_MANIFEST_ENABLED", False
+    )
+
+
 class CapturingCredentialsProvider:
     def __init__(self) -> None:
         self.provider_name: str | None = None
@@ -1128,7 +1135,7 @@ def test_workflow_runtime_missing_drive_mentions_fall_back_to_label_then_decoded
     result = WorkflowAgentRuntimeRequestBuilder(credentials_provider=FakeCredentialsProvider()).build(context)
 
     soul_prompt = next(layer for layer in result.request.composition.layers if layer.name == "agent_soul_prompt")
-    assert soul_prompt.config.prefix == "Use Ghost Skill, Ghost File, and no-label.txt."
+    assert soul_prompt.config.prefix == "Use Ghost Skill, Ghost File, and files/no-label.txt."
     assert "[§" not in soul_prompt.config.prefix
 
 

api/tests/unit_tests/migrations/test_agent_drive_skill_metadata_refactor.py

@@ -64,7 +64,7 @@ def _run_migration_step(module: object, engine: sa.Engine, step_name: str) -> No
             module.op = original_op
 
 
-def test_upgrade_adds_skill_columns_and_index_and_strips_snapshot_data() -> None:
+def test_upgrade_adds_skill_columns_and_index_and_preserves_snapshot_data() -> None:
     engine = sa.create_engine("sqlite:///:memory:")
     _create_pre_upgrade_schema(engine)
     snapshot = {
@@ -91,7 +91,7 @@ def test_upgrade_adds_skill_columns_and_index_and_strips_snapshot_data() -> None
             sa.text("SELECT config_snapshot FROM agent_config_snapshots WHERE id = :id"),
             {"id": "snap-1"},
         ).scalar_one()
-    assert "skills_files" not in json.loads(stored_snapshot)
+    assert json.loads(stored_snapshot) == snapshot
 
 
 def test_downgrade_drops_skill_columns_and_index_without_reconstructing_legacy_data() -> None: