From 39d444d6373b1ce2d495168a96cc5c94b0c11010 Mon Sep 17 00:00:00 2001
From: Victor Woeltjen <victor.woeltjen@nasa.gov>
Date: Wed, 25 Mar 2015 11:01:32 -0700
Subject: [PATCH] [Persistence] Test queue handler

Add test cases for handler for persistence queue, WTD-1033.
---
 .../queue/test/PersistenceQueueHandlerSpec.js | 105 +++++++++++++++++-
 1 file changed, 104 insertions(+), 1 deletion(-)

diff --git a/platform/persistence/queue/test/PersistenceQueueHandlerSpec.js b/platform/persistence/queue/test/PersistenceQueueHandlerSpec.js
index a95631bde5..8a975b1273 100644
--- a/platform/persistence/queue/test/PersistenceQueueHandlerSpec.js
+++ b/platform/persistence/queue/test/PersistenceQueueHandlerSpec.js
@@ -6,8 +6,111 @@ define(
     function (PersistenceQueueHandler) {
         "use strict";
 
-        describe("The persistence queue", function () {
+        var TEST_ERROR = { someKey: "some value" };
 
+        describe("The persistence queue handler", function () {
+            var mockQ,
+                mockFailureHandler,
+                mockPersistences,
+                mockDomainObjects,
+                mockQueue,
+                mockRejection,
+                handler;
+
+            function asPromise(value) {
+                return (value || {}).then ? value : {
+                    then: function (callback) {
+                        return asPromise(callback(value));
+                    }
+                };
+            }
+
+            function makeMockPersistence(id) {
+                var mockPersistence = jasmine.createSpyObj(
+                    'persistence-' + id,
+                    [ 'persist', 'refresh' ]
+                );
+                mockPersistence.persist.andReturn(asPromise(true));
+                return mockPersistence;
+            }
+
+            function makeMockDomainObject(id) {
+                var mockDomainObject = jasmine.createSpyObj(
+                    'domainObject-' + id,
+                    [ 'getId' ]
+                );
+                mockDomainObject.getId.andReturn(id);
+                return mockDomainObject;
+            }
+
+            beforeEach(function () {
+                mockQ = jasmine.createSpyObj('$q', ['all']);
+                mockFailureHandler = jasmine.createSpyObj('handler', ['handle']);
+                mockQueue = jasmine.createSpyObj('queue', ['put']);
+                mockPersistences = {};
+                mockDomainObjects = {};
+                ['a', 'b', 'c'].forEach(function (id) {
+                    mockPersistences[id] = makeMockPersistence(id);
+                    mockDomainObjects[id] = makeMockDomainObject(id);
+                });
+                mockRejection = jasmine.createSpyObj('rejection', ['then']);
+                mockQ.all.andReturn(asPromise([]));
+                mockRejection.then.andCallFake(function (callback, fallback) {
+                    return asPromise(fallback({ someKey: "some value" }));
+                });
+                handler = new PersistenceQueueHandler(mockQ, mockFailureHandler);
+            });
+
+            it("invokes persistence on all members in the group", function () {
+                handler.persist(mockPersistences, mockDomainObjects, mockQueue);
+                expect(mockPersistences.a.persist).toHaveBeenCalled();
+                expect(mockPersistences.b.persist).toHaveBeenCalled();
+                expect(mockPersistences.c.persist).toHaveBeenCalled();
+                // No failures in this group
+                expect(mockFailureHandler.handle).not.toHaveBeenCalled();
+            });
+
+            it("handles failures that occur", function () {
+                mockPersistences.b.persist.andReturn(mockRejection);
+                mockPersistences.c.persist.andReturn(mockRejection);
+                handler.persist(mockPersistences, mockDomainObjects, mockQueue);
+                expect(mockFailureHandler.handle).toHaveBeenCalledWith([
+                    {
+                        id: 'b',
+                        persistence: mockPersistences.b,
+                        domainObject: mockDomainObjects.b,
+                        requeue: jasmine.any(Function),
+                        error: TEST_ERROR
+                    },
+                    {
+                        id: 'c',
+                        persistence: mockPersistences.c,
+                        domainObject: mockDomainObjects.c,
+                        requeue: jasmine.any(Function),
+                        error: TEST_ERROR
+                    }
+                ]);
+            });
+
+            it("provides a requeue method for failures", function () {
+                // This method is needed by PersistenceFailureHandler
+                // to allow requeuing of objects for persistence when
+                // Overwrite is chosen.
+                mockPersistences.b.persist.andReturn(mockRejection);
+                handler.persist(mockPersistences, mockDomainObjects, mockQueue);
+
+                // Verify precondition
+                expect(mockQueue.put).not.toHaveBeenCalled();
+
+                // Invoke requeue
+                mockFailureHandler.handle.mostRecentCall.args[0][0].requeue();
+
+                // Should have returned the object to the queue
+                expect(mockQueue.put).toHaveBeenCalledWith(
+                    mockDomainObjects.b,
+                    mockPersistences.b
+                );
+            });
         });
     }
 );
\ No newline at end of file