Update map and mount defined types to address #59

manifests/map.pp

@@ -24,14 +24,27 @@
 # @param order Order in which entries will appear in the autofs map file.
 
 define autofs::map (
-  String               $mapcontent,
+  Array $mapcontent,
   Stdlib::Absolutepath $mapfile,
-  Integer              $order = 1,
+  Enum['autofs/auto.map.erb', 'autofs/auto.map.exec.erb'] $template = 'autofs/auto.map.erb',
+  String $mapmode                                                   = '0644',
+  Boolean $replace                                                  = true,
+  Integer $order                                                    = 1,
 ) {
 
+  concat { $mapfile:
+    ensure  => present,
+    owner   => 'root',
+    group   => 'root',
+    mode    => $mapmode,
+    replace => $replace,
+    require => Package['autofs'],
+    notify  => Service['autofs'],
+  }
+
   concat::fragment{"${mapfile}_${mapcontent}":
     target  => $mapfile,
-    content => "${mapcontent}\n",
+    content => template($template),
     order   => $order,
   }
 

manifests/mount.pp

@@ -72,7 +72,7 @@
       group          => 'root',
       mode           => '0644',
       ensure_newline => true,
-      notify         => Service[ 'autofs' ],
+      notify         => Service['autofs'],
     }
   }
 
@@ -96,7 +96,7 @@
         target  => $master,
         content => "+dir:${map_dir}",
         order   => $order,
-        require => File[ $map_dir ],
+        require => File[$map_dir],
       }
     }
 
@@ -106,25 +106,18 @@
       group   => 'root',
       mode    => $mapperms,
       content => $contents,
-      require => File[ $map_dir ],
-      notify  => Service[ 'autofs' ],
+      require => File[$map_dir],
+      notify  => Service['autofs'],
     }
   }
 
   if $mapfile {
-    concat { $mapfile:
-      ensure  => present,
-      owner   => 'root',
-      group   => 'root',
-      mode    => $mapperms,
-      replace => $replace,
-      require => Package[ 'autofs' ],
-      notify  => Service[ 'autofs' ],
-    }
-    concat::fragment{"${mapfile}_entries":
-      target  => $mapfile,
-      content => template($maptempl),
-      order   => 1,
+    autofs::map { $title:
+      mapfile    => $mapfile,
+      mapcontent => $mapcontents,
+      replace    => $replace,
+      template   => $maptempl,
+      mapmode    => $mapperms,
     }
   }
 }

spec/acceptance/autofs_init_spec.rb

@@ -222,11 +222,10 @@ class { 'autofs':
         it { is_expected.not_to be_running }
       end
 
-      # Skipped until we can pinpoint why serverspec
-      # doesn't properly check if an Upstart service
-      # is enabled or not
-      describe service('autofs') do
-        xit { is_expected.not_to be_enabled }
+      unless default[:platform] =~ %r{ubuntu-14.04-amd64}
+        describe service('autofs') do
+          it { is_expected.not_to be_enabled }
+        end
       end
     end
   end

spec/acceptance/autofs_map_spec.rb

@@ -5,41 +5,23 @@
     it 'applies' do
       pp = <<-EOS
         class { 'autofs': }
-        autofs::mount { 'data':
-          mount       => '/data',
+        autofs::map { 'datamap':
           mapfile     => '/etc/auto.data',
-          mapcontents => ['dataB -o rw /mnt/dataB'],
-          options     => '--timeout=120',
-          order       => 01
+          mapcontent  => [ 'dataB -o rw /mnt/dataB', 'dataC -o rw /mnt/dataC' ],
         }
-	autofs::map {'dataC':
-	  mapfile => '/etc/auto.data',
-	  mapcontent => 'dataC -o rw /mnt/dataC',
-         }
       EOS
 
       apply_manifest(pp, catch_failures: true)
       apply_manifest(pp, catch_changes: true)
     end
 
-    describe file('/etc/auto.master') do
-      it 'exists and have content' do
-        is_expected.to exist
-        is_expected.to be_owned_by 'root'
-        is_expected.to be_grouped_into 'root'
-        shell('cat /etc/auto.master') do |s|
-          expect(s.stdout).to match(%r{/data /etc/auto.data --timeout=120})
-        end
-      end
-    end
-
     describe file('/etc/auto.data') do
       it 'exists and have content' do
         is_expected.to exist
         is_expected.to be_owned_by 'root'
         is_expected.to be_grouped_into 'root'
         shell('cat /etc/auto.data') do |s|
-          expect(s.stdout).to match(%r{(dataA -o rw /mnt/dataA|dataC -o rw /mnt/dataC)})
+          expect(s.stdout).to match(%r{(dataB -o rw /mnt/dataA|dataC -o rw /mnt/dataC)})
         end
       end
     end

templates/auto.map.erb

@@ -14,6 +14,6 @@
 ###############################################################
 <% end -%>
 
-<% @mapcontents.each do |content| -%>
+<% @mapcontent.each do |content| -%>
 <%= content %>
 <% end -%>

templates/auto.map.exec.erb

@@ -15,6 +15,6 @@
 ###############################################################
 <% end -%>
 
-<% @mapcontents.each do |content| -%>
+<% @mapcontent.each do |content| -%>
 <%= content %>
 <% end -%>