Rewrite the browser/connection/remotes-queue.js file to TypeScript (DevExpress#4607)

* rewrite browser/connection/remotes-queue.js to TypeScript

* fix test

Author: miherlosev

Diff for: @types/time-limit-promise/index.d.ts

@@ -0,0 +1,3 @@
+declare module 'time-limit-promise' {
+    export default function (promise: Promise<any>, timeout: number, options?: { resolveWith: any, rejectWith: any }): Promise<any>;
+}

Diff for: src/browser/connection/index.ts

@@ -56,7 +56,7 @@ export default class BrowserConnection extends EventEmitter {
     private opened: boolean;
     private heartbeatTimeout: NodeJS.Timeout | null;
     private pendingTestRunUrl: string | null;
-    private readonly url: string;
+    public readonly url: string;
     public readonly idleUrl: string;
     private forcedIdleUrl: string;
     private readonly initScriptUrl: string;

Diff for: src/browser/connection/remotes-queue.js renamed to src/browser/connection/remotes-queue.ts

@@ -1,19 +1,29 @@
 import { EventEmitter } from 'events';
 import promisifyEvent from 'promisify-event';
 import getTimeLimitedPromise from 'time-limit-promise';
-
+import { Dictionary } from '../../configuration/interfaces';
+import BrowserConnection from './index';
 
 const REMOTE_REDIRECT_TIMEOUT           = 10000;
 const ADDING_CONNECTION_WAITING_TIMEOUT = 10000;
 
+interface PendingConnection {
+    connection: BrowserConnection;
+    readyPromise: Promise<void>;
+}
+
 export default class RemotesQueue {
-    constructor () {
+    private readonly events: EventEmitter;
+    private shiftingTimeout: Promise<void>;
+    private readonly pendingConnections: Dictionary<PendingConnection>;
+
+    public constructor () {
         this.events             = new EventEmitter();
         this.shiftingTimeout    = Promise.resolve();
         this.pendingConnections = {};
     }
 
-    add (remoteConnection) {
+    public add (remoteConnection: BrowserConnection): void {
         const connectionReadyPromise = promisifyEvent(remoteConnection, 'ready')
             .then(() => this.remove(remoteConnection));
 
@@ -25,11 +35,11 @@ export default class RemotesQueue {
         this.events.emit('connection-added', remoteConnection.id);
     }
 
-    remove (remoteConnection) {
+    public remove (remoteConnection: BrowserConnection): void {
         delete this.pendingConnections[remoteConnection.id];
     }
 
-    shift () {
+    public shift (): Promise<BrowserConnection | null> {
         const shiftingPromise = this.shiftingTimeout
             .then(async () => {
                 let headId = Object.keys(this.pendingConnections)[0];

Diff for: test/server/util-test.js

@@ -72,10 +72,10 @@ describe('Utils', () => {
                 expected: {
                     name:            'Other',
                     version:         '0.0',
-                    platform:        'desktop',
-                    os:              { name: 'Windows', version: '0.0' },
+                    platform:        'other',
+                    os:              { name: 'Other', version: '0.0' },
                     engine:          { name: 'Other', version: '0.0' },
-                    prettyUserAgent: 'Other 0.0 / Windows 0.0',
+                    prettyUserAgent: 'Other 0.0 / Other 0.0',
                     userAgent:       'Windows'
                 }
             },