Coverage detection

.github/workflows/go.yml

@@ -22,7 +22,7 @@ jobs:
         run: go get -v -t -d ./...
 
       - name: Test
-        run: go test -v -race -covermode=atomic -coverprofile=cov ./...
+        run: go test -v -race -covermode=atomic ./...
         env:
           SCOPE_DSN: ${{ secrets.SCOPE_DSN }}
 

instrumentation/testing/coverage.go

@@ -83,6 +83,9 @@ func initCoverage() {
 func startCoverage() {
 	countersMutex.Lock()
 	defer countersMutex.Unlock()
+	if cover.Mode == "" {
+		return
+	}
 	initCoverage()
 
 	for name, counts := range cover.Counters {
@@ -97,6 +100,9 @@ func startCoverage() {
 func restoreCoverageCounters() {
 	countersMutex.Lock()
 	defer countersMutex.Unlock()
+	if cover.Mode == "" {
+		return
+	}
 	for name, counts := range cover.Counters {
 		for i := range counts {
 			atomic.StoreUint32(&counts[i], counters[name][i]+atomic.LoadUint32(&counts[i]))
@@ -108,6 +114,9 @@ func restoreCoverageCounters() {
 func endCoverage() *coverage {
 	countersMutex.Lock()
 	defer countersMutex.Unlock()
+	if cover.Mode == "" {
+		return nil
+	}
 
 	var covSource = map[string][]*blockWithCount{}
 	for name, counts := range cover.Counters {

instrumentation/testing/testing.go

@@ -161,13 +161,14 @@ func (test *Test) end() {
 		LogRecords: logRecords,
 	}
 
-	// Checks if the current test is running parallel to extract the coverage or not
-	if reflection.GetIsParallel(test.t) {
-		instrumentation.Logger().Printf("CodePath in parallel test is not supported: %v\n", test.t.Name())
-		restoreCoverageCounters()
-	} else {
-		cov := endCoverage()
-		test.span.SetTag(tags.Coverage, *cov)
+	if testing.CoverMode() != "" {
+		// Checks if the current test is running parallel to extract the coverage or not
+		if reflection.GetIsParallel(test.t) {
+			instrumentation.Logger().Printf("CodePath in parallel test is not supported: %v\n", test.t.Name())
+			restoreCoverageCounters()
+		} else if cov := endCoverage(); cov != nil {
+			test.span.SetTag(tags.Coverage, *cov)
+		}
 	}
 
 	if r := recover(); r != nil {