fix: 消除旧通知头空值重载歧义

Author: binarywang

weixin-java-pay/src/main/java/com/github/binarywang/wxpay/bean/ecommerce/SignatureHeader.java

@@ -18,7 +18,7 @@
 @NoArgsConstructor
 @AllArgsConstructor
 @Deprecated
-public class SignatureHeader implements Serializable {
+public class SignatureHeader extends com.github.binarywang.wxpay.bean.notify.SignatureHeader implements Serializable {
   private static final long serialVersionUID = -6958015499416059949L;
   /**
    * 时间戳

weixin-java-pay/src/test/java/com/github/binarywang/wxpay/service/LegacyEcommerceApiCompatibilityTest.java

@@ -25,4 +25,23 @@ public void shouldKeepLegacyRefundAndWithdrawNotificationSignatures() throws Exc
     Assert.assertNotNull(EcommerceService.class.getMethod("parseRefundNotifyResult", String.class, legacyHeader));
     Assert.assertNotNull(EcommerceService.class.getMethod("parseWithdrawNotifyResult", String.class, legacyHeader));
   }
+
+  @Test
+  public void shouldMakeLegacyHeaderMoreSpecificAndPreserveSignatureFields() {
+    com.github.binarywang.wxpay.bean.ecommerce.SignatureHeader legacyHeader =
+      new com.github.binarywang.wxpay.bean.ecommerce.SignatureHeader();
+    legacyHeader.setTimeStamp("timestamp");
+    legacyHeader.setNonce("nonce");
+    legacyHeader.setSigned("signed");
+    legacyHeader.setSerialNo("serial-no");
+
+    Assert.assertTrue(com.github.binarywang.wxpay.bean.notify.SignatureHeader.class
+      .isAssignableFrom(legacyHeader.getClass()));
+    com.github.binarywang.wxpay.bean.notify.SignatureHeader unifiedHeader =
+      EcommerceService.toUnifiedSignatureHeader(legacyHeader);
+    Assert.assertEquals(unifiedHeader.getTimeStamp(), "timestamp");
+    Assert.assertEquals(unifiedHeader.getNonce(), "nonce");
+    Assert.assertEquals(unifiedHeader.getSignature(), "signed");
+    Assert.assertEquals(unifiedHeader.getSerial(), "serial-no");
+  }
 }