Fix toString() output

src/fake-timers-src.js

@@ -24,7 +24,7 @@
 /**
  * Queues a function to be called during a browser's idle periods
  * @callback RequestIdleCallback
  * @param {function(IdleDeadline)} callback
  * @param {{timeout: number}} options - an options object
  * @returns {number} the id
  */
@@ -51,13 +51,13 @@
 
 /**
  * @typedef RequestAnimationFrame
  * @property {function(number):void} requestAnimationFrame
  * @returns {number} - the id
  */
 
 /**
  * @typedef Performance
  * @property {function(): number} now
  */
 
 /* eslint-disable jsdoc/require-property-description */
@@ -348,7 +348,7 @@
  return timer && timer.callAt >= from && timer.callAt <= to;
  }
 
  /**
  * @param {Clock} clock
  * @param {Timer} job
  */
@@ -428,7 +428,7 @@
  * @param {number} minute
  * @param {number} second
  * @param {number} ms
  * @returns void
  */
  // eslint-disable-next-line no-unused-vars
  constructor(year, month, date, hour, minute, second, ms) {
@@ -456,11 +456,15 @@
  };
  }
 
+ ClockDate.toString = function toString() {
+ return NativeDate.toString();
+ };
+
  // noinspection UnnecessaryLocalVariableJS
  /**
  * A normal Class constructor cannot be called without `new`, but Date can, so we need
  * to wrap it in a Proxy in order to ensure this functionality of Date is kept intact
  * @type {ClockDate}
  */
  const ClockDateProxy = new Proxy(ClockDate, {
  // handler for [[Call]] invocations (i.e. not using `new`)
@@ -656,7 +660,7 @@
  }
 
  /* eslint consistent-return: "off" */
  /**
  * Timer comparitor
  * @param {Timer} a
  * @param {Timer} b
@@ -787,7 +791,7 @@
  }
  }
 
  /**
  * Gets clear handler name for a given timer type
  * @param {string} ttype
  */
@@ -798,7 +802,7 @@
  return `clear${ttype}`;
  }
 
  /**
  * Gets schedule handler name for a given timer type
  * @param {string} ttype
  */
@@ -809,7 +813,7 @@
  return `set${ttype}`;
  }
 
  /**
  * Creates an anonymous function to warn only once
  */
  function createWarnOnce() {
@@ -1053,6 +1057,9 @@
 
  if (isPresent.setImmediate) {
  timers.setImmediate = _global.setImmediate;
+ }
+
+ if (isPresent.clearImmediate) {
  timers.clearImmediate = _global.clearImmediate;
  }
 
@@ -1073,7 +1080,7 @@
  }
 
  if (isPresent.queueMicrotask) {
- timers.queueMicrotask = true;
+ timers.queueMicrotask = _global.queueMicrotask;
  }
 
  if (isPresent.cancelAnimationFrame) {

test/fake-timers-test.js

@@ -3358,8 +3358,8 @@ describe("FakeTimers", function () {
  });
  });
 
- it("mirrors toString", function () {
- assert.same(this.clock.Date.toString, Date.toString);
+ it("mirrors toString output", function () {
+ assert.same(this.clock.Date.toString(), Date.toString());
  });
  });
 
@@ -6069,12 +6069,18 @@ describe("missing timers", function () {
  });
 
  it(`should ignore timers in toFake that are not present in "global" when passed the ignore flag: [${timer}]`, function () {
- //refute.exception(function () {
  FakeTimers.withGlobal({ Date }).install({
  ignoreMissingTimers: true,
  toFake: [timer],
  });
- //});
  });
  });
+
+ it("should throw on trying to use standard timers that are not present on the custom global", function () {
+ assert.exception(function () {
+ FakeTimers.withGlobal({ setTimeout, Date }).install({
+ toFake: ["setInterval"],
+ });
+ }, /cannot be faked: 'setInterval'/);
+ });
 });