sub test and sub benchmark auto instrumentation

Author: tonyredondo

instrumentation/testing/go_benchmark.go

@@ -74,7 +74,8 @@ func (b *goB) run() bool
 //go:linkname (*goB).add testing.(*B).add
 func (b *goB) add(other testing.BenchmarkResult)
 
-// we clone the same (*testing.B).Run implementation because the Patch overwrite the original implementation with the jump
+// we clone the same (*testing.B).Run implementation because the Patch
+// overwrites the original implementation with the jump
 func (b *goB) Run(name string, f func(b *testing.B)) bool {
 	atomic.StoreInt32(&b.hasSub, 1)
 	benchmarkLock.Unlock()

instrumentation/testing/go_testing.go

@@ -78,7 +78,8 @@ func tRunner(t *goT, fn func(t *goT)) {
 	goTRunner(t.ToTestingT(), func(t *testing.T) { fn(FromTestingT(t)) })
 }
 
-// we clone the same (*testing.T).Run implementation because the Patch overwrite the original implementation with the jump
+// we clone the same (*testing.T).Run implementation because the Patch
+// overwrites the original implementation with the jump
 func (t *goT) Run(name string, f func(t *goT)) bool {
 	atomic.StoreInt32(&t.hasSub, 1)
 	testName, ok, _ := t.context.match.fullName(&t.goCommon, name)

instrumentation/testing/logger.go

@@ -161,7 +161,6 @@ func patch(methodName string, methodBody func(test *Test, argsValues []reflect.V
 func logOnError(err error) {
 	if err != nil {
 		instrumentation.Logger().Println(err)
-		fmt.Println(err)
 	}
 }