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.

Sign up for GitHub

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

Jump to bottom

tweaks to New-TestResources readme #1351

Merged
3 commits merged into from
Feb 1, 2021
Merged

tweaks to New-TestResources readme #1351

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
04a286a
tweaks to New-TestResources readme
christothes Jan 28, 2021
2cb8768
tweaks
christothes Jan 29, 2021
542eb90
prc
christothes Jan 29, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion eng/common/TestResources/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Connect-AzAccount -Subscription 'YOUR SUBSCRIPTION ID'
eng\common\TestResources\New-TestResources.ps1 -ServiceDirectory 'search'
```

The `OutFile` switch would be set if you are running this for a .NET project on Windows. This will save test environment settings
The `OutFile` switch should be set if you are running this for a .NET project on Windows. This will save test environment settings
christothes marked this conversation as resolved.
Show resolved Hide resolved
into a test-resources.json.env file next to test-resources.json. The file is protected via DPAPI.
The environment file would be scoped to the current repository directory and avoids the need to
set environment variables or restart your IDE to recognize them.
Expand Down Expand Up @@ -89,6 +89,12 @@ Remove-AzADServicePrincipal -ApplicationId $sp.ApplicationId -Force

If you persisted environment variables, you should also remove those as well.

Some scripts accept the `AdditionalParameters` parameter to control additional resource configuration options. For example:
christothes marked this conversation as resolved.
Show resolved Hide resolved

```powershell
.\eng\common\TestResources\New-TestResources.ps1 keyvault -AdditionalParameters @{enableHsm = $true}
christothes marked this conversation as resolved.
Show resolved Hide resolved
```

## In CI

Test pipelines should include deploy-test-resources.yml and
Expand Down