Use $DONE instead of $ERROR to avoid unhandled rejections

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);

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);

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);

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);