chef 10.12.0 → 10.14.0.beta.1

data/spec/unit/provider/service/windows_spec.rb

@@ -0,0 +1,235 @@
+#
+# Author:: Nuo Yan <nuo@opscode.com>
+# Author:: Seth Chisamore <schisamo@opscode.com>
+# Copyright:: Copyright (c) 2010-2011 Opscode, Inc
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+
+describe Chef::Provider::Service::Windows, "load_current_resource" do
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+    @new_resource = Chef::Resource::Service.new("chef")
+    @provider = Chef::Provider::Service::Windows.new(@new_resource, @run_context)
+    Object.send(:remove_const, 'Win32') if defined?(Win32)
+    Win32 = Module.new
+    Win32::Service = Class.new
+    Win32::Service::AUTO_START = 0x00000002
+    Win32::Service::DISABLED = 0x00000004
+    Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+      mock("StatusStruct", :current_state => "running"))
+    Win32::Service.stub!(:config_info).with(@new_resource.service_name).and_return(
+      mock("ConfigStruct", :start_type => "auto start"))
+    Win32::Service.stub!(:exists?).and_return(true)
+  end
+
+  it "should set the current resources service name to the new resources service name" do
+    @provider.load_current_resource
+    @provider.current_resource.service_name.should == 'chef'
+  end
+
+  it "should return the current resource" do
+    @provider.load_current_resource.should equal(@provider.current_resource)
+  end
+
+  it "should set the current resources status" do
+    @provider.load_current_resource
+    @provider.current_resource.running.should be_true
+  end
+
+  it "should set the current resources start type" do
+    @provider.load_current_resource
+    @provider.current_resource.enabled.should be_true
+  end
+
+  describe Chef::Provider::Service::Windows, "start_service" do
+    before(:each) do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "stopped"),
+        mock("StatusStruct", :current_state => "running"))
+    end
+
+    it "should call the start command if one is specified" do
+      @new_resource.start_command "sc start chef"
+      @provider.should_receive(:shell_out!).with("#{@new_resource.start_command}").and_return("Starting custom service")
+      @provider.start_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should use the built-in command if no start command is specified" do
+      Win32::Service.should_receive(:start).with(@new_resource.service_name)
+      @provider.start_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should do nothing if the service does not exist" do
+      Win32::Service.stub!(:exists?).with(@new_resource.service_name).and_return(false)
+      Win32::Service.should_not_receive(:start).with(@new_resource.service_name)
+      @provider.start_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+    it "should do nothing if the service is running" do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "running"))
+      @provider.load_current_resource
+      Win32::Service.should_not_receive(:start).with(@new_resource.service_name)
+      @provider.start_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+  end
+
+  describe Chef::Provider::Service::Windows, "stop_service" do
+    
+    before(:each) do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "running"),
+        mock("StatusStruct", :current_state => "stopped"))
+    end
+
+    it "should call the stop command if one is specified" do
+      @new_resource.stop_command "sc stop chef"
+      @provider.should_receive(:shell_out!).with("#{@new_resource.stop_command}").and_return("Stopping custom service")
+      @provider.stop_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should use the built-in command if no stop command is specified" do
+      Win32::Service.should_receive(:stop).with(@new_resource.service_name)
+      @provider.stop_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should do nothing if the service does not exist" do
+      Win32::Service.stub!(:exists?).with(@new_resource.service_name).and_return(false)
+      Win32::Service.should_not_receive(:stop).with(@new_resource.service_name)
+      @provider.stop_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+    it "should do nothing if the service is stopped" do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "stopped"))
+      @provider.load_current_resource
+      Win32::Service.should_not_receive(:stop).with(@new_resource.service_name)
+      @provider.stop_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+  end
+
+  describe Chef::Provider::Service::Windows, "restart_service" do
+
+    it "should call the restart command if one is specified" do
+      @new_resource.restart_command "sc restart"
+      @provider.should_receive(:shell_out!).with("#{@new_resource.restart_command}")
+      @provider.restart_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should stop then start the service if it is running" do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "running"), 
+        mock("StatusStruct", :current_state => "stopped"), 
+        mock("StatusStruct", :current_state => "stopped"), 
+        mock("StatusStruct", :current_state => "running"))
+      Win32::Service.should_receive(:stop).with(@new_resource.service_name)
+      Win32::Service.should_receive(:start).with(@new_resource.service_name)
+      @provider.restart_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should just start the service if it is stopped" do
+      Win32::Service.stub!(:status).with(@new_resource.service_name).and_return(
+        mock("StatusStruct", :current_state => "stopped"), 
+        mock("StatusStruct", :current_state => "stopped"), 
+        mock("StatusStruct", :current_state => "running"))
+      Win32::Service.should_receive(:start).with(@new_resource.service_name)
+      @provider.restart_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should do nothing if the service does not exist" do
+      Win32::Service.stub!(:exists?).with(@new_resource.service_name).and_return(false)
+      Win32::Service.should_not_receive(:stop).with(@new_resource.service_name)
+      Win32::Service.should_not_receive(:start).with(@new_resource.service_name)
+      @provider.restart_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+  end
+
+  describe Chef::Provider::Service::Windows, "enable_service" do
+
+    before(:each) do
+      Win32::Service.stub!(:config_info).with(@new_resource.service_name).and_return(
+        mock("ConfigStruct", :start_type => "disabled"))
+    end
+
+    it "should enable service" do
+      Win32::Service.should_receive(:configure).with(:service_name => @new_resource.service_name, :start_type => Win32::Service::AUTO_START)
+      @provider.enable_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should do nothing if the service does not exist" do
+      Win32::Service.stub!(:exists?).with(@new_resource.service_name).and_return(false)
+      Win32::Service.should_not_receive(:configure)
+      @provider.enable_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+    it "should do nothing if the service is enabled" do
+      Win32::Service.stub!(:config_info).with(@new_resource.service_name).and_return(
+        mock("ConfigStruct", :start_type => "auto start"))
+      Win32::Service.should_not_receive(:configure)
+      @provider.enable_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+  end
+  
+  describe Chef::Provider::Service::Windows, "disable_service" do
+
+    before(:each) do
+      Win32::Service.stub!(:config_info).with(@new_resource.service_name).and_return(
+        mock("ConfigStruct", :start_type => "auto start"))
+    end
+
+    it "should disable service" do
+      Win32::Service.should_receive(:configure).with(:service_name => @new_resource.service_name, :start_type => Win32::Service::DISABLED)
+      @provider.disable_service
+      @new_resource.updated_by_last_action?.should be_true
+    end
+
+    it "should do nothing if the service does not exist" do
+      Win32::Service.stub!(:exists?).with(@new_resource.service_name).and_return(false)
+      Win32::Service.should_not_receive(:configure)
+      @provider.disable_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+    it "should do nothing if the service is disabled" do
+      Win32::Service.stub!(:config_info).with(@new_resource.service_name).and_return(
+        mock("ConfigStruct", :start_type => "disabled"))
+      @provider.load_current_resource
+      Win32::Service.should_not_receive(:configure)
+      @provider.disable_service
+      @new_resource.updated_by_last_action?.should be_false
+    end
+
+  end
+end

data/spec/unit/provider/service_spec.rb

@@ -0,0 +1,169 @@
+#
+# Author:: AJ Christensen (<aj@hjksolutions.com>)
+# Copyright:: Copyright (c) 2008 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+
+describe Chef::Provider::Service do
+  before do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+    @new_resource = Chef::Resource::Service.new("chef")
+    @current_resource = Chef::Resource::Service.new("chef")
+
+    @provider = Chef::Provider::Service.new(@new_resource, @run_context)
+    @provider.current_resource = @current_resource
+    @provider.stub!(:load_current_resource)
+  end
+
+  describe "when enabling the service" do
+    it "should enable the service if disabled and set the resource as updated" do
+      @current_resource.enabled(false)
+      @provider.should_receive(:enable_service).and_return(true)
+      @provider.action_enable
+      @provider.converge
+      @provider.new_resource.should be_updated
+    end
+
+    it "should not enable the service if already enabled" do
+      @current_resource.enabled(true)
+      @provider.should_not_receive(:enable_service)
+      @provider.action_enable
+      @provider.converge
+      @provider.new_resource.should_not be_updated
+    end
+  end
+
+
+  describe "when disabling the service" do
+    it "should disable the service if enabled and set the resource as updated" do
+      @current_resource.stub!(:enabled).and_return(true)
+      @provider.should_receive(:disable_service).and_return(true)
+      @provider.run_action(:disable)
+      @provider.new_resource.should be_updated
+    end
+
+    it "should not disable the service if already disabled" do
+      @current_resource.stub!(:enabled).and_return(false)
+      @provider.should_not_receive(:disable_service).and_return(true)
+      @provider.run_action(:disable)
+      @provider.new_resource.should_not be_updated
+    end
+  end
+
+  describe "action_start" do
+    it "should start the service if it isn't running and set the resource as updated" do
+      @current_resource.running(false)
+      @provider.should_receive(:start_service).with.and_return(true)
+      @provider.run_action(:start)
+      @provider.new_resource.should be_updated
+    end
+
+    it "should not start the service if already running" do
+      @current_resource.running(true)
+      @provider.should_not_receive(:start_service)
+      @provider.run_action(:start)
+      @provider.new_resource.should_not be_updated
+    end
+  end
+
+  describe "action_stop" do
+    it "should stop the service if it is running and set the resource as updated" do
+      @current_resource.stub!(:running).and_return(true)
+      @provider.should_receive(:stop_service).and_return(true)
+      @provider.run_action(:stop)
+      @provider.new_resource.should be_updated
+    end
+
+    it "should not stop the service if it's already stopped" do
+      @current_resource.stub!(:running).and_return(false)
+      @provider.should_not_receive(:stop_service).and_return(true)
+      @provider.run_action(:stop)
+      @provider.new_resource.should_not be_updated
+    end
+  end
+
+  describe "action_restart" do
+    before do
+      @current_resource.supports(:restart => true)
+    end
+
+    it "should restart the service if it's supported and set the resource as updated" do
+      @provider.should_receive(:restart_service).and_return(true)
+      @provider.run_action(:restart)
+      @provider.new_resource.should be_updated
+    end
+
+    it "should restart the service even if it isn't running and set the resource as updated" do
+      @current_resource.stub!(:running).and_return(false)
+      @provider.should_receive(:restart_service).and_return(true)
+      @provider.run_action(:restart)
+      @provider.new_resource.should be_updated
+    end
+  end
+
+  describe "action_reload" do
+    before do
+      @new_resource.supports(:reload => true)
+    end
+
+    it "should raise an exception if reload isn't supported" do
+      @new_resource.supports(:reload => false)
+      @new_resource.stub!(:reload_command).and_return(false)
+      lambda { @provider.run_action(:reload) }.should raise_error(Chef::Exceptions::UnsupportedAction)
+    end
+
+    it "should reload the service if it is running and set the resource as updated" do
+      @current_resource.stub!(:running).and_return(true)
+      @provider.should_receive(:reload_service).and_return(true)
+      @provider.run_action(:reload)
+      @provider.new_resource.should be_updated
+    end
+
+    it "should not reload the service if it's stopped" do
+      @current_resource.stub!(:running).and_return(false)
+      @provider.should_not_receive(:reload_service).and_return(true)
+      @provider.run_action(:stop)
+      @provider.new_resource.should_not be_updated
+    end
+  end
+
+  it "delegates enable_service to subclasses" do
+    lambda { @provider.enable_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+
+  it "delegates disable_service to subclasses" do
+    lambda { @provider.disable_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+
+  it "delegates start_service to subclasses" do
+    lambda { @provider.start_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+
+  it "delegates stop_service to subclasses" do
+    lambda { @provider.stop_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+
+  it "delegates restart_service to subclasses" do
+    lambda { @provider.restart_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+
+  it "delegates reload_service to subclasses" do
+    lambda { @provider.reload_service }.should raise_error(Chef::Exceptions::UnsupportedAction)
+  end
+end

data/spec/unit/provider/subversion_spec.rb

@@ -0,0 +1,281 @@
+#
+# Author:: Daniel DeLeo (<dan@kallistec.com>)
+# Copyright:: Copyright (c) 2008 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+
+require 'spec_helper'
+
+describe Chef::Provider::Subversion do
+
+  before do
+    @resource = Chef::Resource::Subversion.new("my app")
+    @resource.repository "http://svn.example.org/trunk/"
+    @resource.destination "/my/deploy/dir"
+    @resource.revision "12345"
+    @resource.svn_arguments(false)
+    @resource.svn_info_args(false)
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+    @provider = Chef::Provider::Subversion.new(@resource, @run_context)
+  end
+
+  it "converts resource attributes to options for run_command and popen4" do
+    @provider.run_options.should == {}
+    @resource.user 'deployninja'
+    @provider.run_options.should == {:user => "deployninja"}
+  end
+
+  context "determining the revision of the currently deployed code" do
+
+    before do
+      @stdout = mock("stdout")
+      @stderr = mock("stderr")
+      @exitstatus = mock("exitstatus")
+    end
+
+    it "sets the revision to nil if there isn't any deployed code yet" do
+      ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").and_return(false)
+      @provider.find_current_revision.should be_nil
+    end
+
+    it "determines the current revision if there's a checkout with svn data available" do
+      example_svn_info =  "Path: .\n" +
+                          "URL: http://svn.example.org/trunk/myapp\n" +
+                          "Repository Root: http://svn.example.org\n" +
+                          "Repository UUID: d62ff500-7bbc-012c-85f1-0026b0e37c24\n" +
+                          "Revision: 11739\nNode Kind: directory\n" +
+                          "Schedule: normal\n" +
+                          "Last Changed Author: codeninja\n" +
+                          "Last Changed Rev: 11410\n" + # Last Changed Rev is preferred to Revision
+                          "Last Changed Date: 2009-03-25 06:09:56 -0600 (Wed, 25 Mar 2009)\n\n"
+      ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").and_return(true)
+      ::File.should_receive(:directory?).with("/my/deploy/dir").and_return(true)
+      ::Dir.should_receive(:chdir).with("/my/deploy/dir").and_yield
+      @stdout.stub!(:string).and_return(example_svn_info)
+      @stderr.stub!(:string).and_return("")
+      @exitstatus.stub!(:exitstatus).and_return(0)
+      expected_command = ["svn info", {:cwd=>"/my/deploy/dir"}]
+      @provider.should_receive(:popen4).with(*expected_command).
+                                        and_yield("no-pid", "no-stdin", @stdout,@stderr).
+                                        and_return(@exitstatus)
+      @provider.find_current_revision.should eql("11410")
+    end
+
+    it "gives nil as the current revision if the deploy dir isn't a SVN working copy" do
+      example_svn_info = "svn: '/tmp/deploydir' is not a working copy\n"
+      ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").and_return(true)
+      ::File.should_receive(:directory?).with("/my/deploy/dir").and_return(true)
+      ::Dir.should_receive(:chdir).with("/my/deploy/dir").and_yield
+      @stdout.stub!(:string).and_return(example_svn_info)
+      @stderr.stub!(:string).and_return("")
+      @exitstatus.stub!(:exitstatus).and_return(1)
+      @provider.should_receive(:popen4).and_yield("no-pid", "no-stdin", @stdout,@stderr).
+                                        and_return(@exitstatus)
+      @provider.find_current_revision.should be_nil
+    end
+
+    it "finds the current revision when loading the current resource state" do
+      # note: the test is kinda janky, but it provides regression coverage for CHEF-2092
+      @resource.instance_variable_set(:@action, :sync)
+      @provider.should_receive(:find_current_revision).and_return("12345")
+      @provider.load_current_resource
+      @provider.current_resource.revision.should == "12345"
+    end
+  end
+
+  it "creates the current_resource object and sets its revision to the current deployment's revision as long as we're not exporting" do
+    @provider.stub!(:find_current_revision).and_return("11410")
+    @provider.new_resource.instance_variable_set :@action, [:checkout]
+    @provider.load_current_resource
+    @provider.current_resource.name.should eql(@resource.name)
+    @provider.current_resource.revision.should eql("11410")
+  end
+
+  context "resolving revisions to an integer" do
+
+    before do
+      @stdout = mock("stdout")
+      @stderr = mock("stderr")
+      @resource.svn_info_args "--no-auth-cache"
+    end
+
+    it "returns the revision number as is if it's already an integer" do
+      @provider.revision_int.should eql("12345")
+    end
+
+    it "queries the server and resolves the revision if it's not an integer (i.e. 'HEAD')" do
+      example_svn_info =  "Path: .\n" +
+                          "URL: http://svn.example.org/trunk/myapp\n" +
+                          "Repository Root: http://svn.example.org\n" +
+                          "Repository UUID: d62ff500-7bbc-012c-85f1-0026b0e37c24\n" +
+                          "Revision: 11739\nNode Kind: directory\n" +
+                          "Schedule: normal\n" +
+                          "Last Changed Author: codeninja\n" +
+                          "Last Changed Rev: 11410\n" + # Last Changed Rev is preferred to Revision
+                          "Last Changed Date: 2009-03-25 06:09:56 -0600 (Wed, 25 Mar 2009)\n\n"
+      exitstatus = mock("exitstatus")
+      exitstatus.stub!(:exitstatus).and_return(0)
+      @resource.revision "HEAD"
+      @stdout.stub!(:string).and_return(example_svn_info)
+      @stderr.stub!(:string).and_return("")
+      expected_command = ["svn info http://svn.example.org/trunk/ --no-auth-cache  -rHEAD", {:cwd=>Dir.tmpdir}]
+      @provider.should_receive(:popen4).with(*expected_command).
+                                        and_yield("no-pid","no-stdin",@stdout,@stderr).
+                                        and_return(exitstatus)
+      @provider.revision_int.should eql("11410")
+    end
+
+    it "returns a helpful message if data from `svn info` can't be parsed" do
+      example_svn_info =  "some random crap from an error message\n"
+      exitstatus = mock("exitstatus")
+      exitstatus.stub!(:exitstatus).and_return(0)
+      @resource.revision "HEAD"
+      @stdout.stub!(:string).and_return(example_svn_info)
+      @stderr.stub!(:string).and_return("")
+      @provider.should_receive(:popen4).and_yield("no-pid","no-stdin",@stdout,@stderr).
+                                        and_return(exitstatus)
+      lambda {@provider.revision_int}.should raise_error(RuntimeError, "Could not parse `svn info` data: some random crap from an error message")
+
+    end
+
+    it "responds to :revision_slug as an alias for revision_sha" do
+      @provider.should respond_to(:revision_slug)
+    end
+
+  end
+
+  it "generates a checkout command with default options" do
+    @provider.checkout_command.should eql("svn checkout -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir")
+  end
+
+  it "generates a checkout command with authentication" do
+    @resource.svn_username "deployNinja"
+    @resource.svn_password "vanish!"
+    @provider.checkout_command.should eql("svn checkout -q --username deployNinja --password vanish!  " +
+                                          "-r12345 http://svn.example.org/trunk/ /my/deploy/dir")
+  end
+
+  it "generates a checkout command with arbitrary options" do
+    @resource.svn_arguments "--no-auth-cache"
+    @provider.checkout_command.should eql("svn checkout --no-auth-cache -q  -r12345 "+
+                                          "http://svn.example.org/trunk/ /my/deploy/dir")
+  end
+
+  it "generates a sync command with default options" do
+    @provider.sync_command.should eql("svn update -q  -r12345 /my/deploy/dir")
+  end
+
+  it "generates an export command with default options" do
+    @provider.export_command.should eql("svn export --force -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir")
+  end
+
+  it "doesn't try to find the current revision when loading the resource if running an export" do
+    @provider.new_resource.instance_variable_set :@action, [:export]
+    @provider.should_not_receive(:find_current_revision)
+    @provider.load_current_resource
+  end
+
+  it "doesn't try to find the current revision when loading the resource if running a force export" do
+    @provider.new_resource.instance_variable_set :@action, [:force_export]
+    @provider.should_not_receive(:find_current_revision)
+    @provider.load_current_resource
+  end
+
+  it "runs an export with the --force option" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    expected_cmd = "svn export --force -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir"
+    @provider.should_receive(:run_command).with(:command => expected_cmd)
+    @provider.run_action(:force_export)
+    @resource.should be_updated
+  end
+
+  it "runs the checkout command for action_checkout" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    expected_cmd = "svn checkout -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir"
+    @provider.should_receive(:run_command).with(:command => expected_cmd)
+    @provider.run_action(:checkout)
+    @resource.should be_updated
+  end
+
+  it "raises an error if the svn checkout command would fail because the enclosing directory doesn't exist" do
+    lambda {@provider.run_action(:sync)}.should raise_error(Chef::Exceptions::MissingParentDirectory)
+  end
+
+  it "should not checkout if the destination exists or is a non empty directory" do
+    ::File.stub!(:exist?).with("/my/deploy/dir/.svn").and_return(false)
+    ::File.stub!(:exist?).with("/my/deploy/dir").and_return(true)
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::Dir.stub!(:entries).with("/my/deploy/dir").and_return(['.','..','foo','bar'])
+    @provider.should_not_receive(:checkout_command)
+    @provider.run_action(:checkout)
+    @resource.should_not be_updated
+  end
+
+  it "runs commands with the user and group specified in the resource" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    @resource.user "whois"
+    @resource.group "thisis"
+    expected_cmd = "svn checkout -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir"
+    @provider.should_receive(:run_command).with(:command => expected_cmd, :user => "whois", :group => "thisis")
+    @provider.run_action(:checkout)
+    @resource.should be_updated
+  end
+
+  it "does a checkout for action_sync if there's no deploy dir" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").twice.and_return(false)
+    @provider.should_receive(:action_checkout)
+    @provider.run_action(:sync)
+  end
+
+  it "does a checkout for action_sync if the deploy dir exists but is empty" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").twice.and_return(false)
+    @provider.should_receive(:action_checkout)
+    @provider.run_action(:sync) 
+  end
+
+  it "runs the sync_command on action_sync if the deploy dir exists and isn't empty" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").and_return(true)
+    @provider.stub!(:find_current_revision).and_return("11410")
+    @provider.stub!(:current_revision_matches_target_revision?).and_return(false)
+    expected_cmd = "svn update -q  -r12345 /my/deploy/dir"
+    @provider.should_receive(:run_command).with(:command => expected_cmd)
+    @provider.run_action(:sync)
+    @resource.should be_updated
+  end
+
+  it "does not fetch any updates if the remote revision matches the current revision" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    ::File.should_receive(:exist?).with("/my/deploy/dir/.svn").and_return(true)
+    @provider.stub!(:find_current_revision).and_return('12345')
+    @provider.stub!(:current_revision_matches_target_revision?).and_return(true)
+    @provider.run_action(:sync)
+    @resource.should_not be_updated
+  end
+
+  it "runs the export_command on action_export" do
+    ::File.stub!(:directory?).with("/my/deploy").and_return(true)
+    expected_cmd = "svn export --force -q  -r12345 http://svn.example.org/trunk/ /my/deploy/dir"
+    @provider.should_receive(:run_command).with(:command => expected_cmd)
+    @provider.run_action(:export)
+    @resource.should be_updated
+  end
+
+end