chore: Fix typos made during refactoring (#98)

lib/subcommands/bootstatus.js

@@ -5,6 +5,16 @@ import { waitForCondition } from 'asyncbox';
 
 const commands = {};
 
+/**
+ * @typedef {Object} BootMonitorOptions
+ * @property {?number} timeout [240000] - Simulator booting timeout in ms.
+ * @property {?Function} onWaitingDataMigration - This event is fired when data migration stage starts.
+ * @property {?Function} onWaitingSystemApp - This event is fired when system app wait stage starts.
+ * @property {?Function} onFinished - This event is fired when Simulator is fully booted.
+ * @property {?Function} onError - This event is fired when there was an error while monitoring the booting process
+ * or when the timeout has expired.
+ */
+
 /**
  * Start monitoring for boot status of the particular Simulator.
  * If onFinished property is not set then the method will block

lib/subcommands/get_app_container.js

@@ -8,8 +8,6 @@ const commands = {};
  * It is required that Simulator is in _booted_ state.
  *
  * @param {string} bundleId - Bundle identifier of an application.
- * @param {?boolean} logErrors [true] - Whether to include exec's command
- *                                      stderr output into exception message if thrown.
  * @param {?string} containerType - Which container type to return. Possible values
  *                                  are 'app', 'data', 'groups', '<A specific App Group container>'.
  *                                  The default value is 'app'.

lib/subcommands/launch.js

@@ -1,3 +1,4 @@
+import _ from 'lodash';
 import { retryInterval } from 'asyncbox';
 
 const commands = {};
@@ -11,16 +12,18 @@ const commands = {};
  *                            which is going to be removed.
  * @param {number} tries [5] - The maximum number of retries before
  *                             throwing an exception.
+ * @return {string} the actual command output
  * @throws {Error} If the corresponding simctl subcommand command
  *                 returns non-zero return code.
  * @throws {Error} If the `udid` instance property is unset
  */
 commands.launchApp = async function launchApp (bundleId, tries = 5) {
-  await retryInterval(tries, 1000,
-    async () => await this.exec('launch', {
-      args: [this.requireUdid('launch'), bundleId]
-    })
-  );
+  return await retryInterval(tries, 1000, async () => {
+    const {stdout} = await this.exec('launch', {
+      args: [this.requireUdid('launch'), bundleId],
+    });
+    return _.trim(stdout);
+  });
 };
 
 export default commands;

lib/subcommands/spawn.js

@@ -14,8 +14,8 @@ const commands = {};
  *                 returns non-zero return code.
  * @throws {Error} If the `udid` instance property is unset
  */
-commands.spawnProcess = async function spawn (args, env = {}) {
-  if (_.isNil(args)) {
+commands.spawnProcess = async function spawnProcess (args, env = {}) {
+  if (_.isEmpty(args)) {
     throw new Error('Spawn arguments are required');
   }
 
@@ -35,7 +35,7 @@ commands.spawnProcess = async function spawn (args, env = {}) {
  * @throws {Error} If the `udid` instance property is unset
  */
 commands.spawnSubProcess = async function spawnSubProcess (args, env = {}) {
-  if (_.isNil(args)) {
+  if (_.isEmpty(args)) {
     throw new Error('Spawn arguments are required');
   }