From c472aeea4b306b611cae967c17663fa0c380aea7 Mon Sep 17 00:00:00 2001
From: Alexey Shvayka <shvaikalesh@gmail.com>
Date: Tue, 7 Jul 2020 16:03:11 +0300
Subject: [PATCH] Use $DONE instead of $ERROR to avoid unhandled rejections

---
 .../prototype/finally/resolved-observable-then-calls.js  | 3 +--
 .../Promise/prototype/finally/species-constructor.js     | 5 ++---
 .../Promise/prototype/finally/subclass-reject-count.js   | 9 +++++----
 .../Promise/prototype/finally/subclass-resolve-count.js  | 5 ++---
 4 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/test/built-ins/Promise/prototype/finally/resolved-observable-then-calls.js b/test/built-ins/Promise/prototype/finally/resolved-observable-then-calls.js
index abaa8b857d..9d1cf7a184 100644
--- a/test/built-ins/Promise/prototype/finally/resolved-observable-then-calls.js
+++ b/test/built-ins/Promise/prototype/finally/resolved-observable-then-calls.js
@@ -36,5 +36,4 @@ yes.then(function(x) {
 }).then(function() {
   assert.sameValue(sequence.length, 5);
   checkSequence(sequence, "All expected callbacks called in correct order");
-  $DONE();
-}).catch($ERROR);
+}).then($DONE, $DONE);
diff --git a/test/built-ins/Promise/prototype/finally/species-constructor.js b/test/built-ins/Promise/prototype/finally/species-constructor.js
index 1ed3cb3560..6149c38bea 100644
--- a/test/built-ins/Promise/prototype/finally/species-constructor.js
+++ b/test/built-ins/Promise/prototype/finally/species-constructor.js
@@ -20,6 +20,5 @@ class FooPromise extends Promise {
 new FooPromise(r => r())
   .finally(() => {})
   .then(() => {
-    assert.sameValue(count, 6, "6 new promises were created");
-    $DONE();
-  }, $ERROR);
+    assert.sameValue(count, 7, "7 new promises were created");
+  }).then($DONE, $DONE);
diff --git a/test/built-ins/Promise/prototype/finally/subclass-reject-count.js b/test/built-ins/Promise/prototype/finally/subclass-reject-count.js
index 6f9db88fe3..ed90935763 100644
--- a/test/built-ins/Promise/prototype/finally/subclass-reject-count.js
+++ b/test/built-ins/Promise/prototype/finally/subclass-reject-count.js
@@ -16,7 +16,8 @@ class FooPromise extends Promise {
   }
 }
 
-FooPromise.reject().finally(() => {}).then($ERROR).catch(() => {
-  assert.sameValue(7, count);
-  $DONE();
-});
+FooPromise.reject().finally(() => {}).then(value => {
+  throw new Test262Error("Expected Promise to be rejected, got: resolved with " + value);
+}, () => {
+  assert.sameValue(count, 7);
+}).then($DONE, $DONE);
diff --git a/test/built-ins/Promise/prototype/finally/subclass-resolve-count.js b/test/built-ins/Promise/prototype/finally/subclass-resolve-count.js
index 87642c301a..1facba42ba 100644
--- a/test/built-ins/Promise/prototype/finally/subclass-resolve-count.js
+++ b/test/built-ins/Promise/prototype/finally/subclass-resolve-count.js
@@ -17,6 +17,5 @@ class FooPromise extends Promise {
 }
 
 FooPromise.resolve().finally(() => {}).then(() => {
-  assert.sameValue(6, count);
-  $DONE();
-}, $ERROR);
+  assert.sameValue(count, 7);
+}).then($DONE, $DONE);