perf(client): speed up test runs without reducing coverage by apalchys · Pull Request #3111 · rolling-scopes/rsschool-app

apalchys · 2026-09-11T17:12:26Z

Issue:
Optimize tests

Description:
Latest local client coverage run: 2m 26s (145.90s), compared with the recorded 8m 11s baseline (490.72s): 70.3% less time. Both runs used four workers. The latest run includes Vitest 5 and subsequent test cleanup, so this is a historical comparison across toolchain versions.

Enabled dependency optimization and updated Vitest to 5.0.0
Moved 34 DOM-free test files to Node
Consolidated repeated UI renders, preserving assertions
Fixed async flakes and disabled automatic retries
Reduced client warnings from 12,338 to 6 in the full unit test run

Latest coverage run at c5530616:

Command: npm run test:ci --workspace client -- --maxWorkers=4
433 test files and 1,875 tests passed
Coverage: 95.48% statements, 93.09% branches, 95.10% functions, 95.54% lines
All four coverage thresholds remain at 90%

Self-Check:

Database migration added (if required)
Changes tested locally

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

apalchys added 30 commits September 11, 2026 23:26

test(client): wrap team distribution interactions in act

e253224

test(client): wrap CV contact interactions in act

26d2d75

test(client): wrap task submission interactions in act

aa5d8f7

test(client): wrap event modal interactions in act

d0948a1

test(client): wrap criteria interactions in act

f14a922

test(client): wrap education card interactions in act

20eeb08

test(client): wrap about card interactions in act

47a1c76

test(client): wrap message panel interactions in act

0bda433

test(client): wrap tasks page interactions in act

fe03739

test(client): wrap editable criteria updates in act

e3e100b

test(client): wrap join team interactions in act

4732b95

test(client): wrap discipline modal interactions in act

ad5161b

test(client): wrap question list interactions in act

b7f2f72

test(client): wrap obfuscation modal interactions in act

061fc42

test(client): wrap course event interactions in act

a3237be

test(client): consolidate languages card scenarios

c80fb23

test(client): consolidate interview summary scenarios

a12b776

test(client): consolidate student info scenarios

49b43a8

test(client): consolidate mentors hall scenarios

45e0428

test(client): consolidate profile settings modal scenarios

55d4af4

test(client): consolidate student stats modal scenarios

e3444f3

test(client): consolidate legacy feedback scenarios

b36204c

test(client): consolidate additional action scenarios

a26d8be

test(client): consolidate cross-check history scenarios

a55b336

test(client): consolidate score table tab scenarios

b2f23f6

test(client): consolidate stage feedback scenarios

cd26204

test(client): consolidate expired CV tooltip scenarios

7672893

test(client): consolidate timezone settings scenarios

bafdcd3

test(client): consolidate welcome card assertions

71e0421

test(client): consolidate mentor preference scenarios

a951bc4

apalchys added 24 commits September 12, 2026 05:10

test(client): reuse liquid chart render

302f62b

test(client): combine message provider checks

f463abf

test(client): replace student summary polling

69a6b73

test(client): reuse course label render

255bae6

test(client): wrap timer cleanup in act

5a78525

test(client): remove duplicate maps render

1b498a8

test(client): reuse personal section render

bba154f

test(client): reuse dev tools provider render

82c593d

test(client): remove answers polling

321cfb4

test(client): reuse interview details render

a86df31

test(client): reuse contacts section render

1a07007

test(client): reuse mentor registry render

012a6a9

test(client): reuse course stats hook render

6713d82

test(client): remove duplicate places hook renders

96ece2c

test(client): replace view data polling

eb628c2

test(client): reuse modal form hook render

73c2e30

test(client): reuse team score hook renders

da94ff6

test(client): reuse active course hook render

7e9df9c

test(client): call expiration helper directly

4663264

style(client): format education card test

2157f54

style(client): format events admin test

df20222

style(client): format places hook test

0d95228

test(nestjs): fix strict typing in test fixtures and mocks

8dc6b88

test(client): reduce unit test warnings

c553061

apalchys requested a lite review from Copilot September 12, 2026 08:00

Copilot started reviewing on behalf of apalchys September 12, 2026 08:00 View session

Copilot stopped reviewing on behalf of apalchys due to an error September 12, 2026 08:20
Copilot had to stop work due to a timeout.

Copilot AI reviewed Sep 12, 2026

Merge branch 'master' into feature/optimize-tests

d9859c4

AlreadyBored approved these changes Sep 12, 2026

View reviewed changes

Navigation Menu

Uh oh!

perf(client): speed up test runs without reducing coverage - #3111

perf(client): speed up test runs without reducing coverage#3111
apalchys wants to merge 407 commits into
masterfrom
feature/optimize-tests

apalchys commented Sep 11, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

apalchys commented Sep 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

apalchys commented Sep 11, 2026 •

edited

Loading