removed unnecessary code and added unit test case

Author: Ramkishor Chaladi

plugin/commands/user/permissions.go

@@ -84,17 +84,15 @@ func (cmd *PermissionsCommand) Run(args []string) error {
 			}
 
 		}
-		flag := false
+		flag := true
 		arr := []string{"ACCOUNT_SUMMARY_VIEW", "REQUEST_COMPLIANCE_REPORT", "COMPANY_EDIT", "ONE_TIME_PAYMENTS", "UPDATE_PAYMENT_DETAILS",
 			"EU_LIMITED_PROCESSING_MANAGE", "TICKET_ADD", "TICKET_EDIT", "TICKET_SEARCH", "TICKET_VIEW", "TICKET_VIEW_ALL"}
 		for i := 0; i < len(arr); i++ {
 			if *perm.KeyName == arr[i] {
-				flag = true
+				flag = false
 			}
 		}
 		if flag == true {
-			tablePermission.Add("", "", "")
-		} else {
 			tablePermission.Add(utils.FormatStringPointer(perm.Name), utils.FormatStringPointer(perm.KeyName), strconv.FormatBool(assigned))
 		}
 	}

plugin/commands/user/permissions_test.go

@@ -107,6 +107,24 @@ var _ = Describe("User Permissions", func() {
 			})
 		})
 
+		Context("hide user permissions", func() {
+			It("return not equal user permissions", func() {
+				err := testhelpers.RunCobraCommand(cliCommand.Command, "123")
+				Expect(err).NotTo(HaveOccurred())
+				Expect(fakeUI.Outputs()).To(Not(Equal("ACCOUNT_SUMMARY_VIEW")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("REQUEST_COMPLIANCE_REPORT")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("COMPANY_EDIT")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("ONE_TIME_PAYMENTS")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("UPDATE_PAYMENT_DETAILS")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("EU_LIMITED_PROCESSING_MANAGE")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("TICKET_ADD")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("TICKET_EDIT")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("TICKET_SEARCH")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("TICKET_VIEW")))
+				Expect(fakeUI.Outputs()).To(Not(Equal("TICKET_VIEW_ALL")))
+			})
+		})
+
 		Context("user permissions - master account", func() {
 			It("return user permissions", func() {
 				fakeUserManager.GetUserReturns(datatypes.User_Customer{