T1297188 - FileManager inside popup - W1025 (#30401)

Author: Staziz

packages/devextreme/js/ui/file_manager/ui.file_manager.item_list.details.js

@@ -95,6 +95,7 @@ class FileManagerDetailsItemList extends FileManagerItemListBase {
             loadPanel: {
                 shading: true
             },
+            height: '100%',
             showColumnLines: false,
             showRowLines: false,
             columnHidingEnabled: false,

packages/devextreme/testing/tests/DevExpress.ui.widgets/fileManagerParts/markup.tests.js

@@ -1,5 +1,7 @@
 const { test } = QUnit;
 import $ from 'jquery';
+import { extend } from 'core/utils/extend';
+import consoleUtils from '__internal/core/utils/m_console';
 import 'ui/file_manager';
 import fx from 'common/core/animation/fx';
 import windowUtils from 'core/utils/window';
@@ -32,17 +34,49 @@ const prepareFileManager = (context, isPure, options) => {
     context.clock.tick(400);
 };
 
+const moduleConfig_T1297188 = {
+    beforeEach: function() {
+        const markup = '<div id="fileManagerPopup"></div>';
+        $('#qunit-fixture').html(markup);
 
-QUnit.testStart(function() {
-    const markup =
-        '<div id="fileManager"></div>';
+        this.clock = sinon.useFakeTimers();
+        fx.off = true;
+        this.clock.tick(400);
+    },
+    afterEach: function() {
+        this.clock.tick(5000);
+        this.clock.restore();
+        fx.off = false;
+    }
+};
 
-    $('#qunit-fixture').html(markup);
-});
+const createFileManagerInsidePopup = (context, useThumbnailViewMode, extOptions) => {
+    const viewMode = useThumbnailViewMode ? 'thumbnails' : 'details';
+    extOptions = extOptions || {};
+    const options = extend({
+        fileSystemProvider: createTestFileSystem(),
+        itemView: {
+            mode: viewMode
+        }
+    }, extOptions);
+    const fileManagerPopup = $('#fileManagerPopup').dxPopup({
+        visible: true,
+        contentTemplate: function(contentElement) {
+            $('<div id="fileManager"></div>')
+                .appendTo(contentElement)
+                .dxFileManager(options);
+        }
+    }).dxPopup('instance');
+    context.clock.tick(400);
+    return fileManagerPopup;
+};
 
 const moduleConfig = {
 
     beforeEach: function() {
+        const markup = '<div id="fileManager"></div>';
+        $('#qunit-fixture').html(markup);
+
         this.clock = sinon.useFakeTimers();
         fx.off = true;
     },
@@ -207,3 +241,25 @@ QUnit.module('Markup rendering', moduleConfig, () => {
     });
 
 });
+
+QUnit.module('fileManager inside popup not causing any warnings in console (T1297188)', moduleConfig_T1297188, () => {
+    test('no console warnings displayed (T1297188)', function(assert) {
+        const loggerSpy = sinon.spy(consoleUtils.logger, 'warn');
+
+        try {
+            createFileManagerInsidePopup(this, false, { width: '100%', height: '100%' });
+
+            this.clock.tick(400);
+
+            const w1025Warning = loggerSpy.args.find(([warning, ...rest]) =>
+                typeof warning === 'string' && warning.includes('W1025')
+            );
+
+            assert.ok(!w1025Warning, 'W1025 scrolling.mode warning should not appear in console');
+
+        } finally {
+            loggerSpy.restore();
+        }
+    });
+});
+