Skip to content

Commit d42c224

Browse files
change terminology up upstream-diff-to-release (#5085) (#3536)
Signed-off-by: Modular Magician <[email protected]>
1 parent 9c4413e commit d42c224

File tree

3 files changed

+16
-14
lines changed

3 files changed

+16
-14
lines changed

.changelog/5085.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
```release-note:none
2+
3+
```

google-beta/provider_test.go

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -272,8 +272,8 @@ func vcrTest(t *testing.T, c resource.TestCase) {
272272
providers := getTestAccProviders(t.Name())
273273
c.Providers = providers
274274
defer closeRecorder(t)
275-
} else if isUpstreamDiffEnabled() {
276-
c = initializeUpstreamDiffTest(c)
275+
} else if isReleaseDiffEnabled() {
276+
c = initializeReleaseDiffTest(c)
277277
}
278278
resource.Test(t, c)
279279
}
@@ -948,25 +948,25 @@ func sleepInSecondsForTest(t int) resource.TestCheckFunc {
948948
}
949949
}
950950

951-
func isUpstreamDiffEnabled() bool {
952-
upstreamDiff := os.Getenv("UPSTREAM_DIFF")
953-
return upstreamDiff != ""
951+
func isReleaseDiffEnabled() bool {
952+
releaseDiff := os.Getenv("RELEASE_DIFF")
953+
return releaseDiff != ""
954954
}
955955

956-
func initializeUpstreamDiffTest(c resource.TestCase) resource.TestCase {
957-
var upstreamProvider string
956+
func initializeReleaseDiffTest(c resource.TestCase) resource.TestCase {
957+
var releaseProvider string
958958
packagePath := fmt.Sprint(reflect.TypeOf(Config{}).PkgPath())
959959
if strings.Contains(packagePath, "google-beta") {
960-
upstreamProvider = "google-beta"
960+
releaseProvider = "google-beta"
961961
} else {
962-
upstreamProvider = "google"
962+
releaseProvider = "google"
963963
}
964964

965965
if c.ExternalProviders != nil {
966-
c.ExternalProviders[upstreamProvider] = resource.ExternalProvider{}
966+
c.ExternalProviders[releaseProvider] = resource.ExternalProvider{}
967967
} else {
968968
c.ExternalProviders = map[string]resource.ExternalProvider{
969-
upstreamProvider: {},
969+
releaseProvider: {},
970970
}
971971
}
972972

@@ -984,7 +984,7 @@ func initializeUpstreamDiffTest(c resource.TestCase) resource.TestCase {
984984
replacementSteps = append(replacementSteps, teststep)
985985
if teststep.ExpectError == nil && teststep.PlanOnly == false {
986986
newStep := resource.TestStep{
987-
Config: reformConfigWithProvider(ogConfig, upstreamProvider),
987+
Config: reformConfigWithProvider(ogConfig, releaseProvider),
988988
PlanOnly: true,
989989
}
990990
replacementSteps = append(replacementSteps, newStep)

google-beta/resource_dataproc_cluster_test.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,8 @@ import (
1313
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
1414
"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
1515

16-
"google.golang.org/api/googleapi"
17-
1816
dataproc "google.golang.org/api/dataproc/v1beta2"
17+
"google.golang.org/api/googleapi"
1918
)
2019

2120
func TestDataprocExtractInitTimeout(t *testing.T) {

0 commit comments

Comments
 (0)