right_agent 0.5.1

data/lib/right_agent/subprocess.rb

@@ -0,0 +1,38 @@
+#--  -*- mode: ruby; encoding: utf-8 -*-
+# Copyright: Copyright (c) 2011 RightScale, Inc.
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# 'Software'), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+#++
+
+module RightScale
+  module SubprocessFormatting
+    module_function
+
+    SIGNAL_LOOKUP = Signal.list.invert
+
+    def reason(status)
+      if status.exitstatus.nil?
+        "terminated with SIG#{SIGNAL_LOOKUP[status.termsig]}"
+      else
+        "exited with #{status.exitstatus}"
+      end
+    end
+  end
+end

data/lib/right_agent/tracer.rb

@@ -0,0 +1,124 @@
+#
+# Copyright (c) 2009-2011 RightScale Inc
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+# This allows us to define class methods
+class Object
+  def metaclass
+    class << self
+      self
+    end
+  end
+end
+
+module RightScale
+
+  class Tracer
+
+    NON_TRACEABLE_CLASSES = [ 'Kernel', 'Module', 'Object' , 'SyslogLogger', 'RightSupport::SystemLogger' ] +
+                            [ 'RightScale::Tracer', 'RightScale::Multiplexer' ] +
+                            [ 'RightScale::Log', 'RightScale::Log::Formatter' ]
+
+    NON_TRACEABLE_METHODS = [ :metaclass, :method_missing, :method_added, :blank_slate_method_added, :[], :[]= ]
+
+    NON_TRACEABLE_CLASS_METHODS = [ :initialize, :initialize_copy, :inherited, :new, :allocate, :superclass ]
+
+    # Add logs when entering and exiting instance and class methods
+    # defined on given class
+    #
+    # === Parameters
+    # klass(Class):: Class whose methods should be traced
+    #
+    # === Return
+    # true:: Always return true
+    def self.add_tracing_to_class(klass)
+      return true if NON_TRACEABLE_CLASSES.include?(klass.to_s)
+      (klass.public_instance_methods(all=false) + klass.private_instance_methods(all=false) +
+              klass.protected_instance_methods(all=false)).each do |m|
+        if traceable(m)
+          old_m = klass.instance_method(m)
+          klass.module_eval <<-EOM
+            alias :o_l_d_#{m} :#{m}
+            def #{m}(*args, &blk)
+              Log.debug("<<< #{klass}##{m}(" + args.map(&:inspect).join(',') + ")")
+              res = o_l_d_#{m}(*args, &blk)
+              Log.debug(">>> #{klass}##{m}")
+              res
+            end
+          EOM
+        end
+      end
+      (klass.public_methods(all=false) + klass.private_methods(all=false) +
+              klass.protected_methods(all=false)).each do |m|
+        if traceable(m, static=true)
+          old_m = klass.method(m)
+          klass.module_eval <<-EOM
+            class << self
+              alias :o_l_d_#{m} :#{m}
+              def #{m}(*args, &blk)
+                Log.debug("<<< #{klass}.#{m}(" + args.map(&:inspect).join(',') + ")")
+                res = o_l_d_#{m}(*args, &blk)
+                Log.debug(">>> #{klass}.#{m}")
+                res
+              end
+            end
+          EOM
+        end
+      end
+      true
+    end
+
+    # Can method be traced?
+    #
+    # === Parameters
+    # m(String):: Method name
+    # static(Boolean):: Whether method is a class method
+    #
+    # === Return
+    # traceable(Boolean):: true if method can be traced, false otherwise
+    def self.traceable(m, static=false)
+      traceable = !NON_TRACEABLE_METHODS.include?(m.to_sym) && m =~ /[a-zA-Z0-9]$/
+      traceable &&= !NON_TRACEABLE_CLASS_METHODS.include?(m.to_sym) if static
+      traceable
+    end
+
+    # Add tracing to all classes in given namespaces
+    #
+    # === Parameters
+    # namespaces(Array|String):: Namespace(s) of classes whose methods should be traced
+    #
+    # === Return
+    # true:: Always return true
+    def self.add_tracing_to_namespaces(namespaces)
+      namespaces = [ namespaces ] unless namespaces.respond_to?(:inject)
+      regexps = namespaces.inject([]) { |reg, n| reg << "^#{n}::" }
+      unless regexps.empty?
+        ObjectSpace.each_object(Class) do |c|
+          if c.to_s =~ /#{regexps.join('|')}/
+            add_tracing_to_class(c)
+          end
+        end
+      end 
+      true
+    end
+
+  end
+end

data/right_agent.gemspec

@@ -0,0 +1,60 @@
+# -*-ruby-*-
+# Copyright: Copyright (c) 2011 RightScale, Inc.
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# 'Software'), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+# IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
+# CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
+# TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
+# SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+require 'rubygems'
+
+Gem::Specification.new do |spec|
+  spec.name      = 'right_agent'
+  spec.version   = '0.5.1'
+  spec.authors   = ['Lee Kirchhoff', 'Raphael Simon', 'Tony Spataro']
+  spec.email     = 'lee@rightscale.com'
+  spec.homepage  = 'https://github.com/rightscale/right_agent'
+  spec.platform  = Gem::Platform::RUBY
+  spec.summary   = 'Agent for interfacing server with RightScale system'
+  spec.has_rdoc  = true
+  spec.rdoc_options = ["--main", "README.rdoc", "--title", "RightAgent"]
+  spec.extra_rdoc_files = ["README.rdoc"]
+  spec.required_ruby_version = '>= 1.8.7'
+  spec.require_path = 'lib'
+
+  spec.add_dependency('right_support')
+  spec.add_dependency('amqp', "0.6.7")
+  spec.add_dependency('json', [">= 1.4.4", "<= 1.4.6"])
+  spec.add_dependency('eventmachine', "~> 0.12.10")
+  spec.add_dependency('right_popen', "~> 1.0.11")
+  spec.add_dependency('msgpack', "0.4.4")
+
+  spec.description = <<-EOF
+  RightAgent provides a foundation for running an agent on a server to interface
+  in a secure fashion with other agents in the RightScale system. A RightAgent
+  uses RabbitMQ as the message bus and the RightScale mapper as the routing node.
+  Servers running a RightAgent establish a queue on startup for receiving packets
+  routed to it via the mapper. The packets are structured to invoke services in
+  the agent represented by actors and methods. The RightAgent may respond to these
+  requests with a result packet that the mapper then routes to the originator.
+  Similarly a RightAgent can also make requests of other RightAgents in the
+EOF
+
+  candidates = Dir.glob("{lib,spec}/**/*") +
+               ["LICENSE", "README.rdoc", "Rakefile", "right_agent.gemspec"]
+  spec.files = candidates.sort
+end

data/spec/actor_registry_spec.rb

@@ -0,0 +1,81 @@
+#
+# Copyright (c) 2009-2011 RightScale Inc
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+
+describe RightScale::ActorRegistry do
+  
+  class ::WebDocumentImporter
+    include RightScale::Actor
+    expose :import, :cancel
+
+    def import
+      1
+    end
+    def cancel
+      0
+    end
+  end
+
+  module ::Actors
+    class ComedyActor
+      include RightScale::Actor
+      expose :fun_tricks
+      def fun_tricks
+        :rabbit_in_the_hat
+      end
+    end
+  end
+
+  before(:each) do
+    @registry = RightScale::ActorRegistry.new
+  end
+
+  it "should know about all services" do
+    @registry.register(WebDocumentImporter.new, nil)
+    @registry.register(Actors::ComedyActor.new, nil)
+    @registry.services.sort.should == ["/actors/comedy_actor/fun_tricks", "/web_document_importer/cancel", "/web_document_importer/import"]
+  end
+
+  it "should not register anything except RightScale::Actor" do
+    lambda { @registry.register(String.new, nil) }.should raise_error(ArgumentError)
+  end
+
+  it "should register an actor" do
+    importer = WebDocumentImporter.new
+    @registry.register(importer, nil)
+    @registry.actors['web_document_importer'].should == importer
+  end
+
+  it "should log info message that actor was registered" do
+    importer = WebDocumentImporter.new
+    flexmock(RightScale::Log).should_receive(:info).with("[actor] #{importer.class.to_s}").once
+    @registry.register(importer, nil)
+  end
+
+  it "should handle actors registered with a custom prefix" do
+    importer = WebDocumentImporter.new
+    @registry.register(importer, 'monkey')
+    @registry.actor_for('monkey').should == importer
+  end
+  
+end # RightScale::ActorRegistry

data/spec/actor_spec.rb

@@ -0,0 +1,99 @@
+#
+# Copyright (c) 2009-2011 RightScale Inc
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+
+describe RightScale::Actor do
+  class ::WebDocumentImporter
+    include RightScale::Actor
+    expose :import, :cancel
+
+    def import
+      1
+    end
+    def cancel
+      0
+    end
+    def continue
+      1
+    end
+  end
+
+  module ::Actors
+    class ComedyActor
+      include RightScale::Actor
+      expose :fun_tricks
+      def fun_tricks
+        :rabbit_in_the_hat
+      end
+    end
+  end
+
+  class ::Actors::InvalidActor
+    include RightScale::Actor
+    expose :non_existing
+  end
+  
+  describe ".expose" do
+    before :each do
+      @exposed = WebDocumentImporter.instance_variable_get(:@exposed).dup
+    end
+    
+    after :each do
+      WebDocumentImporter.instance_variable_set(:@exposed, @exposed)
+    end
+    
+    
+    it "should single expose method only once" do
+      3.times { WebDocumentImporter.expose(:continue) }
+      WebDocumentImporter.provides_for("webfiles").should == ["/webfiles/import", "/webfiles/cancel", "/webfiles/continue"]
+    end
+  end
+  
+  describe ".default_prefix" do
+    it "is calculated as default prefix as const path of class name" do
+      Actors::ComedyActor.default_prefix.should == "actors/comedy_actor"
+      WebDocumentImporter.default_prefix.should == "web_document_importer"
+    end
+  end
+
+  describe ".provides_for(prefix)" do
+    before :each do
+      @provides = Actors::ComedyActor.provides_for("money")
+    end
+    
+    it "returns an array" do
+      @provides.should be_kind_of(Array)
+    end
+
+    it "maps exposed service methods to prefix" do
+      @provides.should == ["/money/fun_tricks"]
+      wdi_provides = WebDocumentImporter.provides_for("webfiles")
+      wdi_provides.should include("/webfiles/import")
+      wdi_provides.should include("/webfiles/cancel")
+    end
+    
+    it "should not include methods not existing in the actor class" do
+      Actors::InvalidActor.provides_for("money").should_not include("/money/non_existing")
+    end
+  end
+end

data/spec/agent_config_spec.rb

@@ -0,0 +1,226 @@
+#
+# Copyright (c) 2011 RightScale Inc
+#
+# Permission is hereby granted, free of charge, to any person obtaining
+# a copy of this software and associated documentation files (the
+# "Software"), to deal in the Software without restriction, including
+# without limitation the rights to use, copy, modify, merge, publish,
+# distribute, sublicense, and/or sell copies of the Software, and to
+# permit persons to whom the Software is furnished to do so, subject to
+# the following conditions:
+#
+# The above copyright notice and this permission notice shall be
+# included in all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+# MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+# LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+# WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+
+describe RightScale::AgentConfig do
+
+  before(:all) do
+    @agent_config = RightScale::AgentConfig
+    @pwd_dir = Dir.pwd
+    @tmp_dir = RightScale::Platform.filesystem.temp_dir
+    @test_dir = File.join(@tmp_dir, 'agent_config_test')
+    @agent_id1 = "rs-agent-1-1"
+    FileUtils.mkdir_p(@root_dir1 = File.join(@test_dir, 'root1'))
+    FileUtils.mkdir_p(@root_dir2 = File.join(@test_dir, 'root2'))
+    FileUtils.mkdir_p(@root_dir3 = File.join(@test_dir, 'root3'))
+    FileUtils.mkdir_p(@init_dir1 = File.join(@root_dir1, 'init'))
+    FileUtils.mkdir_p(@init_dir2 = File.join(@root_dir2, 'init'))
+    FileUtils.mkdir_p(@init_dir3 = File.join(@root_dir3, 'init'))
+    FileUtils.touch([@config1 = File.join(@init_dir1, 'config.yml'), @config2 = File.join(@init_dir2, 'config.yml')])
+    FileUtils.touch([@init2 = File.join(@init_dir2, 'init.rb'), @init3 = File.join(@init_dir3, 'init.rb')])
+    FileUtils.mkdir_p(@actors1 = File.join(@root_dir1, 'actors'))
+    FileUtils.mkdir_p(@actors3 = File.join(@root_dir3, 'actors'))
+    @actors = File.expand_path(File.join(File.dirname(__FILE__), '..', 'lib', 'right_agent', 'actors'))
+    FileUtils.mkdir_p(@certs1 = File.join(@root_dir1, 'certs'))
+    FileUtils.mkdir_p(@certs2 = File.join(@root_dir2, 'certs'))
+    FileUtils.touch([@mapper_cert1 = File.join(@certs1, 'mapper.cert')])
+    FileUtils.touch([@mapper_cert2 = File.join(@certs2, 'mapper.cert')])
+    FileUtils.touch([@agent_cert2 = File.join(@certs2, 'agent.cert')])
+    FileUtils.touch([@mapper_key2 = File.join(@certs2, 'mapper.key')])
+    FileUtils.mkdir_p(@lib1 = File.join(@root_dir1, 'lib'))
+    FileUtils.mkdir_p(@scripts3 = File.join(@root_dir3, 'scripts'))
+    FileUtils.mkdir_p(@cfg_dir = File.join(@test_dir, 'cfg'))
+    FileUtils.mkdir_p(@cfg_agent1_dir = File.join(@cfg_dir, 'agent_1'))
+    FileUtils.mkdir_p(@pid_dir = File.join(@test_dir, 'pid'))
+    @agent_options1 = {
+      :identity => @agent_id1,
+      :root_dir => [@root_dir1, @root_dir2],
+      :log_dir  => @tmp_dir,
+      :pid_dir  => @pid_dir
+    }
+    File.open(@cfg_agent1 = File.join(@cfg_agent1_dir, 'config.yml'), "w") { |f| f.puts(YAML.dump(@agent_options1)) }
+    @agent_id2 = "rs-agent-2-2"
+    @agent_options2 = {
+      :identity => @agent_id2,
+      :root_dir => @root_dir2,
+    }
+    FileUtils.mkdir_p(@cfg_agent2_dir = File.join(@cfg_dir, 'agent_2'))
+    FileUtils.touch([@cfg_agent2 = File.join(@cfg_agent2_dir, 'config.yml')])
+    @pid = 12345
+    File.open(@cookie_file = File.join(@pid_dir, "#{@agent_id1}.pid"), "w") { |f| f.puts(@pid) }
+    @agent_cookie1 = {
+      :cookie      => "1a2b3c",
+      :listen_port => 70000
+    }
+    File.open(@cookie_file = File.join(@pid_dir, "#{@agent_id1}.cookie"), "w") { |f| f.puts(YAML.dump(@agent_cookie1)) }
+  end
+
+  after(:all) do
+    FileUtils.remove_dir(@test_dir)
+  end
+
+  it 'should return protocol version' do
+    @agent_config.protocol_version.should == RightScale::AgentConfig::PROTOCOL_VERSION
+  end
+
+  it 'should default root directory to current working directory' do
+    @agent_config.root_dir.should == @pwd_dir
+  end
+
+  it 'should set root directory to single directory' do
+    @agent_config.root_dir = @root_dir1
+    @agent_config.root_dir.should == @root_dir1
+  end
+
+  it 'should set root directory to list of directories' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2]
+    @agent_config.root_dirs.should == [@root_dir1, @root_dir2]
+  end
+
+  it 'should be able to change root directory' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2]
+    @agent_config.root_dir.should == [@root_dir1, @root_dir2]
+    @agent_config.root_dir = [@root_dir3, @root_dir2]
+    @agent_config.root_dir.should == [@root_dir3, @root_dir2]
+    @agent_config.root_dir = nil
+    @agent_config.root_dir.should == @pwd_dir
+  end
+
+  it 'should default configuration directory to platform specific directory' do
+    @agent_config.cfg_dir.should == RightScale::Platform.filesystem.cfg_dir
+  end
+
+  it 'should set configuration directory' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.cfg_dir.should == @cfg_dir
+  end
+
+  it 'should default process id directory to platform specific directory' do
+    @agent_config.pid_dir.should == RightScale::Platform.filesystem.pid_dir
+  end
+
+  it 'should set process id directory' do
+    @agent_config.pid_dir = @pid_dir
+    @agent_config.pid_dir.should == @pid_dir
+  end
+
+  it 'should return first init/init.rb file path found' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.init_file.should == @init2
+  end
+
+  it 'should return first init/config.yml file path found' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.init_cfg_file.should == @config1
+  end
+
+  it 'should return actors directory paths containing all in root directory paths with one from gem last' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.actors_dirs.should == [@actors1, @actors3, @actors]
+  end
+
+  it 'should return actors directory paths containing other actors directories' do
+    module RightScale
+      AgentConfig.module_eval { def self.other_actors_dirs; [RightScale::Platform.filesystem.temp_dir] end }
+    end
+    @agent_config.root_dir = [@root_dir1, @root_dir2]
+    @agent_config.actors_dirs.should == [@actors1, @tmp_dir, @actors]
+  end
+
+  it 'should return first certs file path found' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.certs_file('mapper.cert').should == @mapper_cert1
+    @agent_config.certs_file('mapper.key').should == @mapper_key2
+    @agent_config.certs_file('agent.key').should be_nil
+  end
+
+  it 'should return all certs file paths found without any duplicates and by root directory order' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.certs_files('*.cert').should == [@mapper_cert1, @agent_cert2]
+    @agent_config.certs_files('*.key').should == [@mapper_key2]
+    @agent_config.certs_files('*.abc').should == []
+  end
+
+  it 'should return the first lib directory path' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.lib_dir.should == @lib1
+  end
+
+  it 'should return the first scripts directory path' do
+    @agent_config.root_dir = [@root_dir1, @root_dir2, @root_dir3]
+    @agent_config.scripts_dir.should == @scripts3
+  end
+
+  it 'should return the configuration file path for an agent' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.cfg_file('agent_1').should == @cfg_agent1
+    @agent_config.cfg_file('no_agent').should == File.join(@cfg_dir, 'no_agent', 'config.yml')
+  end
+
+  it 'should check if agent configuration file exists if requested to' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.cfg_file('agent_1', exists = true).should == @cfg_agent1
+    @agent_config.cfg_file('no_agent', exists = true).should be_nil
+  end
+
+  it 'should return configuration file paths for all agents' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.cfg_files.should =~ [@cfg_agent1, @cfg_agent2]
+  end
+
+  it 'should return a list of all configured agents' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.cfg_agents.should =~ ['agent_1', 'agent_2']
+  end
+
+  it 'should return agent name for agent identity' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.agent_name(@agent_id1).should == 'agent_1'
+    @agent_config.agent_name("rs-bogus-0-0").should be_nil
+  end
+
+  it 'should load agent options from a configuration file and symbolize the keys' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.load_cfg('agent_1').should == @agent_options1
+    @agent_config.load_cfg("no_agent").should be_nil
+  end
+
+  it 'should store agent options in a configuration file in YAML format' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.store_cfg("agent_2", @agent_options2) == @agent_config.cfg_file("agent_2")
+    @agent_config.load_cfg('agent_2').should == @agent_options2
+  end
+
+  it 'should return the process id file object for an agent' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.pid_file('agent_1').identity.should == @agent_id1
+    @agent_config.pid_file('no_agent').should be_nil
+  end
+
+  it 'should return the agent options retrieved from the configuration file and associated pid file' do
+    @agent_config.cfg_dir = @cfg_dir
+    @agent_config.agent_options('agent_1').should == @agent_options1.merge(@agent_cookie1.merge(:pid => @pid, :log_path => @tmp_dir))
+    @agent_config.agent_options("no_agent").should == {}
+  end
+
+end