This commit cleans up and simplifies the logic in the DSC Base Provider; prior to this commit, the set method included handling for nilis and should states in the change hash; with the current implementation, neither of these values can ever be nil without reaching a runtime error prior to this method being called.
This commit therefore removes the complexity for handling cases where is or should are nil and removes the never-called method create_absent as it is not needed.
This commit further simplifies the set method by removing custom handling for retrieving the name hash as the names for Puppetized DSC Resources are always hashes and any extraneous values are stripped in invoke_set_method itself.
This commit cleans up and simplifies the logic in the DSC Base Provider; prior to this commit, the
setmethod included handling fornilisandshouldstates in the change hash; with the current implementation, neither of these values can ever benilwithout reaching a runtime error prior to this method being called.This commit therefore removes the complexity for handling cases where
isorshouldareniland removes the never-called methodcreate_absentas it is not needed.This commit further simplifies the
setmethod by removing custom handling for retrieving the name hash as the names for Puppetized DSC Resources are always hashes and any extraneous values are stripped ininvoke_set_methoditself.Resolves #141