mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-12-23 09:52:26 +01:00
b1b6d7ae38
The metrics e2e test would fail if the segment events still weren't dispatched when the page loaded. The Segment events are sent on a set interval, so it isn't abnormal for them to lag behind the page load itself. The `waitUntilCalled` utility has been used to wait until all required events have been dispatched. The `wait-until-called` module was converted to an ES5 module, so that it could be used from an e2e test. The optional `callCount` parameter has also been added, to allow waiting for more than one call. The `segmentSpy` had to be converted to a `segmentStub`, to allow the `waitUntilCalled` utility to be used.
216 lines
6.7 KiB
JavaScript
216 lines
6.7 KiB
JavaScript
module.exports = {
|
|
root: true,
|
|
parser: '@babel/eslint-parser',
|
|
parserOptions: {
|
|
sourceType: 'module',
|
|
ecmaVersion: 2017,
|
|
ecmaFeatures: {
|
|
experimentalObjectRestSpread: true,
|
|
impliedStrict: true,
|
|
modules: true,
|
|
blockBindings: true,
|
|
arrowFunctions: true,
|
|
objectLiteralShorthandMethods: true,
|
|
objectLiteralShorthandProperties: true,
|
|
templateStrings: true,
|
|
classes: true,
|
|
jsx: true,
|
|
},
|
|
},
|
|
|
|
ignorePatterns: [
|
|
'!.eslintrc.js',
|
|
'node_modules/**',
|
|
'dist/**',
|
|
'builds/**',
|
|
'test-*/**',
|
|
'docs/**',
|
|
'coverage/',
|
|
'app/scripts/chromereload.js',
|
|
'app/vendor/**',
|
|
'test/e2e/send-eth-with-private-key-test/**',
|
|
'nyc_output/**',
|
|
'.vscode/**',
|
|
],
|
|
|
|
extends: [
|
|
'@metamask/eslint-config',
|
|
'@metamask/eslint-config/config/nodejs',
|
|
'@metamask/eslint-config/config/mocha',
|
|
'plugin:react/recommended',
|
|
'plugin:react-hooks/recommended',
|
|
],
|
|
|
|
plugins: ['@babel', 'react', 'import', 'prettier'],
|
|
|
|
globals: {
|
|
document: 'readonly',
|
|
window: 'readonly',
|
|
},
|
|
|
|
rules: {
|
|
// Prettier changes and reasoning
|
|
|
|
'prettier/prettier': 'error',
|
|
|
|
// Our usage of spaces before *named* function parens is unusual, and
|
|
// doesn't match the prettier spec. prettier does not offer an option
|
|
// to configure this
|
|
'space-before-function-paren': [
|
|
'error',
|
|
{ anonymous: 'always', named: 'never' },
|
|
],
|
|
// Our eslint config has the default setting for this as error. This
|
|
// include beforeBlockComment: true, but in order to match the prettier
|
|
// spec you have to enable before and after blocks, objects and arrays
|
|
// https://github.com/prettier/eslint-config-prettier#lines-around-comment
|
|
'lines-around-comment': [
|
|
'error',
|
|
{
|
|
beforeBlockComment: true,
|
|
afterLineComment: false,
|
|
allowBlockStart: true,
|
|
allowBlockEnd: true,
|
|
allowObjectStart: true,
|
|
allowObjectEnd: true,
|
|
allowArrayStart: true,
|
|
allowArrayEnd: true,
|
|
},
|
|
],
|
|
// Prettier has some opinions on mixed-operators, and there is ongoing work
|
|
// to make the output code clear. It is better today then it was when the first
|
|
// PR to add prettier. That being said, the workaround for keeping this rule enabled
|
|
// requires breaking parts of operations into different variables -- which I believe
|
|
// to be worse. https://github.com/prettier/eslint-config-prettier#no-mixed-operators
|
|
'no-mixed-operators': 'off',
|
|
// Prettier wraps single line functions with ternaries, etc in parens by default, but
|
|
// if the line is long enough it breaks it into a separate line and removes the parens.
|
|
// The second behavior conflicts with this rule. There is some guides on the repo about
|
|
// how you can keep it enabled:
|
|
// https://github.com/prettier/eslint-config-prettier#no-confusing-arrow
|
|
// However, in practice this conflicts with prettier adding parens around short lines,
|
|
// when autofixing in vscode and others.
|
|
'no-confusing-arrow': 'off',
|
|
// There is no configuration in prettier for how it stylizes regexes, which conflicts
|
|
// with wrap-regex.
|
|
'wrap-regex': 'off',
|
|
// Prettier handles all indentation automagically. it can be configured here
|
|
// https://prettier.io/docs/en/options.html#tab-width but the default matches our
|
|
// style.
|
|
indent: 'off',
|
|
// This rule conflicts with the way that prettier breaks code across multiple lines when
|
|
// it exceeds the maximum length. Prettier optimizes for readability while simultaneously
|
|
// maximizing the amount of code per line.
|
|
'function-paren-newline': 'off',
|
|
// This rule throws an error when there is a line break in an arrow function declaration
|
|
// but prettier breaks arrow function declarations to be as readable as possible while
|
|
// still conforming to the width rules.
|
|
'implicit-arrow-linebreak': 'off',
|
|
// This rule would result in an increase in white space in lines with generator functions,
|
|
// which impacts prettier's goal of maximizing code per line and readability. There is no
|
|
// current workaround.
|
|
'generator-star-spacing': 'off',
|
|
'default-param-last': 'off',
|
|
'require-atomic-updates': 'off',
|
|
'import/no-unassigned-import': 'off',
|
|
'prefer-object-spread': 'error',
|
|
'react/no-unused-prop-types': 'error',
|
|
'react/no-unused-state': 'error',
|
|
'react/jsx-boolean-value': 'error',
|
|
'react/jsx-curly-brace-presence': [
|
|
'error',
|
|
{ props: 'never', children: 'never' },
|
|
],
|
|
'react/jsx-equals-spacing': 'error',
|
|
'react/no-deprecated': 'error',
|
|
'react/default-props-match-prop-types': 'error',
|
|
'react/jsx-closing-tag-location': [
|
|
'error',
|
|
{ selfClosing: 'tag-aligned', nonEmpty: 'tag-aligned' },
|
|
],
|
|
'react/jsx-no-duplicate-props': 'error',
|
|
'react/jsx-closing-bracket-location': 'error',
|
|
'react/jsx-first-prop-new-line': ['error', 'multiline'],
|
|
'react/jsx-max-props-per-line': [
|
|
'error',
|
|
{ maximum: 1, when: 'multiline' },
|
|
],
|
|
'react/jsx-tag-spacing': [
|
|
'error',
|
|
{
|
|
closingSlash: 'never',
|
|
beforeSelfClosing: 'always',
|
|
afterOpening: 'never',
|
|
},
|
|
],
|
|
|
|
'no-invalid-this': 'off',
|
|
'@babel/no-invalid-this': 'error',
|
|
|
|
'@babel/semi': ['error', 'never'],
|
|
'mocha/no-setup-in-describe': 'off',
|
|
'node/no-process-env': 'off',
|
|
|
|
// TODO: re-enable these rules
|
|
'node/no-sync': 'off',
|
|
'node/no-unpublished-import': 'off',
|
|
'node/no-unpublished-require': 'off',
|
|
},
|
|
overrides: [
|
|
{
|
|
files: ['test/e2e/**/*.js'],
|
|
rules: {
|
|
'mocha/no-hooks-for-single-case': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['app/scripts/migrations/*.js', '*.stories.js'],
|
|
rules: {
|
|
'import/no-anonymous-default-export': ['error', { allowObject: true }],
|
|
},
|
|
},
|
|
{
|
|
files: ['app/scripts/migrations/*.js'],
|
|
rules: {
|
|
'node/global-require': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['test/**/*-test.js', 'test/**/*.spec.js'],
|
|
rules: {
|
|
// Mocha will re-assign `this` in a test context
|
|
'@babel/no-invalid-this': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: ['development/**/*.js', 'test/e2e/benchmark.js', 'test/helper.js'],
|
|
rules: {
|
|
'node/no-process-exit': 'off',
|
|
'node/shebang': 'off',
|
|
},
|
|
},
|
|
{
|
|
files: [
|
|
'.eslintrc.js',
|
|
'babel.config.js',
|
|
'nyc.config.js',
|
|
'stylelint.config.js',
|
|
'development/**/*.js',
|
|
'test/e2e/**/*.js',
|
|
'test/lib/wait-until-called.js',
|
|
'test/env.js',
|
|
'test/setup.js',
|
|
],
|
|
parserOptions: {
|
|
sourceType: 'script',
|
|
},
|
|
},
|
|
],
|
|
|
|
settings: {
|
|
react: {
|
|
version: 'detect',
|
|
},
|
|
},
|
|
}
|