xen0n / go-workwx

a sensible WeCom (企业微信, Wechat Work, Work Weixin) SDK for Go
https://godoc.org/github.com/xen0n/go-workwx
MIT License
649 stars 164 forks source link

feat: ✨ 修复了news类型无法使用的问题,添加了message的测试,新增模板卡片消息 #177

Closed eryajf closed 9 months ago

eryajf commented 1 year ago

原来提供的发消息能力似乎只有text与markdown还能沿用,另外几个官方都有不小的改动,把news和mpnews做了一下修复。

另外比较重要的是添加了卡片模板的消息,这块儿官方提供了更加丰富的能力,已通过测试可用。

eryajf commented 1 year ago

经过两天的研究,以及调试,总算是把这块儿补齐了。从issue区也看了不少作者的对话,挺能理解你的,毕竟我也有维护一些开源项目,有时候,后期的维护就挺有心无力的了。

关联的issue #96 #76

eryajf commented 1 year ago

测试不应该有副作用(不应该向外部服务发出具体请求),提醒我了,这块我国庆看看能不能抽个 interface 方便 mock

我的打算是:单测用例只去断言预期的请求 URL、头、请求体。后续如果有需求及时发现 API 不兼容变更的话再考虑加 e2e 集测,我去弄个测试用的企业账号和应用啥的。

最后感谢您的贡献!

那这个pr是我修改一下并入呢,还是等你这边修整之后再说呢

eryajf commented 1 year ago

我的go水平也是半路出家,跟着项目也学习了很多东西

xen0n commented 1 year ago

测试不应该有副作用(不应该向外部服务发出具体请求),提醒我了,这块我国庆看看能不能抽个 interface 方便 mock

我的打算是:单测用例只去断言预期的请求 URL、头、请求体。后续如果有需求及时发现 API 不兼容变更的话再考虑加 e2e 集测,我去弄个测试用的企业账号和应用啥的。

最后感谢您的贡献!

那这个pr是我修改一下并入呢,还是等你这边修整之后再说呢

我这两天试着弄弄,要不你暂时把测试代码从这个PR拆出来(至少拆个单独提交)也行,否则CI红的,无法合并

xen0n commented 1 year ago

最近有些其他事情插入了我的 todo list,这两天又不舒服,我会把你的 commits 整理下提交进去。

假期结束前不用等我消息了,可以关掉电脑享受生活了 ;-)

eryajf commented 1 year ago

了解,注意身体,好好休息

tttao7 commented 9 months ago

也挺需要这个模板卡片消息 @xen0n 老哥如果这个没啥问题的话 请提交一下~ 谢谢啦。

xen0n commented 9 months ago

因为我仍然没有时间折腾(十分抱歉),基于对业务研发总会把自己用到的那部分代码跑通的这一份信任 :smiling_imp:,我打算先把这些合并了。后续我再看怎么补测试……

感谢您的贡献!