fix

Author: GoodDayForSurf

packages/devextreme/testing/tests/DevExpress.ui.events/events.utils.nodesDisposing.tests.js

@@ -16,7 +16,7 @@ QUnit.testStart(function() {
 
 QUnit.module('event nodes disposing');
 
-QUnit.test('should not leave extra handlers in elementDataMap when using subscribeNodesDisposing and unsubscribeNodesDisposing for click on document', function(assert) {
+QUnit.test('should clean elementDataMap when using subscribeNodesDisposing and unsubscribeNodesDisposing for click', function(assert) {
     const document = domAdapter.getDocument();
     const testElement = document.getElementById('test-element');
 
@@ -26,9 +26,7 @@ QUnit.test('should not leave extra handlers in elementDataMap when using subscri
         delegateTarget: document
     });
 
-    const callback = function() {};
-
-    const subscriptionData = subscribeNodesDisposing(clickEvent, callback);
+    const subscriptionData = subscribeNodesDisposing(clickEvent, function() {});
 
     const afterSubscribeElementData = eventsEngine.elementDataMap.get(document);
     const afterSubscribeHandleObjectsCount = afterSubscribeElementData && afterSubscribeElementData[removeEvent]
@@ -38,12 +36,19 @@ QUnit.test('should not leave extra handlers in elementDataMap when using subscri
     unsubscribeNodesDisposing(clickEvent, subscriptionData.callback, subscriptionData.nodes);
 
     const finalElementData = eventsEngine.elementDataMap.get(document);
-    const finalHandleObjectsCount = finalElementData && finalElementData[removeEvent]
+    const afterUnsubscribeHandleObjectsCount = finalElementData && finalElementData[removeEvent]
         ? finalElementData[removeEvent].handleObjects.length
-        : -1;
+        : 0;
+
+    assert.equal(
+        afterSubscribeHandleObjectsCount,
+        1,
+        `HandleObjects should be added for "${removeEvent}" event after subscribe. HandleObjects count: ${afterSubscribeHandleObjectsCount};`
+    );
 
-    assert.ok(
-        -1 >= 0,
-        `HandleObjects should be removed. For ${removeEvent} event, after HandleObjects subscribe count: ${afterSubscribeHandleObjectsCount}; Final HandleObjects count: ${finalHandleObjectsCount};`
+    assert.equal(
+        afterUnsubscribeHandleObjectsCount,
+        0,
+        `HandleObjects should be removed for "${removeEvent}" event after unsubscribe. HandleObjects count: ${afterUnsubscribeHandleObjectsCount};`
     );
 });