Skip to content

Commit

Permalink
Enable TestConfigureSimpleValues (#2553)
Browse files Browse the repository at this point in the history
I'm unable to get the flake on my computer, even with `--count 1000`.
The test wasn't previously running in parallel, so it shouldn't be an
interaction with other tests.

I've added better asserts so we can see which result is unexpectedly
nil. I think our best chance at fixing the test is to turn the test back
on and then re-engage if/when it flakes again.

Part of investigating
#2530
  • Loading branch information
iwahbe authored Oct 31, 2024
1 parent b2722f6 commit ed56fc2
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
2 changes: 2 additions & 0 deletions pkg/internal/tests/cross-tests/assert.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ func assertResourceDataEqual(t T, resourceSchema map[string]*schema.Schema, tfRe
// assert.Equal(t, tfResult, puResult)
//
// We make do by comparing slices tfResult and puResult.
require.NotNil(t, tfResult)
require.NotNil(t, puResult)
assertCtyValEqual(t, "RawConfig", tfResult.GetRawConfig(), puResult.GetRawConfig())
assertCtyValEqual(t, "RawPlan", tfResult.GetRawPlan(), puResult.GetRawPlan())
assertCtyValEqual(t, "RawState", tfResult.GetRawState(), puResult.GetRawState())
Expand Down
8 changes: 7 additions & 1 deletion pkg/tfbridge/tests/provider_configure_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,13 @@ import (
)

func TestConfigureSimpleValues(t *testing.T) {
t.Skip("TODO[pulumi/pulumi-terraform-bridge#2530]: flaky")
// TestConfigureSimpleValues was previously flaky:
//
// https://github.com/pulumi/pulumi-terraform-bridge/issues/2530
//
// The investigation was inconclusive, so we are adding back into circulation to
// see if the failures continue with tighter asserts.

t.Run("string", crosstests.MakeConfigure(map[string]*schema.Schema{
"f0": {Type: schema.TypeString, Required: true},
}, cty.ObjectVal(map[string]cty.Value{
Expand Down

0 comments on commit ed56fc2

Please sign in to comment.