fix(miniapp): 修复发布构建中的测试跳过冲突 - #4111
Conversation
Sorry, something went wrong.
|
🤖 Augment PR Summary
总结：将 miniapp 模块 Surefire 的 skip 配置改为读取 ${maven.test.skip}，不再强制执行测试。 🤖 Was this summary useful? React with 👍 or 👎 |
Sorry, something went wrong.
There was a problem hiding this comment.
🟢 Approval recommended
配置已与 Maven 测试跳过参数一致，且没有未解决的审查意见。
Pull request overview修复 miniapp 发布构建中测试跳过配置不一致的问题。
Changes:
- 将 Surefire 的 skip 绑定到 ${maven.test.skip}。
- 保持常规构建执行 TestNG 测试套件。
| File | Description |
|---|---|
| weixin-java-miniapp/pom.xml | 调整 Surefire 测试跳过配置。 |
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6602690e54
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
问题
发布工作流使用 -Dmaven.test.skip=true 跳过测试编译，但 miniapp 模块固定 Surefire 的 <skip>false</skip>，导致 TestNG 执行 suite 时找不到未编译的 WxMaSubscribeServiceImplUrlTest。
修复
让 Surefire 的 skip 读取 ${maven.test.skip}，使发布流程一致跳过测试编译与执行；常规构建仍运行该 TestNG suite。
验证