Merge pull request #164 from puppetlabs/SOLARCH-278

(SOLARCH-278) put pending on these tests due to function issues

Author: davidsandilands

spec/fixtures/plans/pe_conf.json

@@ -0,0 +1,29 @@
+{
+  "puppet_enterprise::profile::master::java_args": { 
+    "Xmx": "2048m",
+    "Xms": "512m"
+  },
+  "puppet_enterprise::profile::console::java_args": {
+    "Xmx": "768m",
+    "Xms": "256m"
+  },
+  "puppet_enterprise::profile::orchestrator::java_args": {
+    "Xmx": "768m",
+    "Xms": "256m"
+  },
+  "puppet_enterprise::profile::puppetdb::java_args": {
+    "Xmx": "768m",
+    "Xms": "256m"
+  },
+  "console_admin_password": "puppetlabs",
+  "puppet_enterprise::puppet_master_host": "pe-server-d8b317-0.us-west1-a.c.davidsand.internal",
+  "puppet_enterprise::database_host": "pe-server-d8b317-0.us-west1-a.c.davidsand.internal",
+  "pe_install::puppet_master_dnsaltnames": [
+    "puppet",
+    "pe-server-d8b317-0.us-west1-a.c.davidsand.internal"
+  ],
+  "puppet_enterprise::profile::master::code_manager_auto_configure": true,
+  "puppet_enterprise::profile::database::puppetdb_hosts": [
+    "pe-server-d8b317-0.us-west1-a.c.davidsand.internal"
+  ]
+}

spec/plans/convert_spec.rb

@@ -0,0 +1,27 @@
+# spec/spec_helper.rb
+
+describe 'peadm::convert' do
+  # Include the BoltSpec library functions
+  include BoltSpec::Plans
+
+  let(:trustedjson) do
+    JSON.parse File.read(File.expand_path(File.join(fixtures, 'plans', 'trusted_facts.json')))
+  end
+
+it 'single primary no dr valid' do
+  expect_out_message()
+  expect_task('peadm::trusted_facts').return_for_targets(
+
+    'primary' => trustedjson,
+  )
+  pending('a lack of support for functions requires a workaround to be written')
+  expect_task('peadm::read_file').always_return({'content' => '2019.2.4'})
+  expect_command('systemctl is-active puppet.service')
+  expect_command('systemctl stop puppet.service')
+  #expect_plan('peadm::util::add_cert_extensions')
+  allow_apply()
+  expect_task('peadm::cert_data')
+  expect_task('peadm::puppet_runonce')
+  expect(run_plan('peadm::convert', 'primary_host' => 'primary')).to be_ok
+  end
+end

spec/plans/util/sanitize_pg_pe_conf_spec.rb

@@ -0,0 +1,20 @@
+# spec/spec_helper.rb
+
+
+describe 'peadm::util::sanitize_pg_pe_conf ' do
+  # Include the BoltSpec library functions
+  include BoltSpec::Plans
+
+  let(:pe_conf_json) do
+    JSON.parse File.read(File.expand_path(File.join(fixtures, 'plans', 'pe_conf.json')))
+  end
+
+it 'Runs' do
+#  1) peadm::util::sanitize_pg_pe_conf  Runs
+#Failure/Error: expect(run_plan('peadm::util::sanitize_pg_pe_conf', 'targets' => 'foo,bar', 'primary_host' => 'pe-server-d8b317-0.us-west1-a.c.davidsand.internal')).to be_ok
+#expected `#<Bolt::PlanResult:0x00007fd37d94b8a0 @value=#<Bolt::PAL::PALError: undefined method `start_with?' for #<Hash:0x00007fd36e30b350>>, @status="failure">.ok?` to be truthy, got false
+  pending('a lack of support for functions requires a workaround to be written')
+  expect_task('peadm::read_file').always_return('content' => pe_conf_json)
+  expect(run_plan('peadm::util::sanitize_pg_pe_conf', 'targets' => 'foo,bar', 'primary_host' => 'pe-server-d8b317-0.us-west1-a.c.davidsand.internal')).to be_ok
+  end
+end