🐛 Ensure the environment.InitFlags() is called to avoid confusion.

README.md

@@ -51,13 +51,11 @@ import (
 // config as well as the parsing level and state flags.
 var global environment.GlobalEnvironment
 
-func init() {
+// TestMain is the first entry point for `go test`.
+func TestMain(m *testing.M) {
 	// environment.InitFlags registers state and level filter flags.
 	environment.InitFlags(flag.CommandLine)
-}
 
-// TestMain is the first entry point for `go test`.
-func TestMain(m *testing.M) {
 	// We get a chance to parse flags to include the framework flags for the
 	// framework as well as any additional flags included in the integration.
 	flag.Parse()

pkg/environment/magic.go

@@ -19,7 +19,6 @@ package environment
 import (
 	"context"
 	"errors"
-	"fmt"
 	"regexp"
 	"testing"
 	"time"
@@ -35,7 +34,13 @@ import (
 )
 
 func NewGlobalEnvironment(ctx context.Context) GlobalEnvironment {
-	fmt.Printf("level %s, state %s, feature %s\n\n", l, s, *f)
+	log := logging.FromContext(ctx)
+	if l.Valid() && s.Valid() {
+		log.Infof("Global environment settings: level %s, state %s, feature %#v", l, s, *f)
+	} else {
+		log.Fatal("Flags are not initialized properly. Ensure to call " +
+			"environment.InitFlags() func.")
+	}
 
 	return &MagicGlobalEnvironment{
 		c:                ctx,

pkg/feature/level.go

@@ -71,6 +71,16 @@ func (l Levels) String() string {
 	return b.String()
 }
 
+func (l Levels) Valid() bool {
+	values := []Levels{Must, MustNot, Should, ShouldNot, May, All}
+	for _, value := range values {
+		if l == value {
+			return true
+		}
+	}
+	return false
+}
+
 var LevelMapping = map[Levels]string{
 	Must:      "MUST",
 	MustNot:   "MUST NOT",

pkg/feature/states.go

@@ -60,6 +60,16 @@ func (s States) String() string {
 	return b.String()
 }
 
+func (s States) Valid() bool {
+	values := []States{Alpha, Beta, Stable, Any}
+	for _, value := range values {
+		if s == value {
+			return true
+		}
+	}
+	return false
+}
+
 var StatesMapping = map[States]string{
 	Alpha:  "Alpha",
 	Beta:   "Beta",

test/e2e/main_test.go

@@ -38,13 +38,11 @@ import (
 // config as well as the parsing level and state flags.
 var global environment.GlobalEnvironment
 
-func init() {
+// TestMain is the first entry point for `go test`.
+func TestMain(m *testing.M) {
 	// environment.InitFlags registers state, level and feature filter flags.
 	environment.InitFlags(flag.CommandLine)
-}
 
-// TestMain is the first entry point for `go test`.
-func TestMain(m *testing.M) {
 	// We get a chance to parse flags to include the framework flags for the
 	// framework as well as any additional flags included in the integration.
 	flag.Parse()
@@ -53,7 +51,7 @@ func TestMain(m *testing.M) {
 	// testing framework for namespace management, and could be leveraged by
 	// features to pull Kubernetes clients or the test environment out of the
 	// context passed in the features.
-	ctx, startInformers := injection.EnableInjectionOrDie(nil, nil) //nolint
+	ctx, startInformers := injection.EnableInjectionOrDie(nil, nil) // nolint
 	startInformers()
 
 	// global is used to make instances of Environments, NewGlobalEnvironment

test/example/main_test.go

@@ -38,13 +38,11 @@ import (
 // config as well as the parsing level and state flags.
 var global environment.GlobalEnvironment
 
-func init() {
+// TestMain is the first entry point for `go test`.
+func TestMain(m *testing.M) {
 	// environment.InitFlags registers state and level filter flags.
 	environment.InitFlags(flag.CommandLine)
-}
 
-// TestMain is the first entry point for `go test`.
-func TestMain(m *testing.M) {
 	// We get a chance to parse flags to include the framework flags for the
 	// framework as well as any additional flags included in the integration.
 	flag.Parse()
@@ -53,7 +51,7 @@ func TestMain(m *testing.M) {
 	// testing framework for namespace management, and could be leveraged by
 	// features to pull Kubernetes clients or the test environment out of the
 	// context passed in the features.
-	ctx, startInformers := injection.EnableInjectionOrDie(nil, nil) //nolint
+	ctx, startInformers := injection.EnableInjectionOrDie(nil, nil) // nolint
 	startInformers()
 
 	// global is used to make instances of Environments, NewGlobalEnvironment