Skip to content

Commit a3a83be

Browse files
authored
Remove actions.preTest in favor of integrationTestProvider (#640)
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)
2 parents cf95e32 + d04a59f commit a3a83be

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

.ci-mgmt.yaml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,10 @@ major-version: 3
33
docker: true
44
makeTemplate: bridged
55
team: ecosystem
6-
actions:
7-
preTest:
8-
- name: Run provider tests
9-
run: |
10-
cd provider && go test -v -json -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 . 2>&1 | tee /tmp/gotest.log | gotestfmt
116
pulumiConvert: 1
127
registryDocs: true
138
plugins:
149
- name: terraform
1510
version: "1.0.16"
1611
kind: converter
12+
integrationTestProvider: true

.github/workflows/test.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,9 @@ jobs:
8080
token: ${{ secrets.GITHUB_TOKEN }}
8181
version: v2.5.0
8282
- name: Run provider tests
83-
run: |
84-
cd provider && go test -v -json -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 . 2>&1 | tee /tmp/gotest.log | gotestfmt
83+
if: matrix.testTarget == 'local'
84+
working-directory: provider
85+
run: go test -v -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -parallel 4 .
8586
- name: Run tests
8687
if: matrix.testTarget == 'local'
8788
run: cd examples && go test -v -count=1 -cover -timeout 2h -tags=${{ matrix.language }} -skip TestPulumiExamples -parallel 4 .

0 commit comments

Comments
 (0)