Skip to content

fix(channel): 修复发布构建中的测试跳过冲突 - #4112

Merged
binarywang merged 1 commit into
developfrom
fix/miniapp-surefire-skip
Aug 22, 2026
Merged

fix(channel): 修复发布构建中的测试跳过冲突#4112
binarywang merged 1 commit into
developfrom
fix/miniapp-surefire-skip

Conversation

@binarywang

@binarywang binarywang commented Aug 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

问题

发布工作流使用 -Dmaven.test.skip=true 跳过测试编译，但 channel 模块固定 Surefire 的 <skip>false</skip>，导致 TestNG 执行 suite 时找不到未编译的 WxChannelProductManagementServiceImplTest。

修复

让 Surefire 的 skip 读取 ${maven.test.skip}，使发布流程一致跳过测试编译与执行；常规构建仍运行对应 TestNG suite。

验证

  • mvn -pl weixin-java-channel clean test -Dmaven.test.skip=true -Dgpg.skip=true --no-transfer-progress
  • mvn -pl weixin-java-channel -am clean test -Dgpg.skip=true --no-transfer-progress（56 tests, 0 failures）

@binarywang
binarywang marked this pull request as ready for review August 22, 2026 14:02
Copilot AI lite review requested due to automatic review settings August 22, 2026 14:02
@augmentcode

augmentcode Bot commented Aug 22, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

总结：将 Channel 模块 Surefire 的 skip 从固定 false 改为读取 ${maven.test.skip}。

目的：使发布使用 -Dmaven.test.skip=true 时测试编译与 TestNG suite 一致跳过，未设置该属性时仍运行模块测试。

🤖 Was this summary useful? React with 👍 or 👎

augmentcode[bot]
augmentcode Bot reviewed Aug 22, 2026
View reviewed changes

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.