updatede command and added unit tets

Author: Brian Flores

plugin/commands/user/details.go

@@ -188,10 +188,14 @@ func baseUserPrint(user datatypes.User_Customer, keys bool, ui terminal.UI) {
 	table.Add(T("ID"), utils.FormatIntPointer(user.Id))
 	table.Add(T("Username"), utils.FormatStringPointer(user.Username))
 
-	if keys {
+	if keys && user.ApiAuthenticationKeys != nil && len(user.ApiAuthenticationKeys) != 0 {
 		for _, key := range user.ApiAuthenticationKeys {
 			table.Add(T("APIKEY"), utils.FormatStringPointer(key.AuthenticationKey))
 		}
+	} else if user.ApiAuthenticationKeys != nil && len(user.ApiAuthenticationKeys) != 0 {
+		table.Add(T("APIKEY"), T("Yes"))
+	} else {
+		table.Add(T("APIKEY"), T("No"))
 	}
 
 	table.Add(T("Name"), utils.FormatStringPointer(user.FirstName)+" "+utils.FormatStringPointer(user.LastName))

plugin/commands/user/details_test.go

@@ -425,5 +425,55 @@ var _ = Describe("Detail", func() {
 				Expect(fakeUI.Outputs()).To(ContainSubstring("2017-11-08T00:00:00Z   IAM Token validation successful   80.80.80.80   test2@test.com   test2_test.com"))
 			})
 		})
+
+		Context("user detail with correct id and without apikey", func() {
+			BeforeEach(func() {
+				created, _ := time.Parse(time.RFC3339, "2017-11-08T00:00:00Z")
+
+				testUser = datatypes.User_Customer{
+					Id:                    sl.Int(5555),
+					Username:              sl.String("ATestUser"),
+					ApiAuthenticationKeys: []datatypes.User_Customer_ApiAuthentication{},
+					FirstName:             sl.String("Name"),
+					LastName:              sl.String("LastName"),
+					Email:                 sl.String("user@email.com"),
+					OpenIdConnectUserName: sl.String("123456"),
+					Address1:              sl.String("addres with number N 123"),
+					CompanyName:           sl.String("NameCompany"),
+					CreateDate:            sl.Time(created),
+					OfficePhone:           sl.String("123456789"),
+					PptpVpnAllowedFlag:    sl.Bool(true),
+					SslVpnAllowedFlag:     sl.Bool(true),
+					Parent: &datatypes.User_Customer{
+						Username: sl.String("ParentName"),
+					},
+					UserStatus: &datatypes.User_Customer_Status{
+						Name: sl.String("ACTIVE"),
+					},
+				}
+				fakeUserManager.GetUserReturns(testUser, nil)
+			})
+
+			It("return a user without apikey", func() {
+				err := testhelpers.RunCobraCommand(cliCommand.Command, "5555", "--keys")
+				Expect(err).NotTo(HaveOccurred())
+				Expect(fakeUI.Outputs()).To(ContainSubstring("5555"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("ATestUser"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("No"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("Name LastName"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("user@email.com"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("123456"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("addres with number N 123 - - - - -"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("NameCompany"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("2017-11-08T00:00:00Z"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("123456789"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("ParentName"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("ACTIVE"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("true"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("true"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("2017-11-08T00:00:00Z"))
+				Expect(fakeUI.Outputs()).To(ContainSubstring("2017-11-08T00:00:00Z"))
+			})
+		})
 	})
 })