beaker-answers 0.12.0 → 0.13.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    Yzc1NGQ4ZmZiOTc3ZjQyYTc2NDQ0MjJhMzk0ZTg4MzRjYzBmNGQxZA==
+    NjEwZDM2NjVhN2I5NmExMTNiNDMzOTE3MDE0Nzk3ODJlMTJlNWYwYg==
   data.tar.gz: !binary |-
-    ZjQyZDIzNjgwYzk3ZGJmZWQxY2YzNzJkZWFmYThmYjY3N2NmNjA1ZQ==
+    ZjVkNzMwZmVmY2ExMmZmMjEyYTFlOTUxM2VjM2M1OWJlNTRlZmIzNw==
 SHA512:
   metadata.gz: !binary |-
-    OWNkZmI1MTNhNDQ3MWQyY2Y5YmU4N2YwYjM4Mjg3MTQ5MzE4ZWFmYzJlNGIw
-    ZDNlNjY3ZTQxZDJlM2VlZTMyMGUyNjY4MjQwNTRmYWY4OTRmMThmMTJlZmMz
-    NGU5Yjk3Nzc5Mzg3MWUzYTdlMTMyNzlkNmFkNWVkMjk2ZDQ5Y2M=
+    ZDM5NDQ0NmQyMzA2ZDUxODY0NTBiZmM0N2ExNzJiNjJiOTRiYzE2NWE4NzE2
+    NWZlZDgzMDBiNDMxMTBkYzI1MzJmYTI4ZGUzOTY1N2I3YTllMmVhZGU5MTli
+    MDIyYzIwM2Y0MDVjNGUwYTI1NGM3OGMxOWRhOGEzYmQ3ZjUxOGE=
   data.tar.gz: !binary |-
-    MjlhZmQ2MTc1NjE0N2Q2NzRjMzMwMjg4ZDEzZjdjOTY3NmI3ZGRhNmVmOTQz
-    NWFhYjc0M2UwZDEwNDYwZWI5Nzg5ZjBmYTBlYjRjYjQ4OGE0ZGE5NGJkNjM3
-    ZGE1YTM0NDUzNWI1YTI0MDFjYjI2MjY3YThhZmE5MmE1ZmQ0OWQ=
+    ODA5MGIyOWQxZGZhNDdkNWFiMTRlNWY2OTg4YTFjOTZlODgzNGZjNjM1MDIx
+    ODIxYzNhNThjM2VhNTA1NjM4ZDQ4NWVlNjQ2Y2E4MWE3YjkxOGY0MWQxYmMz
+    Mjk2NTEzZDU3NjAzZGM3OTllYjNlNjJjMjNhNmRiMDcxODFlMzk=

data/HISTORY.md

@@ -1,6 +1,7 @@
 # default - History
 ## Tags
-* [LATEST - 28 Oct, 2016 (4e1716ad)](#LATEST)
+* [LATEST - 9 Nov, 2016 (f415b5e1)](#LATEST)
+* [0.12.0 - 28 Oct, 2016 (b18d96b7)](#0.12.0)
 * [0.11.0 - 22 Aug, 2016 (39591691)](#0.11.0)
 * [0.10.0 - 15 Aug, 2016 (30507062)](#0.10.0)
 * [0.9.0 - 1 Aug, 2016 (551ce473)](#0.9.0)
@@ -23,7 +24,69 @@
 * [0.1.0 - 26 Aug, 2015 (ef47972d)](#0.1.0)
 
 ## Details
-### <a name = "LATEST">LATEST - 28 Oct, 2016 (4e1716ad)
+### <a name = "LATEST">LATEST - 9 Nov, 2016 (f415b5e1)
+
+* (GEM) update beaker-answers version to 0.13.0 (f415b5e1)
+
+* Merge pull request #28 from jpartlow/issue/2017.1.x/pe-17621-beaker-answers-generates-meep-node-roles (9becfcb3)
+
+
+```
+Merge pull request #28 from jpartlow/issue/2017.1.x/pe-17621-beaker-answers-generates-meep-node-roles
+
+(PE-17621) Beaker answers generates meep node roles
+```
+* (PE-17621) Validate both old and new formats (138e7780)
+
+
+```
+(PE-17621) Validate both old and new formats
+
+in the corresponding PE version tests.
+```
+* (PE-17621) Generate node_roles in 2017.1+ pe_conf (ea795b85)
+
+
+```
+(PE-17621) Generate node_roles in 2017.1+ pe_conf
+
+MEEP 2.0 introduces a node_roles hash in pe.conf for classification.
+This supercedes using the puppet_enterprise host parameters as a de
+facto classification.  The Version20171 class maps existing beaker role
+aliases to the new role strings used for classification in pe.conf.
+
+So, from a MEEP 1.0 version pe.conf such as:
+
+  {
+    "puppet_enterprise::puppet_master_host": "master.net"
+    "puppet_enterprise::puppetdb_host": "puppetdb.net"
+    "puppet_enterprise::console_host": "console.net"
+  }
+
+You now have a 2.0 version that looks like:
+
+  {
+    "node_roles": {
+      "pe_role::split::primary_master": ["master.net"]
+      "pe_role::split::puppetdb": ["puppetdb.net"]
+      "pe_role::split::console": ["console.net"]
+    }
+  }
+
+See the pe_manager and pe_infrastructure modules for more details.
+```
+* (PE-17621) Move shared examples into a separate file (30e27482)
+
+
+```
+(PE-17621) Move shared examples into a separate file
+
+...included by default in spec_helper.rb. Without this, running the
+version20163_spec.rb by itself fails.
+```
+### <a name = "0.12.0">0.12.0 - 28 Oct, 2016 (b18d96b7)
+
+* (HISTORY) update beaker-answers history for gem release 0.12.0 (b18d96b7)
 
 * (GEM) update beaker-answers version to 0.12.0 (4e1716ad)
 

data/lib/beaker-answers/version.rb

@@ -1,5 +1,5 @@
 module BeakerAnswers
   module Version
-    STRING = '0.12.0'
+    STRING = '0.13.0'
   end
 end

data/lib/beaker-answers/versions/version20171.rb

@@ -9,5 +9,75 @@ module BeakerAnswers
     def self.pe_version_matcher
       /\A2017\.1/
     end
+
+    # This lists various aliases for different PE components that might
+    # be installed on a host.
+    #
+    # The first entry in the list is the canonical name for the
+    # component from the PuppetX::Puppetlabs::Meep::Config.pe_components()
+    # from the pe_infrastructure module.
+    #
+    # Secondary aliases are the various working names that have been used
+    # with beaker in CI to establish that a host has that component
+    # installed on it.
+    COMPONENTS = [
+      ["primary_master", "master"],
+      ["primary_master_replica"],
+      # the 'database' alias will be troublesome if/when we begin
+      # breaking out a managed database (as opposed to external)
+      # in future PE layouts.
+      ["puppetdb", "database"],
+      ["console", "classifier", "dashboard"],
+      ["compile_master"],
+      ["mco_hub", "hub"],
+      ["mco_broker", "spoke"],
+    ]
+
+    # @param host_role_aliases [Array<String>] list of strings to search for in
+    #   each host's role array
+    # @return [Array<String>] of hostnames from @hosts that include at least
+    #   one of the passed aliases
+    def all_hosts_with_component(host_role_aliases)
+      hosts = @hosts.select do |h|
+        !(Array(h['roles']) & host_role_aliases).empty?
+      end
+      hosts.map { |h| h.hostname }
+    end
+
+    # This used to generate the profile host parameters, but now generates a MEEP
+    # 2.0 node_roles hash mapping roles -> node certs based on the same host and
+    # role information.
+    def hiera_host_config
+      pe_conf = {
+        "node_roles" => {}
+      }
+      hosts_by_component = {}
+
+      COMPONENTS.each do |aliases|
+        meep_component_name = aliases.first
+        hosts_by_component[meep_component_name] = all_hosts_with_component(aliases)
+      end
+
+      # Reject console and puppetdb lists if they are the same as master
+      hosts_by_component.reject! do |component,hosts|
+        ["puppetdb", "console"].include?(component) &&
+          hosts_by_component["primary_master"].sort == hosts.sort
+      end
+
+      # Which is also sufficient to determine our architecture at the moment
+      architecture = (hosts_by_component.keys & ["puppetdb", "console"]).empty? ?
+        "monolithic" :
+        "split"
+
+      hosts_by_component.reduce(pe_conf) do |conf,entry|
+        component, hosts = entry
+        if !hosts.empty?
+          conf["node_roles"]["pe_role::#{architecture}::#{component}"] = hosts
+        end
+        conf
+      end
+
+      pe_conf
+    end
   end
 end

data/spec/beaker-answers/versions/version20162_spec.rb

@@ -1,182 +1,6 @@
 require 'spec_helper'
 require 'json'
 
-RSpec.shared_examples 'pe.conf' do
-    let( :options )      { { :format => 'hiera' } }
-    let( :answer_hiera ) { answers.answer_hiera }
-    let( :default_password ) { '~!@#$%^*-/ aZ' }
-    let( :gold_db_answers ) do
-      {
-        "puppet_enterprise::activity_database_user" => 'adsfglkj',
-        "puppet_enterprise::classifier_database_user" => 'DFGhjlkj',
-        "puppet_enterprise::orchestrator_database_user" => 'Orc3Str8R',
-        "puppet_enterprise::puppetdb_database_user" => 'mYpdBu3r',
-        "puppet_enterprise::rbac_database_user" => 'RbhNBklm',
-      }
-    end
-    let( :gold_db_password_answers ) do
-      {
-        "puppet_enterprise::activity_database_password" => default_password,
-        "puppet_enterprise::classifier_database_password" => default_password,
-        "puppet_enterprise::orchestrator_database_password" => default_password,
-        "puppet_enterprise::puppetdb_database_password" => default_password,
-        "puppet_enterprise::rbac_database_password" => default_password,
-      }
-    end
-    # This is a set of database parameters specified in host.cfg :answers
-    # (for an external postgres, for instance)
-    let( :overridden_database_parameters ) do
-      {
-        "puppet_enterprise::activity_database_name" => 'custom-activity',
-        "puppet_enterprise::activity_database_user" => 'custom-activity-user',
-        "puppet_enterprise::classifier_database_name" => 'custom-classifier',
-        "puppet_enterprise::classifier_database_user" => 'custom-classifier-user',
-        "puppet_enterprise::orchestrator_database_name" => 'custom-orchestrator',
-        "puppet_enterprise::orchestrator_database_user" => 'custom-orchestrator-user',
-        "puppet_enterprise::puppetdb_database_name" => 'custom-puppetdb',
-        "puppet_enterprise::puppetdb_database_user" => 'custom-puppetdb-user',
-        "puppet_enterprise::rbac_database_name" => 'custom-rbac',
-        "puppet_enterprise::rbac_database_user" => 'custom-rbac-user',
-        "puppet_enterprise::activity_database_password" => 'custom-activity-password',
-        "puppet_enterprise::classifier_database_password" => 'custom-classifier-password',
-        "puppet_enterprise::orchestrator_database_password" => 'custom-orchestrator-password',
-        "puppet_enterprise::puppetdb_database_password" => 'custom-puppetdb-password',
-        "puppet_enterprise::rbac_database_password" => 'custom-rbac-password',
-      }
-    end
-    let( :overridding_parameters ) do
-      {
-        'puppet_enterprise::certificate_authority_host' => 'enterpriseca.vm',
-        'puppet_enterprise::console_host' => 'enterpriseconsole.vm',
-        'console_admin_password' => 'testing123',
-      }
-    end
-    let( :gold_answers_with_overrides ) { gold_role_answers.merge(overridding_parameters) }
-
-    it 'should not have nil keys or values' do
-      answer_hash.each_pair { |k, v|
-        expect([k, v]).not_to include(nil)
-      }
-    end
-
-    it 'has the just the role and values for default install' do
-      expect(answer_hash).to eq(
-        gold_role_answers
-      )
-    end
-
-    context 'when include_legacy_database_defaults' do
-      context 'is false' do
-        let(:options) do
-          {
-            :format => 'hiera',
-            :include_legacy_database_defaults => false,
-          }
-        end
-
-        it 'has only the role values' do
-          expect(answer_hash).to eq(gold_role_answers)
-        end
-
-        it 'also includes any explicitly added database parameters' do
-          options.merge!(:answers => overridden_database_parameters)
-          expect(answer_hash).to eq(
-            gold_role_answers
-              .merge(overridden_database_parameters)
-          )
-        end
-      end
-
-      context 'is true' do
-        let(:options) do
-          {
-            :format => 'hiera',
-            :include_legacy_database_defaults => true,
-          }
-        end
-
-        it 'has the role values and database defaults' do
-          expect(answer_hash).to eq(
-            gold_role_answers
-              .merge(gold_db_answers)
-          )
-        end
-
-        it 'overrides defaults with explicitly added database parameters' do
-          options.merge!(:answers => overridden_database_parameters)
-          expect(answer_hash).to eq(
-            gold_role_answers
-              .merge(overridden_database_parameters)
-          )
-        end
-      end
-    end
-
-    it 'generates valid json if #answer_hiera is called' do
-      expect(answer_hiera).not_to be_empty
-      expect { JSON.load(answer_hiera) }.not_to raise_error
-      expect(answer_hiera).to match "puppet_enterprise::puppet_master_host.*:.*#{basic_hosts[0].hostname}"
-    end
-
-    context 'with legacy answers present' do
-      let(:string_answer) { { 'q_puppet_enterpriseconsole_auth_password' => 'password' } }
-      let(:symbol_answer) { { :q_puppet_enterpriseconsole_auth_password => 'password' } }
-      let(:answers) { BeakerAnswers::Answers.create(ver, hosts, options) }
-
-      context 'when key is a string' do
-        let(:options) { { :format => 'hiera' }.merge(:answers => string_answer) }
-
-        it 'raises a TypeError' do
-          expect { answers.answers }.to raise_error(TypeError, /q_ answers are not supported/)
-        end
-      end
-
-      context 'when key is a symbol' do
-        let(:options) { { :format => 'hiera' }.merge(:answers => symbol_answer) }
-
-        it 'raises a TypeError' do
-          expect { answers.answers }.to raise_error(TypeError, /q_ answers are not supported/)
-        end
-      end
-    end
-
-    context 'when overriding answers' do
-      let( :options ) do
-        {
-          :format  => 'hiera',
-          :answers => {
-            'puppet_enterprise' =>  { 'certificate_authority_host' => 'enterpriseca.vm' },
-            'puppet_enterprise::console_host' => 'enterpriseconsole.vm',
-            'console_admin_password' => 'testing123',
-          }
-        }
-      end
-
-      it 'matches expected answers' do
-        expect(answer_hash).to eq(gold_answers_with_overrides)
-      end
-    end
-
-    context 'when overriding answers using symbolic keys' do
-      let( :options ) do
-        {
-          :format  => 'hiera',
-          :answers => {
-            :puppet_enterprise =>  {
-              :certificate_authority_host => 'enterpriseca.vm',
-              :console_host => 'enterpriseconsole.vm',
-            },
-            :console_admin_password => 'testing123',
-          }
-        }
-      end
-
-      it 'matches expected answers' do
-        expect(answer_hash).to eq(gold_answers_with_overrides)
-      end
-    end
-end
-
 describe BeakerAnswers::Version20162 do
   let( :ver )         { '2016.2.0' }
   let( :options )     { StringifyHash.new }
@@ -202,6 +26,7 @@ describe BeakerAnswers::Version20162 do
       end
 
       include_examples 'pe.conf'
+      include_examples 'valid MEEP 1.0 pe.conf'
     end
 
     context 'for a split install' do
@@ -216,6 +41,7 @@ describe BeakerAnswers::Version20162 do
       end
 
       include_examples 'pe.conf'
+      include_examples 'valid MEEP 1.0 pe.conf'
     end
   end
 end

data/spec/beaker-answers/versions/version20163_spec.rb

@@ -25,6 +25,7 @@ describe BeakerAnswers::Version20163 do
       end
 
       include_examples 'pe.conf'
+      include_examples 'valid MEEP 1.0 pe.conf'
     end
 
     context 'for a split install' do
@@ -38,6 +39,7 @@ describe BeakerAnswers::Version20163 do
       end
 
       include_examples 'pe.conf'
+      include_examples 'valid MEEP 1.0 pe.conf'
     end
   end
 end

data/spec/beaker-answers/versions/version20171_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+require 'json'
+
+describe BeakerAnswers::Version20171 do
+  let( :ver )         { '2017.1.0' }
+  let( :options )     { StringifyHash.new }
+  let( :basic_hosts ) { make_hosts( {'pe_ver' => ver } ) }
+  let( :hosts ) { basic_hosts[0]['roles'] = ['master', 'agent']
+                  basic_hosts[1]['roles'] = ['dashboard', 'agent']
+                  basic_hosts[2]['roles'] = ['database', 'agent']
+                  basic_hosts }
+  let( :answers )     { BeakerAnswers::Answers.create(ver, hosts, options) }
+  let( :answer_hash ) { answers.answers }
+
+  context 'when generating a hiera config' do
+    context 'for a monolithic install' do
+      let( :basic_hosts ) { make_hosts( {'pe_ver' => ver }, 1 ) }
+      let( :hosts ) { basic_hosts[0]['roles'] = ['master', 'agent', 'dashboard', 'database']
+                      basic_hosts }
+      let( :gold_role_answers ) do
+        {
+          "console_admin_password" => default_password,
+          "node_roles" => {
+            "pe_role::monolithic::primary_master" => [basic_hosts[0].hostname],
+          }
+        }
+      end
+
+      it "accepts overrides to node_roles from beaker conf answers"
+
+      include_examples 'pe.conf'
+      include_examples 'valid MEEP 2.0 pe.conf'
+    end
+
+    context 'for a split install' do
+      let( :gold_role_answers ) do
+        {
+          "console_admin_password" => default_password,
+          "node_roles" => {
+            "pe_role::split::primary_master" => [basic_hosts[0].hostname],
+            "pe_role::split::console" => [basic_hosts[1].hostname],
+            "pe_role::split::puppetdb" => [basic_hosts[2].hostname],
+          }
+        }
+      end
+
+      include_examples 'pe.conf'
+      include_examples 'valid MEEP 2.0 pe.conf'
+    end
+  end
+end

data/spec/shared/context.rb

@@ -0,0 +1,186 @@
+RSpec.shared_examples 'pe.conf' do
+  let( :options )      { { :format => 'hiera' } }
+  let( :answer_hiera ) { answers.answer_hiera }
+  let( :default_password ) { '~!@#$%^*-/ aZ' }
+  let( :gold_db_answers ) do
+    {
+      "puppet_enterprise::activity_database_user" => 'adsfglkj',
+      "puppet_enterprise::classifier_database_user" => 'DFGhjlkj',
+      "puppet_enterprise::orchestrator_database_user" => 'Orc3Str8R',
+      "puppet_enterprise::puppetdb_database_user" => 'mYpdBu3r',
+      "puppet_enterprise::rbac_database_user" => 'RbhNBklm',
+    }
+  end
+  let( :gold_db_password_answers ) do
+    {
+      "puppet_enterprise::activity_database_password" => default_password,
+      "puppet_enterprise::classifier_database_password" => default_password,
+      "puppet_enterprise::orchestrator_database_password" => default_password,
+      "puppet_enterprise::puppetdb_database_password" => default_password,
+      "puppet_enterprise::rbac_database_password" => default_password,
+    }
+  end
+  # This is a set of database parameters specified in host.cfg :answers
+  # (for an external postgres, for instance)
+  let( :overridden_database_parameters ) do
+    {
+      "puppet_enterprise::activity_database_name" => 'custom-activity',
+      "puppet_enterprise::activity_database_user" => 'custom-activity-user',
+      "puppet_enterprise::classifier_database_name" => 'custom-classifier',
+      "puppet_enterprise::classifier_database_user" => 'custom-classifier-user',
+      "puppet_enterprise::orchestrator_database_name" => 'custom-orchestrator',
+      "puppet_enterprise::orchestrator_database_user" => 'custom-orchestrator-user',
+      "puppet_enterprise::puppetdb_database_name" => 'custom-puppetdb',
+      "puppet_enterprise::puppetdb_database_user" => 'custom-puppetdb-user',
+      "puppet_enterprise::rbac_database_name" => 'custom-rbac',
+      "puppet_enterprise::rbac_database_user" => 'custom-rbac-user',
+      "puppet_enterprise::activity_database_password" => 'custom-activity-password',
+      "puppet_enterprise::classifier_database_password" => 'custom-classifier-password',
+      "puppet_enterprise::orchestrator_database_password" => 'custom-orchestrator-password',
+      "puppet_enterprise::puppetdb_database_password" => 'custom-puppetdb-password',
+      "puppet_enterprise::rbac_database_password" => 'custom-rbac-password',
+    }
+  end
+  let( :overridding_parameters ) do
+    {
+      'puppet_enterprise::certificate_authority_host' => 'enterpriseca.vm',
+      'puppet_enterprise::console_host' => 'enterpriseconsole.vm',
+      'console_admin_password' => 'testing123',
+    }
+  end
+  let( :gold_answers_with_overrides ) { gold_role_answers.merge(overridding_parameters) }
+
+  it 'should not have nil keys or values' do
+    answer_hash.each_pair { |k, v|
+      expect([k, v]).not_to include(nil)
+    }
+  end
+
+  it 'has just the role and values for default install' do
+    expect(answer_hash).to eq(
+      gold_role_answers
+    )
+  end
+
+  context 'when include_legacy_database_defaults' do
+    context 'is false' do
+      let(:options) do
+        {
+          :format => 'hiera',
+          :include_legacy_database_defaults => false,
+        }
+      end
+
+      it 'has only the role values' do
+        expect(answer_hash).to eq(gold_role_answers)
+      end
+
+      it 'also includes any explicitly added database parameters' do
+        options.merge!(:answers => overridden_database_parameters)
+        expect(answer_hash).to eq(
+          gold_role_answers
+            .merge(overridden_database_parameters)
+        )
+      end
+    end
+
+    context 'is true' do
+      let(:options) do
+        {
+          :format => 'hiera',
+          :include_legacy_database_defaults => true,
+        }
+      end
+
+      it 'has the role values and database defaults' do
+        expect(answer_hash).to eq(
+          gold_role_answers
+            .merge(gold_db_answers)
+        )
+      end
+
+      it 'overrides defaults with explicitly added database parameters' do
+        options.merge!(:answers => overridden_database_parameters)
+        expect(answer_hash).to eq(
+          gold_role_answers
+            .merge(overridden_database_parameters)
+        )
+      end
+    end
+  end
+
+  context 'with legacy answers present' do
+    let(:string_answer) { { 'q_puppet_enterpriseconsole_auth_password' => 'password' } }
+    let(:symbol_answer) { { :q_puppet_enterpriseconsole_auth_password => 'password' } }
+    let(:answers) { BeakerAnswers::Answers.create(ver, hosts, options) }
+
+    context 'when key is a string' do
+      let(:options) { { :format => 'hiera' }.merge(:answers => string_answer) }
+
+      it 'raises a TypeError' do
+        expect { answers.answers }.to raise_error(TypeError, /q_ answers are not supported/)
+      end
+    end
+
+    context 'when key is a symbol' do
+      let(:options) { { :format => 'hiera' }.merge(:answers => symbol_answer) }
+
+      it 'raises a TypeError' do
+        expect { answers.answers }.to raise_error(TypeError, /q_ answers are not supported/)
+      end
+    end
+  end
+
+  context 'when overriding answers' do
+    let( :options ) do
+      {
+        :format  => 'hiera',
+        :answers => {
+          'puppet_enterprise' =>  { 'certificate_authority_host' => 'enterpriseca.vm' },
+          'puppet_enterprise::console_host' => 'enterpriseconsole.vm',
+          'console_admin_password' => 'testing123',
+        }
+      }
+    end
+
+    it 'matches expected answers' do
+      expect(answer_hash).to eq(gold_answers_with_overrides)
+    end
+  end
+
+  context 'when overriding answers using symbolic keys' do
+    let( :options ) do
+      {
+        :format  => 'hiera',
+        :answers => {
+          :puppet_enterprise =>  {
+            :certificate_authority_host => 'enterpriseca.vm',
+            :console_host => 'enterpriseconsole.vm',
+          },
+          :console_admin_password => 'testing123',
+        }
+      }
+    end
+
+    it 'matches expected answers' do
+      expect(answer_hash).to eq(gold_answers_with_overrides)
+    end
+  end
+end
+
+RSpec.shared_examples "valid MEEP 2.0 pe.conf" do
+  it 'generates valid MEEP 2.0 json if #answer_hiera is called' do
+    expect(answer_hiera).not_to be_empty
+    expect { JSON.load(answer_hiera) }.not_to raise_error
+    expect(answer_hiera).to match(%r{"node_roles"\s*:\s*\{})
+    expect(answer_hiera).to match(%r{"pe_role::\w+::primary_master"\s*:\s*\[\s*"#{basic_hosts[0].hostname}"\s*\]}m)
+  end
+end
+
+RSpec.shared_examples "valid MEEP 1.0 pe.conf" do
+  it 'generates valid MEEP 1.0 json if #answer_hiera is called' do
+    expect(answer_hiera).not_to be_empty
+    expect { JSON.load(answer_hiera) }.not_to raise_error
+    expect(answer_hiera).to match(%r{"puppet_enterprise::puppet_master_host"\s*:\s*"#{basic_hosts[0].hostname}"}m)
+  end
+end

data/spec/spec_helper.rb

@@ -1,6 +1,7 @@
 require 'simplecov'
 require 'beaker-answers'
 require 'helpers'
+require 'shared/context.rb'
 
 require 'rspec/its'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: beaker-answers
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.13.0
 platform: ruby
 authors:
 - Puppetlabs
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-28 00:00:00.000000000 Z
+date: 2016-11-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -222,7 +222,9 @@ files:
 - spec/beaker-answers/upgrade_spec.rb
 - spec/beaker-answers/versions/version20162_spec.rb
 - spec/beaker-answers/versions/version20163_spec.rb
+- spec/beaker-answers/versions/version20171_spec.rb
 - spec/helpers.rb
+- spec/shared/context.rb
 - spec/spec_helper.rb
 homepage: https://github.com/puppetlabs/beaker-answers
 licenses: