fix(selectInputsForOutput): send correct message

We were previously sending the result of Array.unshift, which returns the length of the new array,
rather than the new array. This change sends the correct message.

fix #4

Author: hughrawlinson

lib/wekinator-node.js

@@ -185,14 +185,14 @@
     var address = '/selectInputsForOutput';
     self.queuePush({
       address: self.controlEndpoint+address,
-      args: inputs.unshift(output)
+      args: [output].concat(inputs)
     });
   };
   wekinatorNode.prototype.trainOnData = function(data){
     self.startRecording();
     for(var i = 0; i < data.length; i++){
-			// We're not *too* worried about the input arriving before the output
-			// because we handle queueing internally.
+      // We're not *too* worried about the input arriving before the output
+      // because we handle queueing internally.
       self.outputs(data[i].outputs);
       self.inputs(data[i].inputs);
     }

test/wekinator-node.test.js

@@ -237,11 +237,10 @@ test('selectInputsForOutput sends correct OSC message', function() {
 
   wn.connect(() => {
     expect(wn.selectInputsForOutput).toThrow();
-    wn.selectInputsForOutput(0,[1,2,-3]);
-    //TODO: I believe this arg is a bug, see #4
+    wn.selectInputsForOutput(0,[1,2,3]);
     expect(oscMock.UDPPort.prototype.send.mock.calls[0][0]).toEqual({
       address: "/wekinator/control/selectInputsForOutput",
-      args: 4
+      args: [0,1,2,3]
     });
   });
 });