@elliemae/pui-cli 5.21.1 → 5.22.1

package/lib/cli-commands/test.js

@@ -1,6 +1,5 @@
 const { exit } = require('yargs');
 const { exec, logError, logSuccess } = require('./utils');
-const { lintCSS, lintJS } = require('./lint');
 
 const { CI = false } = process.env;
 
@@ -19,17 +18,7 @@ async function handler(argv) {
   if (argv.r) commandOptions += ' --bail --findRelatedTests';
   if (argv.s) commandOptions += ' --silent';
   try {
-    if (!CI) {
-      try {
-        await lintJS();
-        await lintCSS();
-        logSuccess('Linting completed');
-      } catch (err) {
-        logError('Linting failed');
-        exit(-1, err);
-        return -1;
-      }
-    } else {
+    if (CI) {
       await exec('rimraf ./reports');
     }
 

package/lib/testing/jest.config.js

@@ -71,13 +71,12 @@ const jestConfig = {
   },
   setupFilesAfterEnv: [`${getRootDir()}/lib/testing/setup-tests.js`],
   setupFiles: ['raf/polyfill', 'whatwg-fetch'],
-  testRegex: '(app|lib).*/tests/.*\\.test\\.(js|ts)$',
+  testRegex: '(app|lib).*/tests/.*\\.test\\.(js|ts|tsx|jsx)$',
   snapshotSerializers: [],
   testResultsProcessor: 'jest-sonar-reporter',
-  resolver: 'ts-jest-resolver',
+  resolver: path.resolve(__dirname, './resolver.js'),
   transformIgnorePatterns: [
-    'node_modules/(?!(@elliemae/*|lodash-es/*)/)',
-    'node_modules/@elliemae/em-platform-document-viewer',
+    'node_modules/(?!(@elliemae/em-platform-document-viewer|@elliemae/pui-cli|@elliemae/pui-app-widgets|lodash-es|react-spring|react-select|react-dates|@babel/runtime)/)',
   ],
   globals: {
     APP_CONFIG: getAppConfig(),

package/lib/testing/mocks/svg.js

@@ -1,3 +1,5 @@
-module.exports = {
-  ReactComponent: 'IMAGE_MOCK',
-};
+// eslint-disable-next-line no-unused-vars
+import * as React from 'react';
+
+export default 'SvgrURL';
+export const ReactComponent = 'div';

package/lib/testing/resolver.js

@@ -0,0 +1,44 @@
+const resolutions = [
+  {
+    matcher: /\.jsx?$/i,
+    extensions: ['.tsx', '.ts'],
+  },
+  {
+    matcher: /\.mjs$/i,
+    extensions: ['.mts'],
+  },
+  {
+    matcher: /\.cjs$/i,
+    extensions: ['.cts'],
+  },
+];
+
+const importResolver = require('enhanced-resolve').create.sync({
+  conditionNames: ['import', 'node', 'default'],
+  extensions: ['.js', '.jsx', '.json', '.node', '.ts', '.tsx'],
+});
+const requireResolver = require('enhanced-resolve').create.sync({
+  conditionNames: ['require', 'node', 'default'],
+  extensions: ['.js', '.jsx', '.json', '.node', '.ts', '.tsx'],
+});
+
+module.exports = (request, options) => {
+  let resolver = requireResolver;
+  if (options.conditions?.includes('import')) {
+    resolver = importResolver;
+  }
+  const resolution = resolutions.find(({ matcher }) => matcher.test(request));
+  if (resolution) {
+    // eslint-disable-next-line no-restricted-syntax
+    for (const extension of resolution.extensions) {
+      try {
+        return resolver(
+          options.basedir,
+          request.replace(resolution.matcher, extension),
+        );
+        // eslint-disable-next-line no-empty
+      } catch {}
+    }
+  }
+  return resolver(options.basedir, request);
+};

package/lib/testing/setup-tests.js

@@ -5,8 +5,31 @@ import 'regenerator-runtime/runtime';
 import '@testing-library/jest-dom/extend-expect';
 import { toHaveNoViolations } from 'jest-axe';
 import ResizeObserver from 'resize-observer-polyfill';
-import addMatchMedia from './mocks/matchMedia';
-import { logger } from './mocks/pui-diagnostics';
+import addMatchMedia from '@elliemae/pui-cli/lib/testing/mocks/matchMedia';
+import { logger } from '@elliemae/pui-cli/lib/testing/mocks/pui-diagnostics';
+
+// eslint-disable-next-line no-console
+const originalError = console.error;
+// eslint-disable-next-line no-console
+console.error = (...args) => {
+  const ignoreList = [
+    `Warning: Can't perform a React state update on an unmounted component`,
+    `Warning: Function components cannot be given refs`,
+    `Warning: Failed %s type:`,
+    `Warning: Invalid DOM property`,
+    `Warning: Each child in a list should have a unique`,
+    'Warning: Received `%s` for a non-boolean attribute',
+    'Warning: <%s /> is using incorrect casing.',
+    'Warning: The tag <%s> is unrecognized in this browser',
+  ];
+  if (
+    ignoreList.find(
+      (ignoreMsg) => !!args.find((arg) => arg.includes?.(ignoreMsg)),
+    )
+  )
+    return false;
+  return originalError(...args);
+};
 
 if (expect) expect.extend(toHaveNoViolations);
 jest.setTimeout(60000);

package/lib/webpack/webpack.base.babel.js

@@ -7,7 +7,6 @@ const DuplicatePackageCheckerPlugin = require('duplicate-package-checker-webpack
 const CaseSensitivePathsPlugin = require('case-sensitive-paths-webpack-plugin');
 const MomentLocalesPlugin = require('moment-locales-webpack-plugin');
 const { WebpackManifestPlugin } = require('webpack-manifest-plugin');
-const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');
 
 const {
   excludeNodeModulesExcept,
@@ -16,7 +15,6 @@ const {
   getAlias,
   getPaths,
 } = require('./helpers');
-const { isTypeScriptEnabled } = require('../typescript/util');
 
 // get the application configuration
 const devMode = process.env.NODE_ENV !== 'production';
@@ -103,17 +101,6 @@ const plugins = [
   new WebpackManifestPlugin(),
 ];
 
-if (isTypeScriptEnabled()) {
-  plugins.push(
-    new ForkTsCheckerWebpackPlugin({
-      async: devMode,
-      eslint: {
-        files: './app/**/*.{ts,js,tsx,jsx}',
-      },
-    }),
-  );
-}
-
 module.exports = (options) => ({
   mode: options.mode,
   entry: options.entry,
@@ -189,6 +176,7 @@ module.exports = (options) => ({
       {
         test: /\.svg$/i,
         issuer: /\.[jt]sx?$/,
+        resourceQuery: /^((?!url).)*$/,
         use: ['@svgr/webpack'],
       },
       {

package/lib/webpack/webpack.lib.base.babel.js

@@ -10,7 +10,6 @@ const PostcssPresetEnv = require('postcss-preset-env');
 const DuplicatePackageCheckerPlugin = require('duplicate-package-checker-webpack-plugin');
 const CaseSensitivePathsPlugin = require('case-sensitive-paths-webpack-plugin');
 const MomentLocalesPlugin = require('moment-locales-webpack-plugin');
-const ForkTsCheckerWebpackPlugin = require('fork-ts-checker-webpack-plugin');
 const ResolveTypeScriptPlugin = require('resolve-typescript-plugin').default;
 
 const {
@@ -21,7 +20,6 @@ const {
   getAssetPath,
   getAlias,
 } = require('./helpers');
-const { isTypeScriptEnabled } = require('../typescript/util');
 
 const devMode = process.env.NODE_ENV !== 'production';
 const minicssLoader = {
@@ -62,17 +60,6 @@ const plugins = [
   new MomentLocalesPlugin(),
 ];
 
-if (isTypeScriptEnabled()) {
-  plugins.push(
-    new ForkTsCheckerWebpackPlugin({
-      async: devMode,
-      eslint: {
-        files: './lib/**/*.{ts,js,tsx,jsx}',
-      },
-    }),
-  );
-}
-
 module.exports = (options) => ({
   mode: options.mode,
   entry: [path.join(process.cwd(), 'lib/index')],
@@ -173,6 +160,7 @@ module.exports = (options) => ({
       {
         test: /\.svg$/i,
         issuer: /\.[jt]sx?$/,
+        resourceQuery: /^((?!url).)*$/,
         use: ['@svgr/webpack'],
       },
       {

package/lib/webpack/webpack.storybook.js

@@ -95,6 +95,7 @@ const getModuleRules = () => [
   {
     test: /\.svg$/i,
     issuer: /\.[jt]sx?$/,
+    resourceQuery: /^((?!url).)*$/,
     use: ['@svgr/webpack'],
   },
   {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elliemae/pui-cli",
-  "version": "5.21.1",
+  "version": "5.22.1",
   "private": false,
   "description": "EllieMae Platform UI CLI",
   "sideEffects": false,
@@ -93,7 +93,7 @@
     "@testing-library/react": "~12.1.2",
     "@testing-library/react-hooks": "~7.0.2",
     "@types/jest": "~27.0.3",
-    "@types/node": "~16.11.11",
+    "@types/node": "~16.11.12",
     "@types/rimraf": "~3.0.2",
     "@types/testing-library__jest-dom": "~5.14.2",
     "@typescript-eslint/eslint-plugin": "~5.6.0",
@@ -132,7 +132,8 @@
     "dotenv": "~10.0.0",
     "dotenv-webpack": "~7.0.3",
     "duplicate-package-checker-webpack-plugin": "~3.0.0",
-    "eslint": "~8.4.0",
+    "enhanced-resolve": "~5.8.3",
+    "eslint": "~8.4.1",
     "eslint-config-airbnb": "~18.2.1",
     "eslint-config-airbnb-base": "~15.0.0",
     "eslint-config-airbnb-typescript": "~15.0.0",
@@ -179,7 +180,7 @@
     "minimist": "~1.2.5",
     "moment": "~2.29.1",
     "moment-locales-webpack-plugin": "~1.2.0",
-    "msw": "~0.36.0",
+    "msw": "~0.36.2",
     "node-gyp": "~8.4.1",
     "node-plop": "~0.30.0",
     "nodemon": "~2.0.15",
@@ -188,7 +189,7 @@
     "pino": "~7.5.1",
     "pino-pretty": "~7.2.0",
     "pinst": "~2.1.6",
-    "plop": "~3.0.2",
+    "plop": "~3.0.3",
     "postcss": "~8.4.4",
     "postcss-jsx": "~0.36.4",
     "postcss-html": "~1.3.0",
@@ -211,7 +212,7 @@
     "script-loader": "~0.7.2",
     "semantic-release": "~18.0.1",
     "shelljs": "~0.8.4",
-    "slackify-markdown": "~4.3.0",
+    "slackify-markdown": "~4.3.1",
     "storybook-react-router": "~1.0.8",
     "style-loader": "~3.3.1",
     "stylelint": "~14.1.0",
@@ -222,7 +223,6 @@
     "svg-url-loader": "~7.1.1",
     "svgo": "~2.8.0",
     "terser-webpack-plugin": "~5.2.5",
-    "ts-jest-resolver": "~2.0.0",
     "ts-node": "~10.4.0",
     "tsc-alias": "~1.4.2",
     "tsc-files": "~1.1.3",