(bugfix) - fix disable_time_zone_synchronization so that it correctly disables functionality when set to true

[david22swan]

No description provided.

[codecov-commenter]

Codecov Report

Merging #161 into main will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #161   +/-   ##
=======================================
  Coverage   94.97%   94.97%           
=======================================
  Files           6        6           
  Lines         836      836           
=======================================
  Hits          794      794           
  Misses         42       42           

Impacted Files	Coverage Δ
lib/puppet_x/puppetlabs/scheduled_task/trigger.rb	94.77% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2a1622...40bd93b. Read the comment docs.

[sanfrancrisko]

I'm wondering if we should take a look at what the test coverage here given we've got 2 fixes going in to this functionality in as many days?

Would be happy for this to be merged for now, to get the fix out there, but I think we need to look at the test coverage.

If we think enhancing the tests is quick, then let's do it, otherwise we can merge and raise a ticket to address after.