From 7915074b1090e6ba0a0d81515f0b10778fbb2a7e Mon Sep 17 00:00:00 2001
From: Victor Woeltjen <victor.woeltjen@nasa.gov>
Date: Wed, 1 Apr 2015 15:54:59 -0700
Subject: [PATCH] [Policy] Add spec for policy-driven view decorator

Add spec to allow the applicability of views to be determined
by policy decisions, WTD-1062.
---
 .../policy/test/PolicyViewDecoratorSpec.js    | 83 +++++++++++++++++++
 platform/policy/test/suite.json               |  1 +
 2 files changed, 84 insertions(+)
 create mode 100644 platform/policy/test/PolicyViewDecoratorSpec.js

diff --git a/platform/policy/test/PolicyViewDecoratorSpec.js b/platform/policy/test/PolicyViewDecoratorSpec.js
new file mode 100644
index 0000000000..60007fb73b
--- /dev/null
+++ b/platform/policy/test/PolicyViewDecoratorSpec.js
@@ -0,0 +1,83 @@
+/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
+
+define(
+    ["../src/PolicyViewDecorator"],
+    function (PolicyViewDecorator) {
+        "use strict";
+
+        describe("The policy view decorator", function () {
+            var mockPolicyService,
+                mockViewService,
+                mockDomainObject,
+                testViews,
+                decorator;
+
+            beforeEach(function () {
+                mockPolicyService = jasmine.createSpyObj(
+                    'policyService',
+                    ['allow']
+                );
+                mockViewService = jasmine.createSpyObj(
+                    'viewService',
+                    ['getViews']
+                );
+                mockDomainObject = jasmine.createSpyObj(
+                    'domainObject',
+                    ['getId']
+                );
+
+                // Content of actions should be irrelevant to this
+                // decorator, so just give it some objects to pass
+                // around.
+                testViews = [
+                    { someKey: "a" },
+                    { someKey: "b" },
+                    { someKey: "c" }
+                ];
+
+                mockDomainObject.getId.andReturn('xyz');
+                mockViewService.getViews.andReturn(testViews);
+                mockPolicyService.allow.andReturn(true);
+
+                decorator = new PolicyViewDecorator(
+                    mockPolicyService,
+                    mockViewService
+                );
+            });
+
+            it("delegates to its decorated view service", function () {
+                decorator.getViews(mockDomainObject);
+                expect(mockViewService.getViews)
+                    .toHaveBeenCalledWith(mockDomainObject);
+            });
+
+            it("provides views from its decorated view service", function () {
+                // Mock policy service allows everything by default,
+                // so everything should be returned
+                expect(decorator.getActions(mockDomainObject))
+                    .toEqual(testViews);
+            });
+
+            it("consults the policy service for each candidate view", function () {
+                decorator.getViews(mockDomainObject);
+                testViews.forEach(function (testView) {
+                    expect(mockPolicyService.allow).toHaveBeenCalledWith(
+                        'view',
+                        testView,
+                        mockDomainObject
+                    );
+                });
+            });
+
+            it("filters out policy-disallowed views", function () {
+                // Disallow the second action
+                mockPolicyService.allow.andCallFake(function (cat, candidate, ctxt) {
+                    return candidate.someKey !== 'b';
+                });
+                expect(decorator.getViews(mockDomainObject))
+                    .toEqual([ testViews[0], testViews[2] ]);
+            });
+
+        });
+    }
+);
\ No newline at end of file
diff --git a/platform/policy/test/suite.json b/platform/policy/test/suite.json
index 8706198dc6..c695797527 100644
--- a/platform/policy/test/suite.json
+++ b/platform/policy/test/suite.json
@@ -1,4 +1,5 @@
 [
     "PolicyActionDecorator",
+    "PolicyViewDecorator",
     "PolicyProvider"
 ]
\ No newline at end of file