Added the test coverage for completion.go and version.go

Author: Yash Singh

pkg/cli/completion_test.go

@@ -0,0 +1,83 @@
+/*
+Copyright 2022 The Kubernetes Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package cli
+
+import (
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
+)
+
+var _ = Describe("Completion", func() {
+	var (
+		c *CLI
+	)
+
+	BeforeEach(func() {
+		c = &CLI{}
+	})
+
+	When("newBashCmd", func() {
+		It("Testing the BashCompletion", func() {
+			cmd := c.newBashCmd()
+			Expect(cmd).NotTo(BeNil())
+			Expect(cmd.Use).NotTo(Equal(""))
+			Expect(cmd.Use).To(ContainSubstring("bash"))
+			Expect(cmd.Short).NotTo(Equal(""))
+			Expect(cmd.Short).To(ContainSubstring("Load bash completions"))
+			Expect(cmd.Example).NotTo(Equal(""))
+			Expect(cmd.Example).To(ContainSubstring("# To load completion for this session"))
+		})
+	})
+
+	Context("newZshCmd", func() {
+		It("Testing the ZshCompletion", func() {
+			cmd := c.newZshCmd()
+			Expect(cmd).NotTo(BeNil())
+			Expect(cmd.Use).NotTo(Equal(""))
+			Expect(cmd.Use).To(ContainSubstring("zsh"))
+			Expect(cmd.Short).NotTo(Equal(""))
+			Expect(cmd.Short).To(ContainSubstring("Load zsh completions"))
+			Expect(cmd.Example).NotTo(Equal(""))
+			Expect(cmd.Example).To(ContainSubstring("# If shell completion is not already enabled in your environment"))
+		})
+	})
+
+	Context("newFishCmd", func() {
+		It("Testing the FishCompletion", func() {
+			cmd := c.newFishCmd()
+			Expect(cmd).NotTo(BeNil())
+			Expect(cmd.Use).NotTo(Equal(""))
+			Expect(cmd.Use).To(ContainSubstring("fish"))
+			Expect(cmd.Short).NotTo(Equal(""))
+			Expect(cmd.Short).To(ContainSubstring("Load fish completions"))
+			Expect(cmd.Example).NotTo(Equal(""))
+			Expect(cmd.Example).To(ContainSubstring("# To load completion for this session, execute:"))
+		})
+	})
+
+	Context("newPowerShellCmd", func() {
+		It("Testing the PowerShellCompletion", func() {
+			cmd := c.newPowerShellCmd()
+			Expect(cmd).NotTo(BeNil())
+			Expect(cmd.Use).NotTo(Equal(""))
+			Expect(cmd.Use).To(ContainSubstring("powershell"))
+			Expect(cmd.Short).NotTo(Equal(""))
+			Expect(cmd.Short).To(ContainSubstring("Load powershell completions"))
+		})
+
+	})
+})

pkg/cli/version.go

@@ -23,7 +23,7 @@ import (
 )
 
 func (c CLI) newVersionCmd() *cobra.Command {
-	return &cobra.Command{
+	cmd := &cobra.Command{
 		Use:     "version",
 		Short:   fmt.Sprintf("Print the %s version", c.commandName),
 		Long:    fmt.Sprintf("Print the %s version", c.commandName),
@@ -33,4 +33,5 @@ func (c CLI) newVersionCmd() *cobra.Command {
 			return nil
 		},
 	}
+	return cmd
 }

pkg/cli/version_test.go

@@ -0,0 +1,47 @@
+/*
+Copyright 2022 The Kubernetes Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package cli
+
+import (
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
+)
+
+var _ = Describe("Version", func() {
+	var (
+		c *CLI
+	)
+
+	BeforeEach(func() {
+		c = &CLI{}
+	})
+
+	Context("newVersionCmd", func() {
+		It("Test the version", func() {
+			cmd := c.newVersionCmd()
+			Expect(cmd).NotTo(BeNil())
+			Expect(cmd.Use).To(ContainSubstring("version"))
+			Expect(cmd.Use).NotTo(Equal(""))
+			Expect(cmd.Short).NotTo(Equal(""))
+			Expect(cmd.Short).To(ContainSubstring("Print the"))
+			Expect(cmd.Long).NotTo(Equal(""))
+			Expect(cmd.Long).To(ContainSubstring("Print the"))
+			Expect(cmd.Example).NotTo(Equal(""))
+			Expect(cmd.Example).To(ContainSubstring("version"))
+		})
+	})
+})