Skip to content

Commit

Permalink
Remove actions.preTest in favor of integrationTestProvider
Browse files Browse the repository at this point in the history
By using ci-mgmt's intended solution for running integration tests in the `provider/`
directory, we get a maintained test script and avoid overloading the `actions.preTest`
hook to run tests.

Related to pulumi/ci-mgmt#1276, pulumi/ci-mgmt#1274 (comment)
  • Loading branch information
iwahbe committed Jan 10, 2025
1 parent f523215 commit 6e264ec
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
10 changes: 2 additions & 8 deletions .ci-mgmt.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,10 @@ makeTemplate: bridged
plugins:
- name: std
version: "1.6.2"

- name: terraform
version: "1.0.16"
kind: converter

- name: random
- name: random
version: "4.2.0"
- name: aws
version: "4.10.0"
Expand All @@ -24,15 +22,11 @@ plugins:
- name: azure
version: "4.7.0"
team: ecosystem
actions:
preTest:
- name: Run provider tests
run: |
cd provider && go test -v -json -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 . 2>&1 | tee /tmp/gotest.log | gotestfmt
license:
ignore:
# These projects don't have a LICENSE file, but do say mention (in README.md) that
# they are under Apache-2.0.
- github.com/apache/arrow
pulumiConvert: 1
registryDocs: true
integrationTestProvider: true
5 changes: 3 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,9 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}
version: v2.5.0
- name: Run provider tests
run: |
cd provider && go test -v -json -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 . 2>&1 | tee /tmp/gotest.log | gotestfmt
if: matrix.testTarget == 'local'
working-directory: provider
run: go test -v -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 .
- name: Run tests
if: matrix.testTarget == 'local'
run: cd examples && go test -v -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -skip TestPulumiExamples -parallel 4 .
Expand Down

0 comments on commit 6e264ec

Please sign in to comment.