datagrok-tools 4.12.25 → 4.12.27

package/bin/commands/check.js

@@ -253,11 +253,11 @@ function checkFuncSignatures(packagePath, files) {
         tags = _ref5.tags;
       var value = true;
       var message = '';
-      if (tags == null || tags.filter(function (t) {
-        return t.startsWith('fileViewer');
-      }).length < 2) {
+
+      // TODO: GROK-15398: Fix file viewer check in grok check
+      if (tags == null || tags.length !== 1 || tags[0] !== 'fileViewer') {
         value = false;
-        message += 'File viewers must have at least two special tags: "fileViewer" and "fileViewer-<extension>"\n';
+        message += 'File viewers must have only one tag: "fileViewer"\n';
       }
       if (inputs.length !== 1 || inputs[0].type !== 'file') {
         value = false;

package/bin/commands/test.js

@@ -140,31 +140,38 @@ function test(args) {
       var options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
       return testUtils.runWithTimeout(timeout, /*#__PURE__*/(0, _asyncToGenerator2["default"])( /*#__PURE__*/_regenerator["default"].mark(function _callee2() {
         var _process$env$TARGET_P;
-        var consoleLog, targetPackage, r;
+        var consoleLogOutputDir, addLogsToFile, targetPackage, r;
         return _regenerator["default"].wrap(function _callee2$(_context2) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
-              consoleLog = '';
+              addLogsToFile = function _addLogsToFile(msg) {
+                _fs["default"].appendFileSync(consoleLogOutputDir, "".concat(msg));
+              };
+              consoleLogOutputDir = './test-console-output.log';
+              _context2.next = 4;
+              return page.exposeFunction("addLogsToFile", addLogsToFile);
+            case 4:
               if (!options.record) {
-                _context2.next = 7;
+                _context2.next = 11;
                 break;
               }
-              _context2.next = 4;
+              _fs["default"].writeFileSync(consoleLogOutputDir, "");
+              _context2.next = 8;
               return recorder.start('./test-record.mp4');
-            case 4:
+            case 8:
               page.on('console', function (msg) {
-                return consoleLog += "CONSOLE LOG ENTRY: ".concat(msg.text(), "\n");
+                addLogsToFile("CONSOLE LOG ENTRY: ".concat(msg.text(), "\n"));
               });
               page.on('pageerror', function (error) {
-                consoleLog += "CONSOLE LOG ERROR: ".concat(error.message, "\n");
+                addLogsToFile("CONSOLE LOG ERROR: ".concat(error.message, "\n"));
               });
               page.on('response', function (response) {
-                consoleLog += "CONSOLE LOG REQUEST: ".concat(response.status(), ", ").concat(response.url(), "\n");
+                addLogsToFile("CONSOLE LOG REQUEST: ".concat(response.status(), ", ").concat(response.url(), "\n"));
               });
-            case 7:
+            case 11:
               targetPackage = (_process$env$TARGET_P = process.env.TARGET_PACKAGE) !== null && _process$env$TARGET_P !== void 0 ? _process$env$TARGET_P : '#{PACKAGE_NAMESPACE}';
               console.log("Testing ".concat(targetPackage, " package...\n"));
-              _context2.next = 11;
+              _context2.next = 15;
               return page.evaluate(function (targetPackage, options, testContext) {
                 if (options.benchmark) window.DG.Test.isInBenchmark = true;
                 return new Promise(function (resolve, reject) {
@@ -191,7 +198,7 @@ function test(args) {
                     };
                     if (df == null) {
                       failed = true;
-                      failReport = "Fail reason: No package tests found".concat(options.path ? " for path \"".concat(options.path, "\"") : '');
+                      failReport = "Fail reason: No package tests found".concat(options.path ? ' for path "' + options.path + '"' : '');
                       resolve({
                         failReport: failReport,
                         skipReport: skipReport,
@@ -234,34 +241,33 @@ function test(args) {
                       countReport: countReport
                     });
                   })["catch"](function (e) {
-                    var stack = window.DG.Logger.translateStackTrace(e.stack);
-                    resolve({
-                      failReport: "".concat(e.message, "\n").concat(stack),
-                      skipReport: '',
-                      passReport: '',
-                      failed: true,
-                      csv: '',
-                      countReport: {
-                        skip: 0,
-                        pass: 0
-                      }
+                    var stack = window.DG.Logger.translateStackTrace(e.stack).then(function () {
+                      resolve({
+                        failReport: "".concat(e.message, "\n").concat(stack),
+                        skipReport: '',
+                        passReport: '',
+                        failed: true,
+                        csv: '',
+                        countReport: {
+                          skip: 0,
+                          pass: 0
+                        }
+                      });
                     });
                   });
                 });
               }, targetPackage, options, new testUtils.TestContext(options.catchUnhandled, options.report));
-            case 11:
+            case 15:
               r = _context2.sent;
               if (!options.record) {
-                _context2.next = 16;
+                _context2.next = 19;
                 break;
               }
-              _context2.next = 15;
+              _context2.next = 19;
               return recorder.stop();
-            case 15:
-              _fs["default"].writeFileSync('./test-console-output.log', consoleLog);
-            case 16:
+            case 19:
               return _context2.abrupt("return", r);
-            case 17:
+            case 20:
             case "end":
               return _context2.stop();
           }

package/package-template/ts.webpack.config.js

@@ -3,6 +3,9 @@ const FuncGeneratorPlugin = require('datagrok-tools/plugins/func-gen-plugin');
 const packageName = path.parse(require('./package.json').name).name.toLowerCase().replace(/-/g, '');
 
 module.exports = {
+  cache: {
+    type: 'filesystem',
+  },
   mode: 'development',
   entry: {
     test: {filename: 'package-test.js', library: {type: 'var', name: `${packageName}_test`}, import: './src/package-test.ts'},
@@ -13,7 +16,7 @@ module.exports = {
   },
   module: {
     rules: [
-      {test: /\.tsx?$/, loader: 'ts-loader'},
+      {test: /\.tsx?$/, loader: 'ts-loader', options: {allowTsInNodeModules: true}},
     ],
   },
   plugins: [

package/package-template/tsconfig.json

@@ -65,7 +65,7 @@
     "emitDecoratorMetadata": true,                  /* Enables experimental support for emitting type metadata for decorators. */
 
     /* Advanced Options */
-    "skipLibCheck": false,                          /* Skip type checking of declaration files. */
+    "skipLibCheck": true,                          /* Skip type checking of declaration files. */
     "forceConsistentCasingInFileNames": true        /* Disallow inconsistently-cased references to the same file. */
   }
 }

package/package-template/webpack.config.js

@@ -2,7 +2,10 @@ const path = require('path');
 const packageName = path.parse(require('./package.json').name).name.toLowerCase().replace(/-/g, '');
 
 module.exports = {
-  mode: 'development',
+  cache: {
+    type: 'filesystem',
+  },
+  mode: 'production',
   entry: {
     package: './src/package.js',
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "datagrok-tools",
-  "version": "4.12.25",
+  "version": "4.12.27",
   "description": "Utility to upload and publish packages to Datagrok",
   "homepage": "https://github.com/datagrok-ai/public/tree/master/tools#readme",
   "dependencies": {