runat-1128988: Mock 'document' better (Part 2)

For the remote case, we're calling "mocks on" on the server, but we
still need to register the mockCommand converters on the client.

Signed-off-by: Joe Walker <[email protected]>

lib/gcli/test/index.js

@@ -22,6 +22,7 @@ var KeyEvent = require('../util/util').KeyEvent;
 
 var test = require('../commands/test');
 var helpers = require('./helpers');
+var mockCommands = require('./mockCommands');
 var createTerminalAutomator = require('./automators/terminal').createTerminalAutomator;
 require('./suite');
 
@@ -124,6 +125,20 @@ exports.run = function(terminal, isRemote) {
 function setMocks(options, state) {
   var command = 'mocks ' + (state ? 'on' : 'off');
   return options.requisition.updateExec(command).then(function(data) {
+    // We're calling "mocks on" on the server, but we still need to
+    // register the mockCommand converters on the client
+    var requiredConverters = mockCommands.items.filter(function(item) {
+      return item.item === 'converter';
+    });
+
+    if (state) {
+      options.requisition.system.addItems(requiredConverters);
+    }
+    else {
+      options.requisition.system.removeItems(requiredConverters);
+
+    }
+
     if (data.error) {
       throw new Error('Failed to toggle mocks');
     }