Promise.prototype.finally thunks have a length of 0

test/built-ins/Promise/prototype/finally/rejected-observable-then-calls-argument.js

@@ -21,12 +21,22 @@ Promise.reject(new Test262Error())
   .finally(function() {})
   .then($DONE, $DONE);
 
-var then = Promise.prototype.then;
+var calls = 0;
-Promise.prototype.then = function(thrower) {
+var expected = [
-  assert(!isConstructor(thrower));
+  { length: 0, name: '' },
-  assert.sameValue(thrower.length, 1);
+  { length: 1, name: '' }
-  assert.sameValue(thrower.name, '');
+];
-  assert.throws(Test262Error, thrower);
 
-  return then.call(this, thrower);
+var then = Promise.prototype.then;
+Promise.prototype.then = function(resolve) {
+  assert(!isConstructor(resolve));
+  assert.sameValue(resolve.length, expected[calls].length);
+  assert.sameValue(resolve.name, expected[calls].name);
+  if (calls === 0) {
+    assert.throws(Test262Error, resolve);
+  }
+
+  calls += 1;
+
+  return then.call(this, resolve);
 };

test/built-ins/Promise/prototype/finally/resolved-observable-then-calls-argument.js

@@ -23,12 +23,20 @@ Promise.resolve(value)
   .finally(function() {})
   .then($DONE, $DONE);
 
-var then = Promise.prototype.then;
+var calls = 0;
-Promise.prototype.then = function(valueThunk) {
+var expected = [
-  assert(!isConstructor(valueThunk));
+  { length: 0, name: '' },
-  assert.sameValue(valueThunk.length, 1);
+  { length: 1, name: '' }
-  assert.sameValue(valueThunk.name, '');
+];
-  assert.sameValue(valueThunk(), value);
 
-  return then.call(this, valueThunk);
+var then = Promise.prototype.then;
+Promise.prototype.then = function(resolve) {
+  assert(!isConstructor(resolve));
+  assert.sameValue(resolve.length, expected[calls].length);
+  assert.sameValue(resolve.name, expected[calls].name);
+  if (calls === 0) {
+    assert.sameValue(resolve(), value);
+  }
+
+  return then.call(this, resolve);
 };