Migrating all unit tests and types tests to vitest. #1784

siddhantrawal · 2024-01-16T21:03:52Z

Part of #1571

Changes

Followed this guide: https://vitest.dev/guide/migration.html#migrating-from-jest

Testing

All test cases are passing.

Docs

N/A

TODOs after PR Merge:

Handle isJest/isDev issues (Linking to this convo). Added common reliable check to detect jest,vitest and mocha testing framework. #1807
Removing test:types and writing type tests more robustly(Linking to this convo).
Update jsdom to latest version (See convo) Updated jsdom version. #1806

Co-authored-by: Rohan Kadkol <[email protected]>

packages/itwinui-react/tsconfig.json

packages/itwinui-react/vitest.config.ts

Co-authored-by: Rohan Kadkol <[email protected]>

packages/itwinui-react/vitest.config.ts

Co-authored-by: Mayank <[email protected]>

…ersion.

packages/itwinui-react/setupTests.ts

mayank99

LGTM 🚀 It runs faster and feels much nicer overall compared to jest.

r100-stack

LGTM 🎉

packages/itwinui-react/vitest.config.ts

installed vitest and added config file.

9f753c4

siddhantrawal self-assigned this Jan 16, 2024

siddhantrawal and others added 28 commits January 16, 2024 16:04

Merge branch 'main' into siddhant/migrate_tests

66d7b97

Added more things inside config file, fixed Alert unit tests.

9d3ad36

added more things inside config file.

62b1175

replacing jest.fn to vi.fn

6eaa4d6

replacing jest.spyon/mock with vi,spyon/mock

cf0936a

replacing jest.clearAllMocks with vi.clearAllMocks.

ad52a8e

replacing jest.restoreAllMocks with vi.restoreAllMocks

c10ab4b

replacing jest.useFakeTimers with vi.useFakeTimers

5339e7e

replacing jest.useRealTimers with vi.useRealTimers.

e1ffd22

replacing jest.advanceTimersByTime with vi.advanceTimersByTime.

14b9b64

replacing jest.runAllTimers with vi.runAllTimers

47b73d7

Merge branch 'main' into siddhant/migrate_tests

2399d20

Merge branch 'main' into siddhant/migrate_tests

71b8384

Merge branch 'main' into siddhant/migrate_tests

5530f40

Merge branch 'main' into siddhant/migrate_tests

29b3afe

fixed package,json file and updated yarn.lock

ed94e8b

fixed typo

44b7739

replaced jest,fn with vi.fn

a098141

added modulenamemapper replacer for vitest.

63a0837

Co-authored-by: Rohan Kadkol <[email protected]>

removing console.log

92fb4b9

Merge branch 'main' into siddhant/migrate_tests

e652673

removing console.log()

c7b505a

added replacement for spy instance.

120e7d0

removed await from toaster test

9691087

added jest-dom type

40b766e

Merge branch 'main' into siddhant/migrate_tests

d3551a8

added v8 coverage.

6b3af43

added v8 dependency

ae5128f

r100-stack reviewed Jan 26, 2024

View reviewed changes

packages/itwinui-react/tsconfig.json Outdated Show resolved Hide resolved

packages/itwinui-react/vitest.config.ts Outdated Show resolved Hide resolved

Update packages/itwinui-react/tsconfig.json

3a51948

Co-authored-by: Rohan Kadkol <[email protected]>

mayank99 reviewed Jan 26, 2024

View reviewed changes

packages/itwinui-react/vitest.config.ts Show resolved Hide resolved

siddhantrawal and others added 2 commits January 26, 2024 12:45

fixing path of useGlobals.js

7cc0b03

Update packages/itwinui-react/vitest.config.ts

c68b0e5

Co-authored-by: Mayank <[email protected]>

siddhantrawal requested review from mayank99 and r100-stack January 26, 2024 18:22

siddhantrawal added 5 commits January 26, 2024 13:32

adding coverage in test script

36c16d7

added coverage inside vitest, removed flag from package.json

34caaf9

adding globalthis.jest

d312b16

removed coverage from config/build file, updated test-result action v…

04958b5

…ersion.

adding aliase for styles.js, moving __mocks__ folder outside src.

5b3cd7e

mayank99 reviewed Jan 29, 2024

View reviewed changes

packages/itwinui-react/setupTests.ts Outdated Show resolved Hide resolved

siddhantrawal added 3 commits January 29, 2024 10:35

removing textencoder from setupTests.ts

ae4b6c2

adding typecheck.

5e40a96

removing css false and testing load()

72f999e

siddhantrawal requested a review from mayank99 January 29, 2024 16:56

siddhantrawal added 4 commits January 29, 2024 15:04

removing test coverage from package.json

058bfbe

added path for typecheck vitest, fixed test case.,

ddbd532

Merge branch 'main' into siddhant/migrate_tests

6f3011d

renaming jest type to vi.

a2b43fa

mayank99 approved these changes Jan 29, 2024

View reviewed changes

r100-stack approved these changes Jan 29, 2024

View reviewed changes

siddhantrawal merged commit 44b38f8 into main Jan 29, 2024
16 checks passed

siddhantrawal deleted the siddhant/migrate_tests branch January 29, 2024 21:01

mayank99 reviewed Jan 29, 2024

View reviewed changes

packages/itwinui-react/vitest.config.ts Show resolved Hide resolved

This was referenced Jan 29, 2024

Adding typecheck manually for test.tsx files. #1803

Merged

Updated jsdom version. #1806

Merged

Maintenance #1571

Open

Added common reliable check to detect jest,vitest and mocha testing framework. #1807

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrating all unit tests and types tests to vitest. #1784

Migrating all unit tests and types tests to vitest. #1784

siddhantrawal commented Jan 16, 2024 •

edited

Loading

mayank99 left a comment

r100-stack left a comment

Migrating all unit tests and types tests to vitest. #1784

Migrating all unit tests and types tests to vitest. #1784

Conversation

siddhantrawal commented Jan 16, 2024 • edited Loading

Changes

Testing

Docs

TODOs after PR Merge:

mayank99 left a comment

Choose a reason for hiding this comment

r100-stack left a comment

Choose a reason for hiding this comment

siddhantrawal commented Jan 16, 2024 •

edited

Loading