Adding caching to the Environment class

Caching the module path (because we check which
directories exist, and this method can get called
often), and the complete list of modules.

The cache ttl uses the filetimeout, which defaults to
15 seconds.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 16 insertions, 0 deletions

diff --git a/spec/unit/node/environment.rb b/spec/unit/node/environment.rb
index 5fac98b77..3b58a115b 100755
--- a/spec/unit/node/environment.rb
+++ b/spec/unit/node/environment.rb
@@ -6,6 +6,22 @@ require 'puppet/node/environment'
 require 'puppet/util/execution'
 
 describe Puppet::Node::Environment do
+    after do
+        Puppet::Node::Environment.clear
+    end
+
+    it "should include the Cacher module" do
+        Puppet::Node::Environment.ancestors.should be_include(Puppet::Util::Cacher)
+    end
+    
+    it "should use the filetimeout for the ttl for the modulepath" do
+        Puppet::Node::Environment.attr_ttl(:modulepath).should == Integer(Puppet[:filetimeout])
+    end
+    
+    it "should use the filetimeout for the ttl for the module list" do
+        Puppet::Node::Environment.attr_ttl(:modules).should == Integer(Puppet[:filetimeout])
+    end
+
     it "should use the default environment if no name is provided while initializing an environment" do
         Puppet.settings.expects(:value).with(:environment).returns("one")
         Puppet::Node::Environment.new().name.should == :one