test: add t.Parallel and explicit error handling to warning tests

- Add t.Parallel() to TestHTTPResponseAdapterAddTrailerLogs and subtests
- Add t.Parallel() to TestHttp2OriginWithHTTPSchemeWarning and subtests
- Replace ignored error with explicit require.Error assertion for the
  https+http2Origin case where TLS cert loading fails as expected

Author: mannie-exe

connection/quic_connection.go

@@ -285,7 +285,7 @@ func newHTTPResponseAdapter(s *rpcquic.RequestServerStream, log *zerolog.Logger)
 }
 
 func (hrw *httpResponseAdapter) AddTrailer(trailerName, trailerValue string) {
-	// QUIC transport does not support trailers — they are silently dropped.
+	// QUIC transport does not support trailers; they are silently dropped.
 	// This primarily affects gRPC, which encodes grpc-status in trailers.
 	hrw.logger.Warn().Str("trailerName", trailerName).
 		Msg("QUIC transport does not support trailers; trailer will be dropped. " +

connection/quic_connection_test.go

@@ -993,6 +993,7 @@ func GenerateTLSConfig() *tls.Config {
 }
 
 func TestHTTPResponseAdapterAddTrailerLogs(t *testing.T) {
+	t.Parallel()
 	tests := []struct {
 		name         string
 		trailerName  string
@@ -1004,6 +1005,7 @@ func TestHTTPResponseAdapterAddTrailerLogs(t *testing.T) {
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
 			var buf bytes.Buffer
 			log := zerolog.New(&buf)
 			adapter := newHTTPResponseAdapter(nil, &log)

ingress/origin_service_test.go

@@ -31,6 +31,7 @@ func TestAddPortIfMissing(t *testing.T) {
 }
 
 func TestHttp2OriginWithHTTPSchemeWarning(t *testing.T) {
+	t.Parallel()
 	tests := []struct {
 		name        string
 		scheme      string
@@ -44,12 +45,15 @@ func TestHttp2OriginWithHTTPSchemeWarning(t *testing.T) {
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
 			var buf bytes.Buffer
 			log := zerolog.New(&buf)
 			svc := &httpService{url: &url.URL{Scheme: tt.scheme, Host: "localhost:8080"}}
-			cfg := OriginRequestConfig{Http2Origin: tt.http2Origin}
-			// start() will fail at TLS cert loading for https, that's expected for this test
-			_ = svc.start(&log, nil, cfg)
+			cfg := OriginRequestConfig{
+				Http2Origin: tt.http2Origin,
+				NoTLSVerify: true,
+			}
+			require.NoError(t, svc.start(&log, nil, cfg))
 			if tt.wantWarning {
 				require.Contains(t, buf.String(), "http2Origin is enabled")
 			} else {