Files
react/packages/react-dom/src/__tests__/ReactEmptyComponent-test.js
Josh Story cb151849e1 [react-dom] move all client code to react-dom/client (#28271)
This PR reorganizes the `react-dom` entrypoint to only pull in code that
is environment agnostic. Previously if you required anything from this
entrypoint in any environment the entire client reconciler was loaded.
In a prior release we added a server rendering stub which you could
alias in server environments to omit this unecessary code. After landing
this change this entrypoint should not load any environment specific
code.

While a few APIs are truly client (browser) only such as createRoot and
hydrateRoot many of the APIs you import from this package are only
useful in the browser but could concievably be imported in shared code
(components running in Fizz or shared components as part of an RSC app).
To avoid making these require opting into the client bundle we are
keeping them in the `react-dom` entrypoint and changing their
implementation so that in environments where they are not particularly
useful they do something benign and expected.

#### Removed APIs
The following APIs are being removed in the next major. Largely they
have all been deprecated already and are part of legacy rendering modes
where concurrent features of React are not available
* `render`
* `hydrate`
* `findDOMNode`
* `unmountComponentAtNode`
* `unstable_createEventHandle`
* `unstable_renderSubtreeIntoContainer`
* `unstable_runWithPrioirty`

#### moved Client APIs
These APIs were available on both `react-dom` (with a warning) and
`react-dom/client`. After this change they are only available on
`react-dom/client`
* `createRoot`
* `hydrateRoot`

#### retained APIs
These APIs still exist on the `react-dom` entrypoint but have normalized
behavior depending on which renderers are currently in scope
* `flushSync`: will execute the function (if provided) inside the
flushSync implemention of FlightServer, Fizz, and Fiber DOM renderers.
* `unstable_batchedUpdates`: This is a noop in concurrent mode because
it is now the only supported behavior because there is no legacy
rendering mode
* `createPortal`: This just produces an object. It can be called from
anywhere but since you will probably not have a handle on a DOM node to
pass to it it will likely warn in environments other than the browser
* preloading APIS such as `preload`: These methods will execute the
preload across all renderers currently in scope. Since we resolve the
Request object on the server using AsyncLocalStorage or the current
function stack in practice only one renderer should act upon the
preload.

In addition to these changes the server rendering stub now just rexports
everything from `react-dom`. In a future minor we will add a warning
when using the stub and in the next major we will remove the stub
altogether
2024-04-24 08:50:32 -07:00

387 lines
9.8 KiB
JavaScript

/**
* Copyright (c) Meta Platforms, Inc. and affiliates.
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @emails react-core
*/
'use strict';
let React;
let ReactDOM;
let findDOMNode;
let ReactDOMClient;
let TogglingComponent;
let act;
let Scheduler;
let assertLog;
let container;
describe('ReactEmptyComponent', () => {
beforeEach(() => {
jest.resetModules();
React = require('react');
ReactDOM = require('react-dom');
ReactDOMClient = require('react-dom/client');
findDOMNode =
ReactDOM.__DOM_INTERNALS_DO_NOT_USE_OR_WARN_USERS_THEY_CANNOT_UPGRADE
.findDOMNode;
Scheduler = require('scheduler');
const InternalTestUtils = require('internal-test-utils');
act = InternalTestUtils.act;
assertLog = InternalTestUtils.assertLog;
container = document.createElement('div');
TogglingComponent = class extends React.Component {
state = {component: this.props.firstComponent};
componentDidMount() {
Scheduler.log('mount ' + findDOMNode(this)?.nodeName);
this.setState({component: this.props.secondComponent});
}
componentDidUpdate() {
Scheduler.log('update ' + findDOMNode(this)?.nodeName);
}
render() {
const Component = this.state.component;
return Component ? <Component /> : null;
}
};
});
describe.each([null, undefined])('when %s', nullORUndefined => {
it('should not throw when rendering', () => {
function EmptyComponent() {
return nullORUndefined;
}
const root = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root.render(<EmptyComponent />);
});
}).not.toThrowError();
});
it('should not produce child DOM nodes for nullish and false', async () => {
function Component1() {
return nullORUndefined;
}
function Component2() {
return false;
}
const container1 = document.createElement('div');
const root1 = ReactDOMClient.createRoot(container1);
await act(() => {
root1.render(<Component1 />);
});
expect(container1.children.length).toBe(0);
const container2 = document.createElement('div');
const root2 = ReactDOMClient.createRoot(container2);
await act(() => {
root2.render(<Component2 />);
});
expect(container2.children.length).toBe(0);
});
it('should be able to switch between rendering nullish and a normal tag', async () => {
const instance1 = (
<TogglingComponent
firstComponent={nullORUndefined}
secondComponent={'div'}
/>
);
const instance2 = (
<TogglingComponent
firstComponent={'div'}
secondComponent={nullORUndefined}
/>
);
const container2 = document.createElement('div');
const root1 = ReactDOMClient.createRoot(container);
await act(() => {
root1.render(instance1);
});
assertLog(['mount undefined', 'update DIV']);
const root2 = ReactDOMClient.createRoot(container2);
await act(() => {
root2.render(instance2);
});
assertLog(['mount DIV', 'update undefined']);
});
it('should be able to switch in a list of children', async () => {
const instance1 = (
<TogglingComponent
firstComponent={nullORUndefined}
secondComponent={'div'}
/>
);
const root = ReactDOMClient.createRoot(container);
await act(() => {
root.render(
<div>
{instance1}
{instance1}
{instance1}
</div>,
);
});
assertLog([
'mount undefined',
'mount undefined',
'mount undefined',
'update DIV',
'update DIV',
'update DIV',
]);
});
it('should distinguish between a script placeholder and an actual script tag', () => {
const instance1 = (
<TogglingComponent
firstComponent={nullORUndefined}
secondComponent={'script'}
/>
);
const instance2 = (
<TogglingComponent
firstComponent={'script'}
secondComponent={nullORUndefined}
/>
);
const root1 = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root1.render(instance1);
});
}).not.toThrow();
const container2 = document.createElement('div');
const root2 = ReactDOMClient.createRoot(container2);
expect(() => {
ReactDOM.flushSync(() => {
root2.render(instance2);
});
}).not.toThrow();
assertLog([
'mount undefined',
'update SCRIPT',
'mount SCRIPT',
'update undefined',
]);
});
it(
'should have findDOMNode return null when multiple layers of composite ' +
'components render to the same nullish placeholder',
() => {
function GrandChild() {
return nullORUndefined;
}
function Child() {
return <GrandChild />;
}
const instance1 = (
<TogglingComponent firstComponent={'div'} secondComponent={Child} />
);
const instance2 = (
<TogglingComponent firstComponent={Child} secondComponent={'div'} />
);
const root1 = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root1.render(instance1);
});
}).not.toThrow();
const container2 = document.createElement('div');
const root2 = ReactDOMClient.createRoot(container2);
expect(() => {
ReactDOM.flushSync(() => {
root2.render(instance2);
});
}).not.toThrow();
assertLog([
'mount DIV',
'update undefined',
'mount undefined',
'update DIV',
]);
},
);
it('works when switching components', async () => {
let innerRef;
class Inner extends React.Component {
render() {
return <span />;
}
componentDidMount() {
// Make sure the DOM node resolves properly even if we're replacing a
// `null` component
expect(findDOMNode(this)).not.toBe(null);
}
componentWillUnmount() {
// Even though we're getting replaced by `null`, we haven't been
// replaced yet!
expect(findDOMNode(this)).not.toBe(null);
}
}
function Wrapper({showInner}) {
innerRef = React.createRef(null);
return showInner ? <Inner ref={innerRef} /> : nullORUndefined;
}
const el = document.createElement('div');
// Render the <Inner /> component...
const root = ReactDOMClient.createRoot(el);
await act(() => {
root.render(<Wrapper showInner={true} />);
});
expect(innerRef.current).not.toBe(null);
// Switch to null...
await act(() => {
root.render(<Wrapper showInner={false} />);
});
expect(innerRef.current).toBe(null);
// ...then switch back.
await act(() => {
root.render(<Wrapper showInner={true} />);
});
expect(innerRef.current).not.toBe(null);
expect.assertions(6);
});
it('can render nullish at the top level', async () => {
const div = document.createElement('div');
const root = ReactDOMClient.createRoot(div);
await act(() => {
root.render(nullORUndefined);
});
expect(div.innerHTML).toBe('');
});
it('does not break when updating during mount', () => {
class Child extends React.Component {
componentDidMount() {
if (this.props.onMount) {
this.props.onMount();
}
}
render() {
if (!this.props.visible) {
return nullORUndefined;
}
return <div>hello world</div>;
}
}
class Parent extends React.Component {
update = () => {
this.forceUpdate();
};
render() {
return (
<div>
<Child key="1" visible={false} />
<Child key="0" visible={true} onMount={this.update} />
<Child key="2" visible={false} />
</div>
);
}
}
const root = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root.render(<Parent />);
});
}).not.toThrow();
});
it('preserves the dom node during updates', async () => {
function Empty() {
return nullORUndefined;
}
const root = ReactDOMClient.createRoot(container);
await act(() => {
root.render(<Empty />);
});
const noscript1 = container.firstChild;
expect(noscript1).toBe(null);
// This update shouldn't create a DOM node
await act(() => {
root.render(<Empty />);
});
const noscript2 = container.firstChild;
expect(noscript2).toBe(null);
});
it('should not warn about React.forwardRef that returns nullish', () => {
const Empty = () => {
return nullORUndefined;
};
const EmptyForwardRef = React.forwardRef(Empty);
const root = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root.render(<EmptyForwardRef />);
});
}).not.toThrowError();
});
it('should not warn about React.memo that returns nullish', () => {
const Empty = () => {
return nullORUndefined;
};
const EmptyMemo = React.memo(Empty);
const root = ReactDOMClient.createRoot(container);
expect(() => {
ReactDOM.flushSync(() => {
root.render(<EmptyMemo />);
});
}).not.toThrowError();
});
});
});