Fixing #2336 - qualified variables only throw warnings

We were previously throwing exceptions.

This also ports all of the tests for variable lookup
over to rspec.

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

1 files changed, 102 insertions, 4 deletions

diff --git a/spec/unit/parser/scope.rb b/spec/unit/parser/scope.rb
index 13559528b..b0c0a8ef3 100755
--- a/spec/unit/parser/scope.rb
+++ b/spec/unit/parser/scope.rb
@@ -4,13 +4,112 @@ require File.dirname(__FILE__) + '/../../spec_helper'
 
 describe Puppet::Parser::Scope do
     before :each do
-        @scope = Puppet::Parser::Scope.new()
         @topscope = Puppet::Parser::Scope.new()
-        @scope.stubs(:parent).returns(@topscope)
+        # This is necessary so we don't try to use the compiler to discover our parent.
+        @topscope.parent = nil
+        @scope = Puppet::Parser::Scope.new()
+        @scope.parent = @topscope
     end
 
-    describe Puppet::Parser::Scope, "when setvar is called with append=true" do
+    describe "when looking up a variable" do
+        it "should default to an empty string" do
+            @scope.lookupvar("var").should == ""
+        end
+        
+        it "should return an string when asked for a string" do
+            @scope.lookupvar("var", true).should == ""
+        end
+
+        it "should return ':undefined' for unset variables when asked not to return a string" do
+            @scope.lookupvar("var", false).should == :undefined
+        end
+
+        it "should be able to look up values" do
+            @scope.setvar("var", "yep")
+            @scope.lookupvar("var").should == "yep"
+        end
+
+        it "should be able to look up variables in parent scopes" do
+            @topscope.setvar("var", "parentval")
+            @scope.lookupvar("var").should == "parentval"
+        end
+
+        it "should prefer its own values to parent values" do
+            @topscope.setvar("var", "parentval")
+            @scope.setvar("var", "childval")
+            @scope.lookupvar("var").should == "childval"
+        end
+
+        describe "and the variable is qualified" do
+            before do
+                @parser = Puppet::Parser::Parser.new()
+                @compiler = Puppet::Parser::Compiler.new(stub("node", :name => "foonode"), @parser)
+                @scope.compiler = @compiler
+                @scope.parser = @parser
+            end
+
+            def create_class_scope(name)
+                klass = @parser.newclass(name)
+                Puppet::Parser::Resource.new(:type => "class", :title => name, :scope => @scope, :source => mock('source')).evaluate
+
+                return @compiler.class_scope(klass)
+            end
+
+            it "should be able to look up explicitly fully qualified variables from main" do
+                other_scope = create_class_scope("")
+
+                other_scope.setvar("othervar", "otherval")
 
+                @scope.lookupvar("::othervar").should == "otherval"
+            end
+
+            it "should be able to look up explicitly fully qualified variables from other scopes" do
+                other_scope = create_class_scope("other")
+
+                other_scope.setvar("var", "otherval")
+
+                @scope.lookupvar("::other::var").should == "otherval"
+            end
+
+            it "should be able to look up deeply qualified variables" do
+                other_scope = create_class_scope("other::deep::klass")
+
+                other_scope.setvar("var", "otherval")
+
+                @scope.lookupvar("other::deep::klass::var").should == "otherval"
+            end
+
+            it "should return an empty string for qualified variables that cannot be found in other classes" do
+                other_scope = create_class_scope("other::deep::klass")
+
+                @scope.lookupvar("other::deep::klass::var").should == ""
+            end
+
+            it "should warn and return an empty string for qualified variables whose classes have not been evaluated" do
+                klass = @parser.newclass("other::deep::klass")
+                @scope.expects(:warning)
+                @scope.lookupvar("other::deep::klass::var").should == ""
+            end
+
+            it "should warn and return an empty string for qualified variables whose classes do not exist" do
+                @scope.expects(:warning)
+                @scope.lookupvar("other::deep::klass::var").should == ""
+            end
+
+            it "should return ':undefined' when asked for a non-string qualified variable from a class that does not exist" do
+                @scope.stubs(:warning)
+                @scope.lookupvar("other::deep::klass::var", false).should == :undefined
+            end
+
+            it "should return ':undefined' when asked for a non-string qualified variable from a class that has not been evaluated" do
+                @scope.stubs(:warning)
+                klass = @parser.newclass("other::deep::klass")
+                @scope.lookupvar("other::deep::klass::var", false).should == :undefined
+            end
+        end
+    end
+
+    describe Puppet::Parser::Scope, "when setvar is called with append=true" do
         it "should raise error if the variable is already defined in this scope" do
             @scope.setvar("var","1",nil,nil,false)
             lambda { @scope.setvar("var","1",nil,nil,true) }.should raise_error(Puppet::ParseError)
@@ -97,5 +196,4 @@ describe Puppet::Parser::Scope do
             Puppet::Parser::Scope.number?("0x89g").should be_nil
         end
     end
-
 end