Skip to content

Commit d2d5dec

Browse files
committed
fix nil access
1 parent 668b22f commit d2d5dec

3 files changed

Lines changed: 20 additions & 0 deletions

File tree

go/internal/e2e/mcp_and_agents_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,10 @@ func TestMCPServers(t *testing.T) {
9696
t.Fatalf("Failed to send message: %v", err)
9797
}
9898

99+
if message == nil {
100+
t.Fatalf("Expected a message, got nil")
101+
}
102+
99103
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "6") {
100104
t.Errorf("Expected message to contain '6', got: %v", message.Data.Content)
101105
}
@@ -226,6 +230,10 @@ func TestCustomAgents(t *testing.T) {
226230
t.Fatalf("Failed to send message: %v", err)
227231
}
228232

233+
if message == nil {
234+
t.Fatalf("Expected a message, got nil")
235+
}
236+
229237
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "12") {
230238
t.Errorf("Expected message to contain '12', got: %v", message.Data.Content)
231239
}

go/internal/e2e/permissions_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,10 @@ func TestPermissions(t *testing.T) {
165165
t.Fatalf("Failed to send message: %v", err)
166166
}
167167

168+
if message == nil {
169+
t.Fatal("Expected a message, got nil")
170+
}
171+
168172
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "4") {
169173
t.Errorf("Expected message to contain '4', got: %v", message.Data.Content)
170174
}

go/internal/e2e/skills_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,10 @@ func TestSkills(t *testing.T) {
133133
t.Fatalf("Failed to send message: %v", err)
134134
}
135135

136+
if message1 == nil {
137+
t.Fatalf("Expected a message, got nil")
138+
}
139+
136140
if message1.Data.Content != nil && strings.Contains(*message1.Data.Content, skillMarker) {
137141
t.Errorf("Expected message to NOT contain skill marker before skill was added, got: %v", *message1.Data.Content)
138142
}
@@ -155,6 +159,10 @@ func TestSkills(t *testing.T) {
155159
t.Fatalf("Failed to send message: %v", err)
156160
}
157161

162+
if message2 == nil {
163+
t.Fatalf("Expected a message, got nil")
164+
}
165+
158166
if message2.Data.Content == nil || !strings.Contains(*message2.Data.Content, skillMarker) {
159167
t.Errorf("Expected message to contain skill marker '%s' after resume, got: %v", skillMarker, message2.Data.Content)
160168
}

0 commit comments

Comments
 (0)