Fix group ownership on files.

And add a test to stop it being broken again.

manifests/init.pp

@@ -72,7 +72,7 @@ define concat(
 
   $safe_group = $group ? {
     undef   => $concat::setup::root_group,
-    default => $safe_group,
+    default => $group,
   }
 
   case $warn {

spec/defines/init_spec.rb

@@ -106,4 +106,26 @@ describe 'concat' do
 
 end
 
+describe 'concat' do
+  let(:title) { '/etc/foo.bar' }
+  let(:params) { {
+    :group => 'something', 
+    :owner => 'someone', 
+    :mode  => '0755' 
+  } }
+  let(:facts) { {
+    :concat_basedir => '/var/lib/puppet/concat',
+    :id             => 'root',
+  } }
+
+  it do
+    should contain_file("/etc/foo.bar").with( {
+      'ensure' => 'present',
+      'owner'  => 'someone',
+      'group'  => 'something',
+      'mode'   => '0755',
+    } )
+  end
+end
+
 # vim:sw=2:ts=2:expandtab:textwidth=79