noSkippedTests

Это содержимое пока не доступно на вашем языке.

JavaScript (and super languages)

Summary

Rule available since: v1.6.0
Diagnostic Category: lint/suspicious/noSkippedTests
This rule isn’t recommended, so you need to enable it.
This rule has an unsafe fix.
The default severity of this rule is warning.
This rule belongs to the following domains:
- test
Sources:
- Inspired from jest/no-disabled-tests
- Inspired from vitest/no-disabled-tests
- Inspired from playwright/no-skipped-test

How to configure

1
{
2
  "linter": {
3
    "rules": {
4
      "suspicious": {
5
        "noSkippedTests": "error"
6
      }
7
    }
8
  }
9
}

Description

Disallow disabled tests.

Disabled tests are useful when developing and debugging, although they should not be committed in production.

The rule detects the following patterns:

describe.skip, it.skip, test.skip
describe.fixme, it.fixme, test.fixme
test.describe.skip, test.describe.fixme
test.describe.parallel.skip, test.describe.serial.skip
test.describe.parallel.fixme, test.describe.serial.fixme
test.step.skip, test.step.fixme
xdescribe, xit, xtest
Bracket notation: test["skip"], test["fixme"]
Bare test.skip() / test.fixme() calls (0 arguments)

Examples

Invalid

1
describe.skip("test", () => {});

code-block.js:1:10 lint/suspicious/noSkippedTests  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ⚠ Don’t disable tests.
  
  > 1 │ describe.skip(“test”, () => {});
      │          ^^^^
    2 │ 
  
  ℹ Disabling tests is useful when debugging or creating placeholder while working.
  
  ℹ If this is intentional, and you want to commit a disabled test, add a suppression comment.
  
  ℹ Unsafe fix: Enable the test.
  
    1   │ - describe.skip(“test”,·()·=>·{});
      1 │ + describe(“test”,·()·=>·{});
    2 2 │

1
test.skip("test", () => {});

code-block.js:1:6 lint/suspicious/noSkippedTests  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ⚠ Don’t disable tests.
  
  > 1 │ test.skip(“test”, () => {});
      │      ^^^^
    2 │ 
  
  ℹ Disabling tests is useful when debugging or creating placeholder while working.
  
  ℹ If this is intentional, and you want to commit a disabled test, add a suppression comment.
  
  ℹ Unsafe fix: Enable the test.
  
    1   │ - test.skip(“test”,·()·=>·{});
      1 │ + test(“test”,·()·=>·{});
    2 2 │

1
test.fixme("needs fixing", async () => {});

code-block.js:1:6 lint/suspicious/noSkippedTests  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ⚠ Don’t disable tests.
  
  > 1 │ test.fixme(“needs fixing”, async () => {});
      │      ^^^^^
    2 │ 
  
  ℹ Disabling tests is useful when debugging or creating placeholder while working.
  
  ℹ If this is intentional, and you want to commit a disabled test, add a suppression comment.
  
  ℹ Unsafe fix: Enable the test.
  
    1   │ - test.fixme(“needs·fixing”,·async·()·=>·{});
      1 │ + test(“needs·fixing”,·async·()·=>·{});
    2 2 │

Valid

1
test.only("test", () => {});
2
test("test", () => {});