Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[browser] Speed up WasmBuildTests by using the compression switch #110645

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ilonatommy
Copy link
Member

@ilonatommy ilonatommy commented Dec 12, 2024

Attempt 1)
Tests what happens if we switch of compression for all the browser tests. While publish-only tests should be affected positively, publish + run can be affected negatively.

Attempt 2)
In case of visible negative results of attempt 1, try to disable compression manually for all the non-run tests.

We should especially pay attention to duration times of:

ame: ST-Wasm.Build.Tests.MainWithArgsTests, Duration: 00:42:24.8340000, State: True, Details: Workloads: 00:19:41.4750000; NoWebcil: 00:22:43.3590000, NumberOfTests: 16
Name: ST-Wasm.Build.Tests.IcuTests, Duration: 00:43:41.1800000, State: True, Details: Workloads: 00:21:04.0090000; NoWebcil: 00:22:37.1710000, NumberOfTests: 28
Name: ST-Wasm.Build.NativeRebuild.Tests.FlagsChangeRebuildTests, Duration: 00:43:48.3250000, State: True, Details: Workloads: 00:21:59.1170000; NoWebcil: 00:21:49.2080000, NumberOfTests: 12
Name: ST-Wasm.Build.Tests.WasmBuildAppTest, Duration: 00:44:51.5840000, State: True, Details: Workloads: 00:22:42.2910000; NoWebcil: 00:22:09.2930000, NumberOfTests: 18
Name: ST-Wasm.Build.Tests.WasmNativeDefaultsTests, Duration: 00:50:23.6740000, State: True, Details: Workloads: 00:24:17.3640000; NoWebcil: 00:26:06.3100000, NumberOfTests: 158
Name: ST-Wasm.Build.Tests.SatelliteAssembliesTests, Duration: 00:58:33.6940000, State: True, Details: Workloads: 00:28:21.0620000; NoWebcil: 00:30:12.6320000, NumberOfTests: 38
Name: ST-Wasm.Build.Tests.Blazor.BuildPublishTests, Duration: 01:14:41.7980000, State: True, Details: Workloads: 00:37:17.2470000; NoWebcil: 00:37:24.5510000, NumberOfTests: 30

and similar that run long. They are the CI blockers, in comparison to the short running tests, e.g.

Name: ST-Wasm.Build.Templates.Tests.NativeBuildTests, Duration: 00:10:12.1740000, State: True, Details: Workloads: 00:04:54.0100000; NoWebcil: 00:05:18.1640000, NumberOfTests: 8
Name: ST-Wasm.Build.Tests.Blazor.CleanTests, Duration: 00:16:32.4850000, State: True, Details: Workloads: 00:08:39.8330000; NoWebcil: 00:07:52.6520000, NumberOfTests: 12

stats creation app.

@ilonatommy ilonatommy added arch-wasm WebAssembly architecture test-enhancement Improvements of test source code os-browser Browser variant of arch-wasm labels Dec 12, 2024
@ilonatommy ilonatommy added this to the 10.0.0 milestone Dec 12, 2024
@ilonatommy ilonatommy self-assigned this Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch-wasm WebAssembly architecture area-Build-mono os-browser Browser variant of arch-wasm test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant