test: 为覆盖率最低的 weixin-java-open、weixin-java-channel 补充单元测试

[devin-ai-integration]

Summary

统计各模块 src/main/java 与 src/test/java 文件数后，测试占比最低的是 weixin-java-open（240 主类 / 13 测试）和 weixin-java-channel（674 / 36），且这两个模块已有测试几乎全是需要真实微信凭据的 *ServiceImplTest，本地不可执行。本 PR 新增不依赖网络与凭据的纯逻辑单元测试，并顺带修复由此暴露的一个既有反序列化 bug。

新增测试（15 个用例）

• WxOpenCryptUtilTest：encrypt → 解析 <Encrypt>/<MsgSignature>/<TimeStamp>/<Nonce> → decryptXml/decryptContent 往返一致；签名不匹配抛 WxRuntimeException；componentAesKey 含空格时与去空格版本加密结果一致（覆盖构造函数中的 StringUtils.remove(encodingAesKey, " ")）。
• WxOpenXmlMessageTest：fromXml 解析 component_verify_ticket、authorized、快速创建小程序推送（appid/status/auth_code/info.name，含 getRegistAppId() 兼容读取）；fromXml(InputStream)；fromEncryptedXml(String/InputStream) 往返并校验 context 为解密后的原始 xml。
• WxOpenGsonBuilderTest：create() 单例；WxOpenComponentAccessToken、WxOpenAuthorizerAccessToken、WxOpenAuthorizationInfo（含 func_info 缺失 funcscope_category 元素被跳过）、WxOpenQueryAuthResult 各适配器的反序列化；disableHtmlEscaping 行为。
• XmlUtilsTest（channel）：encode、encode(ObjectMapper, obj)、decode(String/TypeReference/InputStream)，以及未知字段被忽略、null/空串/非法 xml 返回 null 的容错分支。
• WxChCryptUtilsTest（channel）：回调消息加解密往返与签名校验失败分支；静态 decrypt(sessionKey, encryptedData, iv) 用本地 AES/CBC 加密数据做往返验证，以及非法密钥抛异常。

测试中的 token / EncodingAESKey 均为明显占位值（AES key 为 base64("0123456789abcdef" * 2) 去掉补位），不含任何真实凭据。

测试接入配置

两个模块的 Surefire 都显式指定 src/test/resources/testng.xml，因此在两个 suite 中新增 Util_Test 注册新增测试类；channel 侧顺带把此前未接入 suite 的 JsonUtilsTest、ResponseUtilsTest 一并加入，保证 mvn test 能真正执行。

修复 WxFastMaCanSetCategoryResult 的 errcode 字段冲突

接入 suite 后暴露既有测试 WxFastMaCanSetCategoryResultTest#testFromJson 失败：该 Bean 自带 @SerializedName("errcode") private int errCode，与父类 WxOpenResult#errcode 映射到同一 JSON 字段名，新版本 Gson 直接抛 IllegalArgumentException: declares multiple JSON fields named 'errcode'，导致该结果类完全无法反序列化。

-  @SerializedName("errcode")
-  private int errCode;
+  /** 已弃用，改用父类 WxOpenResult#getErrcode() */
+  @Deprecated
+  public int getErrCode() { return NumberUtils.toInt(this.errcode); }
+  @Deprecated
+  public void setErrCode(int errCode) { this.errcode = String.valueOf(errCode); }

移除重复字段，改为委托父类 errcode 的过时方法，保留 getErrCode()/setErrCode(int) 的源码兼容性；既有测试补充断言 getErrcode()=="0"、getErrCode()==0、isSuccess() 作为回归保护。

验证

根 pom.xml 中 surefire 配置为 <skip>true</skip>，默认不执行测试。验证时临时将其改为 false 运行，提交前已还原（diff 中不包含 pom.xml）：

mvn -pl weixin-java-open test     # Tests run: 18, Failures: 0
mvn -pl weixin-java-channel test  # Tests run: 15, Failures: 0

本地环境为 JDK 17 + Maven 3.6.3；新增代码仅使用 Java 8 语法与 TestNG，遵循各模块既有测试风格。

Link to Devin session: https://app.devin.ai/sessions/14ac57aaa8c4438da6161b7ddb001650
Requested by: @binarywang

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment, CI, and merge conflict monitoring

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 40f45b483d

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[chatgpt-codex-connector]

将 Open 新增单测加入 TestNG suite

在 weixin-java-open/pom.xml:122-124 中该模块固定通过 src/test/resources/testng.xml 运行 TestNG suite，Surefire 官方文档说明 suite 配置会覆盖默认扫描并运行 suite 文件中的测试（https://maven.apache.org/surefire-archives/surefire-3.5.4/maven-surefire-plugin/examples/testng.html#using-suite-xml-files）；但当前 suite 只列出 3 个 bean.result 测试，没有包含这个类、WxOpenXmlMessageTest 或 WxOpenGsonBuilderTest，所以启用常规 mvn -pl weixin-java-open -am test 时本 PR 新增的 Open 覆盖率不会执行。请把这些类加入 suite，或调整 Surefire 配置让默认扫描生效。

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

将 Channel 新增单测加入 TestNG suite

weixin-java-channel/pom.xml:131-133 同样固定使用 src/test/resources/testng.xml，而该 suite 的 <classes> 在 weixin-java-channel/src/test/resources/testng.xml:5-9 中没有任何启用的测试类；按 Surefire suite XML 语义（https://maven.apache.org/surefire-archives/surefire-3.5.4/maven-surefire-plugin/examples/testng.html#using-suite-xml-files），启用常规 mvn -pl weixin-java-channel -am test 时既不会执行这个类，也不会执行同 PR 新增的 XmlUtilsTest，导致新增 Channel 单测没有回归保护。请将新增类加入 suite，或移除 suiteXmlFiles 让默认扫描生效。

Useful? React with 👍 / 👎.

[binarywang]

需要补充测试接入配置后再合并：

本 PR 新增的 5 个 TestNG 测试类没有加入 weixin-java-open 和 weixin-java-channel 各自的 src/test/resources/testng.xml。两个模块的 Surefire 都显式指定该 suite；open 的 suite 只列出既有测试，channel 的 classes 为空，因此常规 mvn test / CI 不会执行这些新测试，无法形成持续回归保护。

请将新增测试类加入对应 suite（或调整 Surefire 配置为自动发现），并以模块测试命令验证它们确实被执行。

[devin-ai-integration]

需要补充测试接入配置后再合并

已修复（commit 75f347c）：在两个模块的 src/test/resources/testng.xml 中新增 Util_Test，显式注册新增测试类；channel 侧顺带把此前未接入 suite 的 JsonUtilsTest、ResponseUtilsTest 一并加入。

验证（临时将根 pom.xml 的 surefire <skip> 改为 false 执行，验证后已还原，diff 不含 pom.xml）：

• mvn -pl weixin-java-channel test → Tests run: 15, Failures: 0
• mvn -pl weixin-java-open test → Tests run: 18, Failures: 1

open 模块唯一失败的是既有测试 WxFastMaCanSetCategoryResultTest#testFromJson，与本 PR 无关，属历史问题：WxFastMaCanSetCategoryResult#errCode 与父类 WxOpenResult#errcode 在 Gson 下映射到同一字段名，新版 Gson 直接抛 IllegalArgumentException: declares multiple JSON fields named 'errcode'。本 PR 新增的 3 个 open 测试全部通过。是否需要我在本 PR 或单独 PR 中一并修复该 bean 的字段冲突？

[devin-ai-integration]

已按确认修复 WxFastMaCanSetCategoryResult 的 errcode 字段冲突（commit 4e761a2）：移除与父类 WxOpenResult#errcode 重名的 @SerializedName("errcode") private int errCode 字段，改为委托父类 errcode 的 @Deprecated getErrCode() / setErrCode(int)，保留源码兼容性；既有测试补充 getErrcode()、getErrCode()、isSuccess() 断言作为回归保护。

mvn -pl weixin-java-open test → Tests run: 18, Failures: 0（临时将根 pom surefire <skip> 置为 false 执行，已还原）。PR 描述已同步更新。