Fix CI

Author: bernardobelchior

Diff for: src/__tests__/render.js

@@ -277,7 +277,7 @@ describe('render API', () => {
 
     render(ui, {wrapper: WrapperComponent, reactStrictMode: true})
 
-    expect(wrapperComponentMountEffect).toHaveBeenCalledTimes(2);
+    expect(wrapperComponentMountEffect).toHaveBeenCalledTimes(2)
   })
 
   test('reactStrictMode in config is used when renderOptions does not specify reactStrictMode', () => {
@@ -294,6 +294,6 @@ describe('render API', () => {
 
     render(ui, {wrapper: WrapperComponent})
 
-    expect(wrapperComponentMountEffect).toHaveBeenCalledTimes(2);
+    expect(wrapperComponentMountEffect).toHaveBeenCalledTimes(2)
   })
 })

Diff for: src/__tests__/renderHook.js

@@ -1,5 +1,5 @@
-import React from 'react'
-import {renderHook} from '../pure'
+import React, {useEffect} from 'react'
+import {configure, renderHook} from '../pure'
 
 const isReact18 = React.version.startsWith('18.')
 const isReact19 = React.version.startsWith('19.')
@@ -111,3 +111,31 @@ testGateReact19('legacyRoot throws', () => {
     `\`legacyRoot: true\` is not supported in this version of React. If your app runs React 19 or later, you should remove this flag. If your app runs React 18 or earlier, visit https://react.dev/blog/2022/03/08/react-18-upgrade-guide for upgrade instructions.`,
   )
 })
+
+describe('reactStrictMode', () => {
+  let originalConfig
+  beforeEach(() => {
+    // Grab the existing configuration so we can restore
+    // it at the end of the test
+    configure(existingConfig => {
+      originalConfig = existingConfig
+      // Don't change the existing config
+      return {}
+    })
+  })
+
+  afterEach(() => {
+    configure(originalConfig)
+  })
+
+  test('reactStrictMode in renderOptions has precedence over config when rendering', () => {
+    const hookMountEffect = jest.fn()
+    configure({reactStrictMode: false})
+
+    renderHook(() => useEffect(() => hookMountEffect()), {
+      reactStrictMode: true,
+    })
+
+    expect(hookMountEffect).toHaveBeenCalledTimes(2)
+  })
+})

Diff for: src/pure.js

@@ -91,14 +91,24 @@ function wrapUiIfNeeded(innerElement, wrapperComponent) {
 
 function createConcurrentRoot(
   container,
-  {hydrate, onCaughtError, onRecoverableError, ui, wrapper: WrapperComponent, reactStrictMode},
+  {
+    hydrate,
+    onCaughtError,
+    onRecoverableError,
+    ui,
+    wrapper: WrapperComponent,
+    reactStrictMode,
+  },
 ) {
   let root
   if (hydrate) {
     act(() => {
       root = ReactDOMClient.hydrateRoot(
         container,
-        strictModeIfNeeded(wrapUiIfNeeded(ui, WrapperComponent), reactStrictMode),
+        strictModeIfNeeded(
+          wrapUiIfNeeded(ui, WrapperComponent),
+          reactStrictMode,
+        ),
         {onCaughtError, onRecoverableError},
       )
     })
@@ -144,17 +154,31 @@ function createLegacyRoot(container) {
 
 function renderRoot(
   ui,
-  {baseElement, container, hydrate, queries, root, wrapper: WrapperComponent, reactStrictMode},
+  {
+    baseElement,
+    container,
+    hydrate,
+    queries,
+    root,
+    wrapper: WrapperComponent,
+    reactStrictMode,
+  },
 ) {
   act(() => {
     if (hydrate) {
       root.hydrate(
-        strictModeIfNeeded(wrapUiIfNeeded(ui, WrapperComponent), reactStrictMode),
+        strictModeIfNeeded(
+          wrapUiIfNeeded(ui, WrapperComponent),
+          reactStrictMode,
+        ),
         container,
       )
     } else {
       root.render(
-        strictModeIfNeeded(wrapUiIfNeeded(ui, WrapperComponent), reactStrictMode),
+        strictModeIfNeeded(
+          wrapUiIfNeeded(ui, WrapperComponent),
+          reactStrictMode,
+        ),
         container,
       )
     }