Skip to content

Fix runspace tests so that they will run more reliably - #7360

Merged
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
PaulHigin:fix-threadjob-runspace-tests
Jul 26, 2018
Merged

Fix runspace tests so that they will run more reliably#7360
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
PaulHigin:fix-threadjob-runspace-tests

Conversation

@PaulHigin

Copy link
Copy Markdown
Contributor

PR Summary

Update the ThreadJob Runspace count tests so that they run in a separate process. This ensures that the count does not include other session runspace creation/removal and should make the tests reliable.

PR Checklist

'@

$result = & "$PSHOME/pwsh" -c $script
$result | Should Be "True"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should -BeExactly

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

'@

$result = & "$PSHOME/pwsh" -c $script
$result | Should Be "True","True","True"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should -BeExactly

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@anmenaga Andrew (anmenaga) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 20f7577 into PowerShell:master Jul 26, 2018
@PaulHigin
Paul Higinbotham (PaulHigin) deleted the fix-threadjob-runspace-tests branch July 26, 2018 17:13
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WG-Quality-Test issues in a test or in test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants