补齐微信小店商品管理接口并完善契约测试 - #4104
Merged
binarywang merged 6 commits intoAug 22, 2026
Merged
Conversation
🤖 Augment PR Summary摘要:本 PR 补齐微信小店商品管理相关的 13 个服务接口与数据模型。
WxChannelService HTTP 调用与 ResponseUtils 解码链路,保持 Java 8 与模块既有 TestNG 约定。
🤖 Was this summary useful? React with 👍 or 👎 |
There was a problem hiding this comment.
🟡 Changes recommended
新增测试与响应模型中存在可导致 checkstyle/可维护性问题的实现细节,需要先修正后再合并更稳妥。
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
该 PR 面向 weixin-java-channel(视频号小店)模块补齐商品管理缺失能力:新增并落地 #3988 列出的 13 个商品管理接口(含请求/响应模型、URL 常量与 Service 方法),同时通过“无网络调用”的契约回归测试覆盖请求 JSON 与响应解码行为。
Changes:
- 在
WxChannelProductService/WxChannelProductServiceImpl中新增 13 个商品管理接口方法,并补充对应 URL 常量。 - 新增一组商品管理契约测试(TestNG),覆盖 URL、请求 JSON 以及响应解码/嵌套对象序列化。
- 新增/补充商品管理相关的请求参数与响应模型(bean)。
File summaries
| File | Description |
|---|---|
| weixin-java-channel/src/test/resources/testng.xml | 将新增的商品管理契约测试类加入 TestNG suite |
| weixin-java-channel/src/test/java/me/chanjar/weixin/channel/api/impl/WxChannelProductManagementServiceImplTest.java | 新增商品管理接口契约测试(无网络) |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/constant/WxChannelApiUrlConstants.java | 增加商品管理相关接口 URL 常量 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductTimingSaleParam.java | 新增“商品立即开售”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductStockFlowResponse.java | 新增“库存流水”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductStockFlowParam.java | 新增“库存流水”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductSchemeResponse.java | 新增 scheme 响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductSchemeParam.java | 新增 scheme 请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductCategoryPreCheckResponse.java | 新增“发品前校验”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductCategoryPreCheckParam.java | 新增“发品前校验”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductCategoryClassifyResponse.java | 新增“类目推荐”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductCategoryClassifyParam.java | 新增“类目推荐”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductBrandRecommendResponse.java | 新增“品牌推荐”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductBrandRecommendParam.java | 新增“品牌推荐”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductAuditStrategySetParam.java | 新增“设置上架策略”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductAuditStrategyResponse.java | 新增“获取上架策略”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductAuditStrategyInfo.java | 新增上架策略信息模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ProductAuditQuotaResponse.java | 新增“提审限额”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ExternalProductMappingResponse.java | 新增“站内外属性映射”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ExternalProductMappingParam.java | 新增“站内外属性映射”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ExternalProductMappingNewResponse.java | 新增“属性映射及推荐”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/ExternalProductMappingNewParam.java | 新增“属性映射及推荐”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/AddProductThirdPartySourceResponse.java | 新增“新增第三方货源”响应模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/bean/product/AddProductThirdPartySourceParam.java | 新增“新增第三方货源”请求参数模型 |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/WxChannelProductService.java | 在商品 Service 接口中补齐 13 个新方法签名与 Javadoc |
| weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/impl/WxChannelProductServiceImpl.java | 实现新增 13 个接口,并引入通用 post+decode 辅助方法 |
| .gitignore | 忽略本地 docs/superpowers/ 工作目录 |
Review details
- Files reviewed: 26/27 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Owner
Author
|
@copilot resolve the merge conflicts in this pull request |
…annel-product-management-apis # Conflicts: # weixin-java-channel/src/main/java/me/chanjar/weixin/channel/api/impl/WxChannelProductServiceImpl.java Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com>
Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com>
Contributor
已完成合并冲突解决:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
docs/superpowers/工作文档验证
mvn -pl weixin-java-channel test(编译成功;父 POM 的 Surefire 配置跳过测试)git diff --check github/develop...HEADCloses #3988