Added e2e tests for commands showsdks, showdevicetypes, start

Author: shazron

spec/commands.spec.js

@@ -16,15 +16,37 @@
     specific language governing permissions and limitations
     under the License.
 */
+var commands = require('../src/commands');
 
 describe('commands end-to-end', function() {
 
     beforeEach(function() {
+      commands.init();
     });
 
     afterEach(function() {
     });
 
-    // it('', function(done) {
-    // });
-});
+    it('command - showsdks', function() {
+      commands.showsdks({ 'no-output': true });
+    });
+
+    it('command - showdevicetypes', function() {
+      commands.showdevicetypes({ 'no-output': true });;
+    });
+
+    it('command - launch', function() {
+      //TODO: - fail();
+      //TODO: add a fixture Simulator .app to launch
+    });
+
+    it('command - install', function() {
+      //TODO: - fail();
+      //TODO: add a fixture Simulator .app to install
+    });
+
+    it('command - start', function() {
+      var devicetypeid = 'iPhone-6';
+      commands.start({ 'devicetypeid': devicetypeid });
+    });
+});

src/commands.js

@@ -36,13 +36,23 @@ var command_lib = {
 
     //jscs:disable disallowUnusedParams
     showsdks: function(args) {
-        lib.showsdks();
+        args = args || {};
+
+        var output = lib.showsdks();
+        if (!args['no-output']) {
+            console.log(output);
+        }
     },
     //jscs:enable disallowUnusedParams
 
     //jscs:disable disallowUnusedParams
     showdevicetypes: function(args) {
-        lib.showdevicetypes();
+        args = args || {};
+
+        var output = lib.showdevicetypes();
+        if (!args['no-output']) {
+            console.log(output);
+        }
     },
     //jscs:enable disallowUnusedParams
 
@@ -82,4 +92,3 @@ var command_lib = {
 };
 
 module.exports = command_lib;
-

src/lib.js

@@ -327,13 +327,15 @@ var lib = {
         var options = { silent: true, runtimes: true };
         var list = simctl.list(options).json;
 
-        console.log('Simulator SDK Roots:');
+        var output = 'Simulator SDK Roots:\n';
         list.runtimes.forEach(function(runtime) {
             if (runtime.availability === '(available)') {
-                console.log(util.format('"%s" (%s)', runtime.name, runtime.buildversion));
-                console.log(util.format('\t(unknown)'));
+                output += util.format('"%s" (%s)\n', runtime.name, runtime.buildversion);
+                output += util.format('\t(unknown)\n');
             }
         });
+
+        return output;
     },
     //jscs:enable disallowUnusedParams
 
@@ -378,9 +380,12 @@ var lib = {
 
     //jscs:disable disallowUnusedParams
     showdevicetypes: function(args) {
+        var output = '';
         this.getdevicetypes().forEach(function(device) {
-            console.log(device);
+            output += util.format('%s\n', device);
         });
+
+        return output;
     },
     //jscs:enable disallowUnusedParams
 
@@ -475,13 +480,7 @@ var lib = {
     },
 
     start: function(devicetypeid) {
-        var device = {};
-        try {
-            device = getDeviceFromDeviceTypeId(devicetypeid);
-        } catch (e) {
-            console.error(e);
-        }
-
+        var device = getDeviceFromDeviceTypeId(devicetypeid);
         simctl.extensions.start(device.id);
     },