Skip to content

Commit 3652c49

Browse files
committed
Fix test
1 parent 6be703a commit 3652c49

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

services/gitdiff/gitdiff_test.go

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,7 @@ func TestParsePatch_singlefile(t *testing.T) {
216216
wantErr bool
217217
addition int
218218
deletion int
219+
Type DiffFileType
219220
oldFilename string
220221
filename string
221222
}
@@ -237,6 +238,7 @@ func TestParsePatch_singlefile(t *testing.T) {
237238
`,
238239
addition: 4,
239240
deletion: 1,
241+
Type: DiffFileChange,
240242
filename: "README.md",
241243
oldFilename: "README.md",
242244
},
@@ -255,6 +257,7 @@ func TestParsePatch_singlefile(t *testing.T) {
255257
+ cut off`,
256258
addition: 4,
257259
deletion: 1,
260+
Type: DiffFileChange,
258261
filename: "A \\ B",
259262
oldFilename: "A \\ B",
260263
},
@@ -271,6 +274,7 @@ index d2186f1..f5c8ed2 100644
271274
\ No newline at end of file`,
272275
addition: 0,
273276
deletion: 1,
277+
Type: DiffFileChange,
274278
filename: "a b/file b/a a/file",
275279
oldFilename: "a b/file b/a a/file",
276280
},
@@ -290,6 +294,7 @@ index 898651a..0000000
290294
`,
291295
addition: 0,
292296
deletion: 5,
297+
Type: DiffFileDel,
293298
filename: "file with blanks",
294299
oldFilename: "file with blanks",
295300
},
@@ -302,6 +307,7 @@ rename to "a\342\200\224as"
302307
`,
303308
addition: 0,
304309
deletion: 0,
310+
Type: DiffFileRename,
305311
oldFilename: "𣐵b†vs",
306312
filename: "a—as",
307313
},
@@ -312,6 +318,7 @@ similarity index 100%
312318
rename from a b/file b/a a/file
313319
rename to a b/a a/file b/b file
314320
`,
321+
Type: DiffFileRename,
315322
oldFilename: "a b/file b/a a/file",
316323
filename: "a b/a a/file b/b file",
317324
},
@@ -325,6 +332,7 @@ index 92e798b..0000000
325332
@@ -1 +0,0 @@
326333
-b b/b
327334
`,
335+
Type: DiffFileDel,
328336
oldFilename: "b b/b",
329337
filename: "b b/b",
330338
addition: 0,
@@ -340,7 +348,8 @@ index 0000000..92e798b
340348
@@ -0,0 +1 @@
341349
+b b/b
342350
`,
343-
oldFilename: "b b/b",
351+
Type: DiffFileAdd,
352+
oldFilename: "",
344353
filename: "b b/b",
345354
addition: 1,
346355
deletion: 0,
@@ -352,6 +361,7 @@ similarity index 100%
352361
rename from b b/b b/b b/b b/b
353362
rename to b
354363
`,
364+
Type: DiffFileRename,
355365
oldFilename: "b b/b b/b b/b b/b",
356366
filename: "b",
357367
},
@@ -362,6 +372,7 @@ similarity index 100%
362372
rename from b b/b b/b b/b b/b
363373
rename to b
364374
`,
375+
Type: DiffFileRename,
365376
oldFilename: "b b/b b/b b/b b/b",
366377
filename: "b",
367378
},
@@ -372,6 +383,7 @@ similarity index 100%
372383
rename from b b/b b/b b/b
373384
rename to b b/b
374385
`,
386+
Type: DiffFileRename,
375387
oldFilename: "b b/b b/b b/b",
376388
filename: "b b/b",
377389
},
@@ -391,6 +403,7 @@ index 6961180..9ba1a00 100644
391403
+++ 3rd line
392404
+-- 4th line
393405
`,
406+
Type: DiffFileChange,
394407
oldFilename: "minuses-and-pluses",
395408
filename: "minuses-and-pluses",
396409
addition: 4,
@@ -418,6 +431,9 @@ index 6961180..9ba1a00 100644
418431
t.Errorf("ParsePath(%q) did not have correct totalDeletion %d, wanted %d", testcase.name, got.TotalDeletion, testcase.deletion)
419432
}
420433
file := got.Files[0]
434+
if file.Type != testcase.Type {
435+
t.Errorf("ParsePath(%q) did not have correct Type %d, wanted %d", testcase.name, file.Type, testcase.Type)
436+
}
421437
if file.Addition != testcase.addition {
422438
t.Errorf("ParsePath(%q) does not have correct file addition %d, wanted %d", testcase.name, file.Addition, testcase.addition)
423439
}

0 commit comments

Comments
 (0)