fix #217 . reject promise with an error object instead of errCode number

README.md

@@ -298,7 +298,11 @@ try {
   });
   console.log('share text message to time line successful:', result);
 } catch (e) {
-  console.error('share text message to time line failed with:', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 
 // Code example to share image url:
@@ -315,7 +319,11 @@ try {
   });
   console.log('share image url to time line successful:', result);
 } catch (e) {
-  console.log('share image url to time line failed with:', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 
 // Code example to share image file:
@@ -343,7 +351,11 @@ try {
   });
   console.log('share image file to time line successful:', result);
 } catch (e) {
-  console.error('share image file to time line failed with:', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 
 // Code example to share image resource:
@@ -361,7 +373,11 @@ try {
   console.log('share resource image to time line successful', result);
 }
 catch (e) {
-  console.error('share resource image to time line failed', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 
 // Code example to download an word file from web, then share it to WeChat session
@@ -389,7 +405,11 @@ try {
   });
   console.log('share word file to chat session successful', result);
 } catch (e) {
-  console.error('share word file to chat session failed', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 
 //android code use ExternalDirectoryPath
@@ -415,7 +435,11 @@ try {
   console.log('share word file to chat session successful', result);
 }
 catch (e) {
-  console.error('share word file to chat session failed', e);
+  if (e instanceof WeChat.WechatError) {
+    console.error(e.stack);
+  } else {
+    throw e;
+  }
 }
 ```
 

index.js

@@ -154,11 +154,10 @@ export function sendAuthRequest(scopes, state) {
   return new Promise((resolve, reject) => {
     WeChat.sendAuthRequest(scopes, state, () => {});
     emitter.once('SendAuth.Resp', resp => {
-      const result = resp.errCode;
-      if (result === 0) {
+      if (resp.errCode === 0) {
         resolve(resp);
       } else {
-        reject(result);
+        reject(new WechatError(resp));
       }
     });
   });
@@ -181,11 +180,10 @@ export function shareToTimeline(data) {
   return new Promise((resolve, reject) => {
     nativeShareToTimeline(data);
     emitter.once('SendMessageToWX.Resp', resp => {
-      const result = resp.errCode;
-      if (result === 0) {
+      if (resp.errCode === 0) {
         resolve(resp);
       } else {
-        reject(result);
+        reject(new WechatError(resp));
       }
     });
   });
@@ -208,11 +206,10 @@ export function shareToSession(data) {
   return new Promise((resolve, reject) => {
     nativeShareToSession(data);
     emitter.once('SendMessageToWX.Resp', resp => {
-      const result = resp.errCode;
-      if (result === 0) {
+      if (resp.errCode === 0) {
         resolve(resp);
       } else {
-        reject(result);
+        reject(new WechatError(resp));
       }
     });
   });
@@ -235,12 +232,28 @@ export function pay(data) {
       if (result) reject(result);
     });
     emitter.once('PayReq.Resp', resp => {
-      const result = resp.errCode;
-      if (result === 0) {
+      if (resp.errCode === 0) {
         resolve(resp);
       } else {
-        reject(result);
+        reject(new WechatError(resp));
       }
     });
   });
 }
+
+/**
+ * promises will reject with this error when API call finish with an errCode other than zero.
+ */
+export class WechatError extends Error {
+  constructor(resp) {
+    const message = resp.errStr || resp.errCode.toString();
+    super(message);
+    this.name = 'WechatError';
+    this.code = resp.errCode;
+
+    // avoid babel's limition about extending Error class
+    // https://github.com/babel/babel/issues/3083
+    Object.setPrototypeOf(this, WechatError.prototype);
+  }
+}
+