fix(tests): tests ensuring locationType: auto will fail now pass inst…

…ead of hang
isleofcode · Jul 21, 2016 · a40fedf · a40fedf
1 parent 58c721c
commit a40fedf
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 6 deletions.
diff --git a/node-tests/unit/commands/build-test.js b/node-tests/unit/commands/build-test.js
@@ -1,6 +1,8 @@
 'use strict';
 
 const td = require('testdouble');
+
+const ui = require('../../../lib/utils/ui');
 const expect = require('../../helpers/expect');
 const Promise = require('ember-cli/lib/ext/promise');
 
@@ -19,8 +21,8 @@ describe('Build Command', () => {
  BuildCmd.project.config = function() {
  return {
  locationType: 'hash'
- }
- }
+ };
+ };
  });
 
  afterEach(() => {
@@ -93,8 +95,13 @@ describe('Build Command', () => {
  BuildCmd.project.config = function() {
  return {
  locationType: 'auto'
- }
- }
+ };
+ };
+
+ td.replace(ui, 'writeLine', () => {
+ throw new Error('Exit Called');
+ });
+
  });
 
  it('throws', () => {

diff --git a/node-tests/unit/commands/open-test.js b/node-tests/unit/commands/open-test.js
@@ -22,7 +22,6 @@ describe('Open Command', () => {
  });
 
  it('runs Open App Task', () => {
-
  return OpenCmd.run.call({ project: mockProject }, { application: 'dummy', platform: 'ios' })
  .then(function() {
  return true;

diff --git a/node-tests/unit/commands/serve-test.js b/node-tests/unit/commands/serve-test.js
@@ -4,6 +4,7 @@ const td = require('testdouble');
 const expect = require('../../helpers/expect');
 const Promise = require('ember-cli/lib/ext/promise');
 
+const ui = require('../../../lib/utils/ui');
 const ServeCmd = require('../../../lib/commands/serve');
 const BuildWatchTask = require('../../../lib/tasks/ember-build-watch');
 const CdvBuildTask = require('../../../lib/tasks/cordova-build');
@@ -94,8 +95,12 @@ describe('Serve Command', () => {
  ServeCmd.project.config = function() {
  return {
  locationType: 'auto'
- }
+ };
  };
+
+ td.replace(ui, 'writeLine', () => {
+ throw new Error('Exit Called');
+ });
  });
 
  it('throws', () => {