check propagation of acquisitionerror

1cdd261f · Romain Courteaud · Gabriel Monnerat · 617e6614 · 1cdd261f · 1cdd261f
Commit 1cdd261f authored Oct 18, 2021 by Romain Courteaud Committed by Gabriel Monnerat Oct 22, 2021
Show whitespace changes
Inline Side-by-side

Showing with 8 additions and 7 deletions

test/embedded.js test/embedded.js +7 -1

test/renderjs_test.js test/renderjs_test.js +1 -6

No files found.
--- a/test/embedded.js
+++ b/test/embedded.js
@@ -110,7 +110,13 @@
    .declareAcquiredMethod('plugErrorAcquire',
                          'acquireMethodRequestedWithAcquisitionError')
    .declareMethod('callErrorAcquire', function (param1, param2) {
-      return this.plugErrorAcquire(param1, param2);
+      return this.plugErrorAcquire(param1, param2)
+        .push(undefined, function (error) {
+          if (error instanceof renderJS.AcquisitionError) {
+            throw error;
+          }
+          throw new Error('Expected AcquisitionError: ' + error);
+        });
    })
    .declareMethod('triggerMethodToCancel', function () {
      return new RSVP.Promise(function () {

--- a/test/renderjs_test.js
+++ b/test/renderjs_test.js
@@ -5840,12 +5840,7 @@
            ok(false, result);
          })
          .push(undefined, function (error) {
-            equal(
+            ok(error instanceof renderJS.AcquisitionError, error);
-              error,
-              "AcquisitionError: Can not handle " +
-                "acquireMethodRequestedWithAcquisitionError",
-              error
-            );
          })
          // cancel call is correctly propagated by declareMethod