Skip to content

Commit

Permalink
[Go] Exclude "TestFhirIO.*" from Go PostCommit Dataflow ARM test suite (
Browse files Browse the repository at this point in the history
#28215)

* Update integration.go to exclude one failed test

* fix go syntax

* Fix usage of append
  • Loading branch information
celeste-zeng authored Aug 30, 2023
1 parent 45998db commit d616fc2
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion sdks/go/test/integration/integration.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ import (
"strings"
"testing"
"time"
"os"

// common runner flag.
"github.com/apache/beam/sdks/v2/go/pkg/beam/options/jobopts"
Expand Down Expand Up @@ -301,6 +302,12 @@ func CheckFilters(t *testing.T) {
panic("ptest.Main() has not been called: please override TestMain to ensure that the integration test runs properly.")
}

var user = os.Getenv("USER")
// TODO: github-actions service account doesn't have permission to healthcare.fhirStores.create.
if user == "github-actions" {
dataflowFilters = append(dataflowFilters, "TestFhirIO.*")
}

// Check for sickbaying first.
n := t.Name()
for _, f := range sickbay {
Expand All @@ -318,7 +325,7 @@ func CheckFilters(t *testing.T) {
s1 := rand.NewSource(time.Now().UnixNano())
r1 := rand.New(s1)
*jobopts.JobName = fmt.Sprintf("go-%v-%v", strings.ToLower(n), r1.Intn(1000))

// Test for runner-specific skipping second.
var filters []string
runner := *ptest.Runner
Expand Down

0 comments on commit d616fc2

Please sign in to comment.