Skip to content

Commit

Permalink
Re-enable S3 integration test
Browse files Browse the repository at this point in the history
  • Loading branch information
hifi committed Dec 24, 2023
1 parent 69a24af commit 129ad7d
Showing 1 changed file with 22 additions and 25 deletions.
47 changes: 22 additions & 25 deletions .github/workflows/commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,15 +65,6 @@ jobs:

- run: go test -v ./...

# - name: Build integration test
# run: go test -c ./integration
#
# - uses: actions/upload-artifact@v2
# with:
# name: integration.test
# path: integration.test
# if-no-files-found: error

# long-running-test:
# name: Run Long Running Unit Test
# runs-on: ubuntu-latest
Expand Down Expand Up @@ -114,22 +105,28 @@ jobs:

- run: ./etc/s3_mock.py go test -v ./replica_client_test.go -integration s3

# s3-integration-test:
# name: Run S3 Integration Tests
# runs-on: ubuntu-latest
# needs: build
# steps:
# - uses: actions/download-artifact@v2
# with:
# name: integration.test
# - run: chmod +x integration.test
#
# - run: ./integration.test -test.v -test.run=TestReplicaClient -replica-type s3
# env:
# LITESTREAM_S3_ACCESS_KEY_ID: ${{ secrets.LITESTREAM_S3_ACCESS_KEY_ID }}
# LITESTREAM_S3_SECRET_ACCESS_KEY: ${{ secrets.LITESTREAM_S3_SECRET_ACCESS_KEY }}
# LITESTREAM_S3_REGION: us-east-1
# LITESTREAM_S3_BUCKET: integration.litestream.io
s3-integration-test:
name: Run S3 Integration Tests
runs-on: ubuntu-latest
needs: build
if: github.ref == 'refs/heads/main' || github.ref == 'refs/heads/fix/integration-tests'
steps:
- uses: actions/checkout@v4

- uses: actions/setup-go@v4
with:
go-version: ${{ env.GO_VERSION }}

- run: go env

- run: go install ./cmd/litestream

- run: go test -v ./replica_client_test.go -integration s3
env:
LITESTREAM_S3_ACCESS_KEY_ID: ${{ secrets.LITESTREAM_S3_ACCESS_KEY_ID }}
LITESTREAM_S3_SECRET_ACCESS_KEY: ${{ secrets.LITESTREAM_S3_SECRET_ACCESS_KEY }}
LITESTREAM_S3_REGION: us-east-1
LITESTREAM_S3_BUCKET: integration.litestream.io

# gcp-integration-test:
# name: Run GCP Integration Tests
Expand Down

0 comments on commit 129ad7d

Please sign in to comment.