talent-plan/tinykv

test-case疑问

NicoleRobin opened this issue · 3 comments

NicoleRobin commented

tinykv/kv/server/server_test.go

Line 199 in f251031

assert.Equal(t, nil, err)

这个测试用例是不是应该改为判断err是否等于KeyNotFound，因为前面已经执行过删除了，所以此时再去查询应该是key不存在的。

virtuosoo commented

我也觉得

Connor1996 commented

Please check the comment of GetCF()

NicoleRobin commented

OK, thank you for you advice, i will check this function's comment again.