While updating, also addressed some redundant fmt.Sprintf()
opts/throttledevice.go:86:2: Consider pre-allocating `out` (prealloc)
var out []string
^
opts/ulimit.go:37:2: Consider pre-allocating `out` (prealloc)
var out []string
^
opts/ulimit.go:47:2: Consider pre-allocating `ulimits` (prealloc)
var ulimits []*units.Ulimit
^
opts/weightdevice.go:68:2: Consider pre-allocating `out` (prealloc)
var out []string
^
cli/context/store/metadatastore.go:96:2: Consider pre-allocating `res` (prealloc)
var res []Metadata
^
cli/context/store/store.go:127:2: Consider pre-allocating `names` (prealloc)
var names []string
^
cli/compose/loader/loader.go:223:2: Consider pre-allocating `keys` (prealloc)
var keys []string
^
cli/compose/loader/loader.go:397:2: Consider pre-allocating `services` (prealloc)
var services []types.ServiceConfig
^
cli/command/stack/loader/loader.go:63:2: Consider pre-allocating `msgs` (prealloc)
var msgs []string
^
cli/command/stack/loader/loader.go:118:2: Consider pre-allocating `configFiles` (prealloc)
var configFiles []composetypes.ConfigFile
^
cli/command/formatter/container.go:245:2: Consider pre-allocating `joinLabels` (prealloc)
var joinLabels []string
^
cli/command/formatter/container.go:265:2: Consider pre-allocating `mounts` (prealloc)
var mounts []string
^
cli/command/formatter/container.go:316:2: Consider pre-allocating `result` (prealloc)
var result []string
^
cli/command/formatter/displayutils.go:43:2: Consider pre-allocating `display` (prealloc)
var (
^
cli/command/formatter/volume.go:103:2: Consider pre-allocating `joinLabels` (prealloc)
var joinLabels []string
^
cli-plugins/manager/manager_test.go:49:2: Consider pre-allocating `dirs` (prealloc)
var dirs []string
^
cli/command/swarm/init.go:69:2: Consider pre-allocating `defaultAddrPool` (prealloc)
var defaultAddrPool []string
^
cli/command/manifest/push.go:195:2: Consider pre-allocating `blobReqs` (prealloc)
var blobReqs []manifestBlob
^
cli/command/secret/formatter.go:111:2: Consider pre-allocating `joinLabels` (prealloc)
var joinLabels []string
^
cli/command/network/formatter.go:104:2: Consider pre-allocating `joinLabels` (prealloc)
var joinLabels []string
^
cli/command/context/list.go:52:2: Consider pre-allocating `contexts` (prealloc)
var contexts []*formatter.ClientContext
^
cli/command/config/formatter.go:104:2: Consider pre-allocating `joinLabels` (prealloc)
var joinLabels []string
^
cli/command/trust/common_test.go:23:2: Consider pre-allocating `targetNames` (prealloc)
var targetNames []string
^
cli/command/service/generic_resource_opts.go:55:2: Consider pre-allocating `generic` (prealloc)
var generic []swarm.GenericResource
^
cli/command/service/generic_resource_opts.go:98:2: Consider pre-allocating `l` (prealloc)
var l []swarm.GenericResource
^
cli/command/service/opts.go:378:2: Consider pre-allocating `netAttach` (prealloc)
var netAttach []swarm.NetworkAttachmentConfig
^
cli/command/service/update.go:731:2: Consider pre-allocating `limits` (prealloc)
var limits []*units.Ulimit
^
cli/command/service/update.go:1315:2: Consider pre-allocating `newNetworks` (prealloc)
var newNetworks []swarm.NetworkAttachmentConfig
^
cli/command/service/update.go:1514:2: Consider pre-allocating `out` (prealloc)
var out []string
^
cli/compose/convert/service.go:713:2: Consider pre-allocating `ulimits` (prealloc)
var ulimits []*units.Ulimit
^
cli/compose/convert/volume.go:13:2: Consider pre-allocating `mounts` (prealloc)
var mounts []mount.Mount
^
cli/command/stack/swarm/list.go:39:2: Consider pre-allocating `stacks` (prealloc)
var stacks []*formatter.Stack
^
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
167 lines
5.1 KiB
Go
167 lines
5.1 KiB
Go
package manager
|
|
|
|
import (
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/docker/cli/cli/config"
|
|
"github.com/docker/cli/cli/config/configfile"
|
|
"github.com/docker/cli/internal/test"
|
|
"github.com/spf13/cobra"
|
|
"gotest.tools/v3/assert"
|
|
"gotest.tools/v3/fs"
|
|
)
|
|
|
|
func TestListPluginCandidates(t *testing.T) {
|
|
// Populate a selection of directories with various shadowed and bogus/obscure plugin candidates.
|
|
// For the purposes of this test no contents is required and permissions are irrelevant.
|
|
dir := fs.NewDir(t, t.Name(),
|
|
fs.WithDir(
|
|
"plugins1",
|
|
fs.WithFile("docker-plugin1", ""), // This appears in each directory
|
|
fs.WithFile("not-a-plugin", ""), // Should be ignored
|
|
fs.WithFile("docker-symlinked1", ""), // This and ...
|
|
fs.WithSymlink("docker-symlinked2", "docker-symlinked1"), // ... this should both appear
|
|
fs.WithDir("ignored1"), // A directory should be ignored
|
|
),
|
|
fs.WithDir(
|
|
"plugins2",
|
|
fs.WithFile("docker-plugin1", ""),
|
|
fs.WithFile("also-not-a-plugin", ""),
|
|
fs.WithFile("docker-hardlink1", ""), // This and ...
|
|
fs.WithHardlink("docker-hardlink2", "docker-hardlink1"), // ... this should both appear
|
|
fs.WithDir("ignored2"),
|
|
),
|
|
fs.WithDir(
|
|
"plugins3-target", // Will be referenced as a symlink from below
|
|
fs.WithFile("docker-plugin1", ""),
|
|
fs.WithDir("ignored3"),
|
|
fs.WithSymlink("docker-brokensymlink", "broken"), // A broken symlink is still a candidate (but would fail tests later)
|
|
fs.WithFile("non-plugin-symlinked", ""), // This shouldn't appear, but ...
|
|
fs.WithSymlink("docker-symlinked", "non-plugin-symlinked"), // ... this link to it should.
|
|
),
|
|
fs.WithSymlink("plugins3", "plugins3-target"),
|
|
fs.WithFile("/plugins4", ""),
|
|
fs.WithSymlink("plugins5", "plugins5-nonexistent-target"),
|
|
)
|
|
defer dir.Remove()
|
|
|
|
dirs := make([]string, 0, 6)
|
|
for _, d := range []string{"plugins1", "nonexistent", "plugins2", "plugins3", "plugins4", "plugins5"} {
|
|
dirs = append(dirs, dir.Join(d))
|
|
}
|
|
|
|
candidates, err := listPluginCandidates(dirs)
|
|
assert.NilError(t, err)
|
|
exp := map[string][]string{
|
|
"plugin1": {
|
|
dir.Join("plugins1", "docker-plugin1"),
|
|
dir.Join("plugins2", "docker-plugin1"),
|
|
dir.Join("plugins3", "docker-plugin1"),
|
|
},
|
|
"symlinked1": {
|
|
dir.Join("plugins1", "docker-symlinked1"),
|
|
},
|
|
"symlinked2": {
|
|
dir.Join("plugins1", "docker-symlinked2"),
|
|
},
|
|
"hardlink1": {
|
|
dir.Join("plugins2", "docker-hardlink1"),
|
|
},
|
|
"hardlink2": {
|
|
dir.Join("plugins2", "docker-hardlink2"),
|
|
},
|
|
"brokensymlink": {
|
|
dir.Join("plugins3", "docker-brokensymlink"),
|
|
},
|
|
"symlinked": {
|
|
dir.Join("plugins3", "docker-symlinked"),
|
|
},
|
|
}
|
|
|
|
assert.DeepEqual(t, candidates, exp)
|
|
}
|
|
|
|
func TestGetPlugin(t *testing.T) {
|
|
dir := fs.NewDir(t, t.Name(),
|
|
fs.WithFile("docker-bbb", `
|
|
#!/bin/sh
|
|
echo '{"SchemaVersion":"0.1.0"}'`, fs.WithMode(0o777)),
|
|
fs.WithFile("docker-aaa", `
|
|
#!/bin/sh
|
|
echo '{"SchemaVersion":"0.1.0"}'`, fs.WithMode(0o777)),
|
|
)
|
|
defer dir.Remove()
|
|
|
|
cli := test.NewFakeCli(nil)
|
|
cli.SetConfigFile(&configfile.ConfigFile{CLIPluginsExtraDirs: []string{dir.Path()}})
|
|
|
|
plugin, err := GetPlugin("bbb", cli, &cobra.Command{})
|
|
assert.NilError(t, err)
|
|
assert.Equal(t, plugin.Name, "bbb")
|
|
|
|
_, err = GetPlugin("ccc", cli, &cobra.Command{})
|
|
assert.Error(t, err, "Error: No such CLI plugin: ccc")
|
|
assert.Assert(t, IsNotFound(err))
|
|
}
|
|
|
|
func TestListPluginsIsSorted(t *testing.T) {
|
|
dir := fs.NewDir(t, t.Name(),
|
|
fs.WithFile("docker-bbb", `
|
|
#!/bin/sh
|
|
echo '{"SchemaVersion":"0.1.0"}'`, fs.WithMode(0o777)),
|
|
fs.WithFile("docker-aaa", `
|
|
#!/bin/sh
|
|
echo '{"SchemaVersion":"0.1.0"}'`, fs.WithMode(0o777)),
|
|
)
|
|
defer dir.Remove()
|
|
|
|
cli := test.NewFakeCli(nil)
|
|
cli.SetConfigFile(&configfile.ConfigFile{CLIPluginsExtraDirs: []string{dir.Path()}})
|
|
|
|
plugins, err := ListPlugins(cli, &cobra.Command{})
|
|
assert.NilError(t, err)
|
|
|
|
// We're only interested in the plugins we created for testing this, and only
|
|
// if they appear in the expected order
|
|
var names []string
|
|
for _, p := range plugins {
|
|
if p.Name == "aaa" || p.Name == "bbb" {
|
|
names = append(names, p.Name)
|
|
}
|
|
}
|
|
assert.DeepEqual(t, names, []string{"aaa", "bbb"})
|
|
}
|
|
|
|
func TestErrPluginNotFound(t *testing.T) {
|
|
var err error = errPluginNotFound("test")
|
|
err.(errPluginNotFound).NotFound()
|
|
assert.Error(t, err, "Error: No such CLI plugin: test")
|
|
assert.Assert(t, IsNotFound(err))
|
|
assert.Assert(t, !IsNotFound(nil))
|
|
}
|
|
|
|
func TestGetPluginDirs(t *testing.T) {
|
|
cli := test.NewFakeCli(nil)
|
|
|
|
pluginDir, err := config.Path("cli-plugins")
|
|
assert.NilError(t, err)
|
|
expected := append([]string{pluginDir}, defaultSystemPluginDirs...)
|
|
|
|
var pluginDirs []string
|
|
pluginDirs, err = getPluginDirs(cli)
|
|
assert.Equal(t, strings.Join(expected, ":"), strings.Join(pluginDirs, ":"))
|
|
assert.NilError(t, err)
|
|
|
|
extras := []string{
|
|
"foo", "bar", "baz",
|
|
}
|
|
expected = append(extras, expected...)
|
|
cli.SetConfigFile(&configfile.ConfigFile{
|
|
CLIPluginsExtraDirs: extras,
|
|
})
|
|
pluginDirs, err = getPluginDirs(cli)
|
|
assert.DeepEqual(t, expected, pluginDirs)
|
|
assert.NilError(t, err)
|
|
}
|