-
Notifications
You must be signed in to change notification settings - Fork 227
Open
Labels
help wantedThe issue is up for grabs for anyone in the community.The issue is up for grabs for anyone in the community.testsThe issue or pull request is about tests only.The issue or pull request is about tests only.
Description
Description
In PR #2102, temporary changes were made to the Prerequisites integration test to accommodate DSCv3 integration test stages. These changes should be reverted once a separate Prerequisites integration test is created specifically for DSCv3.
Code Changed
From (strict check):
$module | Should -HaveCount 1
$module.Version -ge '1.0.4.1' | Should -BeTrueTo (lenient check):
$module.Count | Should -BeGreaterOrEqual 1
#$module.Version -ge '1.0.4.1' | Should -BeTrueFile: tests/Integration/Commands/Prerequisites.Integration.Tests.ps1
Lines: ~197-198
Expected Resolution
- Create a separate Prerequisites integration test file for DSCv3 integration test stages
- Revert the changes in
tests/Integration/Commands/Prerequisites.Integration.Tests.ps1back to the original strict assertions - Ensure the new DSCv3-specific Prerequisites test has appropriate module version checks for that environment
References
- PR: Add integration test stage for DSCv3 #2102
- Comment: Add integration test stage for DSCv3 #2102 (comment)
Reported by: @johlju
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
help wantedThe issue is up for grabs for anyone in the community.The issue is up for grabs for anyone in the community.testsThe issue or pull request is about tests only.The issue or pull request is about tests only.