Don't mock non-enum accessors

src/lib/__tests__/moduleMocker-test.js

@@ -61,6 +61,25 @@ describe('moduleMocker', function() {
       expect(bar.x()).toBe('Bar');
     });
 
+    it('does not mock non-enumerable getters', function() {
+      var foo = Object.defineProperties({}, {
+        nonEnumMethod: {
+          value: function() {}
+        },
+        nonEnumGetter: {
+          get: function() { throw 1; }
+        }
+      });
+      var fooMock = moduleMocker.generateFromMetadata(
+        moduleMocker.getMetadata(foo)
+      );
+
+      expect(typeof foo.nonEnumMethod).toBe('function');
+
+      expect(fooMock.nonEnumMethod.mock).not.toBeUndefined();
+      expect(fooMock.nonEnumGetter).toBeUndefined();
+    });
+
     it('mocks ES6 non-enumerable methods', function() {
         // ES6: class ClassFoo { foo() { } }
         // Converted with https://babeljs.io/repl/

src/lib/moduleMocker.js

@@ -59,7 +59,10 @@ function getSlots(object) {
   // Simply attempting to access any of these throws an error.
   var forbiddenProps = [ 'caller', 'callee', 'arguments' ];
   var collectProp = function(prop) {
-    if (forbiddenProps.indexOf(prop) === -1) {
+    var propDesc = Object.getOwnPropertyDescriptor(object, prop);
+    if (forbiddenProps.indexOf(prop) === -1 &&
+        // Include only enumerable accessors.
+        (propDesc.enumerable || !propDesc.get)) {
       slots[prop] = true;
     }
   };