From 7b2f877dd3b76367bc4ce5b1e8b9a44cc5bb24fb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mario=20Beltra=CC=81n=20Alarco=CC=81n?= Date: Sun, 21 Jul 2019 19:55:29 +0200 Subject: [PATCH] Wrap tests in describe block --- src/__tests__/useBoolean.test.ts | 6 +- src/__tests__/useToggle.test.ts | 116 ++++++++++++++++--------------- 2 files changed, 63 insertions(+), 59 deletions(-) diff --git a/src/__tests__/useBoolean.test.ts b/src/__tests__/useBoolean.test.ts index 8f633e9e..4dcafae0 100644 --- a/src/__tests__/useBoolean.test.ts +++ b/src/__tests__/useBoolean.test.ts @@ -1,6 +1,8 @@ import useBoolean from '../useBoolean'; import useToggle from '../useToggle'; -it('should be an alias for useToggle ', () => { - expect(useBoolean).toBe(useToggle); +describe('useBoolean hook', () => { + it('should be an alias for useToggle ', () => { + expect(useBoolean).toBe(useToggle); + }); }); diff --git a/src/__tests__/useToggle.test.ts b/src/__tests__/useToggle.test.ts index adafdc3b..32870fb4 100644 --- a/src/__tests__/useToggle.test.ts +++ b/src/__tests__/useToggle.test.ts @@ -1,83 +1,85 @@ import { act, renderHook } from '@testing-library/react-hooks'; import useToggle from '../useToggle'; -const setUp = (initialValue: boolean) => renderHook(() => useToggle(initialValue)); +describe('useToggle hook', () => { + const setUp = (initialValue: boolean) => renderHook(() => useToggle(initialValue)); -it('should init state to true', () => { - const { result } = setUp(true); + it('should init state to true', () => { + const { result } = setUp(true); - expect(result.current[0]).toBe(true); - expect(typeof result.current[1]).toBe('function'); -}); - -it('should init state to false', () => { - const { result } = setUp(false); - - expect(result.current[0]).toBe(false); - expect(result.current[1]).toBeInstanceOf(Function); -}); - -it('should set state to true', () => { - const { result } = setUp(false); - const [, toggle] = result.current; - - expect(result.current[0]).toBe(false); - - act(() => { - toggle(true); + expect(result.current[0]).toBe(true); + expect(typeof result.current[1]).toBe('function'); }); - expect(result.current[0]).toBe(true); -}); + it('should init state to false', () => { + const { result } = setUp(false); -it('should set state to false', () => { - const { result } = setUp(true); - const [, toggle] = result.current; - - expect(result.current[0]).toBe(true); - - act(() => { - toggle(false); + expect(result.current[0]).toBe(false); + expect(result.current[1]).toBeInstanceOf(Function); }); - expect(result.current[0]).toBe(false); -}); + it('should set state to true', () => { + const { result } = setUp(false); + const [, toggle] = result.current; -it('should toggle state from true', () => { - const { result } = setUp(true); - const [, toggle] = result.current; + expect(result.current[0]).toBe(false); - act(() => { - toggle(); + act(() => { + toggle(true); + }); + + expect(result.current[0]).toBe(true); }); - expect(result.current[0]).toBe(false); -}); + it('should set state to false', () => { + const { result } = setUp(true); + const [, toggle] = result.current; -it('should toggle state from false', () => { - const { result } = setUp(false); - const [, toggle] = result.current; + expect(result.current[0]).toBe(true); - act(() => { - toggle(); + act(() => { + toggle(false); + }); + + expect(result.current[0]).toBe(false); }); - expect(result.current[0]).toBe(true); -}); + it('should toggle state from true', () => { + const { result } = setUp(true); + const [, toggle] = result.current; -it('should ignore non-boolean parameters and toggle state', () => { - const { result } = setUp(true); - const [, toggle] = result.current; + act(() => { + toggle(); + }); - act(() => { - toggle('string'); + expect(result.current[0]).toBe(false); }); - expect(result.current[0]).toBe(false); + it('should toggle state from false', () => { + const { result } = setUp(false); + const [, toggle] = result.current; - act(() => { - toggle({}); + act(() => { + toggle(); + }); + + expect(result.current[0]).toBe(true); }); - expect(result.current[0]).toBe(true); + it('should ignore non-boolean parameters and toggle state', () => { + const { result } = setUp(true); + const [, toggle] = result.current; + + act(() => { + toggle('string'); + }); + + expect(result.current[0]).toBe(false); + + act(() => { + toggle({}); + }); + + expect(result.current[0]).toBe(true); + }); });