Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2658 +/- ##
==========================================
- Coverage 77.11% 76.80% -0.32%
==========================================
Files 117 117
Lines 16170 16259 +89
==========================================
+ Hits 12470 12487 +17
- Misses 3700 3772 +72 ☔ View full report in Codecov by Sentry. |
LeoQuote
left a comment
There was a problem hiding this comment.
- 代码复用不够, 这些检查的代码在单个工单的审核和执行中有类似的, 希望能复用一部分, 可以考虑直接把代码挪到 model 层, 作为 workflow 的一个 方法来做
- 请不要再为了同样的case重开pr,会损失之前的讨论
- 审核代码已重构,请考虑使用全新的AuditV2
如果有其他的建议或困难欢迎提出
1、这块不太明白,能否指导下 ? |
LeoQuote
left a comment
There was a problem hiding this comment.
我重新细看了下当前的逻辑,复用方面确实跟以前不一样,提交和取消都是复用了单个工作流的代码,也复用了单个工作流的model,只是多了一个虚的总工单。
总体上对已有的工单代码修改不大,甚至也没有多少的侵入性,不想用这个功能的人不用就行了。
我唯一的建议变成了给代码加上一些单元测试,这有利于代码和功能的长期发展,还希望大佬考虑一下。
以及谢谢大佬的贡献!
单元测试就真不太会了 ,大佬帮忙写一点?这边功能我手动测试是没问题了 |
|
暂时没空帮忙写单元测试,等我什么时候有空吧,欢迎其他人来帮忙补充单元测试 |
|
批量提交SQL页面没有选择库的按钮,一个实例多个库,只需要提交提交其中一个库,该怎么操作呢 @cslingjun |
一个实例有多个库,你可以不用批量提交,按照现有的工单提交模式就好了呀 |
|
单元化的架构,在各个单元都有同样的库,我需要在N和实例中的某一个库发布同样的sql |
|
批量多租户多实例执行 真的刚需,相同sql 在多租户环境,确实面临一次sql提交审核多环境多实例执行问题 pr 看起来问题不多, 为啥没有合并啊 @cslingjun @LeoQuote |
#1469
