From d9171fef3fb1415344641b0ce82c5f4bc53936fc Mon Sep 17 00:00:00 2001
From: Ruben Bridgewater <ruben@bridgewater.de>
Date: Fri, 15 Dec 2017 21:09:12 -0200
Subject: [PATCH] assert: fix throws and doesNotThrow stack frames

The stack frames from .throws and .doesNotThrow got included
even though that was not intended.

PR-URL: https://github.com/nodejs/node/pull/17703
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 lib/assert.js                | 4 ++--
 test/parallel/test-assert.js | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/lib/assert.js b/lib/assert.js
index aca8313a09cbfe..00542b53fc8c87 100644
--- a/lib/assert.js
+++ b/lib/assert.js
@@ -242,7 +242,7 @@ function innerThrows(shouldThrow, block, expected, message) {
         expected,
         operator: 'throws',
         message: `Missing expected exception${details}`,
-        stackStartFn: innerThrows
+        stackStartFn: assert.throws
       });
     }
     if (expected && expectedException(actual, expected) === false) {
@@ -256,7 +256,7 @@ function innerThrows(shouldThrow, block, expected, message) {
         expected,
         operator: 'doesNotThrow',
         message: `Got unwanted exception${details}\n${actual.message}`,
-        stackStartFn: innerThrows
+        stackStartFn: assert.doesNotThrow
       });
     }
     throw actual;
diff --git a/test/parallel/test-assert.js b/test/parallel/test-assert.js
index b64c777bddee9a..6c3aced7b669dc 100644
--- a/test/parallel/test-assert.js
+++ b/test/parallel/test-assert.js
@@ -634,6 +634,7 @@ testAssertionMessage({ a: NaN, b: Infinity, c: -Infinity },
   } catch (e) {
     threw = true;
     assert.strictEqual(e.message, 'Missing expected exception.');
+    assert.ok(!e.stack.includes('throws'), e.stack);
   }
   assert.ok(threw);
 }
@@ -678,6 +679,7 @@ try {
     threw = true;
     assert.ok(e.message.includes(rangeError.message));
     assert.ok(e instanceof assert.AssertionError);
+    assert.ok(!e.stack.includes('doesNotThrow'), e.stack);
   }
   assert.ok(threw);
 }