Lazily eval the empty hash default and remove nil types

We don't need to specify nil anymore since it's the default now.

Signed-off-by: Tim Smith <tsmith@chef.io>

1 files changed, 7 insertions, 6 deletions

diff --git a/lib/chef/resource/cron_d.rb b/lib/chef/resource/cron_d.rb
index 5dacc88fd4..caab0a4b7b 100644
--- a/lib/chef/resource/cron_d.rb
+++ b/lib/chef/resource/cron_d.rb
@@ -135,24 +135,24 @@ class Chef
                description: "The user to run the cron job as.",
                default: "root"
 
-      property :mailto, [String, NilClass],
+      property :mailto, String,
                description: "Set the MAILTO environment variable in the cron.d file."
 
-      property :path, [String, NilClass],
+      property :path, String,
                description: "Set the PATH environment variable in the cron.d file."
 
-      property :home, [String, NilClass],
+      property :home, String,
                description: "Set the HOME environment variable in the cron.d file."
 
-      property :shell, [String, NilClass],
+      property :shell, String,
                description: "Set the HOME environment variable in the cron.d file."
 
-      property :comment, [String, NilClass],
+      property :comment, String,
                description: "A comment to place in the cron.d file."
 
       property :environment, Hash,
                description: "A Hash containing additional arbitrary environment variables under which the cron job will be run.",
-               default: {}
+               default: lazy { Hash.new }
 
       property :mode, [String, Integer],
                description: "The octal mode of the generated crontab file.",
@@ -193,6 +193,7 @@ class Chef
       end
 
       action_class do
+        # @return [String] cron_name property with . replaced with -
         def sanitized_name
           new_resource.cron_name.tr(".", "-")
         end