fix ts errors and tests

lib/dataconnection.ts

@@ -39,7 +39,6 @@ export class DataConnection extends BaseConnection {
     }
   } = {};
 
-  private _peerBrowser: any;
   private _dc: RTCDataChannel;
   private _encodingQueue = new EncodingQueue();
 
@@ -59,10 +58,6 @@ export class DataConnection extends BaseConnection {
     this.serialization = this.options.serialization || SerializationType.Binary;
     this.reliable = !!this.options.reliable;
 
-    if (this.options._payload) {
-      this._peerBrowser = this.options._payload.browser;
-    }
-
     this._encodingQueue.on('done', (ab: ArrayBuffer) => {
       this._bufferedSend(ab);
     });
@@ -319,9 +314,6 @@ export class DataConnection extends BaseConnection {
 
     switch (message.type) {
       case ServerMessageType.Answer:
-        this._peerBrowser = payload.browser;
-
-        // Forward to negotiator
         this._negotiator.handleSDP(message.type, payload.sdp);
         break;
       case ServerMessageType.Candidate:

lib/peer.ts

@@ -453,13 +453,17 @@ export class Peer extends EventEmitter {
   emitError(type: PeerErrorType, err: string | Error): void {
     logger.error("Error:", err);
 
+    let error: Error & { type?: PeerErrorType };
+
     if (typeof err === "string") {
-      err = new Error(err);
+      error = new Error(err);
+    } else {
+      error = err as Error;
     }
 
-    err.type = type;
+    error.type = type;
 
-    this.emit(PeerEventType.Error, err);
+    this.emit(PeerEventType.Error, error);
   }
 
   /**

lib/socket.ts

@@ -11,7 +11,7 @@ export class Socket extends EventEmitter {
   private _id?: string;
   private _messagesQueue: Array<object> = [];
   private _socket?: WebSocket;
-  private _wsPingTimer?: NodeJS.Timeout;
+  private _wsPingTimer?: any;
   private readonly _baseUrl: string;
 
   constructor(

test/peer.ts

@@ -10,7 +10,7 @@ describe("Peer", function () {
             const peer = new Peer();
 
             expect(peer.connections).to.deep.eq({});
-            expect(peer.id).to.be.undefined;
+            expect(peer.id).to.be.null;
             expect(peer.disconnected).to.be.false;
             expect(peer.destroyed).to.be.false;
 
@@ -75,12 +75,11 @@ describe("Peer", function () {
 
             peer1.destroy();
 
-            setTimeout(() => {
-                expect(peer1.disconnected).to.be.true;
-                expect(peer1.open).to.be.false;
+            expect(peer1.disconnected).to.be.true;
+            expect(peer1.destroyed).to.be.true;
+            expect(peer1.open).to.be.false;
 
-                done();
-            }, 0);
+            done();
         });
 
         after(function () {