Skip to content

Commit 0ccae38

Browse files
committed
refactor: change functions to be more consistent between regular functions and arrow functions
1 parent f6096ba commit 0ccae38

File tree

3 files changed

+37
-31
lines changed

3 files changed

+37
-31
lines changed

src/core/index.ts

Lines changed: 27 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -40,38 +40,40 @@ function resultContainer<TValue>(): ResultContainer<TValue> {
4040
}
4141
}
4242

43-
const createRenderHook = <TProps, TResult, TOptions extends {}, TRenderer extends Renderer<TProps>>(
43+
function createRenderHook<TProps, TResult, TOptions extends {}, TRenderer extends Renderer<TProps>>(
4444
createRenderer: CreateRenderer<TProps, TResult, TOptions, TRenderer>
45-
) => (
46-
callback: (props: TProps) => TResult,
47-
options: RenderHookOptions<TProps, TOptions> = {} as RenderHookOptions<TProps, TOptions>
48-
): RenderHook<TProps, TResult, TRenderer> => {
49-
const { result, setValue, setError, addResolver } = resultContainer<TResult>()
50-
const renderProps = { callback, setValue, setError }
51-
let hookProps = options.initialProps
45+
) {
46+
return function renderHook(
47+
callback: (props: TProps) => TResult,
48+
options: RenderHookOptions<TProps, TOptions> = {} as RenderHookOptions<TProps, TOptions>
49+
): RenderHook<TProps, TResult, TRenderer> {
50+
const { result, setValue, setError, addResolver } = resultContainer<TResult>()
51+
const renderProps = { callback, setValue, setError }
52+
let hookProps = options.initialProps
5253

53-
const { render, rerender, unmount, act, ...renderUtils } = createRenderer(renderProps, options)
54+
const { render, rerender, unmount, act, ...renderUtils } = createRenderer(renderProps, options)
5455

55-
render(hookProps)
56+
render(hookProps)
5657

57-
function rerenderHook(newProps = hookProps) {
58-
hookProps = newProps
59-
rerender(hookProps)
60-
}
58+
const rerenderHook = (newProps = hookProps) => {
59+
hookProps = newProps
60+
rerender(hookProps)
61+
}
6162

62-
function unmountHook() {
63-
removeCleanup(unmountHook)
64-
unmount()
65-
}
63+
const unmountHook = () => {
64+
removeCleanup(unmountHook)
65+
unmount()
66+
}
6667

67-
addCleanup(unmountHook)
68+
addCleanup(unmountHook)
6869

69-
return {
70-
result,
71-
rerender: rerenderHook,
72-
unmount: unmountHook,
73-
...asyncUtils(act, addResolver),
74-
...renderUtils
70+
return {
71+
result,
72+
rerender: rerenderHook,
73+
unmount: unmountHook,
74+
...asyncUtils(act, addResolver),
75+
...renderUtils
76+
}
7577
}
7678
}
7779

src/helpers/createTestHarness.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,11 @@ function TestComponent<TProps, TResult>({
2424
return null
2525
}
2626

27-
export const createTestHarness = <TProps, TResult>(
27+
function createTestHarness<TProps, TResult>(
2828
rendererProps: RendererProps<TProps, TResult>,
2929
Wrapper?: WrapperComponent<TProps>,
3030
suspense: boolean = true
31-
) => {
31+
) {
3232
return (props?: TProps) => {
3333
let component = <TestComponent hookProps={props} {...rendererProps} />
3434
if (Wrapper) {
@@ -40,3 +40,5 @@ export const createTestHarness = <TProps, TResult>(
4040
return component
4141
}
4242
}
43+
44+
export { createTestHarness }

src/helpers/promises.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1-
const resolveAfter = (ms: number) =>
2-
new Promise((resolve) => {
1+
function resolveAfter(ms: number) {
2+
return new Promise((resolve) => {
33
setTimeout(resolve, ms)
44
})
5+
}
56

6-
const isPromise = <T>(value: unknown): boolean =>
7-
typeof (value as PromiseLike<T>).then === 'function'
7+
function isPromise<T>(value: unknown): boolean {
8+
return typeof (value as PromiseLike<T>).then === 'function'
9+
}
810

911
export { isPromise, resolveAfter }

0 commit comments

Comments
 (0)