Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add an additional acceptance test for the PowerShellGet module#4531

Merged
adityapatwardhan merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
bmanikm:AddPowerShellGetAcceptanceTestbmanikm/PowerShell:AddPowerShellGetAcceptanceTestCopy head branch name to clipboard
Aug 10, 2017
Merged

Add an additional acceptance test for the PowerShellGet module#4531
adityapatwardhan merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
bmanikm:AddPowerShellGetAcceptanceTestbmanikm/PowerShell:AddPowerShellGetAcceptanceTestCopy head branch name to clipboard

Conversation

@bmanikm

@bmanikm bmanikm commented Aug 8, 2017

Copy link
Copy Markdown
Contributor
  • This acceptance test will ensure that there are no regressions in PowerShell build w.r.t. to adding a .NET type in PowerShellGet module.

function Remove-InstalledModules
{
Get-InstalledModule -Name $ContosoServer -AllVersions -ErrorAction SilentlyContinue | Uninstall-Module -Force
Get-InstalledModule -Name $ContosoServer -AllVersions -ErrorAction SilentlyContinue | PowerShellGet\Uninstall-Module -Force

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This might throw right? Do you want to ignore those errors.

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.

There should not be any errors in uninstalling the test module, otherwise other test cases will fail.

@jianyunt

jianyunt commented Aug 9, 2017

Copy link
Copy Markdown
Contributor

:shipit:

@adityapatwardhan adityapatwardhan merged commit 0bfee35 into PowerShell:master Aug 10, 2017
@bmanikm bmanikm deleted the AddPowerShellGetAcceptanceTest branch August 11, 2017 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.