Fix

- do not change status & value when fulfilled / rejected Refer: 1. https://promisesaplus.com/#point-14 2. https://promisesaplus.com/#point-17
yanguango · Jan 12, 2020 · ee53366 · ee53366
1 parent 4b88e17
commit ee53366
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/dist/src/promise.js b/dist/src/promise.js
@@ -27,7 +27,13 @@ class Promise {
     executor(this._resolve.bind(this), this._reject.bind(this));
   }
 
+  _hasResolved() {
+    return this._isFulfilled() || this._isRejected();
+  }
+
   _resolve(x) {
+    if (this._hasResolved()) return;
+
     if (x === this) {
       throw new TypeError("Resolving object can not be the same object");
     } else if (x instanceof Promise) {
@@ -58,12 +64,16 @@ class Promise {
   }
 
   _fulfill(result) {
+    if (this._hasResolved()) return;
+
     this._state = State.Fulfilled;
     this._value = result;
     this._handlers.forEach(handler => this._callHandler(handler));
   }
 
   _reject(error) {
+    if (this._hasResolved()) return;
+
     this._state = State.Rejected;
     this._value = error;
     this._handlers.forEach(handler => this._callHandler(handler));

diff --git a/src/promise.js b/src/promise.js
@@ -22,7 +22,13 @@ export class Promise {
     executor(this._resolve.bind(this), this._reject.bind(this));
   }
 
+  _hasResolved() {
+    return this._isFulfilled() || this._isRejected()
+  }
+
   _resolve(x) {
+    if (this._hasResolved()) return
+
     if (x === this) {
       throw new TypeError("Resolving object can not be the same object");
     } else if (x instanceof Promise) {
@@ -57,12 +63,16 @@ export class Promise {
   }
 
   _fulfill(result) {
+    if (this._hasResolved()) return
+
     this._state = State.Fulfilled;
     this._value = result;
     this._handlers.forEach(handler => this._callHandler(handler));
   }
 
   _reject(error) {
+    if (this._hasResolved()) return
+
     this._state = State.Rejected;
     this._value = error;
     this._handlers.forEach(handler => this._callHandler(handler));