Files
dingtalk-workspace-cli/internal/cli/contract_final_assembly_test.go
T
玉澜andCursor 32598fb38d test(coverage): close macOS changed-code coverage gaps to 100%
Add TestCrossPlatformCoverage cases for schema overview render failure,
marshalSchemaRaw error path, MCP metadata lookup in contract assembly,
and RegisterFlags MarkRequired+Aliases panic so platform coverage gate passes.

Co-authored-by: Cursor <cursoragent@cursor.com>
2026-08-04 14:56:13 +08:00

343 lines
13 KiB
Go

// Copyright 2026 Alibaba Group
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package cli
import (
"errors"
"strings"
"testing"
"github.com/DingTalk-Real-AI/dingtalk-workspace-cli/internal/corecmd/contract"
"github.com/DingTalk-Real-AI/dingtalk-workspace-cli/internal/corecmd/contractfinal"
"github.com/DingTalk-Real-AI/dingtalk-workspace-cli/internal/corecmd/runtimeannotate"
"github.com/spf13/cobra"
)
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalMCPMetadataLookup(t *testing.T) {
cmd := &cobra.Command{Use: "reply"}
t.Cleanup(func() { contractfinal.ClearRuntimeContractFinalForTest(cmd) })
cmd.Flags().String("text", "", "text")
runtimeannotate.AnnotateRuntimeFlag(cmd, "text", "text", "string", false)
contractfinal.RegisterRuntimeContractFinal(cmd, contract.ContractFinalPayload{
Identity: &contract.ToolIdentitySpec{
ProductID: "chat", Name: "reply_personal_message", CanonicalPath: "chat.reply_personal_message",
CLIPath: "chat reply", PrimaryCLIPath: "chat reply",
},
Interface: &contract.InterfaceSpec{
Mode: contract.InterfaceModeMCP,
Availability: contract.InterfaceAvailable,
Ref: &contract.InterfaceRefSpec{ProductID: "chat", RPCName: "send_personal_message"},
},
})
entry := runtimeSchemaEntry{
ProductID: "chat", ToolName: "reply_personal_message", Command: cmd,
CLIPath: "chat reply", PrimaryCLIPath: "chat reply",
}
metadata := runtimeSchemaMetadataSources{
MCP: embeddedMCPMetadata{Tools: map[string]embeddedMCPToolMetadata{
"chat.send_personal_message": {Parameters: map[string]embeddedMCPParamMeta{
"text": {Type: "string"},
}},
}},
}
if _, err := runtimeToolSpecFromContractFinal(entry, mustFinal(t, cmd), metadata); err != nil {
t.Fatalf("runtimeToolSpecFromContractFinal with MCP metadata = %v", err)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalPassThrough(t *testing.T) {
cmd := &cobra.Command{Use: "create", Short: "s", Long: "l"}
t.Cleanup(func() { contractfinal.ClearRuntimeContractFinalForTest(cmd) })
cmd.Flags().String("mode", "", "usage")
runtimeannotate.AnnotateRuntimeFlag(cmd, "mode", "mode", "string", false)
contractfinal.RegisterRuntimeContractFinal(cmd, contract.ContractFinalPayload{
Title: "Final Title",
Safety: &contract.SafetySpec{
Effect: "write", Confirmation: "user_required", Idempotency: "none",
},
DryRun: &contract.DryRunSpec{PreviewKind: contract.DryRunPreviewInvocation},
Selection: &contract.SelectionSpec{
AgentSummary: "from contract",
UseWhen: []string{"create things"},
},
Identity: &contract.ToolIdentitySpec{
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create",
},
})
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
ProductName: "Dev",
Command: cmd,
Source: "test",
}
spec, err := runtimeToolSpecFromContractFinal(entry, mustFinal(t, cmd), runtimeSchemaMetadataSources{})
if err != nil {
t.Fatal(err)
}
if spec.Title != "Final Title" {
t.Fatalf("title = %q", spec.Title)
}
if spec.Safety.Confirmation != "user_required" || spec.Safety.Idempotency != "none" {
t.Fatalf("safety = %#v", spec.Safety)
}
if spec.DryRun == nil || spec.DryRun.PreviewKind != contract.DryRunPreviewInvocation {
t.Fatalf("dry_run = %#v", spec.DryRun)
}
if spec.Selection.AgentSummary != "from contract" {
t.Fatalf("selection = %#v", spec.Selection)
}
if spec.MetadataSource != "corecmd.contract" {
t.Fatalf("metadata_source = %q", spec.MetadataSource)
}
if len(spec.Parameters) != 1 || spec.Parameters[0].Name != "mode" {
t.Fatalf("parameters = %#v", spec.Parameters)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalIdentityMismatchFails(t *testing.T) {
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Command: &cobra.Command{Use: "create"},
Source: "test",
}
for name, id := range map[string]contract.ToolIdentitySpec{
"wrong name": {Name: "delete_thing"},
"wrong canonical path": {CanonicalPath: "dev.delete_thing"},
"wrong product": {ProductID: "other"},
"wrong cli path": {CLIPath: "dev delete"},
"wrong aliases": {Aliases: []string{"dev rm"}},
} {
t.Run(name, func(t *testing.T) {
_, err := runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{Identity: &id}, runtimeSchemaMetadataSources{})
if err == nil {
t.Fatal("declared identity conflicting with bound entry must fail assembly")
}
})
}
consistent := contract.ContractFinalPayload{Identity: &contract.ToolIdentitySpec{
ProductID: "dev", Name: "create_thing", CLIName: "create",
CanonicalPath: "dev.create_thing", CLIPath: "dev create",
PrimaryCLIPath: "dev create", Source: "test",
}}
if _, err := runtimeToolSpecFromContractFinal(entry, consistent, runtimeSchemaMetadataSources{}); err != nil {
t.Fatalf("declared identity matching bound entry must pass: %v", err)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalRejectsReviewedSelection(t *testing.T) {
reviewed := true
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Command: &cobra.Command{Use: "create"},
Source: "test",
}
_, err := runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{
Identity: &contract.ToolIdentitySpec{
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create",
CLIName: "create", Source: "test",
},
Selection: &contract.SelectionSpec{AgentSummary: "sum", Reviewed: &reviewed},
}, runtimeSchemaMetadataSources{})
if err == nil || !strings.Contains(err.Error(), "must not carry reviewed field") {
t.Fatalf("declaration payload carrying Reviewed must fail assembly, got %v", err)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalOptionalIdentityMismatch(t *testing.T) {
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
Group: "dev thing",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Command: &cobra.Command{Use: "create"},
Source: "test",
}
for name, id := range map[string]contract.ToolIdentitySpec{
"cli_name": {
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create", CLIName: "other", Source: "test",
},
"group": {
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create", CLIName: "create",
Group: "other group", Source: "test",
},
"source": {
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create", CLIName: "create",
Source: "other",
},
} {
t.Run(name, func(t *testing.T) {
_, err := runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{Identity: &id}, runtimeSchemaMetadataSources{})
if err == nil || !strings.Contains(err.Error(), name+":") {
t.Fatalf("optional identity field %s mismatch error = %v", name, err)
}
})
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalIdentityOverridesApplied(t *testing.T) {
cmd := &cobra.Command{Use: "create"}
entry := runtimeSchemaEntry{
ProductID: "dev",
SourceProductID: "src",
ToolName: "create_thing",
CLIName: "create",
Group: "dev thing",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Aliases: []string{"dev alt"},
Command: cmd,
Source: "test",
}
final := contract.ContractFinalPayload{Identity: &contract.ToolIdentitySpec{
ProductID: "dev",
SourceProductID: "src",
Name: "create_thing",
CLIName: "create",
CanonicalPath: "dev.create_thing",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Group: "dev thing",
Aliases: []string{"dev alt"},
Source: "test",
}}
spec, err := runtimeToolSpecFromContractFinal(entry, final, runtimeSchemaMetadataSources{})
if err != nil {
t.Fatalf("consistent declared identity with all override fields failed: %v", err)
}
if spec.Identity.SourceProductID != "src" {
t.Fatalf("source_product_id = %q, want src", spec.Identity.SourceProductID)
}
if spec.Identity.Group != "dev thing" {
t.Fatalf("group = %q, want dev thing", spec.Identity.Group)
}
if len(spec.Identity.Aliases) != 1 || spec.Identity.Aliases[0] != "dev alt" {
t.Fatalf("aliases = %v, want [dev alt]", spec.Identity.Aliases)
}
if spec.Identity.CanonicalPath != "dev.create_thing" || spec.Identity.Source != "test" {
t.Fatalf("identity = %#v", spec.Identity)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalSameLengthAliasMismatchFails(t *testing.T) {
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Aliases: []string{"dev rm"},
Command: &cobra.Command{Use: "create"},
Source: "test",
}
// Same-length alias sets with different members must be detected by the
// element-wise comparison, not only by the length fast-path.
final := contract.ContractFinalPayload{Identity: &contract.ToolIdentitySpec{
ProductID: "dev", Name: "create_thing", CanonicalPath: "dev.create_thing",
CLIPath: "dev create", PrimaryCLIPath: "dev create",
Aliases: []string{"dev other"},
}}
_, err := runtimeToolSpecFromContractFinal(entry, final, runtimeSchemaMetadataSources{})
if err == nil || !strings.Contains(err.Error(), "aliases") {
t.Fatalf("same-length alias mismatch error = %v, want aliases mismatch", err)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalSafetyAnnotationFallbacks(t *testing.T) {
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
CLIName: "create",
CLIPath: "dev create",
PrimaryCLIPath: "dev create",
Source: "test",
}
// Safety nil + Contract Risk annotation: Risk overlay wins.
riskCmd := &cobra.Command{Use: "create"}
riskCmd.Annotations = map[string]string{runtimeannotate.AnnotationRisk: "write"}
entry.Command = riskCmd
spec, err := runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{}, runtimeSchemaMetadataSources{})
if err != nil {
t.Fatalf("risk-annotated declared leaf failed: %v", err)
}
if spec.Safety.Effect != "write" || spec.Safety.Risk != "medium" || spec.Safety.Confirmation != "user_required" {
t.Fatalf("risk fallback safety = %#v", spec.Safety)
}
if spec.Safety.EffectSource != "corecmd.contract" {
t.Fatalf("risk fallback effect source = %q", spec.Safety.EffectSource)
}
// Safety nil + runtime gate annotation: gate overlay wins.
gateCmd := &cobra.Command{Use: "create"}
gateCmd.Annotations = map[string]string{runtimeannotate.AnnotationRuntimeGate: "devAppRequireWriteGuard"}
entry.Command = gateCmd
spec, err = runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{}, runtimeSchemaMetadataSources{})
if err != nil {
t.Fatalf("gate-annotated declared leaf failed: %v", err)
}
if spec.Safety.Confirmation != "user_required" || spec.Safety.Effect != "write" || spec.Safety.Risk != "medium" {
t.Fatalf("gate fallback safety = %#v", spec.Safety)
}
if spec.Safety.EffectSource != "corecmd.contract_gate" {
t.Fatalf("gate fallback effect source = %q", spec.Safety.EffectSource)
}
}
func TestCrossPlatformCoverageRuntimeToolSpecFromContractFinalParameterResolutionError(t *testing.T) {
oldParameters := resolveRuntimeParameters
t.Cleanup(func() { resolveRuntimeParameters = oldParameters })
resolveRuntimeParameters = func(*cobra.Command, string, map[string]embeddedMCPParamMeta, RuntimeSchemaConstraints) ([]ParameterSpec, error) {
return nil, errors.New("parameters failed")
}
entry := runtimeSchemaEntry{
ProductID: "dev",
ToolName: "create_thing",
Command: &cobra.Command{Use: "create"},
}
_, err := runtimeToolSpecFromContractFinal(entry, contract.ContractFinalPayload{}, runtimeSchemaMetadataSources{})
if err == nil || !strings.Contains(err.Error(), "resolve Contract Schema parameters") {
t.Fatalf("parameter resolution error = %v, want resolve Contract Schema parameters", err)
}
}
func mustFinal(t *testing.T, cmd *cobra.Command) contract.ContractFinalPayload {
t.Helper()
final, ok := contractfinal.RuntimeContractFinal(cmd)
if !ok {
t.Fatal("missing final")
}
return final
}