PR suggestions

Author: gururajsh

command/v7/login_command_test.go

@@ -325,13 +325,10 @@ var _ = Describe("login Command", func() {
 			When("targeting the API with --skip-ssl-validation flag", func() {
 				BeforeEach(func() {
 					cmd.APIEndpoint = "api.example.com"
-					// fakeActor.SetTargetReturns(nil, ccerror.UnverifiedServerError{URL: "https://api.example.com"})
 				})
 
 				It("login succeeds", func() {
 					Expect(executeErr).NotTo(HaveOccurred())
-					// Expect(executeErr).To(MatchError(
-					// 	translatableerror.InvalidSSLCertError{URL: "https://api.example.com", SuggestedCommand: "login"}))
 				})
 			})
 		})

integration/v7/isolated/api_command_test.go

@@ -140,7 +140,7 @@ var _ = Describe("api command", func() {
 				Eventually(session.Err).Should(Say("TIP: Use 'cf api --skip-ssl-validation' to continue with an insecure API endpoint"))
 				Eventually(session).Should(Say("FAILED"))
 				Eventually(session).Should(Exit(1))
-				server.Close()
+				defer server.Close()
 			})
 
 			It("sets the API endpoint", func() {