From afe3e701e30f38952319fd032291a13b0182a1e1 Mon Sep 17 00:00:00 2001
From: Nicolas Rodriguez <nicoladmin@free.fr>
Date: Fri, 4 Oct 2019 17:50:26 +0200
Subject: [PATCH] feat(pillar): accept 'true'/True/true for booleans in pillars

---
 exim/defaults.yaml                               | 12 ++++++------
 exim/files/default/update-exim4.conf.conf        |  8 ++++----
 test/integration/default/controls/config_spec.rb |  1 +
 test/salt/pillar/exim.sls                        |  3 ++-
 4 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/exim/defaults.yaml b/exim/defaults.yaml
index f823839..023fc9e 100644
--- a/exim/defaults.yaml
+++ b/exim/defaults.yaml
@@ -7,16 +7,16 @@ exim:
   virtual_aliases: {}
   files: {}
   config:
-    configtype: 'satellite'
+    configtype: satellite
     other_hostnames: ''
     local_interfaces: ''
     relay_domains: ''
     relay_nets: ''
-    use_split_config: 'true'
-    hide_mailname: 'true'
-    ue4c_keepcomments: 'true'
-    localdelivery: 'mail_spool'
-    minimaldns: 'false'
+    use_split_config: true
+    hide_mailname: true
+    ue4c_keepcomments: true
+    localdelivery: mail_spool
+    minimaldns: false
     readhost: ''
     smarthost: ''
     cfilemode: '644'
diff --git a/exim/files/default/update-exim4.conf.conf b/exim/files/default/update-exim4.conf.conf
index 9cdb517..0c05e80 100644
--- a/exim/files/default/update-exim4.conf.conf
+++ b/exim/files/default/update-exim4.conf.conf
@@ -8,11 +8,11 @@ dc_other_hostnames='{{ exim.other_hostnames }}'
 dc_local_interfaces='{{ exim.local_interfaces }}'
 dc_relay_domains='{{ exim.relay_domains }}'
 dc_relay_nets='{{ exim.relay_nets }}'
-dc_use_split_config='{{ exim.use_split_config }}'
-dc_hide_mailname='{{ exim.hide_mailname }}'
-ue4c_keepcomments='{{ exim.ue4c_keepcomments }}'
+dc_use_split_config='{{ exim.use_split_config | lower }}'
+dc_hide_mailname='{{ exim.hide_mailname | lower }}'
+ue4c_keepcomments='{{ exim.ue4c_keepcomments | lower }}'
 dc_localdelivery='{{ exim.localdelivery }}'
-dc_minimaldns='{{ exim.minimaldns }}'
+dc_minimaldns='{{ exim.minimaldns | lower }}'
 dc_readhost='{{ exim.readhost }}'
 dc_smarthost='{{ exim.smarthost }}'
 CFILEMODE='{{ exim.cfilemode }}'
diff --git a/test/integration/default/controls/config_spec.rb b/test/integration/default/controls/config_spec.rb
index 84fa60c..b6ea9c3 100644
--- a/test/integration/default/controls/config_spec.rb
+++ b/test/integration/default/controls/config_spec.rb
@@ -7,6 +7,7 @@
 
     # Custom config
     its('content') { should include "dc_hide_mailname='false'" }
+    its('content') { should include "dc_use_split_config='false'" }
   end
 
   describe file('/etc/mailname') do
diff --git a/test/salt/pillar/exim.sls b/test/salt/pillar/exim.sls
index d8af6b3..4138127 100644
--- a/test/salt/pillar/exim.sls
+++ b/test/salt/pillar/exim.sls
@@ -3,8 +3,9 @@
 ---
 exim:
   config:
-    configtype: 'satellite'
+    configtype: satellite
     hide_mailname: 'false'
+    use_split_config: false
   mailname: foo.bar.baz
   aliases:
     root: root@bar.baz