From: Ramesh Sencha <ramesh.sencha@perforce.com>
Date: Mon, 29 May 2023 08:38:36 +0000 (+0530)
Subject: (CONT-581)  Adding deferred function support for password field
X-Git-Tag: v9.1.0~5^2
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=refs%2Fheads%2FCONT-581;p=puppet-modules%2Fpuppetlabs-apt.git

(CONT-581)  Adding deferred function support for password field
---

diff --git a/manifests/init.pp b/manifests/init.pp
index 7d1baf3..cd408d8 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -366,14 +366,14 @@ class apt (
       default => 'present',
     }
 
-    $auth_conf_tmp = epp('apt/auth_conf.epp')
+    $auth_conf_tmp = stdlib::deferrable_epp('apt/auth_conf.epp', { 'auth_conf_entries' => $auth_conf_entries })
 
     file { '/etc/apt/auth.conf':
       ensure  => $auth_conf_ensure,
       owner   => $auth_conf_owner,
       group   => 'root',
       mode    => '0600',
-      content => Sensitive("${confheadertmp}${auth_conf_tmp}"),
+      content => Sensitive($auth_conf_tmp),
       notify  => Class['apt::update'],
     }
   }
diff --git a/spec/classes/apt_spec.rb b/spec/classes/apt_spec.rb
index cb65c1e..51cc7eb 100644
--- a/spec/classes/apt_spec.rb
+++ b/spec/classes/apt_spec.rb
@@ -415,10 +415,11 @@ describe 'apt' do
             super().merge(manage_auth_conf: true)
           end
 
-          auth_conf_content = "// This file is managed by Puppet. DO NOT EDIT.
-machine deb.example.net login foologin password secret
-machine apt.example.com login aptlogin password supersecret
-"
+          auth_conf_content = <<~CONTENT
+            // This file is managed by Puppet. DO NOT EDIT.
+            machine deb.example.net login foologin password secret
+            machine apt.example.com login aptlogin password supersecret
+          CONTENT
 
           it {
             expect(subject).to contain_file('/etc/apt/auth.conf').with(ensure: 'present',
diff --git a/spec/defines/source_spec.rb b/spec/defines/source_spec.rb
index e9c0cf2..d962717 100644
--- a/spec/defines/source_spec.rb
+++ b/spec/defines/source_spec.rb
@@ -130,7 +130,7 @@ describe 'apt::source' do
 
       it {
         expect(subject).to contain_apt__key("Add key: #{id} from Apt::Source my_source").that_comes_before('Apt::Setting[list-my_source]').with(ensure: 'refreshed',
-                                                                                                                                                id: GPG_KEY_ID,
+                                                                                                                                                id: id,
                                                                                                                                                 server: 'pgp.mit.edu',
                                                                                                                                                 content: 'GPG key content',
                                                                                                                                                 source: 'http://apt.puppetlabs.com/pubkey.gpg',
diff --git a/templates/auth_conf.epp b/templates/auth_conf.epp
index 5b3000f..5dc37e9 100644
--- a/templates/auth_conf.epp
+++ b/templates/auth_conf.epp
@@ -1,5 +1,6 @@
-<% if $apt::auth_conf_entries != [] { -%>
-<% $apt::auth_conf_entries.each | $auth_conf_entry | { -%>
+// This file is managed by Puppet. DO NOT EDIT.
+<% if $auth_conf_entries != [] { -%>
+<% $auth_conf_entries.each | $auth_conf_entry | { -%>
 machine <%= $auth_conf_entry['machine'] %> login <%= $auth_conf_entry['login'] %> password <%= $auth_conf_entry['password'] %>
 <% } -%>
 <% } -%>