Files
dingtalk-workspace-cli/internal/cli/coverage_edges_test.go
玉澜 e135440b98 refactor(cli): retire legacy-overlay schema stack and gate test fixtures
Drop the legacy-metadata overlay path (runtimeToolSpecAllowingLegacy,
assembleSchemaRegistryFromBoundAllowingLegacy, metadata-based agent
selection, dry-run seams) so assembly flows exclusively through
ContractFinal; gate MCP fixture machinery behind a build flag so it can
never enter production assembly. Wire output verified identical before
and after; policy tripwire reports 26 products, 847 tools.
2026-08-04 08:36:22 +08:00

200 lines
6.3 KiB
Go

// Copyright 2026 Alibaba Group
// Licensed under the Apache License, Version 2.0 (the "License");
package cli
import (
"bytes"
"encoding/json"
"os"
"path/filepath"
"testing"
"github.com/spf13/cobra"
)
func TestCrossPlatformCoverageCanonicalCommandsAndFlags(t *testing.T) {
mcp := NewMCPCommand()
mcp.SetOut(&bytes.Buffer{})
if err := mcp.Execute(); err != nil {
t.Fatal(err)
}
}
func TestCrossPlatformCoveragePriorityEdges(t *testing.T) {
cmd := &cobra.Command{Use: "priority"}
SetOverridePriority(cmd, 42)
if got := OverridePriority(cmd); got != 42 {
t.Fatalf("priority = %d", got)
}
}
func TestCrossPlatformCoverageSchemaValidationCompleteMatrix(t *testing.T) {
if err := ValidateInputSchema(nil, nil); err != nil {
t.Fatal(err)
}
if err := ValidateInputSchema(nil, map[string]any{"type": "object", "required": []string{"x"}}); err == nil {
t.Fatal("missing required property accepted")
}
schema := map[string]any{
"type": "object",
"required": []any{"known", 1, ""},
"properties": map[string]any{
"known": map[string]any{"type": "array", "items": map[string]any{"type": "integer"}},
"skip": "invalid",
},
"additionalProperties": false,
}
if err := ValidateInputSchema(map[string]any{"known": []any{1, 2.5}}, schema); err == nil {
t.Fatal("invalid array item accepted")
}
if err := ValidateInputSchema(map[string]any{"known": []any{1}, "extra": true}, schema); err == nil {
t.Fatal("unknown property accepted")
}
additional := map[string]any{
"type": "object",
"properties": map[string]any{"known": map[string]any{"type": "string"}},
"additionalProperties": map[string]any{"type": "number"},
}
if err := ValidateInputSchema(map[string]any{"known": "x", "extra": "bad"}, additional); err == nil {
t.Fatal("invalid additional property accepted")
}
if err := validateSchemaValue("$", "x", nil); err != nil {
t.Fatal(err)
}
if err := validateSchemaValue("$", "x", map[string]any{"enum": []string{"y"}}); err == nil {
t.Fatal("invalid enum accepted")
}
typeCases := []struct {
value any
typ string
want bool
}{
{map[string]any{}, "object", true}, {[]any{}, "array", true}, {"x", "string", true},
{true, "boolean", true}, {1.5, "number", true}, {"x", "number", false},
{2, "integer", true}, {2.5, "integer", false}, {"x", "integer", false},
{nil, "null", true}, {"x", "future", true},
}
for _, tc := range typeCases {
if got := matchesType(tc.value, tc.typ); got != tc.want {
t.Fatalf("matchesType(%#v,%q) = %v, want %v", tc.value, tc.typ, got, tc.want)
}
}
for _, typed := range []any{"", "string", []string{"", "number"}, []any{"", 1, "boolean"}, 1} {
_ = schemaTypes(map[string]any{"type": typed})
}
if hasType([]string{"a", "b"}, "c") || !hasType([]string{"a", "b"}, "b") {
t.Fatal("hasType failed")
}
if matchesAnyType("x", []string{"number", "boolean"}) || !matchesAnyType("x", []string{"number", "string"}) {
t.Fatal("matchesAnyType failed")
}
for _, enum := range []any{[]any{"x"}, []string{"x"}, "invalid"} {
_ = schemaEnum(map[string]any{"enum": enum})
}
if valuesEqual("1", 1) || !valuesEqual(json.Number("1"), float64(1)) {
t.Fatal("enum value equality failed")
}
for _, number := range []any{float64(1), float32(1), int(1), int8(1), int16(1), int32(1), int64(1), uint(1), uint8(1), uint16(1), uint32(1), uint64(1), json.Number("1"), json.Number("x"), "x"} {
_, _ = numberValue(number)
}
if got := schemaProperties(map[string]any{}); len(got) != 0 {
t.Fatal("missing properties were nonempty")
}
for _, required := range []any{[]string{"x"}, []any{"x", 1, ""}, "invalid"} {
_ = schemaRequired(map[string]any{"required": required})
}
for _, additional := range []map[string]any{{}, {"additionalProperties": true}, {"additionalProperties": false}, {"additionalProperties": map[string]any{"type": "string"}}, {"additionalProperties": "invalid"}} {
_, _, _ = additionalProperties(additional)
}
}
func TestCrossPlatformCoverageStdinCompleteMatrix(t *testing.T) {
original := os.Stdin
// Restore/close before t.TempDir cleanup. defer runs on return and on
// t.Fatal (Goexit); open Windows handles otherwise block RemoveAll.
defer func() {
if current := os.Stdin; current != nil && current != original {
os.Stdin = original
_ = current.Close()
return
}
os.Stdin = original
}()
setStdin := func(f *os.File) {
if prev := os.Stdin; prev != nil && prev != original {
_ = prev.Close()
}
os.Stdin = f
}
closed, err := os.CreateTemp(t.TempDir(), "closed")
if err != nil {
t.Fatal(err)
}
_ = closed.Close()
setStdin(closed)
if StdinIsPipe() {
t.Fatal("closed stdin reported as pipe")
}
if _, err := ReadStdin(); err == nil {
t.Fatal("closed stdin read succeeded")
}
stdin, err := os.CreateTemp(t.TempDir(), "stdin")
if err != nil {
t.Fatal(err)
}
if _, err := stdin.WriteString("piped"); err != nil {
t.Fatal(err)
}
_, _ = stdin.Seek(0, 0)
setStdin(stdin)
if !StdinIsPipe() {
t.Fatal("regular file stdin did not report pipe")
}
if got, err := ReadStdinIfPiped(); err != nil || got != "piped" {
t.Fatalf("piped stdin = %q, %v", got, err)
}
_, _ = stdin.Seek(0, 0)
guard := NewStdinGuard()
if got, err := ResolveInputSource("@-", "text", guard); err != nil || got != "piped" {
t.Fatalf("resolved stdin = %q, %v", got, err)
}
if _, err := ResolveInputSource("@-", "other", guard); err == nil {
t.Fatal("stdin reused")
}
if _, err := ResolveInputSource("@-", "text", nil); err == nil {
t.Fatal("nil stdin guard accepted")
}
dir := t.TempDir()
if _, err := readFileBounded(dir); err == nil {
t.Fatal("directory read as file")
}
large := filepath.Join(t.TempDir(), "large")
if err := os.WriteFile(large, bytes.Repeat([]byte("x"), maxStdinSize+1), 0o600); err != nil {
t.Fatal(err)
}
if _, err := readFileBounded(large); err == nil {
t.Fatal("oversized file accepted")
}
largeStdin, err := os.Open(large)
if err != nil {
t.Fatal(err)
}
setStdin(largeStdin)
if _, err := readStdinBounded(); err == nil {
t.Fatal("oversized stdin accepted")
}
for _, value := range []string{"@A", "@a", "@1", "@.", "@/", "@~", "@_", "@-", "@!", "@所有人", "plain"} {
_ = looksLikeFilePath(value)
}
if _, _, err := ReadFileArg("@"); err == nil {
t.Fatal("bare file argument accepted")
}
if _, err := ResolveInputSource("@", "text", guard); err == nil {
t.Fatal("bare input source accepted")
}
}