elasticity 5.0.3 → 6.0

data/spec/lib/elasticity/ganglia_bootstrap_action_spec.rb

@@ -6,8 +6,13 @@ describe Elasticity::GangliaBootstrapAction do
 
   it { should be_a Elasticity::BootstrapAction }
 
-  its(:name) { should == 'Elasticity Bootstrap Action (Install Ganglia)' }
-  its(:arguments) { should == [] }
-  its(:script) { should == 's3://elasticmapreduce/bootstrap-actions/install-ganglia' }
+  describe '.intialize' do
+    it 'should set the fields appropriately' do
+      expect(subject.name).to eql('Elasticity Bootstrap Action (Install Ganglia)')
+      expect(subject.arguments).to eql([])
+      expect(subject.script).to eql('s3://elasticmapreduce/bootstrap-actions/install-ganglia')
+    end
+  end
+
 
 end

data/spec/lib/elasticity/hadoop_bootstrap_action_spec.rb

@@ -6,8 +6,13 @@ describe Elasticity::HadoopBootstrapAction do
 
   it { should be_a Elasticity::BootstrapAction }
 
-  its(:name) { should == 'Elasticity Bootstrap Action (Configure Hadoop)' }
-  its(:arguments) { should == %w(option value) }
-  its(:script) { should == 's3n://elasticmapreduce/bootstrap-actions/configure-hadoop' }
+  describe '.initialize' do
+    it 'should set the fields appropriately' do
+      expect(subject.name).to eql('Elasticity Bootstrap Action (Configure Hadoop)')
+      expect(subject.arguments).to eql(%w(option value))
+      expect(subject.script).to eql('s3n://elasticmapreduce/bootstrap-actions/configure-hadoop')
+    end
+  end
+
 
 end

data/spec/lib/elasticity/hadoop_file_bootstrap_action_spec.rb

@@ -6,8 +6,12 @@ describe Elasticity::HadoopFileBootstrapAction do
 
   it { should be_a Elasticity::BootstrapAction }
 
-  its(:name) { should == 'Elasticity Bootstrap Action (Configure Hadoop via File)' }
-  its(:arguments) { should == %w(--mapred-config-file config_file) }
-  its(:script) { should == 's3n://elasticmapreduce/bootstrap-actions/configure-hadoop' }
+  describe '.initialize' do
+    it 'should set the fields appropriately' do
+      expect(subject.name).to eql('Elasticity Bootstrap Action (Configure Hadoop via File)')
+      expect(subject.arguments).to eql(%w(--mapred-config-file config_file))
+      expect(subject.script).to eql('s3n://elasticmapreduce/bootstrap-actions/configure-hadoop')
+    end
+  end
 
 end

data/spec/lib/elasticity/hive_step_spec.rb

@@ -6,29 +6,33 @@ describe Elasticity::HiveStep do
 
   it { should be_a Elasticity::JobFlowStep }
 
-  its(:name) { should == 'Elasticity Hive Step (script.hql)' }
-  its(:script) { should == 'script.hql' }
-  its(:variables) { should == {} }
-  its(:action_on_failure) { should == 'TERMINATE_JOB_FLOW' }
+  describe '.initialize' do
+    it 'should set the fields appropriately' do
+      expect(subject.name).to eql('Elasticity Hive Step (script.hql)')
+      expect(subject.script).to eql('script.hql')
+      expect(subject.variables).to eql({})
+      expect(subject.action_on_failure).to eql('TERMINATE_JOB_FLOW')
+    end
+  end
 
   describe '#to_aws_step' do
 
     it 'should convert to aws step format' do
-      step = subject.to_aws_step(Elasticity::JobFlow.new('access', 'secret'))
+      step = subject.to_aws_step(Elasticity::JobFlow.new)
       step[:name].should == 'Elasticity Hive Step (script.hql)'
       step[:action_on_failure].should == 'TERMINATE_JOB_FLOW'
       step[:hadoop_jar_step][:jar].should == 's3://elasticmapreduce/libs/script-runner/script-runner.jar'
       step[:hadoop_jar_step][:args].should start_with([
-        's3://elasticmapreduce/libs/hive/hive-script',
-        '--base-path',
-        's3://elasticmapreduce/libs/hive/',
-        '--hive-versions',
-        'latest',
-        '--run-hive-script',
-        '--args',
-        '-f',
-        'script.hql'
-      ])
+            's3://elasticmapreduce/libs/hive/hive-script',
+            '--base-path',
+            's3://elasticmapreduce/libs/hive/',
+            '--hive-versions',
+            'latest',
+            '--run-hive-script',
+            '--args',
+            '-f',
+            'script.hql'
+          ])
     end
 
     context 'when variables are provided' do
@@ -42,7 +46,7 @@ describe Elasticity::HiveStep do
       end
 
       it 'should convert to aws step format' do
-        step = hs_with_variables.to_aws_step(Elasticity::JobFlow.new('access', 'secret'))
+        step = hs_with_variables.to_aws_step(Elasticity::JobFlow.new)
         step[:hadoop_jar_step][:args][9..13].should == %w(-d VAR1=VALUE1 -d VAR2=VALUE2)
       end
     end
@@ -51,7 +55,7 @@ describe Elasticity::HiveStep do
 
   describe '.requires_installation?' do
     it 'should require installation' do
-      Elasticity::HiveStep.requires_installation?.should be_true
+      expect(Elasticity::HiveStep.requires_installation?).to be true
     end
   end
 

data/spec/lib/elasticity/instance_group_spec.rb

@@ -1,10 +1,14 @@
 describe Elasticity::InstanceGroup do
 
-  its(:bid_price) { should == nil }
-  its(:count) { should == 1 }
-  its(:type) { should == 'm1.small' }
-  its(:market) { should == 'ON_DEMAND' }
-  its(:role) { should == 'CORE' }
+  describe '.initialize' do
+    it 'should set the fields appropriately' do
+      expect(subject.bid_price).to eql(nil)
+      expect(subject.count).to eql(1)
+      expect(subject.type).to eql('m1.small')
+      expect(subject.market).to eql('ON_DEMAND')
+      expect(subject.role).to eql('CORE')
+    end
+  end
 
   describe '#count=' do
 

data/spec/lib/elasticity/job_flow_integration_spec.rb

@@ -3,7 +3,7 @@ describe 'Elasticity::JobFlow Integration Examples' do
   let(:emr) { double('Elasticity::EMR') }
 
   before do
-    Elasticity::EMR.should_receive(:new).with('access', 'secret', :region => 'us-west-1').and_return(emr)
+    Elasticity::EMR.should_receive(:new).with(:region => 'us-west-1').and_return(emr)
   end
 
   describe 'Hive' do
@@ -16,7 +16,7 @@ describe 'Elasticity::JobFlow Integration Examples' do
     end
 
     let(:hive_jobflow) do
-      Elasticity::JobFlow.new('access', 'secret').tap do |jf|
+      Elasticity::JobFlow.new.tap do |jf|
         jf.placement = 'us-west-1a'
         jf.log_uri = 's3n://slif-test/output/logs'
         jf.add_step(hive_step)
@@ -101,7 +101,7 @@ describe 'Elasticity::JobFlow Integration Examples' do
     end
 
     let(:pig_jobflow) do
-      Elasticity::JobFlow.new('access', 'secret').tap do |jf|
+      Elasticity::JobFlow.new.tap do |jf|
         jf.placement = 'us-west-1c'
         jf.instance_count = 8
         jf.slave_instance_type = 'm1.xlarge'
@@ -189,7 +189,7 @@ describe 'Elasticity::JobFlow Integration Examples' do
     end
 
     let(:custom_jar_jobflow) do
-      Elasticity::JobFlow.new('access', 'secret').tap do |jf|
+      Elasticity::JobFlow.new.tap do |jf|
         jf.placement = 'us-west-1b'
         jf.log_uri = 's3n://slif-test/output/logs'
         jf.add_step(custom_jar_step)

data/spec/lib/elasticity/job_flow_spec.rb

@@ -1,41 +1,30 @@
 describe Elasticity::JobFlow do
 
   subject do
-    Elasticity::JobFlow.new('access', 'secret')
+    Elasticity::JobFlow.new
   end
 
-  its(:action_on_failure) { should == 'TERMINATE_JOB_FLOW' }
-  its(:ec2_key_name) { should == nil }
-  its(:name) { should == 'Elasticity Job Flow' }
-  its(:instance_count) { should == 2 }
-  its(:log_uri) { should == nil }
-  its(:master_instance_type) { should == 'm1.small' }
-  its(:slave_instance_type) { should == 'm1.small' }
-  its(:ami_version) { should == 'latest' }
-  its(:keep_job_flow_alive_when_no_steps) { should == false }
-  its(:ec2_subnet_id) { should == nil }
-  its(:placement) { should == 'us-east-1a' }
-  its(:region) { should == 'us-east-1' }
-  its(:visible_to_all_users) { should == false }
-  its(:enable_debugging) { should == false }
-  its(:job_flow_role) { should == nil }
-  its(:service_role) { should == nil }
-
-  its(:access_key) { should == 'access' }
-  its(:secret_key) { should == 'secret' }
-
   describe '.initialize' do
-    it 'should set the access and secret keys to nil by default' do
-      Elasticity::JobFlow.new.tap do |j|
-        j.access_key.should == nil
-        j.secret_key.should == nil
-      end
-      Elasticity::JobFlow.new('_') do |j|
-        j.secret_key.should == nil
-      end
+    it 'should set the fields appropriately' do
+      expect(subject.action_on_failure).to eql('TERMINATE_JOB_FLOW')
+      expect(subject.ec2_key_name).to eql(nil)
+      expect(subject.name).to eql('Elasticity Job Flow')
+      expect(subject.instance_count).to eql(2)
+      expect(subject.log_uri).to eql(nil)
+      expect(subject.master_instance_type).to eql('m1.small')
+      expect(subject.slave_instance_type).to eql('m1.small')
+      expect(subject.ami_version).to eql('latest')
+      expect(subject.keep_job_flow_alive_when_no_steps).to eql(false)
+      expect(subject.ec2_subnet_id).to eql(nil)
+      expect(subject.placement).to eql('us-east-1a')
+      expect(subject.region).to eql('us-east-1')
+      expect(subject.visible_to_all_users).to eql(false)
+      expect(subject.enable_debugging).to eql(false)
+      expect(subject.job_flow_role).to eql(nil)
+      expect(subject.service_role).to eql(nil)
     end
   end
-
+  
   describe '#placement=' do
 
     context 'when the placement is set' do
@@ -213,7 +202,7 @@ describe Elasticity::JobFlow do
       let(:emr) { double('Elasticity::EMR', :run_job_flow => 'RUNNING_JOBFLOW_ID') }
 
       let(:running_jobflow) do
-        Elasticity::JobFlow.new('access', 'secret').tap do |jf|
+        Elasticity::JobFlow.new.tap do |jf|
           jf.add_step(Elasticity::PigStep.new('_'))
         end
       end
@@ -230,8 +219,8 @@ describe Elasticity::JobFlow do
 
           it 'should submit the step' do
             emr.should_receive(:add_jobflow_steps).with('RUNNING_JOBFLOW_ID', {
-              :steps => [additional_step.to_aws_step(running_jobflow)]
-            })
+                :steps => [additional_step.to_aws_step(running_jobflow)]
+              })
             running_jobflow.add_step(additional_step)
           end
         end
@@ -241,8 +230,8 @@ describe Elasticity::JobFlow do
 
           it 'should submit the installation step and the step' do
             emr.should_receive(:add_jobflow_steps).with('RUNNING_JOBFLOW_ID', {
-              :steps => Elasticity::HiveStep.aws_installation_steps << additional_step.to_aws_step(running_jobflow)
-            })
+                :steps => Elasticity::HiveStep.aws_installation_steps << additional_step.to_aws_step(running_jobflow)
+              })
             running_jobflow.add_step(additional_step)
           end
         end
@@ -255,8 +244,8 @@ describe Elasticity::JobFlow do
 
         it 'should submit the step' do
           emr.should_receive(:add_jobflow_steps).with('RUNNING_JOBFLOW_ID', {
-            :steps => [additional_step.to_aws_step(running_jobflow)]
-          })
+              :steps => [additional_step.to_aws_step(running_jobflow)]
+            })
           running_jobflow.add_step(additional_step)
         end
 
@@ -280,17 +269,17 @@ describe Elasticity::JobFlow do
 
       let(:jobflow_steps) { [Elasticity::HiveStep.new('script.hql'), Elasticity::PigStep.new('script.pig'), Elasticity::CustomJarStep.new('script.jar')] }
       let(:jobflow_with_steps) do
-        Elasticity::JobFlow.new('_', '_').tap do |jobflow|
+        Elasticity::JobFlow.new.tap do |jobflow|
           jobflow_steps.each { |s| jobflow.add_step(s) }
         end
       end
       let(:aws_steps) do
         [
           Elasticity::HiveStep.aws_installation_steps,
-            jobflow_steps[0].to_aws_step(jobflow_with_steps),
-            Elasticity::PigStep.aws_installation_steps,
-            jobflow_steps[1].to_aws_step(jobflow_with_steps),
-            jobflow_steps[2].to_aws_step(jobflow_with_steps),
+          jobflow_steps[0].to_aws_step(jobflow_with_steps),
+          Elasticity::PigStep.aws_installation_steps,
+          jobflow_steps[1].to_aws_step(jobflow_with_steps),
+          jobflow_steps[2].to_aws_step(jobflow_with_steps),
         ].flatten
       end
 
@@ -325,7 +314,7 @@ describe Elasticity::JobFlow do
 
       context 'when a log URI is specified' do
         let(:jobflow_with_log_uri) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.log_uri = 'LOG_URI'
           end
         end
@@ -336,7 +325,7 @@ describe Elasticity::JobFlow do
 
       context 'when a log URI is not specified' do
         let(:jobflow_with_no_log_uri) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.log_uri = nil
           end
         end
@@ -351,7 +340,7 @@ describe Elasticity::JobFlow do
 
       context 'when a job flow role is specified' do
         let(:jobflow_with_job_flow_role) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.job_flow_role = 'JOB_FLOW_ROLE'
           end
         end
@@ -362,7 +351,7 @@ describe Elasticity::JobFlow do
 
       context 'when a job flow role is not specified' do
         let(:jobflow_with_no_job_flow_role) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.job_flow_role = nil
           end
         end
@@ -377,7 +366,7 @@ describe Elasticity::JobFlow do
 
       context 'when a service role is specified' do
         let(:jobflow_with_service_role) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.service_role = 'SERVICE_ROLE'
           end
         end
@@ -388,7 +377,7 @@ describe Elasticity::JobFlow do
 
       context 'when a service role is not specified' do
         let(:jobflow_with_no_service_role) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             jf.service_role = nil
           end
         end
@@ -410,7 +399,7 @@ describe Elasticity::JobFlow do
           ]
         end
         let(:jobflow_with_bootstrap_actions) do
-          Elasticity::JobFlow.new('_', '_').tap do |jf|
+          Elasticity::JobFlow.new.tap do |jf|
             hadoop_bootstrap_actions.each do |action|
               jf.add_bootstrap_action(action)
             end
@@ -419,8 +408,8 @@ describe Elasticity::JobFlow do
         it 'should include them in the jobflow config' do
           bootstrap_actions = hadoop_bootstrap_actions.map { |a| a.to_aws_bootstrap_action }
           jobflow_with_bootstrap_actions.send(:jobflow_config).should be_a_hash_including({
-            :bootstrap_actions => bootstrap_actions
-          })
+                :bootstrap_actions => bootstrap_actions
+              })
         end
       end
 
@@ -544,7 +533,7 @@ describe Elasticity::JobFlow do
 
     context 'when there are steps added' do
       let(:jobflow_with_steps) do
-        Elasticity::JobFlow.new('STEP_TEST_ACCESS', 'STEP_TEST_SECRET').tap do |jf|
+        Elasticity::JobFlow.new.tap do |jf|
           jf.add_step(Elasticity::CustomJarStep.new('_'))
         end
       end
@@ -553,7 +542,7 @@ describe Elasticity::JobFlow do
         let(:emr) { double('Elasticity::EMR', :run_job_flow => 'JOBFLOW_ID') }
 
         it 'should run the job with the supplied EMR credentials' do
-          Elasticity::EMR.stub(:new).with('STEP_TEST_ACCESS', 'STEP_TEST_SECRET', :region => 'us-east-1').and_return(emr)
+          Elasticity::EMR.stub(:new).with(:region => 'us-east-1').and_return(emr)
           emr.should_receive(:run_job_flow)
           jobflow_with_steps.run
         end
@@ -588,34 +577,68 @@ describe Elasticity::JobFlow do
 
   end
 
-  describe '#status' do
+  describe '#cluster_status' do
 
     context 'before the jobflow has been run' do
       it 'should raise an error' do
         expect {
-          subject.status
+          subject.cluster_status
         }.to raise_error(Elasticity::JobFlowNotStartedError, 'Please #run this job flow before attempting to retrieve status.')
       end
     end
 
     context 'after the jobflow has been run' do
       let(:emr) { double('Elasticity::EMR', :run_job_flow => 'JOBFLOW_ID') }
-      let(:running_jobflow) { Elasticity::JobFlow.new('_', '_') }
-      let(:jobflow_status) do
-        Elasticity::JobFlowStatus.new.tap do |js|
-          js.stub(:state => 'TERMINATED')
-        end
+      let(:running_jobflow) { Elasticity::JobFlow.new }
+      let(:aws_cluster_status) { JSON.parse('{ "Cluster": { "Status": { "State": "TERMINATED" } } }') }
+
+      before do
+        Elasticity::EMR.stub(:new).and_return(emr)
+        running_jobflow.add_step(Elasticity::CustomJarStep.new('_'))
+        running_jobflow.run
+      end
+
+      it 'should return the AWS status' do
+        emr.should_receive(:describe_cluster).with('JOBFLOW_ID').and_return(aws_cluster_status)
+        Elasticity::ClusterStatus.should_receive(:from_aws_data).with(aws_cluster_status).and_return(build(:cluster_status))
+
+        status = running_jobflow.cluster_status
+        expect(status).to be_a(Elasticity::ClusterStatus)
+        expect(status.state).to eql('TERMINATED')
       end
+    end
+
+  end
+
+  describe '#cluster_step_status' do
+
+    context 'before the jobflow has been run' do
+      it 'should raise an error' do
+        expect {
+          subject.cluster_step_status
+        }.to raise_error(Elasticity::JobFlowNotStartedError, 'Please #run this job flow before attempting to retrieve status.')
+      end
+    end
+
+    context 'after the jobflow has been run' do
+      let(:emr) { double('Elasticity::EMR', :run_job_flow => 'JOBFLOW_ID') }
+      let(:running_jobflow) { Elasticity::JobFlow.new }
+
       before do
         Elasticity::EMR.stub(:new).and_return(emr)
         running_jobflow.add_step(Elasticity::CustomJarStep.new('_'))
         running_jobflow.run
       end
+
       it 'should return the AWS status' do
-        emr.should_receive(:describe_jobflow).with('JOBFLOW_ID').and_return(jobflow_status)
-        status = running_jobflow.status
-        status.should be_a(Elasticity::JobFlowStatus)
-        status.state.should == 'TERMINATED'
+        emr.should_receive(:list_steps).with('JOBFLOW_ID').and_return('AWS_CLUSTER_STATUS')
+        Elasticity::ClusterStepStatus.should_receive(:from_aws_list_data).with('AWS_CLUSTER_STATUS').and_return([build(:cluster_step_status)])
+
+        status = running_jobflow.cluster_step_status
+        status.each do |s|
+          expect(s).to be_a(Elasticity::ClusterStepStatus)
+        end
+        expect(status[0].state).to eql('COMPLETED')
       end
     end
 
@@ -624,7 +647,7 @@ describe Elasticity::JobFlow do
   describe '#shutdown' do
 
     context 'when the jobflow has not yet been started' do
-      let(:unstarted_job_flow) { Elasticity::JobFlow.new('_', '_') }
+      let(:unstarted_job_flow) { Elasticity::JobFlow.new }
       it 'should be an error' do
         expect {
           unstarted_job_flow.shutdown
@@ -634,7 +657,7 @@ describe Elasticity::JobFlow do
 
     context 'when the jobflow has been started' do
       let(:emr) { double('Elasticity::EMR', :run_job_flow => 'JOBFLOW_ID') }
-      let(:running_jobflow) { Elasticity::JobFlow.new('_', '_') }
+      let(:running_jobflow) { Elasticity::JobFlow.new }
       before do
         Elasticity::EMR.stub(:new).and_return(emr)
         running_jobflow.add_step(Elasticity::CustomJarStep.new('_'))
@@ -664,7 +687,7 @@ describe Elasticity::JobFlow do
     context 'when the jobflow is active' do
       let(:jobflow_status) { double(:active? => true) }
       before do
-        subject.stub(:status).and_return(jobflow_status)
+        subject.stub(:cluster_status).and_return(jobflow_status)
       end
       it 'returns true and the result of #status' do
         subject.send(:retry_check).should == [true, jobflow_status]
@@ -674,7 +697,7 @@ describe Elasticity::JobFlow do
     context 'when the jobflow is not active' do
       let(:jobflow_status) { double(:active? => false) }
       before do
-        subject.stub(:status).and_return(jobflow_status)
+        subject.stub(:cluster_status).and_return(jobflow_status)
       end
       it 'returns true and the result of #status' do
         subject.send(:retry_check).should == [false, jobflow_status]
@@ -686,24 +709,24 @@ describe Elasticity::JobFlow do
   describe '.from_jobflow_id' do
 
     before do
-      Elasticity::JobFlow.any_instance.stub_chain(:status, :installed_steps => [])
+      Elasticity::JobFlow.any_instance.stub(:cluster_step_status).and_return([])
     end
 
-    let(:jobflow) { Elasticity::JobFlow.from_jobflow_id('ACCESS', 'SECRET', 'JOBFLOW_ID') }
+    let(:jobflow) { Elasticity::JobFlow.from_jobflow_id('JOBFLOW_ID') }
 
     describe 'creating a jobflow with the specified credentials' do
 
       context 'when the region is not specified' do
         it 'should use the default of us-east-1a' do
-          j = Elasticity::JobFlow.from_jobflow_id('ACCESS', 'SECRET', '_')
-          j.send(:emr).should == Elasticity::EMR.new('ACCESS', 'SECRET', :region => 'us-east-1')
+          j = Elasticity::JobFlow.from_jobflow_id('_')
+          j.send(:emr).should == Elasticity::EMR.new(:region => 'us-east-1')
         end
       end
 
       context 'when the region is specified' do
         it 'should use the specified region' do
-          j = Elasticity::JobFlow.from_jobflow_id('ACCESS', 'SECRET', '_', 'us-west-1')
-          j.send(:emr).should == Elasticity::EMR.new('ACCESS', 'SECRET', :region => 'us-west-1')
+          j = Elasticity::JobFlow.from_jobflow_id('_', 'us-west-1')
+          j.send(:emr).should == Elasticity::EMR.new(:region => 'us-west-1')
         end
       end
 
@@ -723,7 +746,7 @@ describe Elasticity::JobFlow do
 
     context 'when no steps have been installed' do
       before do
-        Elasticity::JobFlow.any_instance.should_receive(:status).and_return(double('Elasticity::JobFlowStatus', :installed_steps => []))
+        Elasticity::JobFlow.any_instance.should_receive(:cluster_step_status).and_return([])
       end
       it 'should show that no steps are installed' do
         jobflow.instance_variable_get(:@installed_steps).should == []
@@ -732,7 +755,10 @@ describe Elasticity::JobFlow do
 
     context 'when steps have been installed do' do
       before do
-        Elasticity::JobFlow.any_instance.should_receive(:status).and_return(double('Elasticity::JobFlowStatus', :installed_steps => [Elasticity::HiveStep, Elasticity::PigStep]))
+        Elasticity::JobFlow.any_instance.should_receive(:cluster_step_status).and_return([
+          build(:cluster_step_status, :name => Elasticity::PigStep.aws_installation_step_name),
+          build(:cluster_step_status, :name => Elasticity::HiveStep.aws_installation_step_name)
+          ])
       end
       it 'should show that no steps are installed' do
         jobflow.instance_variable_get(:@installed_steps).should =~ [Elasticity::PigStep, Elasticity::HiveStep]