update app integration test result processing

The TestApplicationGoCounter in app_test.go is a little flaky,
occasionally failing. Seems to be failing more often with the recent
changes in bpfman. Turns out the log processing was always looking at
the oldest retrieved log, so if it didn't pass on the first read, it
would never pass. It was just random timing if the first log contained
packet counts or not.

Signed-off-by: Billy McFall <[email protected]>

Author: Billy99

test/integration/app_test.go

@@ -69,16 +69,17 @@ func TestApplicationGoCounter(t *testing.T) {
 		require.Eventually(t, func() bool {
 			logs, err := req.Stream(ctx)
 			require.NoError(t, err)
-			defer logs.Close()
 			output := new(bytes.Buffer)
 			_, err = io.Copy(output, logs)
 			require.NoError(t, err)
+			logs.Close()
 
 			if f(t, output) {
 				return true
 			}
 			t.Logf("check %d failed retrying, output %s", idx, output.String())
 			return false
 		}, 30*time.Second, time.Second)
+		req.Close()
 	}
 }

test/integration/common.go

@@ -121,8 +121,9 @@ func doAppXdpCheck(t *testing.T, output *bytes.Buffer) bool {
 func doProbeCommonCheck(t *testing.T, output *bytes.Buffer, str string) (bool, int) {
 	want := regexp.MustCompile(str)
 	matches := want.FindAllStringSubmatch(output.String(), -1)
-	if len(matches) >= 1 && len(matches[0]) >= 2 {
-		count, err := strconv.Atoi(matches[0][1])
+	numMatches := len(matches)
+	if numMatches >= 1 && len(matches[numMatches-1]) >= 2 {
+		count, err := strconv.Atoi(matches[numMatches-1][1])
 		require.NoError(t, err)
 		if count > 0 {
 			return true, count