From 8c8336c1650ffb9e471ddae530c170260e427181 Mon Sep 17 00:00:00 2001
From: Zach Bloomquist <github@chary.us>
Date: Mon, 3 Feb 2020 16:26:46 -0500
Subject: [PATCH] Add failing test for #2215

---
 test/util/core/function-to-string-test.js | 28 +++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/test/util/core/function-to-string-test.js b/test/util/core/function-to-string-test.js
index 0953daccd..8a4399707 100644
--- a/test/util/core/function-to-string-test.js
+++ b/test/util/core/function-to-string-test.js
@@ -35,4 +35,32 @@ describe("util/core/functionToString", function() {
 
         assert.equals(functionToString.call(obj.doStuff), "doStuff");
     });
+
+    // https://github.com/sinonjs/sinon/issues/2215
+    it("ignores errors thrown by property accessors on thisValue", function() {
+        var obj = {};
+
+        Object.defineProperty(obj, "foo", {
+            enumerable: true,
+            get: function() {
+                throw new Error();
+            }
+        });
+
+        // this will cause `fn` to be after `foo` when enumerated
+        obj.fn = function() {
+            return "foo";
+        };
+
+        // validate that the keys are in the expected order that will cause the bug
+        var keys = Object.keys(obj);
+        assert.equals(keys[0], "foo");
+        assert.equals(keys[1], "fn");
+
+        var spy = createSpy(obj, "fn");
+
+        obj.fn();
+
+        assert.equals(spy.toString(), "fn");
+    });
 });