test: fix tests - add missing cleanup and wait. (#340)

alan-kut · web-flow · commit 640448d9e1e1 · 2025-04-30T13:16:42.000-04:00
Fix tests: add missing cleanup and wait.

Signed-off-by: Alan Kutniewski &lt;kutniewski@google.com&gt;
diff --git a/openfeature/openfeature_test.go b/openfeature/openfeature_test.go
@@ -721,6 +721,7 @@ func TestDefaultClientUsage(t *testing.T) {
 }
 
 func TestLateBindingOfDefaultProvider(t *testing.T) {
+	defer t.Cleanup(initSingleton)
 	// we are expecting
 	expectedResultUnboundProvider := "default-value-from-unbound-provider"
 	expectedResultFromLateDefaultProvider := "value-from-late-default-provider"
diff --git a/openfeature/testing/testprovider_test.go b/openfeature/testing/testprovider_test.go
@@ -12,7 +12,7 @@ func TestParallelSingletonUsage(t *testing.T) {
 	t.Parallel()
 
 	testProvider := NewTestProvider()
-	err := openfeature.GetApiInstance().SetProvider(testProvider)
+	err := openfeature.GetApiInstance().SetProviderAndWait(testProvider)
 	if err != nil {
 		t.Errorf("unable to set provider")
 	}

Original file line number	Diff line number	Diff line change
`@@ -721,6 +721,7 @@ func TestDefaultClientUsage(t *testing.T) {`
`721`	`721`	`}`
`722`	`722`
`723`	`723`	`func TestLateBindingOfDefaultProvider(t *testing.T) {`
	`724`	`+ defer t.Cleanup(initSingleton)`
`724`	`725`	`// we are expecting`
`725`	`726`	`expectedResultUnboundProvider := "default-value-from-unbound-provider"`
`726`	`727`	`expectedResultFromLateDefaultProvider := "value-from-late-default-provider"`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ func TestParallelSingletonUsage(t *testing.T) {`
`12`	`12`	`t.Parallel()`
`13`	`13`
`14`	`14`	`testProvider := NewTestProvider()`
`15`		`- err := openfeature.GetApiInstance().SetProvider(testProvider)`
	`15`	`+ err := openfeature.GetApiInstance().SetProviderAndWait(testProvider)`
`16`	`16`	`if err != nil {`
`17`	`17`	`t.Errorf("unable to set provider")`
`18`	`18`	`}`