Resolve issues relating to 4-spaces, add crowdin-cli (jestjs#3369)

Adding additional settings to prettier.js to catch unformatted files in packages/*

Running 'yarn prettier'

examples/async/request.js

@@ -11,7 +11,7 @@ export default function request(url) {
  // This module is being mocked in __mocks__/request.js
  http.get({path: url}, response => {
  let data = '';
- response.on('data', _data => data += _data);
+ response.on('data', _data => (data += _data));
  response.on('end', () => resolve(data));
  });
  });

integration_tests/resolve/__tests__/resolve-test.js

@@ -10,7 +10,7 @@
 let platform;
 
 function testRequire(filename) {
- return () => platform = require(filename);
+ return () => (platform = require(filename));
 }
 
 test('should explicitly resolve filename.<platform>.js', () => {

packages/babel-jest/src/index.js

@@ -53,7 +53,9 @@ const getBabelRC = (filename, {useCache}) => {
  // $FlowFixMe
  const packageJsonFileContents = require(packageJsonFilePath);
  if (packageJsonFileContents[BABEL_CONFIG_KEY]) {
- cache[directory] = JSON.stringify(packageJsonFileContents[BABEL_CONFIG_KEY]);
+ cache[directory] = JSON.stringify(
+ packageJsonFileContents[BABEL_CONFIG_KEY],
+ );
  break;
  }
  }

packages/babel-plugin-jest-hoist/src/index.js

@@ -66,7 +66,7 @@ const WHITELISTED_IDENTIFIERS = {
  require: true,
  undefined: true,
 };
-Object.keys(global).forEach(name => WHITELISTED_IDENTIFIERS[name] = true);
+Object.keys(global).forEach(name => (WHITELISTED_IDENTIFIERS[name] = true));
 
 const JEST_GLOBAL = {name: 'jest'};
 const IDVisitor = {

packages/eslint-plugin-jest/src/rules/no-disabled-tests.js

@@ -37,7 +37,8 @@ module.exports = (context: EslintContext) => ({
  }
 
  if (
- callee.type === 'MemberExpression' && isCallToTestSkipFunction(callee)
+ callee.type === 'MemberExpression' &&
+ isCallToTestSkipFunction(callee)
  ) {
  context.report({
  message: 'Unexpected disabled test.',

packages/eslint-plugin-jest/src/rules/no-focused-tests.js

@@ -37,7 +37,8 @@ module.exports = (context: EslintContext) => ({
  }
 
  if (
- callee.type === 'MemberExpression' && isCallToTestOnlyFunction(callee)
+ callee.type === 'MemberExpression' &&
+ isCallToTestOnlyFunction(callee)
  ) {
  context.report({
  message: 'Unexpected focused test.',

packages/jest-changed-files/src/git.js

@@ -31,8 +31,8 @@ function findChangedFiles(
  const child = childProcess.spawn('git', args, {cwd});
  let stdout = '';
  let stderr = '';
- child.stdout.on('data', data => stdout += data);
- child.stderr.on('data', data => stderr += data);
+ child.stdout.on('data', data => (stdout += data));
+ child.stderr.on('data', data => (stderr += data));
  child.on('error', e => reject(e));
  child.on('close', code => {
  if (code === 0) {
@@ -59,7 +59,7 @@ function isGitRepository(cwd: string): Promise<?string> {
  let stdout = '';
  const options = ['rev-parse', '--show-toplevel'];
  const child = childProcess.spawn('git', options, {cwd});
- child.stdout.on('data', data => stdout += data);
+ child.stdout.on('data', data => (stdout += data));
  child.on('error', () => resolve(null));
  child.on('close', code => resolve(code === 0 ? stdout.trim() : null));
  } catch (e) {

packages/jest-changed-files/src/hg.js

@@ -35,8 +35,8 @@ function findChangedFiles(cwd: string, options: Options): Promise<Array<Path>> {
  const child = childProcess.spawn('hg', args, {cwd, env});
  let stdout = '';
  let stderr = '';
- child.stdout.on('data', data => stdout += data);
- child.stderr.on('data', data => stderr += data);
+ child.stdout.on('data', data => (stdout += data));
+ child.stderr.on('data', data => (stderr += data));
  child.on('error', e => reject(e));
  child.on('close', code => {
  if (code === 0) {
@@ -62,7 +62,7 @@ function isHGRepository(cwd: string): Promise<?string> {
  try {
  let stdout = '';
  const child = childProcess.spawn('hg', ['root'], {cwd, env});
- child.stdout.on('data', data => stdout += data);
+ child.stdout.on('data', data => (stdout += data));
  child.on('error', () => resolve(null));
  child.on('close', code => resolve(code === 0 ? stdout.trim() : null));
  } catch (e) {

packages/jest-cli/src/TestSequencer.js

@@ -65,7 +65,7 @@ class TestSequencer {
  cache[test.path] && cache[test.path][0] === FAIL;
  const time = (cache, test) => cache[test.path] && cache[test.path][1];
 
- tests.forEach(test => test.duration = time(this._getCache(test), test));
+ tests.forEach(test => (test.duration = time(this._getCache(test), test)));
  return tests.sort((testA, testB) => {
  const cacheA = this._getCache(testA);
  const cacheB = this._getCache(testB);
@@ -87,7 +87,7 @@ class TestSequencer {
 
  cacheResults(tests: Tests, results: AggregatedResult) {
  const map = Object.create(null);
- tests.forEach(test => map[test.path] = test);
+ tests.forEach(test => (map[test.path] = test));
  results.testResults.forEach(testResult => {
  if (testResult && map[testResult.testFilePath] && !testResult.skipped) {
  const cache = this._getCache(map[testResult.testFilePath]);

packages/jest-cli/src/TestWorker.js

@@ -79,7 +79,7 @@ module.exports = (
  callback: WorkerCallback,
 ) => {
  let parentExited = false;
- const disconnectCallback = () => parentExited = true;
+ const disconnectCallback = () => (parentExited = true);
  const removeListener = () =>
  process.removeListener('disconnect', disconnectCallback);
  process.on('disconnect', disconnectCallback);

packages/jest-cli/src/__tests__/TestRunner-test.js

@@ -19,9 +19,9 @@ let workerFarmMock;
 jest.mock('worker-farm', () => {
  const mock = jest.fn(
  (options, worker) =>
- workerFarmMock = jest.fn((data, callback) =>
+ (workerFarmMock = jest.fn((data, callback) =>
  require(worker)(data, callback),
- ),
+ )),
  );
  mock.end = jest.fn();
  return mock;

packages/jest-cli/src/runJest.js

@@ -26,7 +26,7 @@ const TestSequencer = require('./TestSequencer');
 
 const setConfig = (contexts, newConfig) =>
  contexts.forEach(
- context => context.config = Object.assign({}, context.config, newConfig),
+ context => (context.config = Object.assign({}, context.config, newConfig)),
  );
 
 const formatTestPathPattern = pattern => {

packages/jest-config/src/normalize.js

@@ -90,7 +90,8 @@ const setupBabelJest = (config: InitialConfig) => {
  {basedir},
  );
  if (
- jsTransformer && jsTransformer.includes(NODE_MODULES + 'babel-jest')
+ jsTransformer &&
+ jsTransformer.includes(NODE_MODULES + 'babel-jest')
  ) {
  babelJest = jsTransformer;
  }
@@ -230,7 +231,7 @@ const normalizeArgv = (config: InitialConfig, argv: Object) => {
  if (argv.collectCoverageOnlyFrom) {
  const collectCoverageOnlyFrom = Object.create(null);
  argv.collectCoverageOnlyFrom.forEach(
- path => collectCoverageOnlyFrom[path] = true,
+ path => (collectCoverageOnlyFrom[path] = true),
  );
  config.collectCoverageOnlyFrom = collectCoverageOnlyFrom;
  }

packages/jest-config/src/vendor/jsonlint.js

@@ -309,7 +309,9 @@ var jsonlint = (function() {
 
  // handle parse error
  _handle_error: if (
- typeof action === 'undefined' || !action.length || !action[0]
+ typeof action === 'undefined' ||
+ !action.length ||
+ !action[0]
  ) {
  if (!recovering) {
  // Report error

packages/jest-editor-support/src/parsers/BabylonParser.js

@@ -145,7 +145,8 @@ const parse = (file: string) => {
  ) {
  searchNodes(element.expression.right.body, file);
  } else if (
- element.type === 'ReturnStatement' && element.argument.arguments
+ element.type === 'ReturnStatement' &&
+ element.argument.arguments
  ) {
  element.argument.arguments
  .filter(argument => isFunctionDeclaration(argument.type))