Skip to content

Commit

Permalink
[chore] Skip non-arm compatible integration tests (#22802)
Browse files Browse the repository at this point in the history
Nearly all integration tests in this repo are now compatible with arm processors. There are two remaining tests that are not, and these do not appear to have a clear path forward. Skipping these tests allows those of us with arm processors to locally run make integration-tests-with-cover.
  • Loading branch information
djaglowski committed May 31, 2023
1 parent bc7e409 commit 948f5d8
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions receiver/riakreceiver/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import (
"fmt"
"net"
"path/filepath"
"runtime"
"testing"
"time"

Expand All @@ -24,6 +25,9 @@ import (
const riakPort = "8098"

func TestIntegration(t *testing.T) {
if runtime.GOARCH == "arm64" {
t.Skip("Incompatible with arm64")
}
scraperinttest.NewIntegrationTest(
NewFactory(),
scraperinttest.WithContainerRequest(
Expand Down
4 changes: 4 additions & 0 deletions receiver/sqlqueryreceiver/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ package sqlqueryreceiver
import (
"fmt"
"path/filepath"
"runtime"
"testing"
"time"

Expand Down Expand Up @@ -134,6 +135,9 @@ func TestPostgresqlIntegration(t *testing.T) {
// This test ensures the collector can connect to an Oracle DB, and properly get metrics. It's not intended to
// test the receiver itself.
func TestOracleDBIntegration(t *testing.T) {
if runtime.GOARCH == "arm64" {
t.Skip("Incompatible with arm64")
}
scraperinttest.NewIntegrationTest(
NewFactory(),
scraperinttest.WithContainerRequest(
Expand Down

0 comments on commit 948f5d8

Please sign in to comment.