给 redis_exporter 提交了一个 pull request,老是被作者打回来,求指点目前这个 test 要怎么写? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
Ansen
V2EX    问与答

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

  •  1
     
  •   Ansen 2021-01-07 11:14:50 +08:00 2007 次点击
    这是一个创建于 1804 天前的主题,其中的信息可能已经有所发展或是发生改变。

    [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

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

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

    @shyling #4
    @emeab #5

    嗯,作者非常用心,所以我很过意不去,来 V 站求助
    Thetruechar
        7
    Thetruechar  
       2021-01-07 15:50:17 +08:00 via Android
    @Ansen 走运维开发这条路不错 很吃香
    Ansen
        8
    Ansen  
    OP
       2021-01-07 15:56:41 +08:00
    @Thetruechar #7 是打算走这条路,目前 go 勉强算是半只脚入门吧,这次 pull 的事件,学会了很多,但是单元测试这块确实没接触过,原作者非常用心,但我也不好一直麻烦他,就想着来 V 站请教大家,看看能不能找到解决方案
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1482 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 29ms UTC 16:34 PVG 00:34 LAX 08:34 JFK 11:34
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86