test: fix bolt-modules tests

Author: jay7x

bolt-modules/boltlib/Rakefile

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require 'voxpupuli/test/rake'

bolt-modules/boltlib/spec/functions/add_facts_spec.rb

@@ -5,8 +5,6 @@
 require 'bolt/inventory'
 
 describe 'add_facts' do
-  include PuppetlabsSpec::Fixtures
-
   let(:executor) { Bolt::Executor.new }
   let(:inventory) { Bolt::Inventory.empty }
   let(:target) { inventory.get_target('example') }

bolt-modules/boltlib/spec/functions/add_to_group_spec.rb

@@ -4,8 +4,6 @@
 require 'bolt/inventory'
 
 describe 'add_to_group' do
-  include PuppetlabsSpec::Fixtures
-
   let(:executor) { Bolt::Executor.new }
   let(:inventory) { Bolt::Inventory.empty }
   let(:target) { inventory.get_target('example') }

bolt-modules/boltlib/spec/functions/apply_prep_spec.rb

@@ -11,8 +11,6 @@
 require 'bolt/task'
 
 describe 'apply_prep' do
-  include PuppetlabsSpec::Fixtures
-
   let(:applicator)    { mock('Bolt::Applicator') }
   let(:config)        { Bolt::Config.default }
   let(:executor)      { Bolt::Executor.new }
@@ -32,9 +30,8 @@
   end
 
   context 'with targets' do
-    let(:hostnames)         { %w[a.b.com winrm://x.y.com pcp://foo] }
+    let(:hostnames)         { %w[a.b.com winrm://x.y.com remote://foo] }
     let(:targets)           { hostnames.map { |h| inventory.get_target(h) } }
-    let(:unknown_targets)   { targets.reject { |target| target.protocol == 'pcp' } }
     let(:fact)              { { 'osfamily' => 'none' } }
     let(:custom_facts_task) { Bolt::Task.new('custom_facts_task') }
     let(:version_task)      { Bolt::Task.new('openvox_bootstrap::check') }
@@ -74,7 +71,7 @@
 
       is_expected.to run.with_params(hostnames.join(','))
       targets.each do |target|
-        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'pcp'
+        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'remote'
         expect(inventory.facts(target)).to eq(fact)
       end
     end
@@ -92,7 +89,7 @@
 
       is_expected.to run.with_params(hostnames, '_run_as' => 'root')
       targets.each do |target|
-        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'pcp'
+        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'remote'
         expect(inventory.facts(target)).to eq(fact)
       end
     end
@@ -110,7 +107,7 @@
 
       is_expected.to run.with_params(hostnames, '_noop' => true)
       targets.each do |target|
-        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'pcp'
+        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'remote'
         expect(inventory.facts(target)).to eq(fact)
       end
     end
@@ -199,31 +196,6 @@
     end
   end
 
-  context 'with only pcp targets' do
-    let(:hostnames)         { %w[pcp://foo pcp://bar] }
-    let(:targets)           { hostnames.map { |h| inventory.get_target(h) } }
-    let(:fact)              { { 'osfamily' => 'none' } }
-    let(:custom_facts_task) { Bolt::Task.new('custom_facts_task') }
-
-    before(:each) do
-      applicator.stubs(:build_plugin_tarball).returns(:tarball)
-      applicator.stubs(:custom_facts_task).returns(custom_facts_task)
-    end
-
-    it 'sets feature and gathers facts' do
-      facts = Bolt::ResultSet.new(targets.map { |t| Bolt::Result.new(t, value: fact) })
-      executor.expects(:run_task)
-              .with(targets, custom_facts_task, includes('plugins'), {})
-              .returns(facts)
-
-      is_expected.to run.with_params(hostnames.join(','))
-      targets.each do |target|
-        expect(inventory.features(target)).to include('puppet-agent') unless target.transport == 'pcp'
-        expect(inventory.facts(target)).to eq(fact)
-      end
-    end
-  end
-
   context 'with targets assigned the puppet-agent feature' do
     let(:hostnames)         { %w[foo bar] }
     let(:targets)           { hostnames.map { |h| inventory.get_target(h) } }

bolt-modules/boltlib/spec/functions/background_spec.rb

@@ -6,8 +6,6 @@
 require 'bolt/plan_future'
 
 describe 'background' do
-  include PuppetlabsSpec::Fixtures
-
   let(:name)      { "Pluralize" }
   let(:object)    { "noodle" }
   let(:future)    { Bolt::PlanFuture.new('foo', name, plan_id: 1234) }

bolt-modules/boltlib/spec/functions/download_file_spec.rb

@@ -8,8 +8,6 @@
 require 'bolt/project'
 
 describe 'download_file' do
-  include PuppetlabsSpec::Fixtures
-
   let(:executor)      { Bolt::Executor.new }
   let(:inventory)     { mock('inventory') }
   let(:project)       { Bolt::Project.default_project }

bolt-modules/boltlib/spec/functions/facts_spec.rb

@@ -5,8 +5,6 @@
 require 'bolt/inventory'
 
 describe 'facts' do
-  include PuppetlabsSpec::Fixtures
-
   let(:executor) { Bolt::Executor.new }
   let(:inventory) { Bolt::Inventory.empty }
   let(:hostname) { 'example' }

bolt-modules/boltlib/spec/functions/fail_plan_spec.rb

@@ -5,8 +5,6 @@
 require 'bolt/error'
 
 describe 'fail_plan' do
-  include PuppetlabsSpec::Fixtures
-
   let(:tasks_enabled) { true }
   let(:executor) { Bolt::Executor.new }
 

bolt-modules/boltlib/spec/functions/get_resources_spec.rb

@@ -9,8 +9,6 @@
 require 'bolt/task'
 
 describe 'get_resources' do
-  include PuppetlabsSpec::Fixtures
-
   let(:applicator) { mock('Bolt::Applicator') }
   let(:executor) { Bolt::Executor.new }
   let(:inventory) { Bolt::Inventory.empty }
@@ -26,7 +24,7 @@
   end
 
   context 'with targets' do
-    let(:hostnames) { %w[a.b.com winrm://x.y.com pcp://foo] }
+    let(:hostnames) { %w[a.b.com winrm://x.y.com] }
     let(:targets) { hostnames.map { |h| inventory.get_target(h) } }
     let(:query_resources_task) { Bolt::Task.new('query_resources_task') }
 

bolt-modules/boltlib/spec/functions/get_target_spec.rb

@@ -5,8 +5,6 @@
 require 'bolt/inventory'
 
 describe 'get_target' do
-  include PuppetlabsSpec::Fixtures
-
   let(:executor) { Bolt::Executor.new }
   let(:inventory) { Bolt::Inventory.empty }
   let(:tasks_enabled) { true }