#787 fixed gosec errors, ran go fmt

Author: allmightyspiff

plugin/commands/dedicatedhost/cancel_guest.go

@@ -2,7 +2,7 @@ package dedicatedhost
 
 import (
 	"github.com/spf13/cobra"
-	
+
 	slErr "github.ibm.com/SoftLayer/softlayer-cli/plugin/errors"
 	"github.ibm.com/SoftLayer/softlayer-cli/plugin/metadata"
 
@@ -61,6 +61,8 @@ func (cmd *CancelCommand) Run(args []string) error {
 	if len(listGuest) > 0 {
 		table := cmd.UI.Table([]string{T("Id"), T("Server Name"), T("Status")})
 		for _, guest := range listGuest {
+			//#nosec G601 -- utils.FormatXPointer returns the value of the memory address, so this is not a problem.
+			//Test Issues787 assures this condition.
 			table.Add(utils.FormatIntPointer(&guest.Id), utils.FormatStringPointer(&guest.Fqdn), utils.FormatStringPointer(&guest.Status))
 		}
 		table.Print()

plugin/commands/dedicatedhost/cancel_guest_test.go

@@ -91,5 +91,28 @@ var _ = Describe("Dedicated host create", func() {
 			})
 		})
 
+		Context("Dedicatedhost VS cancel vs successfully - multiple", func() {
+			BeforeEach(func() {
+				FakeDedicatedhostManager.CancelGuestsReturns([]managers.StatusInfo{
+					{
+						Id:     *sl.Int(1234567),
+						Fqdn:   *sl.String("test.softlayer"),
+						Status: *sl.String("Cancelled"),
+					},
+					{
+						Id:     *sl.Int(9999),
+						Fqdn:   *sl.String("qqq.aaa"),
+						Status: *sl.String("Cancelled"),
+					},
+				}, nil)
+			})
+			It("return no error", func() {
+				err := testhelpers.RunCobraCommand(cliCommand.Command, "1234", "-f")
+				Expect(err).NotTo(HaveOccurred())
+				Expect(fakeUI.Outputs()).To(ContainSubstring("1234567"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("9999"))
+			})
+		})
+
 	})
 })

plugin/managers/network.go

@@ -220,6 +220,7 @@ func (n networkManager) AddSubnet(subnetType string, quantity int, vlanID int, v
 			quantity = 0
 			category = "global_ipv4"
 		} else if subnetType == "public" {
+			//#nosec G101 -- Not a credential
 			category = "sov_sec_ip_addresses_pub"
 		}
 	} else {