Test and fix multi-version case

internal/graph/graph.go

@@ -35,7 +35,7 @@ var (
 	// See tests for examples.
 	goRegExp = regexp.MustCompile(`^(?:[\w\-\.]+\/)+(.+)`)
 	// Removes potential module versions in a package path.
-	goVerRegExp = regexp.MustCompile(`^(.*)/v(?:[2-9]|[1-9][0-9]+)([./].*)$`)
+	goVerRegExp = regexp.MustCompile(`^(.*?)/v(?:[2-9]|[1-9][0-9]+)([./].*)$`)
 	// Strips C++ namespace prefix from a C++ function / method name.
 	// NOTE: Make sure to keep the template parameters in the name. Normally,
 	// template parameters are stripped from the C++ names but when

internal/graph/graph_test.go

@@ -475,6 +475,14 @@ func TestShortenFunctionName(t *testing.T) {
 			"example.org/v2xyz.Foo",
 			"v2xyz.Foo",
 		},
+		{
+			"github.com/foo/bar/v4/v4.(*Foo).Bar",
+			"v4.(*Foo).Bar",
+		},
+		{
+			"github.com/foo/bar/v4/foo/bar/v4.(*Foo).Bar",
+			"v4.(*Foo).Bar",
+		},
 		{
 			"java.util.concurrent.ThreadPoolExecutor$Worker.run",
 			"ThreadPoolExecutor$Worker.run",