WIP. check test for click subscription in events_engine (T1307313)"

This reverts commit d0d18c3.

Author: GoodDayForSurf

packages/devextreme/testing/tests/DevExpress.ui.events/event_nodes_disposing.tests.js

@@ -1,4 +1,5 @@
 import eventsEngine from 'common/core/events/core/events_engine';
+import domAdapter from '__internal/core/m_dom_adapter';
 import { removeEvent } from 'common/core/events/remove';
 import {
     subscribeNodesDisposing,
@@ -16,7 +17,8 @@ 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) {
-    const clickEventName = 'click';
+    const clickEventName = 'dxclick';
+    const document = domAdapter.getDocument();
     const testElement = document.getElementById('test-element');
 
     // Get initial state of handleObjects for click event on document

packages/devextreme/testing/tests/DevExpress.ui.events/eventsEngine.tests.js

@@ -1,4 +1,5 @@
 import $ from 'jquery';
+import domAdapter from '__internal/core/m_dom_adapter';
 import eventsEngine from 'common/core/events/core/events_engine';
 import keyboardMock from '../../helpers/keyboardMock.js';
 import registerEvent from 'common/core/events/core/event_registrator';
@@ -568,7 +569,7 @@ QUnit.test('removeEventListener should not be called if native handler is not ex
     delListener.restore();
 });
 
-QUnit.module('Memory');
+QUnit.module('Memory bugs');
 
 QUnit.test('removing subscriptions should remove data from elementDataMap', function(assert) {
     const div = document.createElement('div');
@@ -591,6 +592,33 @@ QUnit.test('removing subscriptions should not remove data from elementDataMap if
     assert.equal(eventsEngine.elementDataMap.has(div), hasData);
 });
 
+QUnit.test('should not leak memory when subscribing on document and clicking elements (T1307313)', function(assert) {
+    const clickEventName = 'dxclick';
+    const document = domAdapter.getDocument();
+    const fixture = document.getElementById('qunit-fixture');
+
+    fixture.innerHTML = '<button id="test-element">Test</button>';
+
+    const testElement = document.getElementById('test-element');
+
+    eventsEngine.on(document, clickEventName, () => {});
+
+    for(let i = 0; i < 100; i++) {
+        testElement.click();
+    }
+
+    const eventData = eventsEngine.elementDataMap.get(document);
+
+    const count = Object.keys(eventData).reduce((res, key) => res + eventData[key].handleObjects?.length, 0);
+
+    assert.ok(
+        eventData?.length <= 5,
+        `Memory should not leak. Element data: ${count}, ${Object.keys(eventData)}, ${eventsEngine.elementDataMap.size}`
+    );
+
+    eventsEngine.off(document);
+});
+
 QUnit.module('Strategy');
 
 QUnit.test('it should be possible to set only one method for strategy', function(assert) {