Skip to content

Commit

Permalink
remove unit tests with race detector temporarily
Browse files Browse the repository at this point in the history
  • Loading branch information
ramin committed Jan 29, 2024
1 parent 1bae060 commit 2e3d3f6
Showing 1 changed file with 21 additions and 15 deletions.
36 changes: 21 additions & 15 deletions .github/workflows/go-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,21 +105,27 @@ jobs:
file: ./coverage.txt
name: coverage-${{ matrix.os }}

unit_test_race:
needs: [lint, go_mod_tidy_check]
name: Unit Tests with Race Detector (ubuntu-latest)
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4

- name: set up go
uses: actions/setup-go@v5
with:
go-version: ${{ inputs.go-version }}

- name: execute test run
run: make test-unit-race ENABLE_VERBOSE=${{ needs.setup.outputs.debug }}
# @ramin - Temporarily removed while we figure out getting
# these unit tests consistently running on ubuntu-latest
# and then enabled for macos-latest. We aren't requiring
# unit_race_test to pass for PRs so lets remove and reintroduce
# once green
#
# unit_test_race:
# needs: [lint, go_mod_tidy_check]
# name: Unit Tests with Race Detector (ubuntu-latest)
# runs-on: ubuntu-latest

# steps:
# - uses: actions/checkout@v4

# - name: set up go
# uses: actions/setup-go@v5
# with:
# go-version: ${{ inputs.go-version }}

# - name: execute test run
# run: make test-unit-race ENABLE_VERBOSE=${{ needs.setup.outputs.debug }}

integration_test:
name: Integration Tests
Expand Down

0 comments on commit 2e3d3f6

Please sign in to comment.