-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: ✨ 修复了news类型无法使用的问题,添加了message的测试,新增模板卡片消息 #177
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
测试不应该有副作用(不应该向外部服务发出具体请求),提醒我了,这块我国庆看看能不能抽个 interface 方便 mock
我的打算是:单测用例只去断言预期的请求 URL、头、请求体。后续如果有需求及时发现 API 不兼容变更的话再考虑加 e2e 集测,我去弄个测试用的企业账号和应用啥的。
最后感谢您的贡献!
那这个pr是我修改一下并入呢,还是等你这边修整之后再说呢 |
我的go水平也是半路出家,跟着项目也学习了很多东西 |
我这两天试着弄弄,要不你暂时把测试代码从这个PR拆出来(至少拆个单独提交)也行,否则CI红的,无法合并 |
最近有些其他事情插入了我的 todo list,这两天又不舒服,我会把你的 commits 整理下提交进去。 假期结束前不用等我消息了,可以关掉电脑享受生活了 ;-) |
了解,注意身体,好好休息 |
也挺需要这个模板卡片消息 @xen0n 老哥如果这个没啥问题的话 请提交一下~ 谢谢啦。 |
因为我仍然没有时间折腾(十分抱歉),基于对业务研发总会把自己用到的那部分代码跑通的这一份信任 😈,我打算先把这些合并了。后续我再看怎么补测试…… 感谢您的贡献! |
原来提供的发消息能力似乎只有text与markdown还能沿用,另外几个官方都有不小的改动,把news和mpnews做了一下修复。
另外比较重要的是添加了卡片模板的消息,这块儿官方提供了更加丰富的能力,已通过测试可用。