[Duplicate] Allow copy across spaces

Fixes #1007
This commit is contained in:
Victor Woeltjen 2016-06-10 10:06:05 -07:00
parent 0263237b2c
commit 5152e64895
2 changed files with 17 additions and 22 deletions

View File

@ -24,10 +24,7 @@ define(
[],
function () {
var DISALLOWED_ACTIONS = [
"move",
"copy"
];
var DISALLOWED_ACTIONS = ["move"];
/**
* This policy prevents performing move/copy/link actions across

View File

@ -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);
});
});