AInoob / NooBox

A Chrome Extension that contains useful tools
https://ainoob.com/project/noobox
GNU General Public License v3.0
491 stars 49 forks source link

test: initial configurations of jest and enzyme #73

Closed orzyyyy closed 5 years ago

orzyyyy commented 5 years ago

ci 推荐 circleci,我可以改个 yaml 出来

codecov 可以加一下,方便看覆盖率

AInoob commented 5 years ago

Hello,

感谢提交,不过暂时不会merge新的pull request了,我准备先重写一下,加入tslint等东西。其实之前很长一段时间我都没有弄NooBox,也准备完全停止开发了。不过最近感觉没有想好什么好的点子,加上感觉有些东西可以优化一下,所以准备重写。

不过关于test这个,说实话要写一个相对完整的测试可能要花很长时间(而且还要确定代码以后不会大改,不然就要重写了),所以目前都是手动测试各种功能的。而且到时候我感觉写完整测试的可能性也不大,毕竟手动测试可能一次5到10分钟,但是如果写测试估计要10~?小时。

再次感谢你一直发的pull requests,不过其实需求是最重要的,NooBox的主要问题是没有找到新的有用的功能,所以如果你有好的idea,咱们可以一起讨论,然后如果你想implement的话,那就完美了(嘿嘿)。

Thanks,

AInoob

orzyyyy commented 5 years ago

没测试是真的不敢 pr,路人并不知道哪些功能是正确的,实际上 idea 都是建立在不停 commit 上的,你这...

测试只要写一遍就够,哪怕花的时间长一点,但每次都能省那 5-10 分钟,路人 pr 也无需担心会影响主要功能

特别是用 ts 重写,要是手动点点点,我是真的不敢想...

AInoob commented 5 years ago

测试从简单的模拟ajax请求,到我觉得很麻烦的测试UI布局,我真的觉得要花很久才能写完。当然我也不是说绝对不弄test。 idea建立在不停commit这个我觉得顺序就错了。一个feature肯定是先有feature request或者use case才会被implement的。然后通过commit来添加feature。怎么可能commit commit的突然发现一个idea。(当然如果你说idea是这个代码应该改改,的确也是一个idea,但并不是产品功能的更改) 之前你的pr我就是手动测试了才merge的。每次省5~10分钟,但是一写test要写很久很久啊,如果按我觉得最短的10小时来算,1060 / 5 = 120. 10 60 / 10 = 60. 你觉得有60~120+次大改(排除那些就改了一个地方,并且早就手动测试了)的概率多大呢?我觉得目前来说不太不划算。