Skip to content

Commit

Permalink
address PR remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
Konrad Jamrozik committed Mar 3, 2023
1 parent a7445ca commit 6b93176
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
4 changes: 3 additions & 1 deletion eng/common-tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ By design, unlike `eng/common`, `eng/common-tests` is not subject to
## When tests in this directory are executed

The PowerShell Pester unit tests will be executed by the [`tools - eng-common-tests`](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=220791) pipeline
upon a PR that makes changes to `eng/common-tests/**`. The pipeline source is `eng/common-tests/ci.yml`.
upon a PR that makes changes to `eng/common/scripts/**` or `eng/common-tests/**`. The pipeline source is `eng/common-tests/ci.yml`.

In addition, specific tools might selectively run a subset of tests during their pipeline run, as e.g. done by `tools\code-owners-parser\ci.yml`.

## How to ensure your PowerShell pester unit test is executed

Expand Down
3 changes: 2 additions & 1 deletion eng/common-tests/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ trigger:
- hotfix/*
paths:
include:
- eng/common-tests/*
- eng/common/scripts
- eng/common-tests

pr:
branches:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ variables:
- template: /eng/pipelines/templates/variables/globals.yml

stages:
- stage: Run_PowerShell_Tests
- stage:
displayName: Run PowerShell Tests
jobs:
- job: Test
strategy:
Expand Down

0 comments on commit 6b93176

Please sign in to comment.