Skip to content

补充 Fund 模块银行卡与二维码接口 URL 前缀回归校验#4019

Closed
Copilot wants to merge 2 commits into
developfrom
copilot/fix-fund-module-url-paths
Closed

补充 Fund 模块银行卡与二维码接口 URL 前缀回归校验#4019
Copilot wants to merge 2 commits into
developfrom
copilot/fix-fund-module-url-paths

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 27, 2026

问题聚焦于 WxChannelApiUrlConstants.Fund 中 7 个银行卡/省市/支行/二维码接口地址前缀应为 /shop/funds/,若误用 /channels/ec/shop/funds/ 会导致请求 404。
本次改动未扩展接口能力,重点是为这 7 个常量补上明确的回归约束,防止后续修改引入错误前缀。

  • 变更范围

    • 新增测试文件:weixin-java-channel/src/test/java/me/chanjar/weixin/channel/constant/WxChannelApiUrlConstantsTest.java
    • 针对以下常量增加固定值断言:
      GET_BANK_BY_NUM_URLGET_BANK_LIST_URLGET_CITY_URLGET_PROVINCE_URLGET_SUB_BANK_URLGET_QRCODE_URLCHECK_QRCODE_URL
  • 回归约束策略

    • 通过常量级断言直接锁定 URL,确保前缀保持为 https://api.weixin.qq.com/shop/funds/...
    • 覆盖银行信息查询、地区查询、支行查询、二维码获取与扫码状态查询四类资金相关场景
  • 示例代码片段

    @Test
    public void testFundBankAndQrCodeUrlPrefix() {
      assertEquals(WxChannelApiUrlConstants.Fund.GET_BANK_BY_NUM_URL,
        "https://api.weixin.qq.com/shop/funds/getbankbynum");
      assertEquals(WxChannelApiUrlConstants.Fund.GET_QRCODE_URL,
        "https://api.weixin.qq.com/shop/funds/qrcode/get");
      assertEquals(WxChannelApiUrlConstants.Fund.CHECK_QRCODE_URL,
        "https://api.weixin.qq.com/shop/funds/qrcode/check");
    }

Copilot AI changed the title [WIP] Fix URL path concatenation errors in Fund module 补充 Fund 模块银行卡与二维码接口 URL 前缀回归校验 May 27, 2026
Copilot AI requested a review from binarywang May 27, 2026 12:58
@binarywang binarywang marked this pull request as ready for review May 27, 2026 13:02
Copilot AI review requested due to automatic review settings May 27, 2026 13:02
Repository owner deleted a comment from chatgpt-codex-connector Bot May 27, 2026
@binarywang
Copy link
Copy Markdown
Owner

早已修复,没必要的代码

@binarywang binarywang closed this May 27, 2026
Copilot AI review requested due to automatic review settings May 27, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Fund模块银行卡相关接口URL路径拼接错误

2 participants