autoexport: Rename IsNone to IsNoneSpanExporter (#4134)

CHANGELOG.md

@@ -12,7 +12,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 - Add `NewMiddleware` function in `go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp`. (#2964)
 - Add the new `go.opentelemetry.io/contrib/instrgen` package to provide auto-generated source code instrumentation. (#3068, #3108)
-- The `go.opentelemetry.io/contrib/exporters/autoexport` package to provide configuration of trace exporters with useful defaults and envar support. (#2753, #4100, #4129, #4132)
+- The `go.opentelemetry.io/contrib/exporters/autoexport` package to provide configuration of trace exporters with useful defaults and envar support. (#2753, #4100, #4129, #4132, #4134)
 - `WithRouteTag` in `go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp` adds HTTP route attribute to metrics. (#615)
 
 ### Fixed

exporters/autoexport/exporter.go

@@ -87,7 +87,7 @@ func WithFallbackSpanExporter(exporter trace.SpanExporter) Option {
 // Use [WithFallbackSpanExporter] option to change the returned exporter
 // when OTEL_TRACES_EXPORTER is unset or empty.
 //
-// Use [IsNone] to check if the retured exporter is a "no operation" exporter.
+// Use [IsNoneSpanExporter] to check if the retured exporter is a "no operation" exporter.
 func NewSpanExporter(ctx context.Context, opts ...Option) (trace.SpanExporter, error) {
 	// prefer exporter configured via environment variables over exporter
 	// passed in via exporter parameter

exporters/autoexport/exporter_test.go

@@ -39,7 +39,7 @@ func TestFallbackExporterReturnedWhenNoEnvExporterConfigured(t *testing.T) {
 	)
 	assert.NoError(t, err)
 	assert.Equal(t, testExporter, exporter)
-	assert.False(t, IsNone(exporter))
+	assert.False(t, IsNoneSpanExporter(exporter))
 }
 
 func TestEnvExporterIsPreferredOverFallbackExporter(t *testing.T) {
@@ -94,7 +94,7 @@ func TestEnvExporterNone(t *testing.T) {
 
 	exporter, err := NewSpanExporter(context.Background())
 	assert.NoError(t, err)
-	assert.True(t, IsNone(exporter))
+	assert.True(t, IsNoneSpanExporter(exporter))
 }
 
 func assertOTLPHTTPExporter(t *testing.T, got trace.SpanExporter) {
@@ -108,7 +108,7 @@ func assertOTLPHTTPExporter(t *testing.T, got trace.SpanExporter) {
 	clientType := reflect.Indirect(reflect.ValueOf(got)).FieldByName("client").Elem().Type().String()
 	assert.Equal(t, "*otlptracehttp.client", clientType)
 
-	assert.False(t, IsNone(got))
+	assert.False(t, IsNoneSpanExporter(got))
 }
 
 func assertOTLPGRPCExporter(t *testing.T, got trace.SpanExporter) {
@@ -122,7 +122,7 @@ func assertOTLPGRPCExporter(t *testing.T, got trace.SpanExporter) {
 	clientType := reflect.Indirect(reflect.ValueOf(got)).FieldByName("client").Elem().Type().String()
 	assert.Equal(t, "*otlptracegrpc.client", clientType)
 
-	assert.False(t, IsNone(got))
+	assert.False(t, IsNoneSpanExporter(got))
 }
 
 type testExporter struct{}

exporters/autoexport/noop.go

@@ -35,9 +35,9 @@ func (e noop) Shutdown(ctx context.Context) error {
 	return nil
 }
 
-// IsNone returns true for the exporter returned by [NewSpanExporter]
+// IsNoneSpanExporter returns true for the exporter returned by [NewSpanExporter]
 // when OTEL_TRACES_EXPORTER environment variable is set to "none".
-func IsNone(e trace.SpanExporter) bool {
+func IsNoneSpanExporter(e trace.SpanExporter) bool {
 	_, ok := e.(noop)
 	return ok
 }