From 3e972650c7bb85a9cb893b1e1906b7f5f97f2ab9 Mon Sep 17 00:00:00 2001
From: Victor Woeltjen <victor.woeltjen@nasa.gov>
Date: Fri, 10 Apr 2015 17:37:11 -0700
Subject: [PATCH] [Containment] Test LinkAction

Add test cases for LinkAction, added to allow composition
changes to be controlled by policy. WTD-962.
---
 .../edit/test/actions/LinkActionSpec.js       | 107 ++++++++++++++++++
 platform/commonUI/edit/test/suite.json        |   1 +
 2 files changed, 108 insertions(+)
 create mode 100644 platform/commonUI/edit/test/actions/LinkActionSpec.js

diff --git a/platform/commonUI/edit/test/actions/LinkActionSpec.js b/platform/commonUI/edit/test/actions/LinkActionSpec.js
new file mode 100644
index 0000000000..ae1985911b
--- /dev/null
+++ b/platform/commonUI/edit/test/actions/LinkActionSpec.js
@@ -0,0 +1,107 @@
+/*global define,describe,it,expect,beforeEach,jasmine,spyOn*/
+
+define(
+    ["../../src/actions/LinkAction"],
+    function (LinkAction) {
+        "use strict";
+
+        describe("The Link action", function () {
+            var mockQ,
+                mockDomainObject,
+                mockParent,
+                mockContext,
+                mockMutation,
+                mockPersistence,
+                mockType,
+                actionContext,
+                model,
+                capabilities,
+                action;
+
+            function mockPromise(value) {
+                return {
+                    then: function (callback) {
+                        return mockPromise(callback(value));
+                    }
+                };
+            }
+
+            beforeEach(function () {
+
+
+                mockDomainObject = jasmine.createSpyObj(
+                    "domainObject",
+                    [ "getId", "getCapability" ]
+                );
+                mockQ = { when: mockPromise };
+                mockParent = {
+                    getModel: function () {
+                        return model;
+                    },
+                    getCapability: function (k) {
+                        return capabilities[k];
+                    },
+                    useCapability: function (k, v) {
+                        return capabilities[k].invoke(v);
+                    }
+                };
+                mockContext = jasmine.createSpyObj("context", [ "getParent" ]);
+                mockMutation = jasmine.createSpyObj("mutation", [ "invoke" ]);
+                mockPersistence = jasmine.createSpyObj("persistence", [ "persist" ]);
+                mockType = jasmine.createSpyObj("type", [ "hasFeature" ]);
+
+                mockDomainObject.getId.andReturn("test");
+                mockDomainObject.getCapability.andReturn(mockContext);
+                mockContext.getParent.andReturn(mockParent);
+                mockType.hasFeature.andReturn(true);
+                mockMutation.invoke.andReturn(mockPromise(true));
+
+
+                capabilities = {
+                    mutation: mockMutation,
+                    persistence: mockPersistence,
+                    type: mockType
+                };
+                model = {
+                    composition: [ "a", "b", "c" ]
+                };
+
+                actionContext = {
+                    domainObject: mockParent,
+                    selectedObject: mockDomainObject
+                };
+
+                action = new LinkAction(actionContext);
+            });
+
+
+            it("mutates the parent when performed", function () {
+                action.perform();
+                expect(mockMutation.invoke)
+                    .toHaveBeenCalledWith(jasmine.any(Function));
+            });
+
+            it("changes composition from its mutation function", function () {
+                var mutator, result;
+                action.perform();
+                mutator = mockMutation.invoke.mostRecentCall.args[0];
+                result = mutator(model);
+
+                // Should not have cancelled the mutation
+                expect(result).not.toBe(false);
+
+                // Simulate mutate's behavior (remove can either return a
+                // new model or modify this one in-place)
+                result = result || model;
+
+                // Should have removed "test" - that was our
+                // mock domain object's id.
+                expect(result.composition).toEqual(["a", "b", "c", "test"]);
+
+                // Finally, should have persisted
+                expect(mockPersistence.persist).toHaveBeenCalled();
+            });
+
+        });
+    }
+);
\ No newline at end of file
diff --git a/platform/commonUI/edit/test/suite.json b/platform/commonUI/edit/test/suite.json
index a73deeb8ee..5e3a909c25 100644
--- a/platform/commonUI/edit/test/suite.json
+++ b/platform/commonUI/edit/test/suite.json
@@ -1,6 +1,7 @@
 [
     "actions/CancelAction",
     "actions/EditAction",
+    "actions/LinkAction",
     "actions/PropertiesAction",
     "actions/PropertiesDialog",
     "actions/RemoveAction",