Skip to content

noSkippedTests (since v1.6.0)

Diagnostic Category: lint/suspicious/noSkippedTests

Sources:

Disallow disabled tests.

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

describe.skip("test", () => {});
suspicious/noSkippedTests.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",·()·=>·{});
          -----                   
test.skip("test", () => {});
suspicious/noSkippedTests.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",·()·=>·{});
      -----                   
test.only("test", () => {});
test("test", () => {});