diff options
author | Markus Roberts <Markus@reality.com> | 2010-06-10 22:51:12 -0700 |
---|---|---|
committer | test branch <puppet-dev@googlegroups.com> | 2010-02-17 06:50:53 -0800 |
commit | ccc869ea48397235d7ba2a5695424eee4923cb9d (patch) | |
tree | ffd534b345923f62733e4e48cc3e36377fa059fe /spec/unit/util/autoload.rb | |
parent | 7c6b8836453b2b1e8679923f98854be3b0022edd (diff) | |
download | puppet-ccc869ea48397235d7ba2a5695424eee4923cb9d.tar.gz puppet-ccc869ea48397235d7ba2a5695424eee4923cb9d.tar.xz puppet-ccc869ea48397235d7ba2a5695424eee4923cb9d.zip |
Part 2 of fix for #1175 (functions in environments)
Jesse and I are shooting for the minimal viable fix here, with the idea that
a great deal of refactoring is needed but isn't appropriate at this time. The
changes in this commit are:
* Index the function-holding modules by environment
* We need to know the "current environment" when we're defining a function so
we can attach it to the proper module, and this information isn't dynamically
available when user-defined functions are being created (we're being called by
user written code that doesn't "know" about environments) so we cheat and
stash the value in Puppet::Node::Environment
* since we must do this anyway, it turns out to be cleaner & safer to do the
same when we are evaluating a functon. This is the main change from the prior
version of this patch.
* Add a special *root* environment for the built in functions, and extend all
scopes with it.
* Index the function characteristics (name, type, docstring, etc.) by environment
* Make the autoloader environment aware, so that it uses the modulepath for the
specified environment rather than the default
* Turn off caching of the modulepath since it potentially changes for each node
* Tweak tests that weren't environment aware
Diffstat (limited to 'spec/unit/util/autoload.rb')
-rwxr-xr-x | spec/unit/util/autoload.rb | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/spec/unit/util/autoload.rb b/spec/unit/util/autoload.rb index 220cb5f68..5862073ea 100755 --- a/spec/unit/util/autoload.rb +++ b/spec/unit/util/autoload.rb @@ -15,10 +15,6 @@ describe Puppet::Util::Autoload do Puppet::Util::Autoload.ancestors.should be_include(Puppet::Util::Cacher) end - it "should use a ttl of 15 for the search path" do - Puppet::Util::Autoload.attr_ttl(:searchpath).should == 15 - end - describe "when building the search path" do it "should collect all of the plugins and lib directories that exist in the current environment's module path" do Puppet.settings.expects(:value).with(:environment).returns "foo" |