Fix connection error handling PR

[olcar]

• merged latest changes from orbotix/sphero.js master
• fixed indentation issues

But still failing 2 tests...

$ grunt
Running "lint" task

lib/adaptors/ble.js
  160:31  warning  Too many nested callbacks (4). Maximum allowed is 3  max-nested-callbacks
  162:23  warning  Too many nested callbacks (5). Maximum allowed is 3  max-nested-callbacks
  167:25  warning  Too many nested callbacks (5). Maximum allowed is 3  max-nested-callbacks
  408:0   warning  Line 408 exceeds the maximum line length of 80       max-len

lib/adaptors/serialport.js
  26:0  warning  Line 26 exceeds the maximum line length of 80  max-len

spec/lib/packet.spec.js
  685:0  warning  Line 685 exceeds the maximum line length of 80  max-len

✖ 6 problems (0 errors, 6 warnings)

Running "test" task


  ․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․
  ․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․
  ․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․․
  ․․․․․․․․․․․․․․․․․․․․․․․․․․․․

  271 passing (665ms)
  1 pending
  2 failing

  1) Serialport Adaptor #open "before each" hook:
     TypeError: port.open is not a function
      at Adaptor.open (lib/adaptors/serialport.js:61:8)
      at Context.<anonymous> (spec/lib/adaptors/serialport.spec.js:41:15)

  2) Core commands #getPowerState calls #command with params:
     AssertionError: expected command to have been called with arguments 0, 32, null, function spy() {}
    command(0, 32, null, function () {})
      at Context.<anonymous> (spec/lib/devices/core.spec.js:66:34)



Warning: Task "test" failed. Use --force to continue.

Aborted due to warnings.