From 73f9eb383b7aabf6cfe46750a6d7732a6f9012f2 Mon Sep 17 00:00:00 2001 From: Ian Howell Date: Mon, 27 Jan 2020 12:08:34 -0600 Subject: [PATCH] Fix an output bug in get-context This fixes an issue where `airshipctl config get-context` would provide no output when there were no contexts in the configuration. Change-Id: I8eaafff8ecee7878f80d22867b0420feba1bf376 --- cmd/config/get_context.go | 4 ++-- .../no-contexts.golden | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/cmd/config/get_context.go b/cmd/config/get_context.go index a4abdda9e..cb902e363 100644 --- a/cmd/config/get_context.go +++ b/cmd/config/get_context.go @@ -95,8 +95,8 @@ func getContexts(out io.Writer, airconfig *config.Config) error { if err != nil { return err } - if contexts == nil { - fmt.Fprint(out, "No Contexts found in the configuration.\n") + if len(contexts) == 0 { + fmt.Fprintln(out, "No Contexts found in the configuration.") } for _, context := range contexts { fmt.Fprintf(out, "%s", context.PrettyString()) diff --git a/cmd/config/testdata/TestNoContextsGetContextCmdGoldenOutput/no-contexts.golden b/cmd/config/testdata/TestNoContextsGetContextCmdGoldenOutput/no-contexts.golden index e69de29bb..e8e4b77ed 100644 --- a/cmd/config/testdata/TestNoContextsGetContextCmdGoldenOutput/no-contexts.golden +++ b/cmd/config/testdata/TestNoContextsGetContextCmdGoldenOutput/no-contexts.golden @@ -0,0 +1 @@ +No Contexts found in the configuration.