Skip to content

Commit 74cc463

Browse files
committed
go/token: add TestRemovedFileFileReturnsNil test
While debugging test issue in the previous CL i noted that we don't have a proper test for RemoveFile. Change-Id: I6a6a6964426ed3cf6725a58ec377686c2900c626 Reviewed-on: https://go-review.googlesource.com/c/go/+/705757 Reviewed-by: Alan Donovan <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Junyang Shao <[email protected]>
1 parent 902dc27 commit 74cc463

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

src/go/token/position_test.go

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -621,3 +621,25 @@ func TestRemoveFileRace(t *testing.T) {
621621
start <- struct{}{}
622622
}
623623
}
624+
625+
func TestRemovedFileFileReturnsNil(t *testing.T) {
626+
fset := NewFileSet()
627+
628+
// Create bunch of files.
629+
var files []*File
630+
for i := range 1000 {
631+
f := fset.AddFile("f", -1, (i+1)*100)
632+
files = append(files, f)
633+
}
634+
635+
rand.Shuffle(len(files), func(i, j int) {
636+
files[i], files[j] = files[j], files[i]
637+
})
638+
639+
for _, f := range files {
640+
fset.RemoveFile(f)
641+
if got := fset.File(Pos(f.Base()) + 10); got != nil {
642+
t.Fatalf("file was not removed correctly; got file with base: %v", got.Base())
643+
}
644+
}
645+
}

0 commit comments

Comments
 (0)