From 5152e64895c1065cc6d460dcd5326b7da4e8ed19 Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Fri, 10 Jun 2016 10:06:05 -0700 Subject: [PATCH] [Duplicate] Allow copy across spaces Fixes #1007 --- .../src/policies/CrossSpacePolicy.js | 5 +-- .../test/policies/CrossSpacePolicySpec.js | 34 +++++++++---------- 2 files changed, 17 insertions(+), 22 deletions(-) diff --git a/platform/entanglement/src/policies/CrossSpacePolicy.js b/platform/entanglement/src/policies/CrossSpacePolicy.js index 29aab5a484..39d05bd532 100644 --- a/platform/entanglement/src/policies/CrossSpacePolicy.js +++ b/platform/entanglement/src/policies/CrossSpacePolicy.js @@ -24,10 +24,7 @@ define( [], function () { - var DISALLOWED_ACTIONS = [ - "move", - "copy" - ]; + var DISALLOWED_ACTIONS = ["move"]; /** * This policy prevents performing move/copy/link actions across diff --git a/platform/entanglement/test/policies/CrossSpacePolicySpec.js b/platform/entanglement/test/policies/CrossSpacePolicySpec.js index ef09a47619..38bb1f9606 100644 --- a/platform/entanglement/test/policies/CrossSpacePolicySpec.js +++ b/platform/entanglement/test/policies/CrossSpacePolicySpec.js @@ -70,27 +70,25 @@ define( policy = new CrossSpacePolicy(); }); - ['move', 'copy'].forEach(function (key) { - describe("for " + key + " actions", function () { - beforeEach(function () { - testActionMetadata.key = key; - }); + describe("for move actions", function () { + beforeEach(function () { + testActionMetadata.key = 'move'; + }); - it("allows same-space changes", function () { - expect(policy.allow(mockAction, sameSpaceContext)) - .toBe(true); - }); + it("allows same-space changes", function () { + expect(policy.allow(mockAction, sameSpaceContext)) + .toBe(true); + }); - it("disallows cross-space changes", function () { - expect(policy.allow(mockAction, crossSpaceContext)) - .toBe(false); - }); + it("disallows cross-space changes", function () { + expect(policy.allow(mockAction, crossSpaceContext)) + .toBe(false); + }); - it("allows actions with no selectedObject", function () { - expect(policy.allow(mockAction, { - domainObject: makeObject('a') - })).toBe(true); - }); + it("allows actions with no selectedObject", function () { + expect(policy.allow(mockAction, { + domainObject: makeObject('a') + })).toBe(true); }); });