test: fix tests for asserting wrong handler callbacks

Signed-off-by: Bernd Warmuth <[email protected]>

openfeature/event_executor_test.go

@@ -988,10 +988,16 @@ func TestEventHandler_HandlersRunImmediately(t *testing.T) {
 			rsp <- e
 		}
 
-		//AddHandler(ProviderReady, &callback) // TODO: hard to test, because callback is executed on handler registration
+		AddHandler(ProviderReady, &callback)
+		<-rsp // ignore first READY event which gets emitted after registration
 		AddHandler(ProviderStale, &callback)
 		AddHandler(ProviderConfigChange, &callback)
 
+		// assert client transitioned to ERROR
+		eventually(t, func() bool {
+			return GetApiInstance().GetClient().State() == ErrorState
+		}, time.Second, time.Millisecond*100, "")
+
 		select {
 		case <-rsp:
 			t.Errorf("event must not emit for this handler")
@@ -1025,10 +1031,16 @@ func TestEventHandler_HandlersRunImmediately(t *testing.T) {
 			rsp <- e
 		}
 
-		//AddHandler(ProviderReady, &callback) // TODO: hard to test, because callback is executed on handler registration
+		AddHandler(ProviderReady, &callback)
+		<-rsp // ignore first READY event which gets emitted after registration
 		AddHandler(ProviderError, &callback)
 		AddHandler(ProviderConfigChange, &callback)
 
+		// assert client transitioned to STALE
+		eventually(t, func() bool {
+			return GetApiInstance().GetClient().State() == StaleState
+		}, time.Second, time.Millisecond*100, "")
+
 		select {
 		case <-rsp:
 			t.Errorf("event must not emit for this handler")