Skip to content

Commit

Permalink
feat: add an option to use native timer functions (#1479)
Browse files Browse the repository at this point in the history
This allows to control the behavior of mocked timers (@sinonjs/fake-timers),
depending on the value of the "useNativeTimers" option:

- true: use native setTimeout function
- false (default): use classic timers, that may be mocked

Related: socketio/engine.io-client@5d1d5be
  • Loading branch information
vartan authored and darrachequesne committed Aug 29, 2021
1 parent f3acddf commit 4e1b656
Show file tree
Hide file tree
Showing 4 changed files with 499 additions and 1,527 deletions.
15 changes: 13 additions & 2 deletions lib/manager.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import * as eio from "engine.io-client";
import { installTimerFunctions } from "engine.io-client/lib/util";
import { Socket, SocketOptions } from "./socket";
import * as parser from "socket.io-parser";
import { Decoder, Encoder, Packet } from "socket.io-parser";
Expand Down Expand Up @@ -203,6 +204,14 @@ interface EngineOptions {
* @default true
*/
closeOnBeforeunload: boolean;

/**
* Whether to always use the native timeouts. This allows the client to
* reconnect when the native timeout functions are overridden, such as when
* mock clocks are installed.
* @default false
*/
useNativeTimers: boolean;
}

export interface ManagerOptions extends EngineOptions {
Expand Down Expand Up @@ -321,6 +330,7 @@ export class Manager<
private nsps: Record<string, Socket> = {};
private subs: Array<ReturnType<typeof on>> = [];
private backoff: Backoff;
private setTimeoutFn: typeof setTimeout;
private _reconnection: boolean;
private _reconnectionAttempts: number;
private _reconnectionDelay: number;
Expand Down Expand Up @@ -358,6 +368,7 @@ export class Manager<

opts.path = opts.path || "/socket.io";
this.opts = opts;
installTimerFunctions(this, opts);
this.reconnection(opts.reconnection !== false);
this.reconnectionAttempts(opts.reconnectionAttempts || Infinity);
this.reconnectionDelay(opts.reconnectionDelay || 1000);
Expand Down Expand Up @@ -542,7 +553,7 @@ export class Manager<
}

// set timer
const timer = setTimeout(() => {
const timer = this.setTimeoutFn(() => {
debug("connect attempt timed out after %d", timeout);
openSubDestroy();
socket.close();
Expand Down Expand Up @@ -769,7 +780,7 @@ export class Manager<
debug("will wait %dms before reconnect attempt", delay);

this._reconnecting = true;
const timer = setTimeout(() => {
const timer = this.setTimeoutFn(() => {
if (self.skipReconnect) return;

debug("attempting reconnect");
Expand Down
Loading

0 comments on commit 4e1b656

Please sign in to comment.