From 0487b664cacacf2440e56ce2c44594fec63651d3 Mon Sep 17 00:00:00 2001
From: Michael Abashian <mabashian@ansible.com>
Date: Tue, 28 Feb 2017 16:30:57 -0500
Subject: [PATCH] Fixed unit tests

---
 .../workflow-results.controller-test.js         | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/awx/ui/tests/spec/workflow--results/workflow-results.controller-test.js b/awx/ui/tests/spec/workflow--results/workflow-results.controller-test.js
index 90bc6f582f..91616b8a03 100644
--- a/awx/ui/tests/spec/workflow--results/workflow-results.controller-test.js
+++ b/awx/ui/tests/spec/workflow--results/workflow-results.controller-test.js
@@ -7,7 +7,6 @@ describe('Controller: workflowResults', () => {
     let $controller;
     let workflowResults;
     let $rootScope;
-    let ParseVariableString;
     let workflowResultsService;
     let $interval;
 
@@ -17,10 +16,8 @@ describe('Controller: workflowResults', () => {
         }
     };
 
-    beforeEach(angular.mock.module('VariablesHelper'));
-
     beforeEach(angular.mock.module('workflowResults', ($provide) => {
-        ['PromptDialog', 'Prompt', 'Wait', 'Rest', '$state', 'ProcessErrors', 
+        ['PromptDialog', 'Prompt', 'Wait', 'Rest', '$state', 'ProcessErrors',
          'InitiatePlaybookRun', 'jobLabels', 'workflowNodes', 'count',
         ].forEach((item) => {
             $provide.value(item, {});
@@ -30,8 +27,9 @@ describe('Controller: workflowResults', () => {
         $provide.value('workflowData', workflow_job_json);
         $provide.value('workflowDataOptions', workflow_job_options_json);
         $provide.value('ParseTypeChange', function() {});
+        $provide.value('ParseVariableString', function() {});
         $provide.value('i18n', { '_': (a) => { return a; } });
-        $provide.provider('$stateProvider', { '$get': function() { return function() {} } });
+        $provide.provider('$stateProvider', { '$get': function() { return function() {}; } });
         $provide.service('WorkflowService', function($q) {
             return {
                 buildTree: function() {
@@ -39,14 +37,13 @@ describe('Controller: workflowResults', () => {
                     deferred.resolve(treeData);
                     return deferred.promise;
                 }
-            }
+            };
         });
     }));
 
-    beforeEach(angular.mock.inject(function(_$controller_, _$rootScope_, _ParseVariableString_, _workflowResultsService_, _$interval_){
+    beforeEach(angular.mock.inject(function(_$controller_, _$rootScope_, _workflowResultsService_, _$interval_){
         $controller = _$controller_;
         $rootScope = _$rootScope_;
-        ParseVariableString = _ParseVariableString_;
         workflowResultsService = _workflowResultsService_;
         $interval = _$interval_;
 
@@ -102,7 +99,7 @@ describe('Controller: workflowResults', () => {
             describe('job waiting', () => {
                 beforeEach(() => {
                     jobWaitingWorkflowResultsControllerFixture(null, 'waiting');
-                });           
+                });
 
                 it('should not start elapsed timer', () => {
                     expect(workflowResultsService.createOneSecondTimer).not.toHaveBeenCalled();
@@ -133,4 +130,4 @@ describe('Controller: workflowResults', () => {
             });
         });
     });
-});
\ No newline at end of file
+});