Skip to content

Commit

Permalink
test: update matrix for Go 1.20 (#1130)
Browse files Browse the repository at this point in the history
* test: update matrix for Go 1.20

* test: update coverage expecation for Go 1.20
  • Loading branch information
blgm committed Feb 9, 2023
1 parent a774638 commit 4890a62
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,14 @@ jobs:
steps:
- uses: actions/setup-go@v3
with:
go-version: '1.18'
go-version: '1.19'
- uses: actions/checkout@v3
- run: go mod tidy && git diff --exit-code go.mod go.sum
build:
runs-on: ubuntu-latest
strategy:
matrix:
version: [ '1.18', '1.19' ]
version: [ '1.19', '1.20' ]
name: Go ${{ matrix.version }}
steps:
- uses: actions/setup-go@v3
Expand Down
4 changes: 2 additions & 2 deletions integration/profiling_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,13 +105,13 @@ var _ = Describe("Profiling Specs", func() {
coverPkgFlag := fmt.Sprintf("-coverpkg=%s,%s", fm.PackageNameFor("coverage"), fm.PackageNameFor("coverage/external_coverage"))
seriesSession := startGinkgo(fm.PathTo("coverage"), coverPkgFlag)
Eventually(seriesSession).Should(gexec.Exit(0))
Ω(seriesSession.Out).Should(gbytes.Say("coverage: 71.4% of statements in"))
Ω(seriesSession.Out).Should(gbytes.Say(`coverage: (80\.0|71\.4)% of statements in`))
seriesCoverage := processCoverageProfile(fm.PathTo("coverage", "coverprofile.out"))
fm.RemoveFile("coverage", "coverprofile.out")

parallelSession := startGinkgo(fm.PathTo("coverage"), "--no-color", "--procs=2", coverPkgFlag)
Eventually(parallelSession).Should(gexec.Exit(0))
Ω(parallelSession.Out).Should(gbytes.Say(`coverage: 71\.4% of statements`))
Ω(parallelSession.Out).Should(gbytes.Say(`coverage: (80\.0|71\.4)% of statements`))
parallelCoverage := processCoverageProfile(fm.PathTo("coverage", "coverprofile.out"))

Ω(parallelCoverage).Should(Equal(seriesCoverage))
Expand Down

0 comments on commit 4890a62

Please sign in to comment.