added unit test case and updated the code as suggested

Author: Ramkishor Chaladi

SoftLayer/CLI/user/permissions.py

@@ -49,9 +49,7 @@ def permission_table(user_permissions, all_permissions):
                                 'COMPANY_EDIT', 'ONE_TIME_PAYMENTS', 'UPDATE_PAYMENT_DETAILS',
                                 'EU_LIMITED_PROCESSING_MANAGE', 'TICKET_ADD', 'TICKET_EDIT',
                                 'TICKET_SEARCH', 'TICKET_VIEW', 'TICKET_VIEW_ALL']
-        if perm['keyName'] in hide_permission_list:
-            table.add_row(["", "", ""])
-        else:
+        if perm['keyName'] not in hide_permission_list:
             table.add_row([perm['name'], perm['keyName'], assigned])
     return table
 

tests/managers/user_tests.py

@@ -183,6 +183,24 @@ def test_format_permission_object_all(self):
         self.assert_called_with(service_name, 'getAllObjects')
         self.assertEqual(expected, result)
 
+    def test_hide_permissions(self):
+        result = self.manager.get_all_permissions()
+        hide_permissions = [
+            {'keyName': 'ACCOUNT_SUMMARY_VIEW'},
+            {'keyName': 'REQUEST_COMPLIANCE_REPORT'},
+            {'keyName': 'COMPANY_EDIT'},
+            {'keyName': 'ONE_TIME_PAYMENTS'},
+            {'keyName': 'UPDATE_PAYMENT_DETAILS'},
+            {'keyName': 'EU_LIMITED_PROCESSING_MANAGE'},
+            {'keyName': 'TICKET_ADD'},
+            {'keyName': 'TICKET_EDIT'},
+            {'keyName': 'TICKET_SEARCH'},
+            {'keyName': 'TICKET_VIEW'},
+            {'keyName': 'TICKET_VIEW_ALL'}
+        ]
+        self.assert_called_with('SoftLayer_User_Permission_Action', 'getAllObjects')
+        self.assertNotEqual(hide_permissions, result)
+
     def test_get_current_user(self):
         result = self.manager.get_current_user()
         self.assert_called_with('SoftLayer_Account', 'getCurrentUser', mask=mock.ANY)