#no-conditional-tests

#Configuration

import { defineConfig, rstestPlugin } from '@rslint/core';

export default defineConfig([
  rstestPlugin.configs.recommended,
  {
    rules: {
      'rstest/no-conditional-tests': 'error',
    },
  },
]);

#Rule Details

Disallows registering a test or a suite from inside an if statement. A test that only exists on some runs is a test that silently stops covering anything when the condition flips, and the report shows a shrinking suite rather than a failure.

A registration is reported when it is reached through the then branch or the else branch of an if; a call in the if's own condition runs every time and is not reported. Only if is reported — conditional expressions, switch, and the logical operators are left alone. Hooks such as beforeEach are not registrations and are not reported.

The rule does not carry an enclosing if through code that runs later. A registration in a function or method body, a parameter default, or an instance field initializer is left alone, because an if around the declaration does not decide whether that code runs — the later call or construction does. Class static blocks, static field initializers, computed member names, and decorators run while the class is defined, so registrations in those positions remain conditional on an enclosing if.

The rule does not follow a function to its call sites, so a registration in a helper is never reported, whether or not the helper is only ever called from inside an if. The same boundary means a nested pair such as if (x) { describe('a', () => { test('b', fn) }) } reports only the outer describe. Conditions written inside a test body are covered separately by rstest/no-conditional-in-test.

A conditionally-run test should be registered with test.skipIf(condition) or test.runIf(condition) instead, so the suite keeps its shape and the runner decides at execution time whether to run it. Both modifiers are also available on describe.

#Incorrect

if (process.env.CI) {
  test('uploads the report', async () => {
    await expect(upload()).resolves.toBe(true);
  });
}

#Correct

test.skipIf(!process.env.CI)('uploads the report', async () => {
  await expect(upload()).resolves.toBe(true);
});