Skip to content

Commit f2958cb

Browse files
committed
test coverage
1 parent df02594 commit f2958cb

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

internal/resource/nginx_instance_process_operator_test.go

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,13 @@ import (
1010
"context"
1111
"errors"
1212
"fmt"
13+
"testing"
14+
"time"
15+
1316
"github.com/nginx/agent/v3/internal/datasource/host/exec/execfakes"
1417
"github.com/nginx/agent/v3/pkg/nginxprocess"
1518
"github.com/stretchr/testify/assert"
1619
"github.com/stretchr/testify/require"
17-
"testing"
18-
"time"
1920
)
2021

2122
func TestProcessOperator_FindParentProcessID(t *testing.T) {
@@ -31,12 +32,11 @@ func TestProcessOperator_FindParentProcessID(t *testing.T) {
3132
configure arguments: ` + configArgs
3233

3334
tests := []struct {
34-
expectedLog string
35-
name string
36-
instanceID string
37-
expectErr error
35+
name string // 16 bytes
36+
instanceID string // 16 bytes
37+
expectErr error // 16 bytes (interface)
38+
nginxProcesses []*nginxprocess.Process // 24 bytes (slice header)
3839
expectedPPID int32
39-
nginxProcesses []*nginxprocess.Process
4040
}{
4141
{
4242
name: "Test 1: Found parent process",
@@ -168,13 +168,12 @@ func TestProcessOperator_FindParentProcessID(t *testing.T) {
168168
ppid, err := processOperator.FindParentProcessID(ctx, test.instanceID, test.nginxProcesses, mockExec)
169169

170170
if test.expectErr != nil {
171-
assert.Error(tt, err)
171+
require.Error(tt, err)
172172
} else {
173173
require.NoError(tt, err)
174174
}
175175

176176
assert.Equal(tt, test.expectedPPID, ppid)
177-
178177
})
179178
}
180179
}

0 commit comments

Comments
 (0)