Fix test

gjulianm · gjulianm · commit abd697b3db8c · 2025-06-30T05:19:11.000-07:00
diff --git a/pkg/ebpf/uprobes/attacher_test.go b/pkg/ebpf/uprobes/attacher_test.go
@@ -1095,15 +1095,15 @@ func (s *SharedLibrarySuite) TestMultipleLibsets() {
 				}
 			},
 			3, 10*time.Millisecond, 500*time.Millisecond, "did not catch %s process, received calls %v", tc.description, mockRegistry.Calls)
+
+		require.NotNil(t, cmd)
+		require.NotNil(t, cmd.Process)
 		commands = append(commands, cmd)
 	}
 
 	for i, cmd := range commands {
 		mockRegistry.AssertCalled(t, "Register", testCases[i].libPath, uint32(cmd.Process.Pid), mock.Anything, mock.Anything, mock.Anything)
-
-		if cmd != nil && cmd.Process != nil {
-			cmd.Process.Kill()
-		}
+		cmd.Process.Kill()
 	}
 
 	// Verify unregister calls for all processes

Original file line number	Diff line number	Diff line change
`@@ -1095,15 +1095,15 @@ func (s *SharedLibrarySuite) TestMultipleLibsets() {`
`1095`	`1095`	`}`
`1096`	`1096`	`},`
`1097`	`1097`	`3, 10time.Millisecond, 500time.Millisecond, "did not catch %s process, received calls %v", tc.description, mockRegistry.Calls)`
	`1098`	`+`
	`1099`	`+ require.NotNil(t, cmd)`
	`1100`	`+ require.NotNil(t, cmd.Process)`
`1098`	`1101`	`commands = append(commands, cmd)`
`1099`	`1102`	`}`
`1100`	`1103`
`1101`	`1104`	`for i, cmd := range commands {`
`1102`	`1105`	`mockRegistry.AssertCalled(t, "Register", testCases[i].libPath, uint32(cmd.Process.Pid), mock.Anything, mock.Anything, mock.Anything)`
`1103`		`-`
`1104`		`- if cmd != nil && cmd.Process != nil {`
`1105`		`- cmd.Process.Kill()`
`1106`		`- }`
	`1106`	`+ cmd.Process.Kill()`
`1107`	`1107`	`}`
`1108`	`1108`
`1109`	`1109`	`// Verify unregister calls for all processes`