[Representation] Spec for gesture provider
Add spec for GestureProvider, introduced to separate out the attachment of gestures from the creation of representations. WTD-521.
This commit is contained in:
@@ -15,12 +15,10 @@ define(
|
|||||||
function GestureProvider(gestures) {
|
function GestureProvider(gestures) {
|
||||||
var gestureMap = {};
|
var gestureMap = {};
|
||||||
|
|
||||||
function releaseGestures(gestures) {
|
function releaseGesture(gesture) {
|
||||||
gestures.forEach(function (gesture) {
|
|
||||||
if (gesture && gesture.destroy) {
|
if (gesture && gesture.destroy) {
|
||||||
gesture.destroy();
|
gesture.destroy();
|
||||||
}
|
}
|
||||||
});
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function attachGestures(element, domainObject, gestureKeys) {
|
function attachGestures(element, domainObject, gestureKeys) {
|
||||||
@@ -36,7 +34,7 @@ define(
|
|||||||
|
|
||||||
return {
|
return {
|
||||||
destroy: function () {
|
destroy: function () {
|
||||||
releaseGestures(attachedGestures);
|
attachedGestures.forEach(releaseGesture);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|||||||
70
platform/representation/test/gestures/GestureProviderSpec.js
Normal file
70
platform/representation/test/gestures/GestureProviderSpec.js
Normal file
@@ -0,0 +1,70 @@
|
|||||||
|
/*global define,Promise,describe,it,expect,beforeEach,waitsFor,jasmine*/
|
||||||
|
|
||||||
|
/**
|
||||||
|
* GestureProviderSpec. Created by vwoeltje on 11/6/14.
|
||||||
|
*/
|
||||||
|
define(
|
||||||
|
["../../src/gestures/GestureProvider"],
|
||||||
|
function (GestureProvider) {
|
||||||
|
"use strict";
|
||||||
|
|
||||||
|
var JQLITE_FUNCTIONS = [ "on", "off", "attr", "removeAttr" ],
|
||||||
|
GESTURE_KEYS = ["a", "b", "c", "d", "e"],
|
||||||
|
DOMAIN_OBJECT_METHODS = [ "getId", "getModel", "getCapability", "hasCapability", "useCapability"];
|
||||||
|
|
||||||
|
describe("The gesture provider", function () {
|
||||||
|
var mockGestures,
|
||||||
|
mockDestroys,
|
||||||
|
mockElement,
|
||||||
|
mockDomainObject,
|
||||||
|
provider;
|
||||||
|
|
||||||
|
beforeEach(function () {
|
||||||
|
mockElement = jasmine.createSpyObj("element", JQLITE_FUNCTIONS);
|
||||||
|
mockDomainObject = jasmine.createSpyObj("domainObject", DOMAIN_OBJECT_METHODS);
|
||||||
|
|
||||||
|
mockDestroys = {};
|
||||||
|
mockGestures = {};
|
||||||
|
GESTURE_KEYS.forEach(function (key) {
|
||||||
|
mockDestroys[key] = jasmine.createSpy("destroy-" + key);
|
||||||
|
mockGestures[key] = jasmine.createSpy("gesture-" + key);
|
||||||
|
mockGestures[key].andReturn({ destroy: mockDestroys[key] });
|
||||||
|
mockGestures[key].key = key;
|
||||||
|
});
|
||||||
|
|
||||||
|
provider = new GestureProvider(GESTURE_KEYS.map(function (key) {
|
||||||
|
return mockGestures[key];
|
||||||
|
}));
|
||||||
|
});
|
||||||
|
|
||||||
|
it("attaches matching to an element", function () {
|
||||||
|
provider.attachGestures(mockElement, mockDomainObject, ["a", "c", "e"]);
|
||||||
|
|
||||||
|
expect(mockGestures.a).toHaveBeenCalledWith(mockElement, mockDomainObject);
|
||||||
|
expect(mockGestures.c).toHaveBeenCalledWith(mockElement, mockDomainObject);
|
||||||
|
expect(mockGestures.e).toHaveBeenCalledWith(mockElement, mockDomainObject);
|
||||||
|
expect(mockGestures.b).not.toHaveBeenCalled();
|
||||||
|
expect(mockGestures.d).not.toHaveBeenCalled();
|
||||||
|
|
||||||
|
// No destroys should have been called - let's check
|
||||||
|
GESTURE_KEYS.forEach(function (key) {
|
||||||
|
expect(mockDestroys[key]).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("detaches gestures on request", function () {
|
||||||
|
provider.attachGestures(
|
||||||
|
mockElement,
|
||||||
|
mockDomainObject,
|
||||||
|
["b", "d", "e"]
|
||||||
|
).destroy();
|
||||||
|
|
||||||
|
expect(mockDestroys.b).toHaveBeenCalled();
|
||||||
|
expect(mockDestroys.d).toHaveBeenCalled();
|
||||||
|
expect(mockDestroys.e).toHaveBeenCalled();
|
||||||
|
expect(mockDestroys.a).not.toHaveBeenCalled();
|
||||||
|
expect(mockDestroys.c).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
});
|
||||||
|
}
|
||||||
|
);
|
||||||
@@ -2,6 +2,7 @@
|
|||||||
"gestures/ContextMenuGesture",
|
"gestures/ContextMenuGesture",
|
||||||
"gestures/DragGesture",
|
"gestures/DragGesture",
|
||||||
"gestures/DropGesture",
|
"gestures/DropGesture",
|
||||||
|
"gestures/GestureProvider",
|
||||||
"MCTInclude",
|
"MCTInclude",
|
||||||
"MCTRepresentation"
|
"MCTRepresentation"
|
||||||
]
|
]
|
||||||
Reference in New Issue
Block a user