(MAINT) Minimally Parallelize pester tests #1166
Open
+205
−60
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
michaeltlombardi
force-pushed
the
maint/main/speed-builds
branch
6 times, most recently
from
8b9a3eb to
184a2dc
Compare
|
To see the run times: https://github.com/PowerShell/DSC/actions/runs/18358507549 
This is a substantial improvement and shows that the majority of the remaining performance to squeeze from CI right now is parallelizing the windows tests for the |
Prior to this change, the pester tests made up the majority of the build time in CI. This change attempts to speed up the pester tests by parallelizing them roughly by folder: - The tests in the `dsc` folder run in their own cell. - The tests in the `adapters` folder run in their own cell. - The tests in the `extensions` folder run in their own cell. - The tests in the `resources` folder run in their own cell. This should substantially reduce the time spent on tests, particularly on Windows, where the CLI tests take much longer. This change makes minor changes to the new build script and helpers to support targeted test execution.
michaeltlombardi
force-pushed
the
maint/main/speed-builds
branch
from
184a2dc to
405ed02
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Summary
Prior to this change, the pester tests made up the majority of the build time in CI. This change attempts to speed up the pester tests by parallelizing them roughly by folder:
dscfolder run in their own cell.adaptersfolder run in their own cell.extensionsfolder run in their own cell.resourcesfolder run in their own cell.This should substantially reduce the time spent on tests, particularly on Windows, where the CLI tests take much longer.
This change makes minor changes to the new build script and helpers to support targeted test execution.
PR Context
Depends on #1161