Skip to content

Commit 2a672c5

Browse files
authored
style: clean up test code formatting for improved readability (#198)
- Remove unnecessary blank lines from test functions for improved code clarity Signed-off-by: appleboy <appleboy.tw@gmail.com>
1 parent 5e8b5b0 commit 2a672c5

File tree

2 files changed

+22
-24
lines changed

2 files changed

+22
-24
lines changed

internal/ai/ollama_test.go

Lines changed: 21 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,8 @@ func TestNewOllamaProvider_InvalidEndpoints(t *testing.T) {
119119
cfg := &Config{
120120
Endpoint: tt.endpoint,
121121
Model: "test-model",
122-
MinCases: 3,
123-
MaxCases: 3,
122+
MinCases: 3,
123+
MaxCases: 3,
124124
MaxRetries: 3,
125125
RequestTimeout: 60,
126126
HealthTimeout: 2,
@@ -192,8 +192,8 @@ func TestOllamaProvider_IsAvailable(t *testing.T) {
192192
provider, err := NewOllamaProvider(&Config{
193193
Endpoint: server.URL,
194194
Model: "test-model",
195-
MinCases: 3,
196-
MaxCases: 3,
195+
MinCases: 3,
196+
MaxCases: 3,
197197
MaxRetries: 3,
198198
RequestTimeout: 60,
199199
HealthTimeout: 2,
@@ -214,8 +214,8 @@ func TestOllamaProvider_IsAvailable(t *testing.T) {
214214
provider, err := NewOllamaProvider(&Config{
215215
Endpoint: "http://invalid-host-that-does-not-exist:9999",
216216
Model: "test-model",
217-
MinCases: 3,
218-
MaxCases: 3,
217+
MinCases: 3,
218+
MaxCases: 3,
219219
MaxRetries: 3,
220220
RequestTimeout: 60,
221221
HealthTimeout: 2,
@@ -236,7 +236,7 @@ func TestOllamaProvider_generateGo(t *testing.T) {
236236
responseBody map[string]interface{}
237237
statusCode int
238238
wantErr bool
239-
wantCases int
239+
wantCases int
240240
validateResult func(*testing.T, []TestCase)
241241
}{
242242
{
@@ -270,9 +270,9 @@ func TestAdd(t *testing.T) {
270270
}
271271
}`,
272272
},
273-
statusCode: http.StatusOK,
274-
wantErr: false,
275-
wantCases: 2,
273+
statusCode: http.StatusOK,
274+
wantErr: false,
275+
wantCases: 2,
276276
validateResult: func(t *testing.T, cases []TestCase) {
277277
if len(cases) != 2 {
278278
t.Errorf("got %d cases, want 2", len(cases))
@@ -307,9 +307,9 @@ func TestAdd(t *testing.T) {
307307
responseBody: map[string]interface{}{
308308
"response": "```go\npackage testdata\n\nimport \"testing\"\n\ntype args struct { x int }\n\nfunc Test(t *testing.T) {\n\ttests := []struct {\n\t\tname string\n\t\targs args\n\t\twant int\n\t}{\n\t\t{name: \"test1\", args: args{x: 10}, want: 20},\n\t}\n}\n```",
309309
},
310-
statusCode: http.StatusOK,
311-
wantErr: false,
312-
wantCases: 1,
310+
statusCode: http.StatusOK,
311+
wantErr: false,
312+
wantCases: 1,
313313
validateResult: func(t *testing.T, cases []TestCase) {
314314
if len(cases) != 1 {
315315
t.Errorf("got %d cases, want 1", len(cases))
@@ -337,8 +337,8 @@ func TestAdd(t *testing.T) {
337337
provider, err := NewOllamaProvider(&Config{
338338
Endpoint: server.URL,
339339
Model: "test-model",
340-
MinCases: 3,
341-
MaxCases: 3,
340+
MinCases: 3,
341+
MaxCases: 3,
342342
MaxRetries: 3,
343343
RequestTimeout: 60,
344344
HealthTimeout: 2,
@@ -407,8 +407,8 @@ func TestAdd(t *testing.T) {
407407
provider, err := NewOllamaProvider(&Config{
408408
Endpoint: server.URL,
409409
Model: "test-model",
410-
MinCases: 3,
411-
MaxCases: 3,
410+
MinCases: 3,
411+
MaxCases: 3,
412412
MaxRetries: 3,
413413
RequestTimeout: 60,
414414
HealthTimeout: 2,
@@ -430,7 +430,6 @@ func TestAdd(t *testing.T) {
430430

431431
ctx := context.Background()
432432
cases, err := provider.GenerateTestCases(ctx, fn)
433-
434433
if err != nil {
435434
t.Fatalf("GenerateTestCases() error = %v", err)
436435
}
@@ -484,8 +483,8 @@ func Test(t *testing.T) {
484483
provider, err := NewOllamaProvider(&Config{
485484
Endpoint: server.URL,
486485
Model: "test-model",
487-
MinCases: 3,
488-
MaxCases: 3,
486+
MinCases: 3,
487+
MaxCases: 3,
489488
MaxRetries: 3,
490489
RequestTimeout: 60,
491490
HealthTimeout: 2,
@@ -506,7 +505,6 @@ func Test(t *testing.T) {
506505

507506
ctx := context.Background()
508507
cases, err := provider.GenerateTestCases(ctx, fn)
509-
510508
if err != nil {
511509
t.Fatalf("GenerateTestCases() error = %v", err)
512510
}
@@ -534,8 +532,8 @@ func Test(t *testing.T) {
534532
provider, err := NewOllamaProvider(&Config{
535533
Endpoint: server.URL,
536534
Model: "test-model",
537-
MinCases: 3,
538-
MaxCases: 3,
535+
MinCases: 3,
536+
MaxCases: 3,
539537
MaxRetries: 3,
540538
RequestTimeout: 60,
541539
HealthTimeout: 2,

internal/ai/validator_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ func TestTypeError(t *testing.T) {
157157
errMsg: "type check error",
158158
},
159159
{
160-
name: "empty_test_code",
160+
name: "empty_test_code",
161161
testCode: ``,
162162
pkgName: "testpkg",
163163
wantErr: true,

0 commit comments

Comments
 (0)