给 redis_exporter 提交了一个 pull request，老是被作者打回来，求指点目前这个 test 要怎么写？
Ansen · 2021-01-07 11:14:50 +08:00 · 1134 次点击这是一个创建于 1197 天前的主题，其中的信息可能已经有所发展或是发生改变。
[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
 |
1
Thetruechar 2021-01-07 12:20:50 +08:00 via Android
开源项目测试看的很重 合进去不能降低项目的测试覆盖率是底线
|
 |
2
Ansen OP @Thetruechar #1 嗯，能理解，但是作为小白的我，技术有点跟不上了
 |
|
3
Thetruechar 2021-01-07 15:31:23 +08:00 via Android
@Ansen 写个测试就技术跟不上了？这就是个开发理念问题 估计你平时开发功能也不写测试的吧 写完拉到
|
 |
4
shyling 2021-01-07 15:36:49 +08:00
感觉作者挺用心的
|
 |
5
emeab 2021-01-07 15:43:27 +08:00
看了眼 pr 感觉作者几乎是带着楼主写了..
|
|
6
Ansen OP @Thetruechar #3
确实没有写测试的习惯，加上我并不是开发，运维一枚，平时也就写点 python 脚本什么的 @shyling #4 @emeab #5 嗯，作者非常用心，所以我很过意不去，来 V 站求助 |
|
7
Thetruechar 2021-01-07 15:50:17 +08:00 via Android
@Ansen 走运维开发这条路不错 很吃香
|
|
8
Ansen OP @Thetruechar #7 是打算走这条路，目前 go 勉强算是半只脚入门吧，这次 pull 的事件，学会了很多，但是单元测试这块确实没接触过，原作者非常用心，但我也不好一直麻烦他，就想着来 V 站请教大家，看看能不能找到解决方案
|
Select Language