rw2senderfix: rename and skip all "old" tests all by default

remotewrite/sender/backoff_test.go

@@ -53,7 +53,9 @@ func (ttr *TimestampTrackingReceiver) GetTimestamps() []time.Time {
 }
 
 // TestBackoffBehavior validates backoff and exponential backoff implementation.
-func TestBackoffBehavior(t *testing.T) {
+func TestBackoffBehavior_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []struct {
 		name        string
 		description string

remotewrite/sender/batching_test.go

@@ -23,7 +23,8 @@ import (
 )
 
 // TestBatchingBehavior validates sender batching and queueing behavior.
-func TestBatchingBehavior(t *testing.T) {
+func TestBatchingBehavior_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
 	tests := []TestCase{
 		{
 			Name:        "multiple_series_per_request",
@@ -176,7 +177,9 @@ memory_usage_bytes 1048576
 }
 
 // TestConcurrentRequests validates parallel request handling.
-func TestConcurrentRequests(t *testing.T) {
+func TestConcurrentRequests_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "MAY")
 	t.Attr("description", "Sender MAY send multiple requests in parallel")
 

remotewrite/sender/combined_test.go

@@ -21,7 +21,9 @@ import (
 )
 
 // TestCombinedFeatures validates integration of multiple Remote Write 2.0 features.
-func TestCombinedFeatures(t *testing.T) {
+func TestCombinedFeatures_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "samples_with_metadata",

remotewrite/sender/edge_cases_test.go

@@ -23,7 +23,9 @@ import (
 )
 
 // TestEdgeCases validates sender behavior in edge case scenarios.
-func TestEdgeCases(t *testing.T) {
+func TestEdgeCases_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "empty_scrape",
@@ -360,7 +362,9 @@ rpc_duration_count 1000
 }
 
 // TestRobustnessUnderLoad validates sender behavior under stress.
-func TestRobustnessUnderLoad(t *testing.T) {
+func TestRobustnessUnderLoad_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "SHOULD")
 	t.Attr("description", "Sender SHOULD remain stable under load")
 

remotewrite/sender/error_handling_test.go

@@ -23,7 +23,9 @@ import (
 )
 
 // TestErrorHandling validates sender error handling in various failure scenarios.
-func TestErrorHandling(t *testing.T) {
+func TestErrorHandling_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []struct {
 		name        string
 		description string
@@ -195,7 +197,9 @@ func TestErrorHandling(t *testing.T) {
 }
 
 // TestNetworkErrors validates handling of network-level errors.
-func TestNetworkErrors(t *testing.T) {
+func TestNetworkErrors_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "SHOULD")
 	t.Attr("description", "Sender SHOULD handle network errors gracefully")
 

remotewrite/sender/exemplars_test.go

@@ -19,7 +19,9 @@ import (
 )
 
 // TestExemplarEncoding validates exemplar encoding in Remote Write 2.0.
-func TestExemplarEncoding(t *testing.T) {
+func TestExemplarEncoding_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "exemplar_with_trace_id",

remotewrite/sender/fallback_test.go

@@ -66,7 +66,9 @@ func (ftr *FallbackTrackingReceiver) ShouldReturn415() bool {
 }
 
 // TestFallbackBehavior validates RW 2.0 to RW 1.0 fallback on 415 response.
-func TestFallbackBehavior(t *testing.T) {
+func TestFallbackBehavior_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []struct {
 		name        string
 		description string
@@ -282,7 +284,9 @@ func TestFallbackBehavior(t *testing.T) {
 }
 
 // TestNoFallbackOn2xx validates that fallback doesn't happen on success.
-func TestNoFallbackOn2xx(t *testing.T) {
+func TestNoFallbackOn2xx_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "MUST")
 	t.Attr("description", "Sender MUST NOT fallback when receiving 2xx success responses")
 

remotewrite/sender/histograms_test.go

@@ -18,7 +18,9 @@ import (
 )
 
 // TestHistogramEncoding validates native histogram encoding.
-func TestHistogramEncoding(t *testing.T) {
+func TestHistogramEncoding_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "native_histogram_structure",

remotewrite/sender/labels_test.go

@@ -20,7 +20,9 @@ import (
 )
 
 // TestLabelValidation validates label encoding and formatting.
-func TestLabelValidation(t *testing.T) {
+func TestLabelValidation_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "label_lexicographic_ordering",

remotewrite/sender/metadata_test.go

@@ -21,7 +21,9 @@ import (
 )
 
 // TestMetadataEncoding validates metric metadata encoding.
-func TestMetadataEncoding(t *testing.T) {
+func TestMetadataEncoding_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "metadata_type_counter",

remotewrite/sender/protocol_test.go

@@ -22,7 +22,9 @@ import (
 )
 
 // TestProtocolCompliance validates HTTP protocol requirements for Remote Write 2.0 senders.
-func TestProtocolCompliance(t *testing.T) {
+func TestProtocolCompliance_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "content_type_protobuf",
@@ -130,7 +132,9 @@ func TestProtocolCompliance(t *testing.T) {
 }
 
 // TestHTTPMethod validates that senders use POST method for remote write.
-func TestHTTPMethod(t *testing.T) {
+func TestHTTPMethod_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "MUST")
 	t.Attr("description", "Sender MUST use POST method for remote write requests")
 

remotewrite/sender/response_test.go

@@ -23,7 +23,9 @@ import (
 )
 
 // TestResponseProcessing validates sender response header processing.
-func TestResponseProcessing(t *testing.T) {
+func TestResponseProcessing_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []struct {
 		name        string
 		description string
@@ -252,7 +254,9 @@ sample_3 3
 }
 
 // TestContentTypeNegotiation validates content-type handling.
-func TestContentTypeNegotiation(t *testing.T) {
+func TestContentTypeNegotiation_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "SHOULD")
 	t.Attr("description", "Sender SHOULD handle content-type negotiation")
 

remotewrite/sender/retry_test.go

@@ -23,7 +23,9 @@ import (
 )
 
 // TestRetryBehavior validates sender retry behavior on different error responses.
-func TestRetryBehavior(t *testing.T) {
+func TestRetryBehavior_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []struct {
 		name        string
 		description string

remotewrite/sender/rw1_compat_test.go

@@ -23,7 +23,9 @@ import (
 // TestRemoteWrite1Compatibility validates RW 1.0 backward compatibility.
 // Note: These tests require sender to be configured for RW 1.0 mode.
 // Most senders default to RW 2.0, so RW 1.0 tests are informational.
-func TestRemoteWrite1Compatibility(t *testing.T) {
+func TestRemoteWrite1Compatibility_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "rw1_version_header",
@@ -210,7 +212,9 @@ http_requests{method="POST",status="200"} 50
 }
 
 // TestRemoteWrite1Configuration tests if sender can be configured for RW 1.0.
-func TestRemoteWrite1Configuration(t *testing.T) {
+func TestRemoteWrite1Configuration_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "SHOULD")
 	t.Attr("description", "Sender SHOULD support RW 1.0 configuration for backward compatibility")
 

remotewrite/sender/samples_test.go

@@ -23,7 +23,9 @@ import (
 )
 
 // TestSampleEncoding validates that senders correctly encode float samples.
-func TestSampleEncoding(t *testing.T) {
+func TestSampleEncoding_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "float_value_encoding",
@@ -224,7 +226,9 @@ func TestSampleEncoding(t *testing.T) {
 }
 
 // TestSampleOrdering validates timestamp ordering in samples.
-func TestSampleOrdering(t *testing.T) {
+func TestSampleOrdering_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "MUST")
 	t.Attr("description", "Sender MUST send samples with older timestamps before newer ones within a series")
 

remotewrite/sender/symbols_test.go

@@ -21,7 +21,9 @@ import (
 )
 
 // TestSymbolTable validates symbol table requirements for Remote Write 2.0.
-func TestSymbolTable(t *testing.T) {
+func TestSymbolTable_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "empty_string_at_index_zero",
@@ -105,7 +107,9 @@ another_metric{foo="bar"} 3
 }
 
 // TestSymbolTableEfficiency validates that symbol tables are efficiently constructed.
-func TestSymbolTableEfficiency(t *testing.T) {
+func TestSymbolTableEfficiency_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	t.Attr("rfcLevel", "RECOMMENDED")
 	t.Attr("description", "Symbol table should be efficiently constructed with good compression")
 

remotewrite/sender/timestamps_test.go

@@ -19,7 +19,9 @@ import (
 )
 
 // TestTimestampEncoding validates timestamp encoding and handling.
-func TestTimestampEncoding(t *testing.T) {
+func TestTimestampEncoding_Old(t *testing.T) {
+	t.Skip("TODO: Revise and move to a new framework")
+
 	tests := []TestCase{
 		{
 			Name:        "timestamp_int64_milliseconds",