Merge pull request #2070 from ramkishor-ch/issue_1956

Enable all user permissions doesn't get all permissions

Author: allmightyspiff

SoftLayer/CLI/user/permissions.py

@@ -45,7 +45,12 @@ def permission_table(user_permissions, all_permissions):
     table.align['Assigned'] = 'l'
     for perm in all_permissions:
         assigned = user_permissions.get(perm['keyName'], False)
-        table.add_row([perm['name'], perm['keyName'], assigned])
+        hide_permission_list = ['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']
+        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)