back to no metric test - just for sanity which apparently is now sore…

…ly lacking

metrics_test.go

@@ -1,14 +1,5 @@
 package unleash
 
-import (
-	"testing"
-	"time"
-
-	"github.com/Unleash/unleash-client-go/v4/api"
-	"github.com/stretchr/testify/assert"
-	"gopkg.in/h2non/gock.v1"
-)
-
 // func TestMetrics_RegisterInstance(t *testing.T) {
 // 	assert := assert.New(t)
 // 	defer gock.OffAll()
@@ -123,42 +114,42 @@ import (
 // 	assert.True(gock.IsDone(), "there should be no more mocks")
 // }
 
-func TestMetrics_DisabledMetrics(t *testing.T) {
-	assert := assert.New(t)
-	defer gock.OffAll()
-
-	gock.New(mockerServer).
-		Get("/client/features").
-		Reply(200).
-		JSON(api.FeatureResponse{})
-
-	mockListener := &DebugListener{}
-	// mockListener.On("OnReady").Return()
-
-	client, err := NewClient(
-		WithUrl(mockerServer),
-		WithDisableMetrics(false),
-		WithMetricsInterval(100*time.Millisecond),
-		WithAppName(mockAppName),
-		WithInstanceId(mockInstanceId),
-		WithListener(mockListener),
-	)
-	assert.Nil(err, "client should not return an error")
-
-	client.IsEnabled("bar")
-
-	assert.True(true)
-
-	client.Close()
-	// client.WaitForReady()
-	// client.IsEnabled("foo")
-	// client.IsEnabled("bar")
-	// client.IsEnabled("baz")
-
-	// time.Sleep(300 * time.Millisecond)
-	// client.Close()
-	// assert.True(gock.IsDone(), "there should be no more mocks")
-}
+// func TestMetrics_DisabledMetrics(t *testing.T) {
+// assert := assert.New(t)
+// defer gock.OffAll()
+
+// gock.New(mockerServer).
+// 	Get("/client/features").
+// 	Reply(200).
+// 	JSON(api.FeatureResponse{})
+
+// mockListener := &DebugListener{}
+// // mockListener.On("OnReady").Return()
+
+// client, err := NewClient(
+// 	WithUrl(mockerServer),
+// 	WithDisableMetrics(false),
+// 	WithMetricsInterval(100*time.Millisecond),
+// 	WithAppName(mockAppName),
+// 	WithInstanceId(mockInstanceId),
+// 	WithListener(mockListener),
+// )
+// assert.Nil(err, "client should not return an error")
+
+// client.IsEnabled("bar")
+
+// assert.True(true)
+
+// client.Close()
+// client.WaitForReady()
+// client.IsEnabled("foo")
+// client.IsEnabled("bar")
+// client.IsEnabled("baz")
+
+// time.Sleep(300 * time.Millisecond)
+// client.Close()
+// assert.True(gock.IsDone(), "there should be no more mocks")
+// }
 
 // // TestMetrics_SendMetricsFail tests that no metrics are lost if /client/metrics
 // // fails temporarily.