Merge pull request #94 from bastelfreak/name

node_group type: output wrong environment name

Author: Ramesh7

lib/puppet/type/node_group.rb

@@ -123,7 +123,7 @@ def insync?(is)
     desc 'Environment for this group'
     defaultto :production
     validate do |value|
-      raise('Invalid environment name') unless value =~ (%r{\A[a-zA-Z0-9_]+\Z}) || (value == 'agent-specified')
+      raise("Invalid environment name: #{value}") unless value =~ (%r{\A[a-zA-Z0-9_]+\Z}) || (value == 'agent-specified')
     end
   end
   newproperty(:classes) do

spec/unit/puppet/type/node_group_spec.rb

@@ -80,7 +80,7 @@
         name: 'stubname',
         environment: 'not-a-valid-name',
       )
-    }.to raise_error(%r{Invalid environment name})
+    }.to raise_error(%r{Invalid environment name: not-a-valid-name})
   end
 
   it 'allows name with symbols, numbers, and whitespace' do