-
Notifications
You must be signed in to change notification settings - Fork 991
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fixes #37637 - Test plugins from foreman core
- Loading branch information
Showing
9 changed files
with
300 additions
and
8 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| module.exports = { | ||
| presets: [require.resolve('@theforeman/builder/babel')], | ||
| plugins: [require.resolve('babel-plugin-dynamic-import-node')], | ||
| }; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,135 @@ | ||
| #!/usr/bin/env node | ||
| /* eslint-disable import/no-dynamic-require */ | ||
| /* eslint-disable no-console */ | ||
| /* eslint-disable no-var */ | ||
|
|
||
| /* This script is used to run tests for all plugins that have a `lint` script defined in their package.json | ||
| To run tests for an individual plugin, pass the plugin name as the first argument to the script | ||
| For example, to run tests for the `foreman-tasks` plugin, run: `npm run test-plugin foreman-tasks` | ||
| To pass arguments to jest, pass them after the plugin name like so: `npm run test-plugin foreman-tasks -- --debug` | ||
| */ | ||
|
|
||
| var fs = require('fs'); | ||
| var path = require('path'); | ||
| var lodash = require('lodash'); | ||
| var childProcess = require('child_process'); | ||
| var { packageJsonDirs } = require('./plugin_webpack_directories'); | ||
|
|
||
| const passedArgs = process.argv.slice(2); | ||
| const coreConfigPath = path.resolve(__dirname, '../webpack/jest.config.js'); | ||
| const coreConfig = require(coreConfigPath); | ||
|
|
||
| function runChildProcess(args, pluginPath) { | ||
| return new Promise((resolve, reject) => { | ||
| const child = childProcess.spawn('npx', args, { | ||
| shell: true, | ||
| }); | ||
| // this is needed to make sure the output is not cut | ||
| let stdoutBuffer = ''; | ||
| child.stdout.on('data', data => { | ||
| stdoutBuffer += data.toString(); | ||
| const lines = stdoutBuffer.split('\n'); | ||
| stdoutBuffer = lines.pop(); | ||
| }); | ||
|
|
||
| let stderrBuffer = `${pluginPath}: \n`; | ||
| child.stderr.on('data', data => { | ||
| stderrBuffer += data.toString(); | ||
| const lines = stderrBuffer.split('\n'); | ||
| stderrBuffer = lines.pop(); | ||
| lines.forEach(line => console.error(line)); | ||
| }); | ||
| child.on('close', code => { | ||
| if (stdoutBuffer) console.log(stdoutBuffer); | ||
| if (stderrBuffer) console.error(stderrBuffer); | ||
| if (code === 0) { | ||
| resolve(); | ||
| } else { | ||
| reject(new Error(`Child process exited with code ${code}`)); | ||
| } | ||
| }); | ||
| }); | ||
| } | ||
| const runTests = async () => { | ||
| function pluginDefinesLint(pluginPath) { | ||
| var packageHasNodeModules = fs.existsSync(`${pluginPath}/node_modules`); // skip gems | ||
| var packageData = JSON.parse(fs.readFileSync(`${pluginPath}/package.json`)); | ||
|
|
||
| return ( | ||
| packageHasNodeModules && packageData.scripts && packageData.scripts.lint | ||
| ); | ||
| } | ||
| var dirs = packageJsonDirs(); | ||
| if (passedArgs[0] && passedArgs[0][0] !== '-') { | ||
| dirs = dirs.filter(dir => dir.endsWith(passedArgs[0])); | ||
| passedArgs.shift(); | ||
| } | ||
| for (const pluginPath of dirs) { | ||
| if (pluginDefinesLint(pluginPath)) { | ||
| const testSetupFiles = [ | ||
| path.resolve(__dirname, '../webpack/global_test_setup.js'), | ||
| ]; | ||
| const testSetupPath = path.join(pluginPath, 'webpack', 'test_setup.js'); | ||
| if (fs.existsSync(testSetupPath)) { | ||
| testSetupFiles.unshift(testSetupPath); | ||
| } | ||
| const pluginConfigPath = path.join(pluginPath, 'jest.config.js'); | ||
| const combinedConfigPath = path.join( | ||
| pluginPath, | ||
| 'combined.jest.config.js' | ||
| ); | ||
|
|
||
| if (fs.existsSync(pluginConfigPath)) { | ||
| // eslint-disable-next-line global-require | ||
| const pluginConfig = require(pluginConfigPath); | ||
| function customizer(objValue, srcValue) { | ||
| if (lodash.isArray(objValue)) { | ||
| return lodash.uniq(objValue.concat(srcValue)); | ||
| } | ||
| } | ||
|
|
||
| const combinedConfig = lodash.mergeWith( | ||
| pluginConfig, | ||
| { | ||
| ...coreConfig, | ||
| setupFilesAfterEnv: [ | ||
| path.resolve(__dirname, '../webpack/global_test_setup.js'), | ||
| ], | ||
| }, | ||
| customizer | ||
| ); | ||
| combinedConfig.snapshotSerializers = coreConfig.snapshotSerializers; | ||
| fs.writeFileSync( | ||
| combinedConfigPath, | ||
| `module.exports = ${JSON.stringify(combinedConfig, null, 2)};`, | ||
| 'utf8' | ||
| ); | ||
| } | ||
| const pluginConfigOverride = fs.existsSync(pluginConfigPath); | ||
| const configPath = pluginConfigOverride | ||
| ? combinedConfigPath | ||
| : coreConfigPath; | ||
| const corePath = path.resolve(__dirname, '../'); | ||
| const args = [ | ||
| 'jest', | ||
| `${pluginPath}/webpack`, | ||
| '--roots', | ||
| pluginPath, | ||
| corePath, | ||
| `--config=${configPath}`, | ||
| pluginConfigOverride | ||
| ? '' | ||
| : `--setupFilesAfterEnv ${testSetupFiles.join(' ')}`, | ||
| '--color', | ||
| ...passedArgs, | ||
| ]; | ||
|
|
||
| await runChildProcess(args, pluginPath); // Run every plugin test in a separate process | ||
| if(fs.existsSync(combinedConfigPath)) { | ||
| fs.unlinkSync(combinedConfigPath); | ||
| } | ||
| } | ||
| } | ||
| }; | ||
|
|
||
| runTests(); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| // eslint-disable-next-line import/no-unresolved, import/extensions | ||
| import 'core-js/shim'; | ||
| // eslint-disable-next-line import/no-extraneous-dependencies | ||
| import 'regenerator-runtime/runtime'; | ||
|
|
||
| const { configure } = require('./theforeman-test'); | ||
| const Adapter = require('enzyme-adapter-react-16'); | ||
|
|
||
| configure({ adapter: new Adapter() }); | ||
|
|
||
| // https://github.com/facebook/jest/issues/6121 | ||
| // eslint-disable-next-line no-console | ||
| const { error } = console; | ||
| // eslint-disable-next-line no-console | ||
| console.error = (message, ...args) => { | ||
| error.apply(console, args); // keep default behaviour | ||
| const err = message instanceof Error ? message : new Error(message); | ||
| throw err; | ||
| }; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,71 @@ | ||
| /* eslint-disable spellcheck/spell-checker */ | ||
| const fs = require('fs'); | ||
| const path = require('path'); | ||
|
|
||
| const nodeModules = path.resolve(__dirname, '..', 'node_modules'); | ||
| const packageJsonPath = path.resolve(__dirname, '..', 'package.json'); | ||
| const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8')); | ||
| const vendorCorePackageJsonPath = path.resolve(nodeModules, '@theforeman/vendor-core', 'package.json'); | ||
| const vendorCorePackageJson = JSON.parse(fs.readFileSync(vendorCorePackageJsonPath, 'utf8')); | ||
| const dependencies = { | ||
| ...packageJson.dependencies, | ||
| ...packageJson.devDependencies, | ||
| ...vendorCorePackageJson.dependencies, | ||
| '@apollo/client/testing': '@apollo/client/testing', | ||
| }; // Use shared dependencies from foreman node_modules and not plugin node_modules to avoid jest errors due to multiple instances of same package | ||
|
|
||
| const moduleNameMapper = {}; | ||
| Object.keys(dependencies).forEach(dep => { | ||
| moduleNameMapper[`^${dep}$`] = path.resolve(nodeModules, dep); | ||
| }); | ||
|
|
||
| const foremanReactFull = path.resolve( | ||
| __dirname, | ||
| 'assets/javascripts/react_app' | ||
| ); | ||
| const foremanTest = path.resolve(__dirname, 'theforeman-test.js'); | ||
|
|
||
| module.exports = { | ||
| verbose: true, | ||
| logHeapUsage: true, | ||
| maxWorkers: 2, | ||
| collectCoverage: true, | ||
| coverageReporters: ['lcov'], | ||
| coverageDirectory: `../coverage`, | ||
| setupFiles: [require.resolve('jest-prop-type-error')], | ||
| testPathIgnorePatterns: [ | ||
| '/node_modules/', | ||
| '<rootDir>/foreman/', | ||
| '<rootDir>/.+fixtures.+', | ||
| 'foreman/webpack', // dont test foreman core in plugins | ||
| ], | ||
| testMatch: ['**/*.test.js'], | ||
| moduleDirectories: [ | ||
| `node_modules`, | ||
| `<rootDir>/node_modules/@theforeman/vendor-core/node_modules`, | ||
| `node_modules/@theforeman/vendor-core/node_modules`, | ||
| '<rootDir>/node_modules', | ||
| ], | ||
| transform: { | ||
| '^.+\\.js?$': 'babel-jest', | ||
| '\\.(gql|graphql)$': require.resolve('jest-transform-graphql'), // for graphql-tag | ||
| }, | ||
| snapshotSerializers: [require.resolve('enzyme-to-json/serializer')], | ||
| moduleNameMapper: { | ||
| '^.+\\.(png|gif|css|scss)$': 'identity-obj-proxy', | ||
| ...moduleNameMapper, | ||
| '^dnd-core$': `${nodeModules}/dnd-core/dist/cjs`, | ||
| '^react-dnd$': `${nodeModules}/react-dnd/dist/cjs`, | ||
| '^react-dnd-html5-backend$': `${nodeModules}/react-dnd-html5-backend/dist/cjs`, | ||
| '^react-dnd-touch-backend$': `${nodeModules}/react-dnd-touch-backend/dist/cjs`, | ||
| '^react-dnd-test-backend$': `${nodeModules}/react-dnd-test-backend/dist/cjs`, | ||
| '^react-dnd-test-utils$': `${nodeModules}/react-dnd-test-utils/dist/cjs`, | ||
| '^foremanReact(.*)$': `${foremanReactFull}/$1`, | ||
| '^@theforeman/test$': foremanTest, | ||
| '^victory(.*)$': `${nodeModules}/victory$1`, | ||
| }, | ||
| globals: { | ||
| __testing__: true, | ||
| URL_PREFIX: '', | ||
| }, | ||
| }; |
Oops, something went wrong.