fix: make fakeAsync test spec timer id global (#1207)

There is a possibility that a single test gets timer id-s from two
separate fakeAsync test zones and the id-s could clash.

This changes makes it such that the next timer id is global and the can
never clash.

Author: vikerman

lib/zone-spec/fake-async-test.ts

@@ -64,7 +64,7 @@ const timers = {
 
 class Scheduler {
   // Next scheduler id.
-  public nextId: number = 1;
+  public static nextId: number = 1;
 
   // Scheduler queue with the tuple of end time and callback function - sorted by end time.
   private _schedulerQueue: ScheduledFunction[] = [];
@@ -90,7 +90,7 @@ class Scheduler {
   scheduleFunction(
       cb: Function, delay: number, args: any[] = [], isPeriodic: boolean = false,
       isRequestAnimationFrame: boolean = false, id: number = -1): number {
-    let currentId: number = id < 0 ? this.nextId++ : id;
+    let currentId: number = id < 0 ? Scheduler.nextId++ : id;
     let endTime = this._currentTime + delay;
 
     // Insert so that scheduler queue remains sorted by end time.
@@ -292,7 +292,7 @@ class FakeAsyncTestZoneSpec implements ZoneSpec {
   }
 
   private _setTimeout(fn: Function, delay: number, args: any[], isTimer = true): number {
-    let removeTimerFn = this._dequeueTimer(this._scheduler.nextId);
+    let removeTimerFn = this._dequeueTimer(Scheduler.nextId);
     // Queue the callback and dequeue the timer on success and error.
     let cb = this._fnAndFlush(fn, {onSuccess: removeTimerFn, onError: removeTimerFn});
     let id = this._scheduler.scheduleFunction(cb, delay, args, false, !isTimer);
@@ -308,7 +308,7 @@ class FakeAsyncTestZoneSpec implements ZoneSpec {
   }
 
   private _setInterval(fn: Function, interval: number, args: any[]): number {
-    let id = this._scheduler.nextId;
+    let id = Scheduler.nextId;
     let completers = {onSuccess: null as any, onError: this._dequeuePeriodicTimer(id)};
     let cb = this._fnAndFlush(fn, completers);