Skip to content

Commit 814b7b9

Browse files
remove bridgedProvider utility
1 parent 380c0e9 commit 814b7b9

File tree

13 files changed

+16
-77
lines changed

13 files changed

+16
-77
lines changed

pkg/pf/tests/autonaming_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ func TestAutonaming(t *testing.T) {
2727
},
2828
})
2929

30-
prov := bridgedProvider(provBuilder)
30+
prov := provBuilder.ToProviderInfo()
3131
prov.Resources["testprovider_test"] = &tfbridge.ResourceInfo{
3232
Tok: "testprovider:index:Test",
3333
Fields: map[string]*tfbridge.SchemaInfo{

pkg/pf/tests/config_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ func TestAccProviderConfigureSecretsPluginFramework(t *testing.T) {
475475
},
476476
})
477477

478-
bp := bridgedProvider(tfp)
478+
bp := tfp.ToProviderInfo()
479479
bp.Config = map[string]*info.Schema{
480480
"basic_strlist_config": {
481481
// Prevent basicStrlistConfigs pluralization

pkg/pf/tests/diff_list_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,6 @@ func TestDetailedDiffList(t *testing.T) {
237237

238238
diff := crosstests.Diff(
239239
t, res, map[string]cty.Value{"key": initialValue}, map[string]cty.Value{"key": changeValue},
240-
crosstests.DisableAccurateBridgePreview(),
241240
)
242241

243242
autogold.ExpectFile(t, testOutput{

pkg/pf/tests/diff_map_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,6 @@ func TestDetailedDiffMap(t *testing.T) {
187187

188188
diff := crosstests.Diff(
189189
t, schemaValueMakerPair.res, map[string]cty.Value{"key": initialValue}, map[string]cty.Value{"key": changeValue},
190-
crosstests.DisableAccurateBridgePreview(),
191190
)
192191

193192
autogold.ExpectFile(t, testOutput{

pkg/pf/tests/diff_object_test.go

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -337,10 +337,7 @@ func TestDetailedDiffObject(t *testing.T) {
337337
res := pb.NewResource(pb.NewResourceArgs{
338338
ResourceSchema: schema.schema,
339339
})
340-
diff := crosstests.Diff(
341-
t, res, initialValue, changeValue,
342-
crosstests.DisableAccurateBridgePreview(),
343-
)
340+
diff := crosstests.Diff(t, res, initialValue, changeValue)
344341
autogold.ExpectFile(t, testOutput{
345342
initialValue: scenario.initialValue,
346343
changeValue: scenario.changeValue,
@@ -360,10 +357,7 @@ func TestDetailedDiffObject(t *testing.T) {
360357
res := pb.NewResource(pb.NewResourceArgs{
361358
ResourceSchema: schema.schema,
362359
})
363-
diff := crosstests.Diff(
364-
t, res, initialValue, changeValue,
365-
crosstests.DisableAccurateBridgePreview(),
366-
)
360+
diff := crosstests.Diff(t, res, initialValue, changeValue)
367361
autogold.ExpectFile(t, testOutput{
368362
initialValue: scenario.initialValue,
369363
changeValue: scenario.changeValue,

pkg/pf/tests/diff_secret_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ func TestSecretBasic(t *testing.T) {
3232
},
3333
})
3434

35-
prov := bridgedProvider(provBuilder)
35+
prov := provBuilder.ToProviderInfo()
3636

3737
program := `
3838
name: test
@@ -82,7 +82,7 @@ func TestSecretSet(t *testing.T) {
8282
},
8383
})
8484

85-
prov := bridgedProvider(provBuilder)
85+
prov := provBuilder.ToProviderInfo()
8686

8787
program := `
8888
name: test
@@ -177,7 +177,7 @@ func TestSecretObjectBlock(t *testing.T) {
177177
},
178178
})
179179

180-
prov := bridgedProvider(provBuilder)
180+
prov := provBuilder.ToProviderInfo()
181181

182182
program := `
183183
name: test
@@ -273,7 +273,7 @@ func TestSecretPulumiSchema(t *testing.T) {
273273
},
274274
})
275275

276-
prov := bridgedProvider(provBuilder)
276+
prov := provBuilder.ToProviderInfo()
277277

278278
prov.Resources["testprovider_test"].Fields = map[string]*info.Schema{
279279
"s": {Secret: tfbridge0.True()},

pkg/pf/tests/diff_set_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -597,7 +597,6 @@ func TestDetailedDiffSet(t *testing.T) {
597597

598598
diff := crosstests.Diff(
599599
t, schemaValueMakerPair.res, map[string]cty.Value{"key": initialValue}, map[string]cty.Value{"key": changeValue},
600-
crosstests.DisableAccurateBridgePreview(),
601600
)
602601

603602
autogold.ExpectFile(t, testOutput{

pkg/pf/tests/diff_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,6 @@ func TestDetailedDiffStringAttribute(t *testing.T) {
197197
})
198198
diff := crosstests.Diff(
199199
t, res, map[string]cty.Value{"key": initialValue}, map[string]cty.Value{"key": changeValue},
200-
crosstests.DisableAccurateBridgePreview(),
201200
)
202201

203202
autogold.ExpectFile(t, testOutput{

pkg/pf/tests/internal/cross-tests/diff.go

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,7 @@ func Diff(t T, res pb.Resource, tfConfig1, tfConfig2 map[string]cty.Value, optio
109109
require.NoError(t, err)
110110
t.Logf("Pulumi.yaml:\n%s", string(bytes))
111111

112-
pt, err := pulcheck.PulCheck(
113-
t, bridgedProvider(prov, bridgedProviderOpts{enableAccurateBridgePreview: !opts.disableAccurateBridgePreview}), string(bytes))
112+
pt, err := pulcheck.PulCheck(t, prov.ToProviderInfo(), string(bytes))
114113
require.NoError(t, err)
115114
pt.Up(t)
116115

@@ -134,8 +133,7 @@ func Diff(t T, res pb.Resource, tfConfig1, tfConfig2 map[string]cty.Value, optio
134133
}
135134

136135
type diffOpts struct {
137-
resourceInfo map[string]*info.Schema
138-
disableAccurateBridgePreview bool
136+
resourceInfo map[string]*info.Schema
139137
}
140138

141139
type DiffOption func(*diffOpts)
@@ -144,8 +142,3 @@ type DiffOption func(*diffOpts)
144142
func DiffProviderInfo(info map[string]*info.Schema) DiffOption {
145143
return func(o *diffOpts) { o.resourceInfo = info }
146144
}
147-
148-
// DisableAccurateBridgePreview disables the accurate bridge preview feature.
149-
func DisableAccurateBridgePreview() DiffOption {
150-
return func(o *diffOpts) { o.disableAccurateBridgePreview = true }
151-
}

pkg/pf/tests/internal/cross-tests/util.go

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,6 @@ import (
2424
"github.com/pulumi/pulumi/sdk/v3/go/common/resource"
2525

2626
crosstestsimpl "github.com/pulumi/pulumi-terraform-bridge/v3/pkg/internal/tests/cross-tests/impl"
27-
"github.com/pulumi/pulumi-terraform-bridge/v3/pkg/pf/internal/providerbuilder"
28-
"github.com/pulumi/pulumi-terraform-bridge/v3/pkg/pf/tfbridge"
29-
tfbridge0 "github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfbridge"
30-
"github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfbridge/info"
31-
"github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfbridge/tokens"
3227
)
3328

3429
type T = crosstestsimpl.T
@@ -57,23 +52,3 @@ func skipUnlessLinux(t T) {
5752
t.Skip("Skipping on non-Linux platforms as our CI does not yet install Terraform CLI required for these tests")
5853
}
5954
}
60-
61-
type bridgedProviderOpts struct {
62-
enableAccurateBridgePreview bool
63-
}
64-
65-
func bridgedProvider(prov *providerbuilder.Provider, opts bridgedProviderOpts) info.Provider {
66-
shimProvider := tfbridge.ShimProvider(prov)
67-
68-
provider := tfbridge0.ProviderInfo{
69-
P: shimProvider,
70-
Name: prov.TypeName,
71-
Version: prov.Version,
72-
MetadataInfo: &tfbridge0.MetadataInfo{},
73-
EnableAccurateBridgePreview: opts.enableAccurateBridgePreview,
74-
}
75-
76-
provider.MustComputeTokens(tokens.SingleModule(prov.TypeName, "index", tokens.MakeStandard(prov.TypeName)))
77-
78-
return provider
79-
}

0 commit comments

Comments
 (0)