Merge pull request #3886 from thaJeztah/context_lazy_evaluate_step3

cli/command/context: "docker context show": don't validate context
This commit is contained in:
Sebastiaan van Stijn
2022-11-28 11:52:32 +01:00
committed by GitHub
2 changed files with 5 additions and 11 deletions
+4 -9
View File
@@ -16,19 +16,14 @@ func newShowCommand(dockerCli command.Cli) *cobra.Command {
Short: "Print the name of the current context",
Args: cli.NoArgs,
RunE: func(cmd *cobra.Command, args []string) error {
return runShow(dockerCli)
runShow(dockerCli)
return nil
},
ValidArgsFunction: completion.NoComplete,
}
return cmd
}
func runShow(dockerCli command.Cli) error {
context := dockerCli.CurrentContext()
metadata, err := dockerCli.ContextStore().GetMetadata(context)
if err != nil {
return err
}
fmt.Fprintln(dockerCli.Out(), metadata.Name)
return nil
func runShow(dockerCli command.Cli) {
fmt.Fprintln(dockerCli.Out(), dockerCli.CurrentContext())
}
+1 -2
View File
@@ -3,7 +3,6 @@ package context
import (
"testing"
"gotest.tools/v3/assert"
"gotest.tools/v3/golden"
)
@@ -13,6 +12,6 @@ func TestShow(t *testing.T) {
cli.SetCurrentContext("current")
cli.OutBuffer().Reset()
assert.NilError(t, runShow(cli))
runShow(cli)
golden.Assert(t, cli.OutBuffer().String(), "show.golden")
}