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 f3dbde5 commit 882c811
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 13 deletions.
12 changes: 1 addition & 11 deletions .ci-mgmt.yaml
Original file line number Diff line number Diff line change
@@ -1,24 +1,14 @@
provider: sdwan
major-version: 0
makeTemplate: bridged
# TODO: Enable this once our random provider is updated with framework support
# generate-nightly-test-workflow: true
plugins:
- name: std
version: "1.6.2"

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

team: ecosystem
providerDefaultBranch: main

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
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 @@ -78,8 +78,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 882c811

Please sign in to comment.