force preprocess_deferred

Author: LukasAud

spec/acceptance/deferred_spec.rb

@@ -17,6 +17,25 @@ def read_win_file_if_exists(path)
   { exists: exists, content: exists ? body : '' }
 end
 
+# ---- NEW: wrappers that always pass the required flags ----
+def puppet_flags
+  [
+    '--preprocess_deferred', 'true',
+    '--preferred_serialization_format', 'json',
+    '--color', 'false',
+    '--detailed-exitcodes'
+  ]
+end
+
+def apply_manifest_with_flags(manifest, opts = {})
+  apply_manifest(manifest, opts.merge(puppet_options: puppet_flags))
+end
+
+def idempotent_apply_with_flags(manifest, opts = {})
+  idempotent_apply(manifest, opts.merge(puppet_options: puppet_flags))
+end
+# ----------------------------------------------------------
+
 RSpec.configure do |config|
   config.before(:all) do
     # Ensure Deferred values are preprocessed before provider runs (Puppet 8 default is false)
@@ -29,6 +48,10 @@ def read_win_file_if_exists(path)
     fmt = run_shell('puppet config print preferred_serialization_format --section agent').stdout.strip
     raise 'preprocess_deferred not true on agent!' unless pp.casecmp('true').zero?
     raise 'preferred_serialization_format not json on agent!' unless fmt.casecmp('json').zero?
+
+    # Extra visibility: show puppet version & confdir in logs
+    run_shell('puppet --version')
+    run_shell('puppet config print confdir')
   end
 end
 
@@ -39,16 +62,28 @@ def read_win_file_if_exists(path)
   let(:dsc_deferred_stringified) { read_fixture('03a_dsc_deferred_stringified.pp') }
   let(:dsc_deferred_bad_unwrap)  { read_fixture('03b_dsc_deferred_bad_unwrap.pp') }
 
+  # ---- NEW: diagnostic example to prove effective settings for this process ----
+  it '00: confirms preprocess_deferred=true and JSON are active in the current apply context' do
+    print_pp = <<-PP
+      notify { "preprocess_deferred=#{Puppet.lookup(:settings).value('preprocess_deferred')}": }
+      notify { "preferred_serialization_format=#{Puppet.lookup(:settings).value('preferred_serialization_format')}": }
+    PP
+    res = apply_manifest_with_flags(print_pp)
+    expect(res.stdout).to match(%r{preprocess_deferred=true}i)
+    expect(res.stdout).to match(%r{preferred_serialization_format=json}i)
+  end
+  # ------------------------------------------------------------------------------
+
   it 'control (01): native file + Deferred resolves to hello-file' do
-    result = idempotent_apply(control_manifest)
+    result = idempotent_apply_with_flags(control_manifest)
     expect(result.exit_code).to eq(0)
     out = read_win_file_if_exists('C:/Temp/deferred_ok.txt')
     expect(out[:exists]).to be(true)
     expect(out[:content].strip).to eq('hello-file')
   end
 
   it '02: passing Deferred via variable to DSC resolves to hello-dsc (otherwise flag bug)' do
-    apply = apply_manifest(dsc_deferred_direct)
+    apply = apply_manifest_with_flags(dsc_deferred_direct)
     out   = read_win_file_if_exists('C:/Temp/from_dsc.txt')
     content = out[:content].strip
     if out[:exists] && content == 'hello-dsc'
@@ -62,7 +97,7 @@ def read_win_file_if_exists(path)
 
   # NEW 02b: inline Deferred on the DSC property (no variable intermediary)
   it '02b: passing Deferred inline to DSC resolves to hello-dsc-inline (otherwise flag bug)' do
-    apply = apply_manifest(dsc_deferred_inline)
+    apply = apply_manifest_with_flags(dsc_deferred_inline)
     out   = read_win_file_if_exists('C:/Temp/from_dsc_inline.txt')
     content = out[:content].strip
     if out[:exists] && content == 'hello-dsc-inline'
@@ -75,15 +110,15 @@ def read_win_file_if_exists(path)
   end
 
   it '03a: stringifying a Deferred writes the function form (reproduces customer report)' do
-    apply_manifest(dsc_deferred_stringified)
+    apply_manifest_with_flags(dsc_deferred_stringified)
     out = read_win_file_if_exists('C:/Temp/from_dsc_var_string.txt')
     expect(out[:exists]).to be(true)
     expect(out[:content]).to match(%r{Deferred\s*\(|Puppet::Pops::Types::Deferred}i)
     expect(out[:content]).not_to match(%r{\bhello-var\b})
   end
 
   it '03b: unwrap on a non‑Sensitive is a no‑op; also writes the function form' do
-    apply_manifest(dsc_deferred_bad_unwrap)
+    apply_manifest_with_flags(dsc_deferred_bad_unwrap)
     out   = read_win_file_if_exists('C:/Temp/from_dsc_var_bad_unwrap.txt')
     out   = read_win_file_if_exists('C:/Temp/from_dsc_var.txt') unless out[:exists]
     expect(out[:exists]).to be(true)