oldme

Results 50 comments of oldme

> > @gqcn 因为已经使用了标准库 AST 来做语法分析,所以这个测试用例不应该属于框架职责范畴。 Because GoFrame have used AST for parsing,the testing unit case should not be the responsibility of GoFrame. > > 用 AST 能保证解析正确,但不能保证生成正确。测的不是 AST,测的是生成逻辑。 The...

> @oldme-git @wln32 我看了下,这里加一个单例来保证正确性验证也挺好的。我顺便看了下咱们生成`ctrl`的命令逻辑,似乎有一块没有用`ast`做替换,是否这里统一使用`ast`对对后续维护来说更友好一些? @gqcn 之前用正则的时候,有很多 `issue` 都说过关于注释生成有误的问题,如果要加上这块的单测,直接放在 `testdata/genctrl/api` 下当做基础用例或许更好,而且,`ctrl` 的其他基础测试用例也不完善,在未来也应该补充进去,可以参考 #3488 中关于 logic 的测试用例。 关于 `ctrl` 的 `AST` 重构计划和更多的基础单测用例,是准备在合并 #3488 后进行的。单纯的针对单个 `issue` 编写不复杂且基础的测试用例,不会为框架带来长久的影响。

> @oldme-git 我看改动的文件比较多,都是把下划线改为了大驼峰,但在单测命名这块本身没有规范,这么大量的修改看不出来能给项目带来什么收益。 单测命名这块的规范官方给的示例都是 TestXxx,第三方规范也都是 TestXxx,哪怕不批量改成TestXxx,也要统一一种风格比较好吧,你看呢

@gqcn 我详细看了一下 go 标准库,`Example` 确实会使用下划线,但是单测 Test 没有使用下划线的案例,另外,其他的开源项目,如 `opentelemetry-go`, `gin`,都是使用的 `TestXxx`,当前 gf 两种方式混用,项目大了还是注意一下规范比较好,对齐开源社区的规范也不会带来什么风险

@gqcn 另外,虽然没有正式的 go 官方标准,但是在开发文档中还是提及了 Test 的命名标准。参见: https://pkg.go.dev/testing#section-documentation

@shuqingzai 从配置文件调整到调用,暂时没有做这个功能,它不像lv都是php文件

@alaywn 请提供更多代码,v2.7.2 没有复现

@gqcn ci failed

`XML` 依赖其他三方库,等待第三方库 PR 通过后升级依赖版本修复此 BUG。或者更换第三方库

信息补充:不再使用第三方库处理 XML,切换成标准库 `encoding/xml`