Address comments + other tidying up

Author: KaptenJansson

.gitignore

@@ -1,4 +1,5 @@
 browsers/
+browsers-tmp
 firefox-*.tar.bz2
 .DS_Store
 node_modules/

test/run-tests

@@ -1,7 +1,12 @@
-#!/bin/sh
 #
-# Run testling with a default set of parameters
+#  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
 #
+#  Use of this source code is governed by a BSD-style license
+#  that can be found in the LICENSE file in the root of the source
+#  tree.
+#!/bin/sh
+
+# Run testling with a default set of parameters
 BINDIR=./browsers/bin
 export BROWSER=${BROWSER-chrome}
 export BVER=${BVER-stable}

test/run-tests.js

@@ -5,7 +5,7 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* jshint node: true */
+/* eslint-env node */
 
 'use strict';
 var test = require('tape');

test/test.js

@@ -5,7 +5,7 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* jshint node: true */
+/* eslint-env node */
 
 'use strict';
 
@@ -32,18 +32,23 @@ test('Check Selenium lib buildDriver method', function(t) {
 });
 
 test('Check Selenium lib getStats method', function(t) {
+  if (process.env.BROWSER === 'firefox') {
+    t.skip('getStats not supported on Firefox.');
+    t.end();
+    return;
+  }
   var driver = require('../main.js').seleniumLib.buildDriver();
   var getStats = require('../main.js').seleniumLib.getStats;
 
   driver.get('file://' + process.cwd() + '/test/testpage.html')
   .then(function() {
     t.plan(3);
     t.pass('Page loaded');
-    return driver.executeScript('window.pc1 = new RTCPeerConnection;' +
+    return driver.executeScript('window.pc1 = new RTCPeerConnection();' +
         'return window.pc1;');
   })
   .then(function(peerConnection) {
-    if (typeof peerConnection === 'object') {
+    if (typeof peerConnection.remoteDescription === 'object') {
       t.pass('PeerConnection created, calling on getStats.')
       return getStats(driver, 'pc1');
     }