Remove node version from bundle

Author: pharret31

packages/devextreme/build/gulp/npm.js

@@ -70,7 +70,6 @@ const distGlobsPattern = (jsFolder, exclude) => [
     `!${jsFolder}/dx-diagram*`,
     `!${jsFolder}/dx-gantt*`,
     `!${jsFolder}/dx-quill*`,
-    `!${jsFolder}/vectormap-utils/**/*`,
 ];
 
 const srcGlobs = esmSrcGlobs;

packages/devextreme/build/gulp/vectormap.js

@@ -24,13 +24,29 @@ function transformFileName(fileName) {
 
 gulp.task('vectormap-utils', function() {
     return merge(
-        createVectorMapUtilsStream('browser', '.debug', false),
-        createVectorMapUtilsStream('browser', '', true),
-        createVectorMapUtilsStream('node', '', false)
+        createBrowserVectorMapUtilsStream('.debug', false),
+        createBrowserVectorMapUtilsStream('', true),
     );
 });
 
-gulp.task('vectormap-data', gulp.series('vectormap-utils', function() {
+function buildNodeUtilsForDataGeneration() {
+    const settings = require(path.join('../..', VECTORMAP_UTILS_PATH, '_settings.json'));
+    const part = settings['node'];
+    return gulp.src(settings.commonFiles.concat(part.files).map(transformFileName))
+        .pipe(concat(part.fileName + '.js'))
+        .pipe(headerPipes.useStrict())
+        .pipe(gulp.dest(VECTORMAP_UTILS_RESULT_PATH));
+}
+
+function cleanNodeUtils(done) {
+    const nodeFile = path.join(VECTORMAP_UTILS_RESULT_PATH, 'dx.vectormaputils.node.js');
+    if(fs.existsSync(nodeFile)) {
+        fs.unlinkSync(nodeFile);
+    }
+    done();
+}
+
+gulp.task('vectormap-data', gulp.series(buildNodeUtilsForDataGeneration, function() {
     const stream = merge();
     const processFiles = require(path.join('../..', VECTORMAP_UTILS_RESULT_PATH, 'dx.vectormaputils.node.js')).processFiles;
 
@@ -57,7 +73,7 @@ gulp.task('vectormap-data', gulp.series('vectormap-utils', function() {
         });
     });
     return stream;
-}));
+}, cleanNodeUtils));
 
 function patchVectorMapUtilsStream(stream, isMinify) {
     return stream.pipe(headerPipes.useStrict())
@@ -67,20 +83,17 @@ function patchVectorMapUtilsStream(stream, isMinify) {
         .pipe(gulp.dest(VECTORMAP_UTILS_RESULT_PATH));
 }
 
-function createVectorMapUtilsStream(name, suffix, isMinify) {
+function createBrowserVectorMapUtilsStream(suffix, isMinify) {
     const settings = require(path.join('../..', VECTORMAP_UTILS_PATH, '_settings.json'));
-    const part = settings[name];
+    const part = settings['browser'];
     const stream = gulp.src(settings.commonFiles.concat(part.files).map(transformFileName))
         .pipe(concat(part.fileName + suffix + '.js'));
 
-    if(name === 'browser') {
-        return stream.pipe(tap(file => {
-            patchVectorMapUtilsStream(gulp.src('build/gulp/vectormaputils-template.jst')
-                .pipe(template({ data: file.contents }))
-                .pipe(rename(path.basename(file.path))), isMinify);
-        }));
-    }
-    return patchVectorMapUtilsStream(stream, isMinify);
+    return stream.pipe(tap(file => {
+        patchVectorMapUtilsStream(gulp.src('build/gulp/vectormaputils-template.jst')
+            .pipe(template({ data: file.contents }))
+            .pipe(rename(path.basename(file.path))), isMinify);
+    }));
 }
 
 gulp.task('vectormap', gulp.series('vectormap-utils', 'vectormap-data'));

packages/devextreme/ports.json

@@ -1,4 +1,3 @@
 {
-  "qunit": "20060",
-  "vectormap-utils-tester": "20061"
+  "qunit": "20060"
 }

packages/devextreme/project.json

@@ -685,6 +685,7 @@
         "{projectRoot}/build/gulp/vectormap.js"
       ],
       "outputs": [
+        "{projectRoot}/artifacts/js/vectormap-utils",
         "{projectRoot}/artifacts/js/vectormap-data"
       ]
     },
@@ -732,7 +733,6 @@
         "{projectRoot}/build/gulp/vendor.js"
       ],
       "outputs": [
-        "{projectRoot}/artifacts/js/vectormap-utils",
         "{projectRoot}/artifacts/js/cldr"
       ]
     },

packages/devextreme/testing/helpers/vectormaputils-tester.js

@@ -21,7 +21,6 @@ import {
   parseNumber,
   readBodyText,
   readFormBody,
-  resolveNodePath,
   safeDecodeURIComponent,
   safeReadFile,
   splitCommaList,
@@ -35,7 +34,6 @@ import { createVectorMapService } from './lib/vectormap';
 import {
   sendHtml,
   sendJson,
-  sendJsonText,
   sendNotFound,
   sendText,
   sendXml,
@@ -77,9 +75,6 @@ const RUN_FLAGS = {
 
 const PORTS = loadPorts(path.join(PACKAGE_ROOT, 'ports.json'));
 const QUNIT_PORT = Number(PORTS.qunit);
-const VECTOR_MAP_TESTER_PORT = Number(PORTS['vectormap-utils-tester']);
-
-const PATH_TO_NODE = resolveNodePath();
 
 const logger = createRunnerLogger(RAW_LOG_FILENAME);
 const templates = createTemplateRenderer(TEMPLATES_ROOT, escapeHtml);
@@ -102,8 +97,6 @@ const vectorMapService = createVectorMapService({
   packageRoot: PACKAGE_ROOT,
   testingRoot: TESTING_ROOT,
   vectorDataDirectory: VECTOR_DATA_DIRECTORY,
-  vectorMapTesterPort: VECTOR_MAP_TESTER_PORT,
-  pathToNode: PATH_TO_NODE,
 });
 const staticFiles = createStaticFileService({
   escapeHtml,
@@ -312,35 +305,6 @@ async function handleRequest(req: http.IncomingMessage, res: http.ServerResponse
     return;
   }
 
-  if (req.method === 'GET') {
-    const parseBufferMatch = /^\/TestVectorMapData\/ParseBuffer\/(.+)$/i.exec(pathname);
-    if (parseBufferMatch) {
-      const id = safeDecodeURIComponent(parseBufferMatch[1]);
-      const responseText = await vectorMapService.redirectRequestToVectorMapNodeServer('parse-buffer', id);
-      sendJsonText(res, responseText);
-      return;
-    }
-  }
-
-  if (req.method === 'GET') {
-    const readAndParseMatch = /^\/TestVectorMapData\/ReadAndParse\/(.+)$/i.exec(pathname);
-    if (readAndParseMatch) {
-      const id = safeDecodeURIComponent(readAndParseMatch[1]);
-      const responseText = await vectorMapService.redirectRequestToVectorMapNodeServer('read-and-parse', id);
-      sendJsonText(res, responseText);
-      return;
-    }
-  }
-
-  if (req.method === 'GET') {
-    const executeConsoleAppMatch = /^\/TestVectorMapData\/ExecuteConsoleApp(?:\/(.*))?$/i.exec(pathname);
-    if (executeConsoleAppMatch) {
-      const result = vectorMapService.executeVectorMapConsoleApp(requestUrl.searchParams);
-      sendJson(res, result);
-      return;
-    }
-  }
-
   if (staticFiles.tryServeStatic(req, res, pathname, requestUrl.searchParams)) {
     return;
   }

packages/devextreme/testing/runner/index.ts

@@ -100,16 +100,9 @@ export interface VectorMapDataItem {
   expected: string;
 }
 
-export interface VectorMapOutputItem {
-  file: string;
-  variable: string | null;
-  content: JsonValue;
-}
-
 export interface PortsMap {
   [key: string]: number | string;
   qunit: number | string;
-  'vectormap-utils-tester': number | string;
 }
 
 export type TemplateVarValue = JsonValue | bigint | undefined;

packages/devextreme/testing/runner/lib/types.ts

@@ -45,13 +45,12 @@ export function loadPorts(filePath: string): PortsMap {
     throw new Error(`Invalid ports definition: ${filePath}`);
   }
 
-  if (!isPortValue(parsed.qunit) || !isPortValue(parsed['vectormap-utils-tester'])) {
+  if (!isPortValue(parsed.qunit)) {
     throw new Error(`Required ports are missing in ${filePath}`);
   }
 
   const portsMap: PortsMap = {
     qunit: parsed.qunit,
-    'vectormap-utils-tester': parsed['vectormap-utils-tester'],
   };
 
   Object.entries(parsed).forEach(([key, value]) => {