Added Test for http.Handler returned from zpages.NewTracezHandler (#8108)

Co-authored-by: Damien Mathieu <[email protected]>

Author: 2 people

otelconf/config_json.go

@@ -194,11 +194,6 @@ func (j *LogRecordExporter) UnmarshalJSON(b []byte) error {
 	return nil
 }
 
-// MarshalJSON implements json.Marshaler.
-func (j *AttributeType) MarshalJSON() ([]byte, error) {
-	return json.Marshal(j.Value)
-}
-
 // UnmarshalJSON implements json.Unmarshaler.
 func (j *TextMapPropagator) UnmarshalJSON(b []byte) error {
 	type Plain TextMapPropagator

otelconf/config_test.go

@@ -394,6 +394,51 @@ func TestNewSDK(t *testing.T) {
 			wantMeterProvider:  metricnoop.NewMeterProvider(),
 			wantLoggerProvider: lognoop.NewLoggerProvider(),
 		},
+		{
+			name: "invalid resource",
+			cfg: []ConfigurationOption{
+				WithContext(t.Context()),
+				WithOpenTelemetryConfiguration(OpenTelemetryConfiguration{
+					TracerProvider: &TracerProviderJson{},
+					MeterProvider:  &MeterProviderJson{},
+					LoggerProvider: &LoggerProviderJson{},
+					Resource:       &LoggerProviderJson{},
+				}),
+			},
+			wantErr:            newErrInvalid("resource"),
+			wantTracerProvider: tracenoop.NewTracerProvider(),
+			wantMeterProvider:  metricnoop.NewMeterProvider(),
+			wantLoggerProvider: lognoop.NewLoggerProvider(),
+		},
+		{
+			name: "invalid logger provider",
+			cfg: []ConfigurationOption{
+				WithContext(t.Context()),
+				WithOpenTelemetryConfiguration(OpenTelemetryConfiguration{
+					TracerProvider: &TracerProviderJson{},
+					MeterProvider:  &MeterProviderJson{},
+					LoggerProvider: &ResourceJson{},
+					Resource:       &ResourceJson{},
+				}),
+			},
+			wantErr:            newErrInvalid("logger_provider"),
+			wantTracerProvider: tracenoop.NewTracerProvider(),
+			wantMeterProvider:  metricnoop.NewMeterProvider(),
+			wantLoggerProvider: lognoop.NewLoggerProvider(),
+		},
+		{
+			name: "invalid tracer provider",
+			cfg: []ConfigurationOption{
+				WithContext(t.Context()),
+				WithOpenTelemetryConfiguration(OpenTelemetryConfiguration{
+					TracerProvider: &ResourceJson{},
+				}),
+			},
+			wantErr:            newErrInvalid("tracer_provider"),
+			wantTracerProvider: tracenoop.NewTracerProvider(),
+			wantMeterProvider:  metricnoop.NewMeterProvider(),
+			wantLoggerProvider: lognoop.NewLoggerProvider(),
+		},
 	}
 	for _, tt := range tests {
 		sdk, err := NewSDK(tt.cfg...)
@@ -975,38 +1020,69 @@ var v100OpenTelemetryConfigEnvParsing = OpenTelemetryConfiguration{
 
 func TestUnmarshalOpenTelemetryConfiguration(t *testing.T) {
 	tests := []struct {
-		name     string
-		input    string
-		wantErr  error
-		wantType *OpenTelemetryConfiguration
+		name       string
+		jsonConfig []byte
+		yamlConfig []byte
+		wantErr    error
+		wantType   *OpenTelemetryConfiguration
 	}{
 		{
-			name:     "invalid config missing required file_format",
-			input:    `missing_file_format`,
-			wantErr:  newErrRequired(&OpenTelemetryConfiguration{}, "file_format"),
-			wantType: &OpenTelemetryConfiguration{},
-		},
-		{
-			name:     "file_format invalid",
-			input:    `invalid_file_format`,
-			wantErr:  newErrUnmarshal(&OpenTelemetryConfiguration{}),
-			wantType: &OpenTelemetryConfiguration{},
-		},
-		{
-			name:  "valid defaults config",
-			input: `valid_defaults`,
+			name:       "valid defaults config",
+			jsonConfig: []byte(`{"file_format": "1.0"}`),
+			yamlConfig: []byte("file_format: 1.0"),
 			wantType: &OpenTelemetryConfiguration{
 				Disabled:   ptr(false),
 				FileFormat: "1.0",
 				LogLevel:   ptr("info"),
 			},
 		},
 		{
-			name:     "invalid config",
-			input:    "invalid_bool",
-			wantErr:  newErrUnmarshal(&OpenTelemetryConfiguration{}),
-			wantType: &OpenTelemetryConfiguration{},
+			name:       "invalid config missing required file_format",
+			jsonConfig: []byte(`{"disabled": false}`),
+			yamlConfig: []byte("disabled: false"),
+			wantErr:    newErrRequired(&OpenTelemetryConfiguration{}, "file_format"),
+			wantType:   &OpenTelemetryConfiguration{},
+		},
+		{
+			name:       "file_format invalid",
+			jsonConfig: []byte(`{"file_format":[], "disabled": false}`),
+			yamlConfig: []byte("file_format: []\ndisabled: false"),
+			wantErr:    newErrUnmarshal(&OpenTelemetryConfiguration{}),
+			wantType:   &OpenTelemetryConfiguration{},
 		},
+		{
+			name:       "invalid config",
+			jsonConfig: []byte(`{"file_format": "yaml", "disabled": "notabool"}`),
+			yamlConfig: []byte("file_format: []\ndisabled: notabool"),
+			wantErr:    newErrUnmarshal(&OpenTelemetryConfiguration{}),
+			wantType:   &OpenTelemetryConfiguration{},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run("yaml "+tt.name, func(t *testing.T) {
+			got, err := ParseYAML(tt.yamlConfig)
+			require.ErrorIs(t, err, tt.wantErr)
+			if tt.wantErr == nil {
+				assert.Equal(t, tt.wantType, got)
+			}
+		})
+		t.Run("json "+tt.name, func(t *testing.T) {
+			var got OpenTelemetryConfiguration
+			err := json.Unmarshal(tt.jsonConfig, &got)
+			require.ErrorIs(t, err, tt.wantErr)
+			assert.Equal(t, tt.wantType, &got)
+		})
+	}
+}
+
+func TestParseFiles(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    string
+		wantErr  error
+		wantType *OpenTelemetryConfiguration
+	}{
 		{
 			name:     "invalid nil name",
 			input:    "v1.0.0_invalid_nil_name",

otelconf/config_yaml.go

@@ -96,10 +96,10 @@ func (j *OpenTelemetryConfiguration) UnmarshalYAML(node *yaml.Node) error {
 		sh.Plain.InstrumentationDevelopment = sh.InstrumentationDevelopment
 	}
 
-	if sh.Plain.LogLevel == nil {
+	if sh.LogLevel == nil {
 		// Configure the log level of the internal logger used by the SDK.
 		// If omitted, info is used.
-		sh.Plain.LogLevel = ptr("info")
+		sh.LogLevel = ptr("info")
 	}
 
 	*j = OpenTelemetryConfiguration(sh.Plain)

otelconf/example_test.go

@@ -12,11 +12,11 @@ import (
 	"go.opentelemetry.io/otel"
 	"go.opentelemetry.io/otel/log/global"
 
-	otelconf "go.opentelemetry.io/contrib/otelconf/v0.3.0"
+	"go.opentelemetry.io/contrib/otelconf"
 )
 
 func Example() {
-	b, err := os.ReadFile(filepath.Join("testdata", "v0.3.yaml"))
+	b, err := os.ReadFile(filepath.Join("testdata", "v1.0.0.yaml"))
 	if err != nil {
 		log.Fatal(err)
 	}