Merge pull request #1025 from wyardley/wyardley/fix_test

Disallow empty string in `package_apt_pin`

Author: wyardley

REFERENCE.md

@@ -725,7 +725,7 @@ Default value: `undef`
 
 ##### <a name="-rabbitmq--package_apt_pin"></a>`package_apt_pin`
 
-Data type: `Optional[Variant[Numeric, String]]`
+Data type: `Optional[Variant[Numeric, String[1]]]`
 
 Whether to pin the package to a particular source
 

manifests/init.pp

@@ -378,7 +378,7 @@
   Boolean $management_ssl                                                                          = true,
   Optional[String] $management_hostname                                                            = undef,
   Optional[String] $node_ip_address                                                                = undef,
-  Optional[Variant[Numeric, String]] $package_apt_pin                                              = undef,
+  Optional[Variant[Numeric, String[1]]] $package_apt_pin                                           = undef,
   String $package_ensure                                                                           = 'installed',
   Optional[String] $package_gpg_key                                                                = undef,
   Optional[String] $repo_gpg_key                                                                   = undef,

spec/acceptance/clustering_spec.rb

@@ -68,6 +68,10 @@ class { 'rabbitmq':
   context 'rabbitmq::cluster[:local_node] = foobar' do
     it 'runs successfully' do
       pp = <<-EOS
+      # Needed to avoid nxdomain error
+      host { 'foobar':
+        ip => '127.0.0.1',
+      }
       class { 'rabbitmq':
         cluster                  => { 'name' => 'rabbit_cluster', 'init_node' => 'foobar', 'local_node' => 'foobar' },
         config_cluster           => true,
@@ -78,7 +82,7 @@ class { 'rabbitmq':
       }
       EOS
 
-      apply_manifest(pp, expect_failures: true)
+      apply_manifest(pp, catch_failures: true)
     end
   end
 end

spec/classes/rabbitmq_spec.rb

@@ -83,7 +83,7 @@
       end
 
       context 'with no pin', if: os_facts['os']['family'] == 'Debian' do
-        let(:params) { { repos_ensure: true, package_apt_pin: '' } }
+        let(:params) { { repos_ensure: true, package_apt_pin: nil } }
 
         describe 'it sets up an apt::source' do
           it {