SMC-5125: Review changes

happner · Feb 3, 2022 · 84fc4bb · 84fc4bb
1 parent f68248f
commit 84fc4bb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/lib/services/security/checkpoint.js b/lib/services/security/checkpoint.js
@@ -175,10 +175,7 @@ CheckPoint.prototype.__loadPermissionSet = function(identity, callback) {
         permissionSetStore.setSync(
           identity.permissionSetKey,
           identity.user.username,
-          permissionSet,
-          {
-            clone: false
-          }
+          permissionSet
         );
         callback(null, permissionSet);
       }

diff --git a/test/integration/security/caches.js b/test/integration/security/caches.js
@@ -61,9 +61,6 @@ describe(testHelper.testName(__filename, 3), function() {
         expect(
           serviceInstance.services.security.checkpoint.__cache_checkpoint_authorization.__cache.max
         ).to.be(5);
-        // expect(
-        //   serviceInstance.services.security.checkpoint.__cache_checkpoint_permissionset.__cache.max
-        // ).to.be(6);
 
         serviceInstance.services.session
           .localClient({
@@ -297,10 +294,6 @@ describe(testHelper.testName(__filename, 3), function() {
           serviceInstance.services.security.checkpoint.__cache_checkpoint_authorization.__cache.values()
             .length
         ).to.be(5);
-        // expect(
-        //   serviceInstance.services.security.checkpoint.__cache_checkpoint_permissionset.__cache.values()
-        //     .length
-        // ).to.be(6);
         expect(
           serviceInstance.services.security.groups.permissionManager.cache.__cache.values().length
         ).to.be(5);