Update PR puppetlabs-toy-chest#132 for new resource types and resolve…

… branch conflict

lib/puppet/type/ec2_instance.rb

@@ -1,4 +1,5 @@
 require_relative '../../puppet_x/puppetlabs/property/tag.rb'
+require_relative '../../puppet_x/puppetlabs/property/region'
 
 Puppet::Type.newtype(:ec2_instance) do
   @doc = 'Type representing an EC2 instance.'
@@ -93,12 +94,8 @@ def insync?(is)
     end
   end
 
-  newproperty(:region) do
+  newproperty(:region, :parent => PuppetX::Property::AwsRegion) do
     desc 'The region in which to launch the instance.'
-    validate do |value|
-      fail 'region should not contain spaces' if value =~ /\s/
-      fail 'region should be a String' unless value.is_a?(String)
-    end
   end
 
   newproperty(:image_id) do

lib/puppet/type/rds_db_parameter_group.rb

@@ -1,3 +1,5 @@
+require_relative '../../puppet_x/puppetlabs/property/region'
+
 Puppet::Type.newtype(:rds_db_parameter_group) do
   @doc = 'Type representing an RDS DB Parameter group.'
 
@@ -22,12 +24,8 @@
     end
   end
 
-  newproperty(:region) do
+  newproperty(:region, :parent => PuppetX::Property::AwsRegion) do
     desc 'The region in which to create the db_parameter_group.'
-    validate do |value|
-      fail 'region should be a String' unless value.is_a?(String)
-      fail 'region should not contain spaces' if value =~ /\s/
-    end
   end
 
 end

lib/puppet/type/rds_db_securitygroup.rb

@@ -1,3 +1,5 @@
+require_relative '../../puppet_x/puppetlabs/property/region'
+
 Puppet::Type.newtype(:rds_db_securitygroup) do
   @doc = 'Type representing an RDS instance.'
 
@@ -32,12 +34,8 @@
     end
   end
 
-  newproperty(:region) do
+  newproperty(:region, :parent => PuppetX::Property::AwsRegion) do
     desc 'The region in which to create the db_securitygroup.'
-    validate do |value|
-      fail 'region should be a String' unless value.is_a?(String)
-      fail 'region should not contain spaces' if value =~ /\s/
-    end
   end
 
   newproperty(:ip_ranges, :array_matching => :all) do

lib/puppet/type/rds_instance.rb

@@ -1,3 +1,5 @@
+require_relative '../../puppet_x/puppetlabs/property/region'
+
 Puppet::Type.newtype(:rds_instance) do
   @doc = 'Type representing an RDS instance.'
 
@@ -49,12 +51,8 @@
 Not applicable. Must be null.'
   end
 
-  newproperty(:region) do
+  newproperty(:region, :parent => PuppetX::Property::AwsRegion) do
     desc 'The region in which to launch the instance.'
-    validate do |value|
-      fail 'region should be a String' unless value.is_a?(String)
-      fail 'region should not contain spaces' if value =~ /\s/
-    end
   end
 
   newproperty(:db_instance_class) do

lib/puppet/type/sqs_queue.rb

@@ -1,3 +1,5 @@
+require_relative '../../puppet_x/puppetlabs/property/region'
+
 Puppet::Type.newtype(:sqs_queue) do
   @doc = "Type representing a SQS Queue"
   ensurable
@@ -43,13 +45,8 @@
      munge(&:to_s)
   end
 
-  newproperty(:region) do
+  newproperty(:region, :parent => PuppetX::Property::AwsRegion) do
     desc 'The name of the region in which the SQS queue is located'
-    validate do |value|
-      fail 'region should be a String' unless value.is_a?(String)
-      fail 'You must provide a non-blank region name for SQS Queues' if value.nil? || value.empty?
-      fail 'The name of a region should contain only alphanumeric characters or dashes' unless value =~ /^([a-zA-Z]+-+)+\d$/
-    end
   end
 
   newproperty(:visibility_timeout) do

spec/unit/provider/rds_instance/v2_spec.rb

@@ -12,7 +12,7 @@
     Puppet::Type.type(:rds_instance).new(
       ensure: 'present',
       name: 'awesome-db-5',
-      region: 'us-west-1',
+      region: 'sa-east-1',
       db_name:  'mysqldbname3',
       engine: 'mysql',
       engine_version: '5.6.19a',

spec/unit/type/sqs_queue_spec.rb

@@ -29,46 +29,46 @@
     end
 
     it 'should create a valid instance' do
-      type_class.new({name: 'name', region: 'us-east-1'})
+      type_class.new({name: 'name', region: 'sa-east-1'})
     end
 
     it 'should set delay seconds should get set with a valid value' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1', delay_seconds: 450})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1', delay_seconds: 450})
       expect(queue[:delay_seconds]).to eq("450")
     end
 
     it 'delay seconds should default to 0' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1'})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1'})
       expect(queue[:delay_seconds]).to eq("0")
     end
 
     it 'visibility_timeout should default to 30' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1'})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1'})
       expect(queue[:visibility_timeout]).to eq('30')
     end
 
     it 'should set visibility_timeout with a valid value' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1', visibility_timeout: 123})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1', visibility_timeout: 123})
       expect(queue[:visibility_timeout]).to eq("123")
     end
 
     it 'should set message_retention_period should get set with a valid value' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1', message_retention_period: 360})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1', message_retention_period: 360})
       expect(queue[:message_retention_period]).to eq("360")
     end
 
     it 'should set message_retention_period to default to 345600' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1'})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1'})
       expect(queue[:message_retention_period]).to eq("345600")
     end
 
     it 'should set maximum_message_size should get set with a valid value' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1', maximum_message_size: 2048})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1', maximum_message_size: 2048})
       expect(queue[:maximum_message_size]).to eq("2048")
     end
 
     it 'should set maximum_message_size to default to 262144' do
-      queue = type_class.new({name: 'queue', region: 'us-east-1'})
+      queue = type_class.new({name: 'queue', region: 'sa-east-1'})
       expect(queue[:maximum_message_size]).to eq("262144")
     end
 
@@ -81,26 +81,23 @@
   context 'erroring values' do
     it 'should error with incorrect range value for delay seconds' do
       expect do
-        type_class.new({name: 'queue', region: 'us-east-1', delay_seconds: 1024})
+        type_class.new({name: 'queue', region: 'sa-east-1', delay_seconds: 1024})
       end.to raise_error(Puppet::Error, /delay_seconds must be an integer between 0 and 900/)
     end
 
      it 'should require something that looks like a region' do
       expect do
-        type_class.new ({name: 'somename', :region => 'us_east_1'})
-      end.to raise_error(Puppet::Error, /The name of a region should contain only alphanumeric characters or dashes/)
+        type_class.new ({name: 'somename', :region => 'sa-east-1 '})
+      end.to raise_error(Puppet::Error, /region should not contain spaces/)
       expect do
-        type_class.new ({name: 'somename', :region => '123'})
-      end.to raise_error(Puppet::Error, /The name of a region should contain only alphanumeric characters or dashes/)
-      expect do
-        type_class.new ({name: 'somename', :region => 'europe'})
-      end.to raise_error(Puppet::Error, /The name of a region should contain only alphanumeric characters or dashes/)
+        type_class.new ({name: 'somename', :region => 1})
+      end.to raise_error(Puppet::Error, /region should be a String/)
      end
 
      it 'should require a non-blank region' do
       expect do
         type_class.new ({name: 'somename', region: ''})
-      end.to raise_error(Puppet::Error, /You must provide a non-blank region name for SQS Queues/)
+      end.to raise_error(Puppet::Error, /region should not be blank/)
     end
     it 'should require a name' do
       expect do