给 redis_exporter 提交了一个 pull request,老是被作者打回来,求指点目前这个 test 要怎么写?

2021-01-07 11:14:50 +08:00
 Ansen

[TOC]

背景

最近给部署了 Prometheus 监控系统,有用到 redis_exporter 这个组件,发现不同密码的 redis 实例 需要单独部署一个 exporter, 就用自己半吊子的 golang 水平 修改了一下源码,顺手提交了 pull 到官方,然后一次次的被打回来,原因当然是因为我代码写得太差了,作者也给也非常实用的建议,我也学到了很多

问题

由于之前没写过单元测试代码,所以目前卡在单元测试这里了,求指点

相关代码

官方库:https://github.com/oliver006/redis_exporter

我的 fork 地址:https://github.com/Ansen/redis_exporter

pull request 详情: https://github.com/oliver006/redis_exporter/pull/465

1140 次点击
所在节点    问与答
8 条回复
Thetruechar
2021-01-07 12:20:50 +08:00
开源项目测试看的很重 合进去不能降低项目的测试覆盖率是底线
Ansen
2021-01-07 12:47:15 +08:00
@Thetruechar #1 嗯,能理解,但是作为小白的我,技术有点跟不上了
Thetruechar
2021-01-07 15:31:23 +08:00
@Ansen 写个测试就技术跟不上了?这就是个开发理念问题 估计你平时开发功能也不写测试的吧 写完拉到
shyling
2021-01-07 15:36:49 +08:00
感觉作者挺用心的
emeab
2021-01-07 15:43:27 +08:00
看了眼 pr 感觉作者几乎是带着楼主写了..
Ansen
2021-01-07 15:47:05 +08:00
@Thetruechar #3

确实没有写测试的习惯,加上我并不是开发,运维一枚,平时也就写点 python 脚本什么的

@shyling #4
@emeab #5

嗯,作者非常用心,所以我很过意不去,来 V 站求助
Thetruechar
2021-01-07 15:50:17 +08:00
@Ansen 走运维开发这条路不错 很吃香
Ansen
2021-01-07 15:56:41 +08:00
@Thetruechar #7 是打算走这条路,目前 go 勉强算是半只脚入门吧,这次 pull 的事件,学会了很多,但是单元测试这块确实没接触过,原作者非常用心,但我也不好一直麻烦他,就想着来 V 站请教大家,看看能不能找到解决方案

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/742469

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX