puppet 3.1.0.rc1 → 3.1.0.rc2

data/man/man8/puppet-parser.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-PARSER" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-PARSER" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-parser\fR \- Interact directly with the parser\.
@@ -10,16 +10,12 @@
 puppet parser \fIaction\fR
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .

data/man/man8/puppet-plugin.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-PLUGIN" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-PLUGIN" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-plugin\fR \- Interact with the Puppet plugin system\.
@@ -16,16 +16,12 @@ This subcommand provides network access to the puppet master\'s store of plugins
 The puppet master serves Ruby code collected from the \fBlib\fR directories of its modules\. These plugins can be used on agent nodes to extend Facter and implement custom types and providers\. Plugins are normally downloaded by puppet agent during the course of a run\.
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .

data/man/man8/puppet-queue.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-QUEUE" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-QUEUE" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-queue\fR \- Queuing daemon for asynchronous storeconfigs

data/man/man8/puppet-report.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-REPORT" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-REPORT" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-report\fR \- Create, display, and submit reports\.
@@ -10,16 +10,12 @@
 puppet report \fIaction\fR [\-\-terminus TERMINUS] [\-\-extra HASH]
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .
@@ -84,7 +80,7 @@ puppet report info [\-\-terminus TERMINUS] [\-\-extra HASH]
 \fBDESCRIPTION\fR
 .
 .IP
-Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-mode\' option\.
+Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-run_mode\' option\.
 .
 .TP
 \fBsave\fR \- API only: submit a report\.

data/man/man8/puppet-resource.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-RESOURCE" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-RESOURCE" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-resource\fR \- The resource abstraction layer shell

data/man/man8/puppet-resource_type.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-RESOURCE_TYPE" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-RESOURCE_TYPE" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-resource_type\fR \- View classes, defined resource types, and nodes from all manifests\.
@@ -16,16 +16,12 @@ This subcommand reads information about the resource collections (classes, nodes
 It will eventually be extended to examine native resource types\.
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .
@@ -105,7 +101,7 @@ puppet resource_type info [\-\-terminus TERMINUS] [\-\-extra HASH]
 \fBDESCRIPTION\fR
 .
 .IP
-Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-mode\' option\.
+Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-run_mode\' option\.
 .
 .TP
 \fBsave\fR \- Invalid for this subcommand\.

data/man/man8/puppet-secret_agent.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-SECRET_AGENT" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-SECRET_AGENT" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-secret_agent\fR \- Mimics puppet agent\.
@@ -13,16 +13,12 @@ puppet secret_agent \fIaction\fR
 This subcommand currently functions as a proof of concept, demonstrating how the Faces API exposes Puppet\'s internal systems to application logic; compare the actual code for puppet agent\. It will eventually replace puppet agent entirely, and can provide a template for users who wish to implement agent\-like functionality with non\-standard application logic\.
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .

data/man/man8/puppet-status.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET\-STATUS" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET\-STATUS" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\-status\fR \- View puppet server status\.
@@ -10,16 +10,12 @@
 puppet status \fIaction\fR [\-\-terminus TERMINUS] [\-\-extra HASH]
 .
 .SH "OPTIONS"
-Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR is a valid configuration parameter, so you can specify \fB\-\-server <servername>\fR as an argument\.
+Note that any configuration parameter that\'s valid in the configuration file is also a valid long argument, although it may or may not be relevant to the present action\. For example, \fBserver\fR and \fBrun_mode\fR are valid configuration parameters, so you can specify \fB\-\-server <servername>\fR, or \fB\-\-run_mode <runmode>\fR as an argument\.
 .
 .P
 See the configuration file documentation at \fIhttp://docs\.puppetlabs\.com/references/stable/configuration\.html\fR for the full list of acceptable parameters\. A commented list of all configuration options can also be generated by running puppet with \fB\-\-genconfig\fR\.
 .
 .TP
-\-\-mode MODE
-The run mode to use for the current action\. Valid modes are \fBuser\fR, \fBagent\fR, and \fBmaster\fR\.
-.
-.TP
 \-\-render\-as FORMAT
 The format in which to render output\. The most common formats are \fBjson\fR, \fBs\fR (string), \fByaml\fR, and \fBconsole\fR, but other options such as \fBdot\fR are sometimes available\.
 .
@@ -102,7 +98,7 @@ puppet status info [\-\-terminus TERMINUS] [\-\-extra HASH]
 \fBDESCRIPTION\fR
 .
 .IP
-Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-mode\' option\.
+Prints the default terminus class for this subcommand\. Note that different run modes may have different default termini; when in doubt, specify the run mode with the \'\-\-run_mode\' option\.
 .
 .TP
 \fBsave\fR \- Invalid for this subcommand\.

data/man/man8/puppet.8

@@ -1,7 +1,7 @@
 .\" generated with Ronn/v0.7.3
 .\" http://github.com/rtomayko/ronn/tree/0.7.3
 .
-.TH "PUPPET" "8" "May 2012" "Puppet Labs, LLC" "Puppet manual"
+.TH "PUPPET" "8" "January 2013" "Puppet Labs, LLC" "Puppet manual"
 .
 .SH "NAME"
 \fBpuppet\fR

data/spec/integration/parser/ruby_manifest_spec.rb

@@ -0,0 +1,128 @@
+#! /usr/bin/env ruby
+require 'spec_helper'
+
+require 'tempfile'
+require 'puppet_spec/files'
+
+describe "Pure ruby manifests" do
+  include PuppetSpec::Files
+
+  before do
+    @node = Puppet::Node.new "testnode"
+
+    @scope_resource = stub 'scope_resource', :builtin? => true, :finish => nil, :ref => 'Class[main]'
+    @scope = stub 'scope', :resource => @scope_resource, :source => mock("source")
+    @test_dir = tmpdir('ruby_manifest_test')
+    Puppet.expects(:deprecation_warning).at_least(1)
+  end
+
+  after do
+    Puppet.settings.clear
+  end
+
+  def write_file(name, contents)
+    path = File.join(@test_dir, name)
+    File.open(path, "w") { |f| f.write(contents) }
+    path
+  end
+
+  def compile(contents)
+    Puppet[:code] = contents
+    Dir.chdir(@test_dir) do
+      Puppet::Parser::Compiler.compile(Puppet::Node.new("mynode"))
+    end
+  end
+
+  it "should allow classes" do
+    write_file('foo.rb', ["hostclass 'one' do notify('one_notify') end",
+                          "hostclass 'two' do notify('two_notify') end"].join("\n"))
+    catalog = compile("import 'foo'\ninclude one")
+    catalog.resource("Notify[one_notify]").should_not be_nil
+    catalog.resource("Notify[two_notify]").should be_nil
+  end
+
+  it "should allow defines" do
+    write_file('foo.rb', 'define "bar", :arg do notify("bar_#{@name}_#{@arg}") end')
+    catalog = compile("import 'foo'\nbar { instance: arg => 'xyz' }")
+    catalog.resource("Notify[bar_instance_xyz]").should_not be_nil
+    catalog.resource("Bar[instance]").should_not be_nil
+  end
+
+  it "should allow node declarations" do
+    write_file('foo.rb', "node 'mynode' do notify('mynode') end")
+    catalog = compile("import 'foo'")
+    node_declaration = catalog.resource("Notify[mynode]")
+    node_declaration.should_not be_nil
+    node_declaration.title.should == 'mynode'
+  end
+
+  it "should allow access to the environment" do
+    write_file('foo.rb', ["hostclass 'bar' do",
+                          "  if environment.is_a? Puppet::Node::Environment",
+                          "    notify('success')",
+                          "  end",
+                          "end"].join("\n"))
+    compile("import 'foo'\ninclude bar").resource("Notify[success]").should_not be_nil
+  end
+
+  it "should allow creation of resources of built-in types" do
+    write_file('foo.rb', "hostclass 'bar' do file 'test_file', :owner => 'root', :mode => '644' end")
+    catalog = compile("import 'foo'\ninclude bar")
+    file = catalog.resource("File[test_file]")
+    file.should be_a(Puppet::Resource)
+    file.type.should == 'File'
+    file.title.should == 'test_file'
+    file.exported.should_not be
+    file.virtual.should_not be
+    file[:owner].should == 'root'
+    file[:mode].should == '644'
+    file[:stage].should be_nil # TODO: is this correct behavior?
+  end
+
+  it "should allow calling user-defined functions" do
+    write_file('foo.rb', "hostclass 'bar' do user_func 'name', :arg => 'xyz' end")
+    catalog = compile(['define user_func($arg) { notify {"n_$arg": } }',
+                       'import "foo"',
+                       'include bar'].join("\n"))
+    catalog.resource("Notify[n_xyz]").should_not be_nil
+    catalog.resource("User_func[name]").should_not be_nil
+  end
+
+  it "should be properly cached for multiple compiles" do
+    # Note: we can't test this by calling compile() twice, because
+    # that sets Puppet[:code], which clears out all cached
+    # environments.
+    Puppet[:filetimeout] = 1000
+    write_file('foo.rb', "hostclass 'bar' do notify('success') end")
+    Puppet[:code] = "import 'foo'\ninclude bar"
+
+    # Compile the catalog and check it
+    catalog = Dir.chdir(@test_dir) do
+      Puppet::Parser::Compiler.compile(Puppet::Node.new("mynode"))
+    end
+    catalog.resource("Notify[success]").should_not be_nil
+
+    # Secretly change the file to make it invalid.  This change
+    # shouldn't be noticed because the we've set a high
+    # Puppet[:filetimeout].
+    write_file('foo.rb', "raise 'should not be executed'")
+
+    # Compile the catalog a second time and make sure it's still ok.
+    catalog = Dir.chdir(@test_dir) do
+      Puppet::Parser::Compiler.compile(Puppet::Node.new("mynode"))
+    end
+    catalog.resource("Notify[success]").should_not be_nil
+  end
+
+  it "should be properly reloaded when stale" do
+    Puppet[:filetimeout] = -1 # force stale check to happen all the time
+    write_file('foo.rb', "hostclass 'bar' do notify('version1') end")
+    catalog = compile("import 'foo'\ninclude bar")
+    catalog.resource("Notify[version1]").should_not be_nil
+    sleep 1 # so that timestamp will change forcing file reload
+    write_file('foo.rb', "hostclass 'bar' do notify('version2') end")
+    catalog = compile("import 'foo'\ninclude bar")
+    catalog.resource("Notify[version1]").should be_nil
+    catalog.resource("Notify[version2]").should_not be_nil
+  end
+end

data/spec/lib/puppet_spec/compiler.rb

@@ -3,15 +3,4 @@ module PuppetSpec::Compiler
     Puppet[:code] = string
     Puppet::Parser::Compiler.compile(node)
   end
-
-  def compile_ruby_to_catalog(string = nil, node = Puppet::Node.new('foonode'))
-    File.stubs(:read).returns(string)
-    Puppet::Parser::Compiler.compile(node)
-  end
-
-  def prepare_compiler
-    let(:compiler) { Puppet::Parser::Compiler.new(Puppet::Node.new("floppy", :environment => 'production')) }
-    let(:scope)    { Puppet::Parser::Scope.new compiler }
-  end
-
 end

data/spec/spec_helper.rb

@@ -1,7 +1,6 @@
-# NOTE: a lot of the stuff in this file is duplicated in the
-# "puppet_spec_helper" in the project puppetlabs_spec_helper. We should
-# probably eat our own dog food and get rid of most of this from here, and
-# have the puppet core itself use puppetlabs_spec_helper
+# NOTE: a lot of the stuff in this file is duplicated in the "puppet_spec_helper" in the project
+#  puppetlabs_spec_helper.  We should probably eat our own dog food and get rid of most of this from here,
+#  and have the puppet core itself use puppetlabs_spec_helper
 
 dir = File.expand_path(File.dirname(__FILE__))
 $LOAD_PATH.unshift File.join(dir, 'lib')

data/spec/unit/dsl/resource_api_spec.rb

@@ -0,0 +1,180 @@
+#! /usr/bin/env ruby
+require 'spec_helper'
+
+require 'puppet/dsl/resource_api'
+
+describe Puppet::DSL::ResourceAPI do
+  before do
+    @compiler = Puppet::Parser::Compiler.new(Puppet::Node.new("foo"))
+    @scope = Puppet::Parser::Scope.new(@compiler, :source => "foo")
+    @resource = Puppet::Parser::Resource.new(:mytype, "myresource", :scope => @scope)
+    @api = Puppet::DSL::ResourceAPI.new(@resource, @scope, proc { })
+  end
+
+  it "should include the resource type collection helper" do
+    Puppet::DSL::ResourceAPI.ancestors.should be_include(Puppet::Resource::TypeCollectionHelper)
+  end
+
+  it "should use the scope's environment as its environment" do
+    @scope.expects(:environment).returns "myenv"
+    @api.environment.should == "myenv"
+  end
+
+  it "should be able to set all of its parameters as instance variables" do
+    @resource["foo"] = "myval"
+    @api.set_instance_variables
+    @api.instance_variable_get("@foo").should == "myval"
+  end
+
+  describe "when calling a function" do
+    it "should return false if the function does not exist" do
+      Puppet::Parser::Functions.expects(:function).with("myfunc").returns nil
+      @api.call_function("myfunc", "foo").should be_false
+    end
+
+    it "should use the scope the call the provided function with the provided arguments and return the results" do
+      scope = stub 'scope'
+      @api.stubs(:scope).returns scope
+      Puppet::Parser::Functions.expects(:function).with("myfunc").returns "myfunc_method"
+
+      scope.expects(:myfunc_method).with("one", "two")
+      @api.call_function("myfunc", ["one", "two"])
+    end
+
+    it "should call 'include' when asked to call 'acquire'" do
+      scope = stub 'scope'
+      @api.stubs(:scope).returns scope
+      @api.stubs(:valid_type?).returns false
+
+      scope.expects(:function_include).with("one", "two")
+      @api.acquire("one", "two")
+    end
+  end
+
+  describe "when determining if a provided name is a valid type" do
+    it "should be valid if it's :class" do
+      @api.should be_valid_type(:class)
+    end
+
+    it "should be valid if it's :node" do
+      @api.should be_valid_type(:node)
+    end
+
+    it "should be valid if it's a builtin type" do
+      Puppet::Type.expects(:type).with(:mytype).returns "whatever"
+      @api.should be_valid_type(:mytype)
+    end
+
+    it "should be valid if it's a defined resource type in the environment's known resource types" do
+      collection = stub 'collection'
+      @api.stubs(:known_resource_types).returns collection
+      collection.expects(:definition).with(:mytype).returns "whatever"
+      @api.should be_valid_type(:mytype)
+    end
+
+    it "should not be valid unless it's a node, class, builtin type, or defined resource" do
+      collection = stub 'collection'
+      @api.stubs(:known_resource_types).returns collection
+      collection.expects(:definition).returns nil
+      Puppet::Type.expects(:type).returns nil
+      @api.should_not be_valid_type(:mytype)
+    end
+  end
+
+  describe "when creating a resource" do
+    before do
+      @api.scope.stubs(:source).returns stub("source")
+      @api.scope.compiler.stubs(:add_resource)
+      @created_resource = Puppet::Parser::Resource.new("yay", "eh", :scope => @api.scope)
+    end
+
+    it "should create and return a resource of the type specified" do
+      Puppet::Parser::Resource.expects(:new).with { |type, title, args| type == "mytype" }.returns @created_resource
+      @api.create_resource("mytype", "myname", {:foo => "bar"}).should == [@created_resource]
+    end
+
+    it "should use the name from the first element of the provided argument array" do
+      Puppet::Parser::Resource.expects(:new).with { |type, title, args| title == "myname" }.returns @created_resource
+      @api.create_resource("mytype", "myname", {:foo => "bar"})
+    end
+
+    it "should create multiple resources if the first element of the argument array is an array" do
+      second_resource = Puppet::Parser::Resource.new('yay', "eh", :scope => @api.scope)
+      Puppet::Parser::Resource.expects(:new).with { |type, title, args| title == "first" }.returns @created_resource
+      Puppet::Parser::Resource.expects(:new).with { |type, title, args| title == "second" }.returns @created_resource
+      @api.create_resource("mytype", ["first", "second"], {:foo => "bar"})
+    end
+
+    it "should provide its scope as the scope" do
+      Puppet::Parser::Resource.expects(:new).with { |type, title, args| args[:scope] == @api.scope }.returns @created_resource
+      @api.create_resource("mytype", "myname", {:foo => "bar"})
+    end
+
+    it "should set each provided argument as a parameter on the created resource" do
+      result = @api.create_resource("mytype", "myname", {"foo" => "bar", "biz" => "baz"}).shift
+      result["foo"].should == "bar"
+      result["biz"].should == "baz"
+    end
+
+    it "should add the resource to the scope's copmiler" do
+      Puppet::Parser::Resource.expects(:new).returns @created_resource
+      @api.scope.compiler.expects(:add_resource).with(@api.scope, @created_resource)
+      @api.create_resource("mytype", "myname", {:foo => "bar"})
+    end
+
+    it "should fail if the resource parameters are not a hash" do
+      lambda { @api.create_resource("mytype", "myname", %w{foo bar}) }.should raise_error(ArgumentError)
+    end
+  end
+
+  describe "when an unknown method is called" do
+    it "should create a resource if the method name is a valid type" do
+      @api.expects(:valid_type?).with(:mytype).returns true
+      @api.expects(:create_resource).with(:mytype, "myname", {:foo => "bar"}).returns true
+
+      @api.mytype("myname", :foo => "bar")
+    end
+
+    it "should call any function whose name matches the undefined method if the name is not a valid type" do
+      @api.expects(:valid_type?).with(:myfunc).returns false
+      @api.expects(:create_resource).never
+
+      Puppet::Parser::Functions.expects(:function).with(:myfunc).returns true
+
+      @api.expects(:call_function).with(:myfunc, %w{foo bar})
+
+      @api.myfunc("foo", "bar")
+    end
+
+    it "should raise a method missing error if the method is neither a type nor a function" do
+      @api.expects(:valid_type?).with(:myfunc).returns false
+      @api.expects(:create_resource).never
+
+      Puppet::Parser::Functions.expects(:function).with(:myfunc).returns false
+
+      @api.expects(:call_function).never
+
+      lambda { @api.myfunc("foo", "bar") }.should raise_error(NoMethodError)
+    end
+  end
+
+  it "should mark the specified resource as exported when creating a single exported resource" do
+    resources = @api.export @api.file("/my/file", :ensure => :present)
+    resources[0].should be_exported
+  end
+
+  it "should mark all created resources as exported when creating exported resources using a block" do
+    @compiler.expects(:add_resource).with { |s, res| res.exported == true }
+    @api.export { file "/my/file", :ensure => :present }
+  end
+
+  it "should mark the specified resource as virtual when creating a single virtual resource" do
+    resources = @api.virtual @api.file("/my/file", :ensure => :present)
+    resources[0].should be_virtual
+  end
+
+  it "should mark all created resources as virtual when creating virtual resources using a block" do
+    @compiler.expects(:add_resource).with { |s, res| res.virtual == true }
+    @api.virtual { file "/my/file", :ensure => :present }
+  end
+end