ab120af649
The --registry flag value was previously ignored and only used as a switch. Now the flag value is properly used as the registry name. Fixes #1104 Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
97 lines
2.2 KiB
Go
97 lines
2.2 KiB
Go
package skills
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestNewInstallSubcommand(t *testing.T) {
|
|
cmd := newInstallCommand(nil)
|
|
|
|
require.NotNil(t, cmd)
|
|
|
|
assert.Equal(t, "install", cmd.Use)
|
|
assert.Equal(t, "Install skill from GitHub", cmd.Short)
|
|
|
|
assert.Nil(t, cmd.Run)
|
|
assert.NotNil(t, cmd.RunE)
|
|
|
|
assert.True(t, cmd.HasExample())
|
|
assert.False(t, cmd.HasSubCommands())
|
|
|
|
assert.True(t, cmd.HasFlags())
|
|
assert.NotNil(t, cmd.Flags().Lookup("registry"))
|
|
|
|
assert.Len(t, cmd.Aliases, 0)
|
|
}
|
|
|
|
func TestInstallCommandArgs(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
args []string
|
|
registry string
|
|
expectError bool
|
|
errorMsg string
|
|
}{
|
|
{
|
|
name: "no registry, one arg",
|
|
args: []string{"sipeed/picoclaw-skills/weather"},
|
|
registry: "",
|
|
expectError: false,
|
|
},
|
|
{
|
|
name: "no registry, no args",
|
|
args: []string{},
|
|
registry: "",
|
|
expectError: true,
|
|
errorMsg: "exactly 1 argument is required: <github>",
|
|
},
|
|
{
|
|
name: "no registry, too many args",
|
|
args: []string{"arg1", "arg2"},
|
|
registry: "",
|
|
expectError: true,
|
|
errorMsg: "exactly 1 argument is required: <github>",
|
|
},
|
|
{
|
|
name: "with registry, one arg",
|
|
args: []string{"weather-skill"},
|
|
registry: "clawhub",
|
|
expectError: false,
|
|
},
|
|
{
|
|
name: "with registry, no args",
|
|
args: []string{},
|
|
registry: "clawhub",
|
|
expectError: true,
|
|
errorMsg: "when --registry is set, exactly 1 argument is required: <slug>",
|
|
},
|
|
{
|
|
name: "with registry, too many args",
|
|
args: []string{"arg1", "arg2"},
|
|
registry: "clawhub",
|
|
expectError: true,
|
|
errorMsg: "when --registry is set, exactly 1 argument is required: <slug>",
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
cmd := newInstallCommand(nil)
|
|
|
|
if tt.registry != "" {
|
|
require.NoError(t, cmd.Flags().Set("registry", tt.registry))
|
|
}
|
|
|
|
err := cmd.Args(cmd, tt.args)
|
|
if tt.expectError {
|
|
require.Error(t, err)
|
|
assert.Equal(t, tt.errorMsg, err.Error())
|
|
} else {
|
|
require.NoError(t, err)
|
|
}
|
|
})
|
|
}
|
|
}
|