Commit 993d2c5

committed

test: refator and added comment for clarity about it

1 parent e51f5c4 commit 993d2c5Copy full SHA for 993d2c5

File tree

1 file changed

-6

lines changed

src/__tests__/utils
- runForRenderers.ts

1 file changed

-6

lines changed

`‎src/tests/utils/runForRenderers.ts`

Lines changed: 7 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -32,10 +32,10 @@ function requireRenderer<TRendererType extends ReactHooksRenderer = ReactHooksRe`
`32`	`32`	`return require(requirePath) as TRendererType`
`33`	`33`	`}`
`34`	`34`
`35`		`-functionhydratedServerRenderer(pure: boolean): ReactHooksRenderer{`
`36`		`-const{ renderHook, ...otherImports}=requireRenderer<ReactHooksServerRenderer>(`
`37`		`-pure ? 'server/pure' : 'server'`
`38`		`- )`
	`35`	+// This render turns the `server` renderer into a client renderer as many of the tests only
	`36`	`+// require hydration after the hook is renderer to be able to be reused for all the renderers`
	`37`	`+functionhydratedServerRenderer(baseRenderer: 'server'\|'server/pure'): ReactHooksRenderer{`
	`38`	`+ const{ renderHook, ...otherImports}=requireRenderer<ReactHooksServerRenderer>(baseRenderer)`
`39`	`39`
`40`	`40`	`return {`
`41`	`41`	`renderHook<TProps, TResult>(`
`@@ -61,8 +61,8 @@ const rendererResolvers = {`
`61`	`61`	`'dom/pure': () => requireRenderer('default/pure'),`
`62`	`62`	`'native/pure': () => requireRenderer('default/pure'),`
`63`	`63`	`'server/pure': () => requireRenderer<ReactHooksServerRenderer>('server/pure'),`
`64`		`- 'server/hydrated': () => hydratedServerRenderer(false),`
`65`		`- 'server/hydrated/pure': () => hydratedServerRenderer(true)`
	`64`	`+ 'server/hydrated': () => hydratedServerRenderer('server'),`
	`65`	`+ 'server/hydrated/pure': () => hydratedServerRenderer('server/pure')`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`global.runForRenderers = function runForRenderers<TRenderer extends Renderer>(`
`@@ -77,6 +77,7 @@ global.runForLazyRenderers = function runForLazyRenderers<TRenderer extends Rend`
`77`	`77`	`fn: (getRenderer: () => InferredRenderer<TRenderer>, rendererName: Renderer) => void`
`78`	`78`	`): void {`
`79`	`79`	`renderers.forEach((renderer) => {`
	`80`	`+ // eslint-disable-next-line jest/valid-title`
`80`	`81`	`describe(renderer, () => {`
`81`	`82`	`fn(() => rendererResolvers[renderer]() as InferredRenderer<TRenderer>, renderer)`
`82`	`83`	`})`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 993d2c5

File tree

1 file changed

1 file changed

`‎src/tests/utils/runForRenderers.ts`

0 commit comments

File tree

1 file changed

1 file changed

‎src/__tests__/utils/runForRenderers.ts

0 commit comments

`‎src/tests/utils/runForRenderers.ts`