ecodeclub / ecache

统一缓存 API
Apache License 2.0
12 stars 15 forks source link

测试代码重复问题 #26

Open longyue0521 opened 1 year ago

longyue0521 commented 1 year ago

仅限中文

在提之前请先查找已有 issues,避免重复上报。

并且确保自己已经:

你的问题

当前仓库中测试代码有很多重复,在我提交的PR #25 ,我试着通过传入不同的具体实现来得到ecode.Cache实例, 然后针对ecode.Cache实例编写测试代码发现只需要一套e2e测试代码即可

详见memory/lru/cache_test.go的TestCache_Delete与redis/cache_e2e_test.go的TestCache_e2e_Delete. 经过抽象两者只有newCache的实现不一样.

建议使用testify维护一套测试e2e测试套件针对ecode.Cache实例测试即可, 然后传入不同的具体实例调用同一套测试套件即可.

你使用的是 ecache 哪个版本?

你设置的的 Go 环境?

上传 go env 的结果

flycash commented 4 months ago

这个确实是需要重构的