g

internal/mx/mx_test.go

@@ -408,52 +408,7 @@ func TestMux(t *testing.T) {
 	})
 }
 
-func getManyRoutes(b *testing.B) []Route {
-	b.Helper()
-
-	routes := []Route{}
-
-	for i := 0; i <= 200; i++ {
-		uri := fmt.Sprintf("uri-%d", i)
-		rt, err := NewRoute(
-			uri,
-			MethodAll,
-			someMuxHandler(uri),
-		)
-		attest.Ok(b, err)
-		routes = append(
-			routes,
-			rt,
-		)
-	}
-
-	return routes
-}
-
-var result Muxer //nolint:gochecknoglobals
-
-func BenchmarkMuxNew(b *testing.B) {
-	var r Muxer
-
-	l := log.New(context.Background(), &bytes.Buffer{}, 500)
-
-	b.ReportAllocs()
-	b.ResetTimer()
-	for range b.N {
-		mux, err := New(
-			config.WithOpts("localhost", 443, tst.SecretKey(), config.DirectIpStrategy, l),
-			nil,
-			getManyRoutes(b)...,
-		)
-		attest.Ok(b, err)
-		r = mux
-	}
-	// always store the result to a package level variable
-	// so the compiler cannot eliminate the Benchmark itself.
-	result = r
-}
-
-func TestMuxKomu(t *testing.T) {
+func TestMuxFlexiblePattern(t *testing.T) {
 	t.Parallel()
 
 	tr := &http.Transport{
@@ -467,7 +422,7 @@ func TestMuxKomu(t *testing.T) {
 
 	l := log.New(context.Background(), &bytes.Buffer{}, 500)
 
-	t.Run("todo", func(t *testing.T) {
+	t.Run("flexible pattern accepts all uris", func(t *testing.T) {
 		t.Parallel()
 
 		msg := "hello world"
@@ -510,4 +465,78 @@ func TestMuxKomu(t *testing.T) {
 			attest.Equal(t, string(rb), msg)
 		}
 	})
+
+	t.Run("conflict", func(t *testing.T) {
+		t.Parallel()
+
+		msg := "hello world"
+
+		rt1, err := NewRoute(
+			"/*",
+			MethodGet,
+			someMuxHandler(msg),
+		)
+		attest.Ok(t, err)
+
+		rt2, err := NewRoute(
+			"/hi",
+			MethodGet,
+			thisIsAnotherMuxHandler(),
+		)
+		attest.Ok(t, err)
+
+		_, err = New(
+			config.WithOpts(domain, httpsPort, tst.SecretKey(), config.DirectIpStrategy, l),
+			nil,
+			rt1,
+			rt2,
+		)
+		attest.Error(t, err)
+		attest.Subsequence(t, err.Error(), "would conflict")
+	})
+}
+
+func getManyRoutes(b *testing.B) []Route {
+	b.Helper()
+
+	routes := []Route{}
+
+	for i := 0; i <= 200; i++ {
+		uri := fmt.Sprintf("uri-%d", i)
+		rt, err := NewRoute(
+			uri,
+			MethodAll,
+			someMuxHandler(uri),
+		)
+		attest.Ok(b, err)
+		routes = append(
+			routes,
+			rt,
+		)
+	}
+
+	return routes
+}
+
+var result Muxer //nolint:gochecknoglobals
+
+func BenchmarkMuxNew(b *testing.B) {
+	var r Muxer
+
+	l := log.New(context.Background(), &bytes.Buffer{}, 500)
+
+	b.ReportAllocs()
+	b.ResetTimer()
+	for range b.N {
+		mux, err := New(
+			config.WithOpts("localhost", 443, tst.SecretKey(), config.DirectIpStrategy, l),
+			nil,
+			getManyRoutes(b)...,
+		)
+		attest.Ok(b, err)
+		r = mux
+	}
+	// always store the result to a package level variable
+	// so the compiler cannot eliminate the Benchmark itself.
+	result = r
 }