Simplify memoryMode assertions to exact value

trask · trask · commit 2b13219ae438 · 2025-11-18T10:37:43.000-08:00
All builder classes hardcode DEFAULT_MEMORY_MODE = MemoryMode.REUSABLE_DATA,
so the regex alternation (IMMUTABLE_DATA|REUSABLE_DATA) is unnecessary.
Using the exact value makes the tests more precise.
diff --git a/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/http/metrics/OtlpHttpMetricExporterOkHttpSenderTest.java b/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/http/metrics/OtlpHttpMetricExporterOkHttpSenderTest.java
@@ -103,7 +103,7 @@ void stringRepresentation() {
                   + "internalTelemetrySchemaVersion=LEGACY, "
                   + "aggregationTemporalitySelector=AggregationTemporalitySelector\\{.*\\}, "
                   + "defaultAggregationSelector=DefaultAggregationSelector\\{.*\\}, "
-                  + "memoryMode=(IMMUTABLE_DATA|REUSABLE_DATA)"
+                  + "memoryMode=REUSABLE_DATA"
                   + "\\}");
     }
   }
diff --git a/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/http/trace/OtlpHttpSpanExporterOkHttpSenderTest.java b/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/http/trace/OtlpHttpSpanExporterOkHttpSenderTest.java
@@ -50,7 +50,7 @@ void stringRepresentation() {
                   + "componentLoader=.*, "
                   + "exporterType=OTLP_HTTP_SPAN_EXPORTER, "
                   + "internalTelemetrySchemaVersion=LEGACY, "
-                  + "memoryMode=(IMMUTABLE_DATA|REUSABLE_DATA)"
+                  + "memoryMode=REUSABLE_DATA"
                   + "\\}");
     }
   }
diff --git a/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/metrics/OtlpGrpcMetricExporterTest.java b/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/metrics/OtlpGrpcMetricExporterTest.java
@@ -103,7 +103,7 @@ void stringRepresentation() {
                   + "internalTelemetrySchemaVersion=LEGACY, "
                   + "aggregationTemporalitySelector=AggregationTemporalitySelector\\{.*\\}, "
                   + "defaultAggregationSelector=DefaultAggregationSelector\\{.*\\}, "
-                  + "memoryMode=(IMMUTABLE_DATA|REUSABLE_DATA)"
+                  + "memoryMode=REUSABLE_DATA"
                   + "\\}");
     }
   }
diff --git a/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/traces/OtlpGrpcSpanExporterTest.java b/exporters/otlp/all/src/testDefaultSender/java/io/opentelemetry/exporter/otlp/traces/OtlpGrpcSpanExporterTest.java
@@ -50,7 +50,7 @@ void stringRepresentation() {
                   + "componentLoader=.*, "
                   + "exporterType=OTLP_GRPC_SPAN_EXPORTER, "
                   + "internalTelemetrySchemaVersion=LEGACY, "
-                  + "memoryMode=(IMMUTABLE_DATA|REUSABLE_DATA)"
+                  + "memoryMode=REUSABLE_DATA"
                   + "\\}");
     }
   }
diff --git a/exporters/otlp/all/src/testJdkHttpSender/java/io/opentelemetry/exporter/otlp/http/metrics/OtlpHttpMetricExporterJdkSenderTest.java b/exporters/otlp/all/src/testJdkHttpSender/java/io/opentelemetry/exporter/otlp/http/metrics/OtlpHttpMetricExporterJdkSenderTest.java
@@ -104,7 +104,7 @@ void stringRepresentation() {
                   + "internalTelemetrySchemaVersion=LEGACY, "
                   + "aggregationTemporalitySelector=AggregationTemporalitySelector\\{.*\\}, "
                   + "defaultAggregationSelector=DefaultAggregationSelector\\{.*\\}, "
-                  + "memoryMode=(IMMUTABLE_DATA|REUSABLE_DATA)"
+                  + "memoryMode=REUSABLE_DATA"
                   + "\\}");
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ void stringRepresentation() {`
`103`	`103`	`+ "internalTelemetrySchemaVersion=LEGACY, "`
`104`	`104`	`+ "aggregationTemporalitySelector=AggregationTemporalitySelector\\{.*\\}, "`
`105`	`105`	`+ "defaultAggregationSelector=DefaultAggregationSelector\\{.*\\}, "`
`106`		`- + "memoryMode=(IMMUTABLE_DATA\|REUSABLE_DATA)"`
	`106`	`+ + "memoryMode=REUSABLE_DATA"`
`107`	`107`	`+ "\\}");`
`108`	`108`	`}`
`109`	`109`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ void stringRepresentation() {`
`50`	`50`	`+ "componentLoader=.*, "`
`51`	`51`	`+ "exporterType=OTLP_HTTP_SPAN_EXPORTER, "`
`52`	`52`	`+ "internalTelemetrySchemaVersion=LEGACY, "`
`53`		`- + "memoryMode=(IMMUTABLE_DATA\|REUSABLE_DATA)"`
	`53`	`+ + "memoryMode=REUSABLE_DATA"`
`54`	`54`	`+ "\\}");`
`55`	`55`	`}`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ void stringRepresentation() {`
`104`	`104`	`+ "internalTelemetrySchemaVersion=LEGACY, "`
`105`	`105`	`+ "aggregationTemporalitySelector=AggregationTemporalitySelector\\{.*\\}, "`
`106`	`106`	`+ "defaultAggregationSelector=DefaultAggregationSelector\\{.*\\}, "`
`107`		`- + "memoryMode=(IMMUTABLE_DATA\|REUSABLE_DATA)"`
	`107`	`+ + "memoryMode=REUSABLE_DATA"`
`108`	`108`	`+ "\\}");`
`109`	`109`	`}`
`110`	`110`	`}`