feedback

Author: rolodato

client_test.go

@@ -145,7 +145,7 @@ func TestGetFlags(t *testing.T) {
 	// When
 	client := flagsmith.MustNewClient(fixtures.EnvironmentAPIKey, flagsmith.WithBaseURL(server.URL+"/api/v1/"))
 
-	flags, err := client.GetEnvironmentFlags(context.Background())
+	flags, err := client.GetFlags(context.Background(), flagsmith.EvaluationContext{})
 
 	// Then
 	assert.NoError(t, err)

offline_handler.go

@@ -8,14 +8,14 @@ import (
 )
 
 type Environment interface {
-	Environment() *environments.EnvironmentModel
+	GetEnvironment() *environments.EnvironmentModel
 }
 
 type environment struct {
 	model *environments.EnvironmentModel
 }
 
-func (e environment) Environment() *environments.EnvironmentModel {
+func (e environment) GetEnvironment() *environments.EnvironmentModel {
 	return e.model
 }
 

offline_handler_test.go

@@ -28,5 +28,5 @@ func TestLocalFileHandlerGetEnvironment(t *testing.T) {
 
 	// Then
 	assert.NoError(t, err)
-	assert.NotEmpty(t, env.Environment().APIKey)
+	assert.NotEmpty(t, env.GetEnvironment().APIKey)
 }

options.go

@@ -112,7 +112,7 @@ func WithProxy(url string) Option {
 // WithOfflineEnvironment sets the current environment and prevents Client from making network requests.
 func WithOfflineEnvironment(env Environment) Option {
 	return func(c *Client) {
-		c.state.SetOfflineEnvironment(env.Environment())
+		c.state.SetOfflineEnvironment(env.GetEnvironment())
 	}
 }