Allow custom ports when usinge Sauce or remote grid

Fixes jasmine#48.

README.md

@@ -226,6 +226,12 @@ exception of the `url` will be used as `capabilties` sent to the grid hub url.
 if no value is specified for the `url` then a default of
 `http://localhost:4445/wd/hub` is used. 
 
+It's common for remote grids to support only a limited set of ports. Check your
+remote grid's documentation to make sure that the port you're using is 
+supported. When using a remote grid, `jasmine-browser-runner` will run on port 
+5555 unless you use the `--port` command line option or specify a port in the
+second parameter to`startServer`.
+
 ## Want more control?
 
 ```javascript

index.js

@@ -84,16 +84,10 @@ module.exports = {
     const useRemote = options.browser && options.browser.useRemoteSeleniumGrid;
     let portRequest;
 
-    if (useSauce || useRemote) {
-      if (options.port) {
-        throw new Error(
-          "Can't specify a port when browser.useSauce or browser.useRemoteSeleniumGrid is true"
-        );
-      }
-
-      portRequest = 5555;
-    } else if (options.port) {
+    if (options.port) {
       portRequest = options.port;
+    } else if (useSauce || useRemote) {
+      portRequest = 5555;
     } else {
       portRequest = 0;
     }

spec/indexSpec.js

@@ -35,7 +35,7 @@ describe('index', function() {
         };
       });
 
-      describe('When not using Sauce Connect', function() {
+      describe('When not using Sauce Connect or remote grid', function() {
         it('uses the specified port', async function() {
           const promise = runSpecs({ port: 12345 }, this.deps);
           await this.waitForServerStart(promise);
@@ -51,6 +51,38 @@ describe('index', function() {
         });
       });
 
+      describe('When using remote grid', function() {
+        it('uses port 5555', async function() {
+          const promise = runSpecs(
+            {
+              browser: {
+                useRemoteSeleniumGrid: true,
+              },
+            },
+            this.deps
+          );
+          await this.waitForServerStart(promise);
+
+          expect(this.server.start).toHaveBeenCalledWith({ port: 5555 });
+        });
+
+        it('uses the specified port', async function() {
+          spyOn(console, 'warn');
+          const promise = runSpecs(
+            {
+              browser: {
+                useRemoteSeleniumGrid: true,
+              },
+              port: 1234,
+            },
+            this.deps
+          );
+          await this.waitForServerStart(promise);
+
+          expect(this.server.start).toHaveBeenCalledWith({ port: 1234 });
+        });
+      });
+
       describe('When using Sauce Connect', function() {
         it('uses port 5555', async function() {
           const promise = runSpecs(
@@ -66,7 +98,8 @@ describe('index', function() {
           expect(this.server.start).toHaveBeenCalledWith({ port: 5555 });
         });
 
-        it('throws if a port is specified', async function() {
+        it('uses the specified port', async function() {
+          spyOn(console, 'warn');
           const promise = runSpecs(
             {
               browser: {
@@ -76,11 +109,9 @@ describe('index', function() {
             },
             this.deps
           );
+          await this.waitForServerStart(promise);
 
-          await expectAsync(promise).toBeRejectedWithError(
-            "Can't specify a port when browser.useSauce or browser.useRemoteSeleniumGrid is true"
-          );
-          expect(this.server.start).not.toHaveBeenCalled();
+          expect(this.server.start).toHaveBeenCalledWith({ port: 1234 });
         });
       });
     });