eslint-plugin-jest 27.2.2 → 27.2.3

package/lib/rules/__tests__/unbound-method.test.js

@@ -1,518 +0,0 @@
-"use strict";
-
-var _path = _interopRequireDefault(require("path"));
-var _utils = require("@typescript-eslint/utils");
-var _dedent = _interopRequireDefault(require("dedent"));
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-function getFixturesRootDir() {
-  return _path.default.join(__dirname, 'fixtures');
-}
-const rootPath = getFixturesRootDir();
-const ruleTester = new _utils.ESLintUtils.RuleTester({
-  parser: '@typescript-eslint/parser',
-  parserOptions: {
-    sourceType: 'module',
-    tsconfigRootDir: rootPath,
-    project: './tsconfig.json'
-  }
-});
-const ConsoleClassAndVariableCode = (0, _dedent.default)`
-  class Console {
-    log(str) {
-      process.stdout.write(str);
-    }
-  }
-
-  const console = new Console();
-`;
-const toThrowMatchers = ['toThrow', 'toThrowError', 'toThrowErrorMatchingSnapshot', 'toThrowErrorMatchingInlineSnapshot'];
-const validTestCases = [...['expect(Console.prototype.log).toHaveBeenCalledTimes(1);', 'expect(Console.prototype.log).not.toHaveBeenCalled();', 'expect(Console.prototype.log).toStrictEqual(somethingElse);'].map(code => [ConsoleClassAndVariableCode, code].join('\n')), (0, _dedent.default)`
-    expect(() => {
-      ${ConsoleClassAndVariableCode}
-
-      expect(Console.prototype.log).toHaveBeenCalledTimes(1);
-    }).not.toThrow();
-  `, 'expect(() => Promise.resolve().then(console.log)).not.toThrow();', ...toThrowMatchers.map(matcher => `expect(console.log).not.${matcher}();`), ...toThrowMatchers.map(matcher => `expect(console.log).${matcher}();`)];
-const invalidTestCases = [{
-  code: (0, _dedent.default)`
-      ${ConsoleClassAndVariableCode}
-
-      expect(Console.prototype.log)
-    `,
-  errors: [{
-    line: 9,
-    messageId: 'unboundWithoutThisAnnotation'
-  }]
-}, {
-  code: 'expect(Console.prototype.log).toHaveBeenCalledTimes',
-  errors: [{
-    line: 1,
-    messageId: 'unboundWithoutThisAnnotation'
-  }]
-}, {
-  code: (0, _dedent.default)`
-      expect(() => {
-        ${ConsoleClassAndVariableCode}
-
-        Promise.resolve().then(console.log);
-      }).not.toThrow();
-    `,
-  errors: [{
-    line: 10,
-    messageId: 'unboundWithoutThisAnnotation'
-  }]
-},
-// toThrow matchers call the expected value (which is expected to be a function)
-...toThrowMatchers.map(matcher => ({
-  code: (0, _dedent.default)`
-      ${ConsoleClassAndVariableCode}
-
-      expect(console.log).${matcher}();
-    `,
-  errors: [{
-    line: 9,
-    messageId: 'unboundWithoutThisAnnotation'
-  }]
-})),
-// toThrow matchers call the expected value (which is expected to be a function)
-...toThrowMatchers.map(matcher => ({
-  code: (0, _dedent.default)`
-      ${ConsoleClassAndVariableCode}
-
-      expect(console.log).not.${matcher}();
-    `,
-  errors: [{
-    line: 9,
-    messageId: 'unboundWithoutThisAnnotation'
-  }]
-}))];
-const requireRule = throwWhenRequiring => {
-  jest.resetModules();
-  TSESLintPluginRef.throwWhenRequiring = throwWhenRequiring;
-
-  // eslint-disable-next-line @typescript-eslint/no-require-imports,node/no-missing-require
-  return require('../unbound-method').default;
-};
-const TSESLintPluginRef = {
-  throwWhenRequiring: false
-};
-jest.mock('@typescript-eslint/eslint-plugin', () => {
-  if (TSESLintPluginRef.throwWhenRequiring) {
-    throw new class extends Error {
-      constructor(message) {
-        super(message);
-        this.code = 'MODULE_NOT_FOUND';
-      }
-    }();
-  }
-  return jest.requireActual('@typescript-eslint/eslint-plugin');
-});
-describe('error handling', () => {
-  describe('when an error is thrown accessing the base rule', () => {
-    it('re-throws the error', () => {
-      jest.mock('@typescript-eslint/eslint-plugin', () => {
-        throw new Error('oh noes!');
-      });
-      jest.resetModules();
-
-      // eslint-disable-next-line @typescript-eslint/no-require-imports,node/no-missing-require
-      expect(() => require('../unbound-method').default).toThrow(/oh noes!/iu);
-    });
-  });
-  describe('when @typescript-eslint/eslint-plugin is not available', () => {
-    const ruleTester = new _utils.ESLintUtils.RuleTester({
-      parser: '@typescript-eslint/parser',
-      parserOptions: {
-        sourceType: 'module',
-        tsconfigRootDir: rootPath,
-        project: './tsconfig.json'
-      }
-    });
-    ruleTester.run('unbound-method jest edition without type service', requireRule(true), {
-      valid: validTestCases.concat(invalidTestCases.map(({
-        code
-      }) => code)),
-      invalid: []
-    });
-  });
-});
-ruleTester.run('unbound-method jest edition', requireRule(false), {
-  valid: validTestCases,
-  invalid: invalidTestCases
-});
-function addContainsMethodsClass(code) {
-  return `
-class ContainsMethods {
-  bound?: () => void;
-  unbound?(): void;
-
-  static boundStatic?: () => void;
-  static unboundStatic?(): void;
-}
-
-let instance = new ContainsMethods();
-
-const arith = {
-  double(this: void, x: number): number {
-    return x * 2;
-  }
-};
-
-${code}
-  `;
-}
-function addContainsMethodsClassInvalid(code) {
-  return code.map(c => ({
-    code: addContainsMethodsClass(c),
-    errors: [{
-      line: 18,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }));
-}
-ruleTester.run('unbound-method', requireRule(false), {
-  valid: ['Promise.resolve().then(console.log);', "['1', '2', '3'].map(Number.parseInt);", '[5.2, 7.1, 3.6].map(Math.floor);', 'const x = console.log;', ...['instance.bound();', 'instance.unbound();', 'ContainsMethods.boundStatic();', 'ContainsMethods.unboundStatic();', 'const bound = instance.bound;', 'const boundStatic = ContainsMethods;', 'const { bound } = instance;', 'const { boundStatic } = ContainsMethods;', '(instance.bound)();', '(instance.unbound)();', '(ContainsMethods.boundStatic)();', '(ContainsMethods.unboundStatic)();', 'instance.bound``;', 'instance.unbound``;', 'if (instance.bound) { }', 'if (instance.unbound) { }', 'if (instance.bound !== undefined) { }', 'if (instance.unbound !== undefined) { }', 'if (ContainsMethods.boundStatic) { }', 'if (ContainsMethods.unboundStatic) { }', 'if (ContainsMethods.boundStatic !== undefined) { }', 'if (ContainsMethods.unboundStatic !== undefined) { }', 'if (ContainsMethods.boundStatic && instance) { }', 'if (ContainsMethods.unboundStatic && instance) { }', 'if (instance.bound || instance) { }', 'if (instance.unbound || instance) { }', 'ContainsMethods.unboundStatic && 0 || ContainsMethods;', '(instance.bound || instance) ? 1 : 0', '(instance.unbound || instance) ? 1 : 0', 'while (instance.bound) { }', 'while (instance.unbound) { }', 'while (instance.bound !== undefined) { }', 'while (instance.unbound !== undefined) { }', 'while (ContainsMethods.boundStatic) { }', 'while (ContainsMethods.unboundStatic) { }', 'while (ContainsMethods.boundStatic !== undefined) { }', 'while (ContainsMethods.unboundStatic !== undefined) { }', 'instance.bound as any;', 'ContainsMethods.boundStatic as any;', 'instance.bound++;', '+instance.bound;', '++instance.bound;', 'instance.bound--;', '-instance.bound;', '--instance.bound;', 'instance.bound += 1;', 'instance.bound -= 1;', 'instance.bound *= 1;', 'instance.bound /= 1;', 'instance.bound || 0;', 'instance.bound && 0;', 'instance.bound ? 1 : 0;', 'instance.unbound ? 1 : 0;', 'ContainsMethods.boundStatic++;', '+ContainsMethods.boundStatic;', '++ContainsMethods.boundStatic;', 'ContainsMethods.boundStatic--;', '-ContainsMethods.boundStatic;', '--ContainsMethods.boundStatic;', 'ContainsMethods.boundStatic += 1;', 'ContainsMethods.boundStatic -= 1;', 'ContainsMethods.boundStatic *= 1;', 'ContainsMethods.boundStatic /= 1;', 'ContainsMethods.boundStatic || 0;', 'instane.boundStatic && 0;', 'ContainsMethods.boundStatic ? 1 : 0;', 'ContainsMethods.unboundStatic ? 1 : 0;', "typeof instance.bound === 'function';", "typeof instance.unbound === 'function';", "typeof ContainsMethods.boundStatic === 'function';", "typeof ContainsMethods.unboundStatic === 'function';", 'instance.unbound = () => {};', 'instance.unbound = instance.unbound.bind(instance);', 'if (!!instance.unbound) {}', 'void instance.unbound', 'delete instance.unbound', 'const { double } = arith;'].map(addContainsMethodsClass), `
-interface RecordA {
-  readonly type: 'A';
-  readonly a: {};
-}
-interface RecordB {
-  readonly type: 'B';
-  readonly b: {};
-}
-type AnyRecord = RecordA | RecordB;
-
-function test(obj: AnyRecord) {
-  switch (obj.type) {
-  }
-}
-    `,
-  // https://github.com/typescript-eslint/typescript-eslint/issues/496
-  `
-class CommunicationError {
-  constructor() {
-    const x = CommunicationError.prototype;
-  }
-}
-    `, `
-class CommunicationError {}
-const x = CommunicationError.prototype;
-    `,
-  // optional chain
-  `
-class ContainsMethods {
-  bound?: () => void;
-  unbound?(): void;
-
-  static boundStatic?: () => void;
-  static unboundStatic?(): void;
-}
-
-function foo(instance: ContainsMethods | null) {
-  instance?.bound();
-  instance?.unbound();
-
-  instance?.bound++;
-
-  if (instance?.bound) {
-  }
-  if (instance?.unbound) {
-  }
-
-  typeof instance?.bound === 'function';
-  typeof instance?.unbound === 'function';
-}
-    `,
-  // https://github.com/typescript-eslint/typescript-eslint/issues/1425
-  `
-interface OptionalMethod {
-  mightBeDefined?(): void;
-}
-
-const x: OptionalMethod = {};
-declare const myCondition: boolean;
-if (myCondition || x.mightBeDefined) {
-  console.log('hello world');
-}
-    `,
-  // https://github.com/typescript-eslint/typescript-eslint/issues/1256
-  `
-class A {
-  unbound(): void {
-    this.unbound = undefined;
-    this.unbound = this.unbound.bind(this);
-  }
-}
-    `, 'const { parseInt } = Number;', 'const { log } = console;', `
-let parseInt;
-({ parseInt } = Number);
-    `, `
-let log;
-({ log } = console);
-    `, `
-const foo = {
-  bar: 'bar',
-};
-const { bar } = foo;
-    `, `
-class Foo {
-  unbnound() {}
-  bar = 4;
-}
-const { bar } = new Foo();
-    `, `
-class Foo {
-  bound = () => 'foo';
-}
-const { bound } = new Foo();
-    `,
-  // https://github.com/typescript-eslint/typescript-eslint/issues/1866
-  `
-class BaseClass {
-  x: number = 42;
-  logThis() {}
-}
-class OtherClass extends BaseClass {
-  superLogThis: any;
-  constructor() {
-    super();
-    this.superLogThis = super.logThis;
-  }
-}
-const oc = new OtherClass();
-oc.superLogThis();
-    `],
-  invalid: [{
-    code: `
-class Console {
-  log(str) {
-    process.stdout.write(str);
-  }
-}
-
-const console = new Console();
-
-Promise.resolve().then(console.log);
-      `,
-    errors: [{
-      line: 10,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-import { console } from './class';
-const x = console.log;
-      `,
-    errors: [{
-      line: 3,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: addContainsMethodsClass(`
-function foo(arg: ContainsMethods | null) {
-  const unbound = arg?.unbound;
-  arg.unbound += 1;
-  arg?.unbound as any;
-}
-      `),
-    errors: [{
-      line: 20,
-      messageId: 'unboundWithoutThisAnnotation'
-    }, {
-      line: 21,
-      messageId: 'unboundWithoutThisAnnotation'
-    }, {
-      line: 22,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, ...addContainsMethodsClassInvalid(['const unbound = instance.unbound;', 'const unboundStatic = ContainsMethods.unboundStatic;', 'const { unbound } = instance;', 'const { unboundStatic } = ContainsMethods;', '<any>instance.unbound;', 'instance.unbound as any;', '<any>ContainsMethods.unboundStatic;', 'ContainsMethods.unboundStatic as any;', 'instance.unbound || 0;', 'ContainsMethods.unboundStatic || 0;', 'instance.unbound ? instance.unbound : null']), {
-    code: `
-class ContainsMethods {
-  unbound?(): void;
-
-  static unboundStatic?(): void;
-}
-
-new ContainsMethods().unbound;
-
-ContainsMethods.unboundStatic;
-      `,
-    options: [{
-      ignoreStatic: true
-    }],
-    errors: [{
-      line: 8,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  },
-  // https://github.com/typescript-eslint/typescript-eslint/issues/496
-  {
-    code: `
-class CommunicationError {
-  foo() {}
-}
-const x = CommunicationError.prototype.foo;
-      `,
-    errors: [{
-      line: 5,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    // Promise.all is not auto-bound to Promise
-    code: 'const x = Promise.all;',
-    errors: [{
-      line: 1,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound() {}
-}
-const instance = new Foo();
-
-let x;
-
-x = instance.unbound; // THIS SHOULD ERROR
-instance.unbound = x; // THIS SHOULD NOT
-      `,
-    errors: [{
-      line: 9,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound = function () {};
-}
-const unbound = new Foo().unbound;
-      `,
-    errors: [{
-      line: 5,
-      messageId: 'unbound'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound() {}
-}
-const { unbound } = new Foo();
-      `,
-    errors: [{
-      line: 5,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound = function () {};
-}
-const { unbound } = new Foo();
-      `,
-    errors: [{
-      line: 5,
-      messageId: 'unbound'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound() {}
-}
-let unbound;
-({ unbound } = new Foo());
-      `,
-    errors: [{
-      line: 6,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-class Foo {
-  unbound = function () {};
-}
-let unbound;
-({ unbound } = new Foo());
-      `,
-    errors: [{
-      line: 6,
-      messageId: 'unbound'
-    }]
-  }, {
-    code: `
-class CommunicationError {
-  foo() {}
-}
-const { foo } = CommunicationError.prototype;
-      `,
-    errors: [{
-      line: 5,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-class CommunicationError {
-  foo() {}
-}
-let foo;
-({ foo } = CommunicationError.prototype);
-      `,
-    errors: [{
-      line: 6,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: `
-import { console } from './class';
-const { log } = console;
-      `,
-    errors: [{
-      line: 3,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }, {
-    code: 'const { all } = Promise;',
-    errors: [{
-      line: 1,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  },
-  // https://github.com/typescript-eslint/typescript-eslint/issues/1866
-  {
-    code: `
-class BaseClass {
-  logThis() {}
-}
-class OtherClass extends BaseClass {
-  constructor() {
-    super();
-    const x = super.logThis;
-  }
-}
-      `,
-    errors: [{
-      line: 8,
-      column: 15,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  },
-  // https://github.com/typescript-eslint/typescript-eslint/issues/1866
-  {
-    code: `
-class BaseClass {
-  logThis() {}
-}
-class OtherClass extends BaseClass {
-  constructor() {
-    super();
-    let x;
-    x = super.logThis;
-  }
-}
-      `,
-    errors: [{
-      line: 9,
-      column: 9,
-      messageId: 'unboundWithoutThisAnnotation'
-    }]
-  }]
-});