Merge pull request #53 from bmclaughlin/miq_pr_13152

Enable provision instances button via providers

app/helpers/application_helper/toolbar_builder.rb

@@ -426,7 +426,7 @@ def hide_button_ops(id)
   def hide_button?(id)
     # need to hide add buttons when on sub-list view screen of a CI.
     return true if id.ends_with?("_new", "_discover") &&
-                   @lastaction == "show" && !["main", "vms"].include?(@display)
+                   @lastaction == "show" && !%w(main vms instances).include?(@display)
 
     # user can see the buttons if they can get to Policy RSOP/Automate Simulate screen
     return false if ["miq_ae_tools"].include?(@layout)

spec/helpers/application_helper/toolbar_builder_spec.rb

@@ -304,13 +304,17 @@
       end
     end
 
-    context "when with vm_miq_request_new" do
-      it "and @lastaction = show, @display = vms" do
-        @id = "vm_miq_request_new"
-        @lastaction = "show"
-        @display = "vms"
-        stub_user(:features => :all)
-        expect(subject).to be_falsey
+    context 'last action set to show' do
+      let(:lastaction) { 'show' }
+      context 'requested to display instances' do
+        let(:display) { 'instances' }
+        it 'returns with false' do
+          @lastaction = lastaction
+          @display = display
+          @id = 'vm_miq_request_new'
+          stub_user(:features => :all)
+          expect(subject).to be_falsey
+        end
       end
     end