fixes when the mapname is too long

when the mapname (or any other field in the ident file) exceeds the
predefined width, the values are 'glued' together. This renders the file
invalid.

similar like this issue:
#733 we now force a space
between the values if the exceed the column width.

Author: kristofvandam

libraries/ident.rb

@@ -184,6 +184,7 @@ def to_s
             ENTRY_FIELD_FORMAT.each do |field, ljust_count|
               field = respond_to?(field) ? send(field) : ''
               field_string = field.to_s.ljust(ljust_count)
+              field_string += ' ' unless field_string.include?(' ') || field == :comment
               entry_string.concat(field_string)
             end
 

test/cookbooks/test/recipes/ident.rb

@@ -54,6 +54,14 @@
   action :delete
 end
 
+postgresql_ident 'map with very long name' do
+  map_name 'this_is_a_very_long_map_name_that_should_be_handled_correctly_by_the_postgresql_ident_resource'
+  system_username 'shef'
+  database_username 'sous_chef'
+
+  notifies :reload, 'postgresql_service[postgresql]', :delayed
+end
+
 postgresql_access 'postgresql host superuser' do
   type 'host'
   database 'all'

test/integration/ident/controls/ident_map.rb

@@ -31,3 +31,13 @@
     its('output') { should include 'sous_chef' }
   end
 end
+
+control 'postgresql-ident-long-map-name' do
+  impact 1.0
+  desc 'This test ensures postgres configures ident access with long map names correctly'
+
+  describe postgres_ident_conf.where { map_name == 'this_is_a_very_long_map_name_that_should_be_handled_correctly_by_the_postgresql_ident_resource' } do
+    its('system_username') { should cmp 'shef' }
+    its('pg_username') { should cmp 'sous_chef' }
+  end
+end