testing-library / testing-library/react-testing-library

bug: calling configure() without reactStrictMode resets it to undefined, silently disabling strict mode

Open
#1,466 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
19.7k
Forks
1.2k
Avg merge
3d 16h
Merged PRs (30d)
1

Description

Bug

Calling configure() with any config object that omits reactStrictMode silently resets the previously-configured strict mode flag to undefined.

Root cause

In src/config.js:

function configure(newConfig) {
  if (typeof newConfig === 'function') {
    newConfig = newConfig(getConfig())
  }

  const {reactStrictMode, ...configForDTL} = newConfig

  configureDTL(configForDTL)

  configForRTL = {
    ...configForRTL,
    reactStrictMode,          // <-- always overwrites, even when undefined
  }
}

When newConfig does not include a reactStrictMode key, destructuring yields reactStrictMode = undefined. The spread ...configForRTL correctly carries the previously saved value, but the explicit reactStrictMode property after it overwrites it with undefined.

Reproduction

import { configure, getConfig } from '@testing-library/react'

// Enable strict mode
configure({ reactStrictMode: true })
console.log(getConfig().reactStrictMode)  // true ✓

// Later, an unrelated configure() call — e.g. from a setup file
configure({ asyncUtilTimeout: 2000 })
console.log(getConfig().reactStrictMode)  // undefined ✗  (expected: true)

The second configure() call erases the strict-mode setting. The function-form callback is affected identically:

configure(prev => ({ ...prev, asyncUtilTimeout: 2000 }))
// Works because prev spreads reactStrictMode — but only if callers remember to spread prev.
// The plain-object form has no such safety net.

Impact

Tests that rely on global strict-mode configuration (e.g. set in a setupFilesAfterFramework) silently lose that configuration if any later configure() call omits reactStrictMode. This is particularly easy to hit when test utilities or libraries call configure() with their own options.

Fix

Only write reactStrictMode into configForRTL when it is explicitly present in newConfig:

configForRTL = {
  ...configForRTL,
  ...(reactStrictMode !== undefined && {reactStrictMode}),
}

PR #1461 implements exactly this fix.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/config.js at configure(), then review how getConfig() exposes the saved reactStrictMode value. The fix is done when a later configure() call that omits reactStrictMode preserves the prior setting, including the plain-object case described in the reproduction.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.