right_scraper 1.0.26 → 3.0.0

data/spec/git/scraper_spec.rb

@@ -0,0 +1,136 @@
+#--
+# Copyright: Copyright (c) 2010-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 'stringio'
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
+require File.expand_path(File.join(File.dirname(__FILE__), 'retriever_spec_helper'))
+require 'tmpdir'
+require 'flexmock'
+
+describe RightScraper::Scraper do
+  include RightScraper::SpecHelpers::DevelopmentModeEnvironment
+
+  include RightScraper::SharedExamples
+
+  before(:each) do
+    @stream = StringIO.new()
+    @tmpdir = Dir.mktmpdir
+    @scraper = RightScraper::Scraper.new(:basedir => @tmpdir, :kind => :cookbook)
+  end
+
+  after(:each) do
+    FileUtils.remove_entry_secure @tmpdir
+  end
+
+  it 'starts out successful' do
+    @scraper.succeeded?.should be_true
+    @scraper.errors.should == []
+  end
+
+  context 'given a Git repository' do
+    before(:each) do
+      @helper = RightScraper::GitRetrieverSpecHelper.new
+      @helper.setup_cookbooks
+      @repo = @helper.repo
+    end
+
+    after(:each) do
+      @helper.close
+    end
+
+    it_should_behave_like "a normal repository"
+
+    it 'should log correctly as it scrapes' do
+      callback = flexmock("callback")
+      callback.should_receive(:call).with(:begin, :retrieving, "from #{@repo}", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :initialize, String, nil).once.ordered
+      callback.should_receive(:call).with(:commit, :initialize, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :checkout, "", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :cloning, "to #{RightScraper::Retrievers::Base.repo_dir(@tmpdir, @repo)}", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :cloning, "to #{RightScraper::Retrievers::Base.repo_dir(@tmpdir, @repo)}", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :fetch, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :fetch, "", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :checkout_revision, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :checkout_revision, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :checkout, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :retrieving, "from #{@repo}", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :scraping, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :finding_next_cookbook, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :reading_cookbook, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :scanning_filesystem, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :metadata_parsing, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :metadata_parsing, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :scanning_filesystem, String, nil).once.ordered
+      callback.should_receive(:call).with(:commit, :reading_cookbook, String, nil).once.ordered
+      callback.should_receive(:call).with(:commit, :finding_next_cookbook, String, nil).once.ordered
+      callback.should_receive(:call).with(:begin, :next, "", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :searching, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :searching, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :next, "", nil).once.ordered
+      callback.should_receive(:call).with(:begin, :next, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :next, "", nil).once.ordered
+      callback.should_receive(:call).with(:commit, :scraping, String, nil).once.ordered
+
+      @scraper.scrape(@repo) do |phase, operation, explanation, exception|
+        callback.call(phase, operation, explanation, exception)
+      end
+      @scraper.succeeded?.should be_true
+    end
+
+    context 'with tag being an empty string' do
+      before(:each) do
+        @repo.tag = ""
+      end
+
+      it_should_behave_like "a normal repository"
+    end
+
+    context 'with some additional commits' do
+      before(:each) do
+        @helper.create_file_layout(@helper.repo_path, @helper.branch_content)
+        @helper.commit_content("change to master")
+      end
+
+      context 'after a successful scraper run' do
+        before(:each) do
+          @scraper.scrape(@repo)
+          @scraper.succeeded?.should be_true
+        end
+
+        context 'with some incompatible changes' do
+          before(:each) do
+            @helper.create_file_layout(@helper.repo_path, [{'other_branch_folder' => ['file7']}])
+            @helper.commit_content("2nd change to master")
+            @helper.force_rebase('master^', 'master^^')
+          end
+
+          it 'should still be successful' do
+            @scraper.scrape(@repo)
+            @scraper.errors.should == []
+            @scraper.succeeded?.should be_true
+          end
+        end
+      end
+    end
+  end
+end

data/spec/git/ssh_spec.rb

@@ -0,0 +1,170 @@
+#--
+# Copyright: Copyright (c) 2010-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 RightScraper::Processes::SSHAgent do
+  shared_examples_for 'a process that sets environment variables' do
+    def setvar(name, value)
+      if value.nil?
+        ENV.delete name
+      else
+        ENV[name] = value
+      end
+    end
+
+    it 'should set SSH_AUTH_SOCK' do
+      oldsock = ENV['SSH_AUTH_SOCK']
+      RightScraper::Processes::SSHAgent.with do |agent|
+        ENV.should have_key('SSH_AUTH_SOCK')
+        ENV['SSH_AUTH_SOCK'].should_not be_empty
+        File.exists?(ENV['SSH_AUTH_SOCK']).should == true
+      end
+      ENV['SSH_AUTH_SOCK'].should == oldsock
+    end
+
+    it 'should set SSH_AGENT_PID' do
+      oldpid = ENV['SSH_AUTH_PID']
+      RightScraper::Processes::SSHAgent.with do |agent|
+        ENV.should have_key('SSH_AGENT_PID')
+        ENV['SSH_AGENT_PID'].should_not be_empty
+        # This is a Unixism; sending signal 0 to a process tests whether
+        # it exists, but has no effect on the process.  I have no idea
+        # how to express this on Windows.
+        Process.kill(0, ENV['SSH_AGENT_PID'].to_i).should be_true
+      end
+      ENV['SSH_AUTH_PID'].should == oldpid
+    end
+
+    it 'should set SSH_ASKPASS' do
+      oldpass = ENV['SSH_ASKPASS']
+      RightScraper::Processes::SSHAgent.with do |agent|
+        ENV.should have_key('SSH_ASKPASS')
+        ENV['SSH_ASKPASS'].should_not be_empty
+
+        script = File.expand_path(File.join(File.dirname(__FILE__), '..', '..',
+                                            'scripts', 'stub_ssh_askpass'))
+        ENV['SSH_ASKPASS'].should == script
+      end
+      ENV['SSH_ASKPASS'].should == oldpass
+    end
+
+    it 'should set HOME' do
+      oldhome = ENV['HOME']
+      RightScraper::Processes::SSHAgent.with do |agent|
+        ENV.should have_key('HOME')
+        ENV['HOME'].should_not be_empty
+        ENV['HOME'].should == "/dev/null"
+      end
+      ENV['HOME'].should == oldhome
+    end
+  end
+
+  context 'with no relevant environment variables' do
+    before(:each) do
+      @display = ENV['DISPLAY']
+      @askpass = ENV['SSH_ASKPASS']
+      @sshauth = ENV['SSH_AUTH_SOCK']
+      @agentpid = ENV['SSH_AGENT_PID']
+      @home = ENV['HOME']
+      ENV.delete 'DISPLAY'
+      ENV.delete 'SSH_ASKPASS'
+      ENV.delete 'SSH_AUTH_SOCK'
+      ENV.delete 'SSH_AGENT_PID'
+      ENV.delete 'HOME'
+    end
+
+    after(:each) do
+      setvar 'DISPLAY', @display
+      setvar 'SSH_ASKPASS', @askpass
+      setvar 'SSH_AUTH_SOCK', @sshauth
+      setvar 'SSH_AGENT_PID', @agentpid
+      setvar 'HOME', @home
+    end
+
+    it_should_behave_like 'a process that sets environment variables'
+  end
+
+  context 'with relevant environment variables set' do
+    before(:each) do
+      @display = ENV['DISPLAY']
+      @askpass = ENV['SSH_ASKPASS']
+      @sshauth = ENV['SSH_AUTH_SOCK']
+      @agentpid = ENV['SSH_AGENT_PID']
+      @home = ENV['HOME']
+      ENV['DISPLAY'] = "foo"
+      ENV['SSH_ASKPASS'] = "bar"
+      ENV['SSH_AUTH_SOCK'] = "baz"
+      ENV['SSH_AGENT_PID'] = "quux"
+      ENV['HOME'] = "fred"
+    end
+
+    after(:each) do
+      setvar 'DISPLAY', @display
+      setvar 'SSH_ASKPASS', @askpass
+      setvar 'SSH_AUTH_SOCK', @sshauth
+      setvar 'SSH_AGENT_PID', @agentpid
+      setvar 'HOME', @home
+    end
+
+    it_should_behave_like 'a process that sets environment variables'
+  end
+
+  it 'should be able to load the demo key' do
+    RightScraper::Processes::SSHAgent.with do |agent|
+      demofile = File.expand_path(File.join(File.dirname(__FILE__), 'demokey'))
+      File.chmod(0600, demofile)
+      demofile = File.join(File.dirname(__FILE__), 'demokey')
+      agent.add_keyfile(demofile)
+      `ssh-add -l`.should == "2048 3d:6a:4f:8b:ec:35:da:e9:7e:cc:e8:2d:03:2f:6f:23 #{demofile} (RSA)\n"
+      `ssh-add -L`.should == <<FULLOUTPUT
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAxJsM8sZ6++Nky/ogLEYhKtKivB37sPB9M6Un0z6PkIRgUsGdntMJqP1U6820jH+n1/lOH/MnlUsvzoo8DnOdbe9kGOHBmtWmNcjqacZUn9DbpbjvlI7RUUmZ5OBKn8Pjt2qbSXnnci9Q5j5Rgh6DR8A70S04FIUP8AGpCIO23BhA928CiM18zN5mBvzET7L2DYiNhJJFsFWMbN13CdukTjNVNLETEusNVUU09G1NxX4esKky7tHh1c9APFvu98KjYOHkv1o7dB7T4dO3KaKCNWINCHeeoE+QmAkhAZwI72ijRkPxH+QMisMsHucPFvgOVVObxHWu9hRlNWIOodANHQ== #{demofile}
+FULLOUTPUT
+    end
+  end
+
+  it 'should fail on the passworded key' do
+    pid = nil
+    lambda {
+      RightScraper::Processes::SSHAgent.with do |agent|
+        pid = ENV['SSH_AGENT_PID'].to_i
+        demofile = File.expand_path(File.join(File.dirname(__FILE__), 'password_key'))
+        File.chmod(0600, demofile)
+        agent.add_keyfile(demofile)
+      end
+    }.should raise_exception(ProcessWatcher::NonzeroExitCode, /Attempted to use credentials that require passwords; bailing/)
+    lambda {
+      Process.kill(0, pid)
+    }.should raise_exception(Errno::ESRCH)
+  end
+
+  it 'should be able to load the demo key from memory' do
+    RightScraper::Processes::SSHAgent.with do |agent|
+      demofile = File.join(File.dirname(__FILE__), 'demokey')
+      demodata = File.open(demofile).read
+      agent.add_key(demodata)
+      `ssh-add -l`.should =~ /^2048 3d:6a:4f:8b:ec:35:da:e9:7e:cc:e8:2d:03:2f:6f:23 .*? \(RSA\)\n$/
+      `ssh-add -L`.should =~ %r{^ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAxJsM8sZ6\+\+Nky/ogLEYhKtKivB37sPB9M6Un0z6PkIRgUsGdntMJqP1U6820jH\+n1/lOH/MnlUsvzoo8DnOdbe9kGOHBmtWmNcjqacZUn9DbpbjvlI7RUUmZ5OBKn8Pjt2qbSXnnci9Q5j5Rgh6DR8A70S04FIUP8AGpCIO23BhA928CiM18zN5mBvzET7L2DYiNhJJFsFWMbN13CdukTjNVNLETEusNVUU09G1NxX4esKky7tHh1c9APFvu98KjYOHkv1o7dB7T4dO3KaKCNWINCHeeoE\+QmAkhAZwI72ijRkPxH\+QMisMsHucPFvgOVVObxHWu9hRlNWIOodANHQ== .*\n$}
+    end
+  end
+end

data/spec/git/url_spec.rb

@@ -0,0 +1,103 @@
+#--
+# Copyright: Copyright (c) 2010-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 RightScraper::Repositories::Base do
+  def make_repo(url)
+    @repo = RightScraper::Repositories::Base.from_hash(:display_name => 'test repo',
+                                             :repo_type => :git,
+                                             :url => url,
+                                             :first_credential => "foo")
+  end
+  include RightScraper::SpecHelpers::ProductionModeEnvironment
+
+  context 'with Git URIs' do
+    it 'should not throw an error for http URIs' do
+      lambda do
+        make_repo "http://rightscale.com/%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for git URIs' do
+      lambda do
+        make_repo "git://rightscale.com/%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for absolute SCP URIs' do
+      lambda do
+        make_repo "git@rightscale.com:/%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for SCP URIs with dashes in the username' do
+      lambda do
+        make_repo "git-foo@rightscale.com:/%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for relative SCP URIs' do
+      lambda do
+        make_repo "git-foo@rightscale.com:%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for git+ssh URIs' do
+      lambda do
+        make_repo "git+ssh://rightscale.com/%20CBLAH"
+      end.should_not raise_exception
+    end
+    it 'should not throw an error for ssh URIs' do
+      lambda do
+        make_repo "ssh://rightscale.com/%20CBLAH"
+      end.should_not raise_exception
+    end
+  end
+
+  context '#to_url' do
+    it 'should correctly convert http URIs' do
+      make_repo("http://rightscale.com/%20CBLAH").to_url.to_s.should ==
+        "http://foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert git URIs' do
+      make_repo("git://rightscale.com/%20CBLAH").to_url.to_s.should ==
+        "git://foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert SCP URIs' do
+      make_repo("git@rightscale.com:/%20CBLAH").to_url.to_s.should ==
+        "ssh://git:foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert SCP URIs with dashes in the username' do
+      make_repo("git-foo@rightscale.com:/%20CBLAH").to_url.to_s.should ==
+        "ssh://git-foo:foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert relative SCP URIs' do
+      make_repo("git-foo@rightscale.com:%20CBLAH").to_url.to_s.should ==
+        "ssh://git-foo:foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert git+ssh URIs' do
+      make_repo("git+ssh://rightscale.com/%20CBLAH").to_url.to_s.should ==
+        "git+ssh://foo@rightscale.com/%20CBLAH"
+    end
+    it 'should correctly convert ssh URIs' do
+      make_repo("ssh://rightscale.com/%20CBLAH").to_url.to_s.should ==
+        "ssh://foo@rightscale.com/%20CBLAH"
+    end
+  end
+end

data/spec/logger_spec.rb

@@ -0,0 +1,185 @@
+#--
+# Copyright: Copyright (c) 2010-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 'stringio'
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'lib', 'right_scraper', 'loggers', 'noisy'))
+
+describe RightScraper::Logger do
+  before(:each) do
+    @stream = StringIO.new()
+    @logger = RightScraper::Logger.new(@stream)
+    @logger.level = Logger::WARN
+  end
+
+  after(:each) do
+    @logger.close
+  end
+
+  def match_log_entry(entry, type, message)
+    shortform, longform = case type
+                          when Logger::DEBUG then ["D", "DEBUG"]
+                          when Logger::INFO then ["I", "INFO"]
+                          when Logger::WARN then ["W", "WARN"]
+                          when Logger::ERROR then ["E", "ERROR"]
+                          when Logger::FATAL then ["F", "FATAL"]
+                          when Logger::FATAL then ["U", "UNKNOWN"]
+                          end
+    datestamp = /[[:digit:]]{4}-[[:digit:]]{2}-[[:digit:]]{2}/
+    timestamp = /[[:digit:]]{2}:[[:digit:]]{2}:[[:digit:]]{2}.[[:digit:]]+/
+    datetimestamp = /#{datestamp}T#{timestamp}/
+    entry.should =~ %r{^#{shortform}, \[#{datetimestamp} \#[[:digit:]]+\] #{longform} -- : #{message}\n$}
+  end
+
+  it 'should log to the stream' do
+    @logger.debug("foo")
+    @logger.info("first")
+    @logger.error("baz")
+    @logger.fatal("quux")
+    @stream.rewind
+    match_log_entry(@stream.readline, Logger::ERROR, "baz")
+    match_log_entry(@stream.readline, Logger::FATAL, "quux")
+    @stream.eof?.should be_true
+  end
+
+  it 'should note errors properly' do
+    @logger.note_error(Exception.new("foo!"), :baz)
+    match_log_entry(@stream.string, Logger::ERROR, "Saw foo! during baz")
+  end
+
+  it 'should note errors with an explanation properly' do
+    @logger.note_error(Exception.new("foo!"), :baz, "an explanation")
+    match_log_entry(@stream.string, Logger::ERROR, "Saw foo! during baz: an explanation")
+  end
+
+  it 'should pass values through the block' do
+    result = @logger.operation(:passthrough) { 4 }
+    result.should == 4
+  end
+end
+
+describe RightScraper::Loggers::NoisyLogger do
+  before(:each) do
+    @stream = StringIO.new()
+    @logger = RightScraper::Loggers::NoisyLogger.new(@stream)
+  end
+
+  after(:each) do
+    @logger.close
+  end
+
+  def match_log_entry(entry, type, message)
+    shortform, longform = case type
+                          when Logger::DEBUG then ["D", "DEBUG"]
+                          when Logger::INFO then ["I", "INFO"]
+                          when Logger::WARN then ["W", "WARN"]
+                          when Logger::ERROR then ["E", "ERROR"]
+                          when Logger::FATAL then ["F", "FATAL"]
+                          when Logger::FATAL then ["U", "UNKNOWN"]
+                          end
+    datestamp = /[[:digit:]]{4}-[[:digit:]]{2}-[[:digit:]]{2}/
+    timestamp = /[[:digit:]]{2}:[[:digit:]]{2}:[[:digit:]]{2}.[[:digit:]]+/
+    datetimestamp = /#{datestamp}T#{timestamp}/
+    entry.should =~ %r{^#{shortform}, \[#{datetimestamp} \#[[:digit:]]+\] #{longform} -- : #{message}\n$}
+  end
+
+  context 'with the level set to WARN' do
+    before(:each) do
+      @logger.level = Logger::WARN
+    end
+
+    it 'should not log operations' do
+      @logger.operation(:foo) do
+        @logger.operation(:bar) do
+        end
+      end
+      @stream.string.should == ""
+    end
+
+    it 'should note errors with context' do
+      begin
+        @logger.operation(:foo) do
+          @logger.operation(:bar, "blah") do
+            @logger.note_error("an exception", :note, "bar")
+          end
+        end
+      rescue
+      end
+      @stream.rewind
+      match_log_entry(@stream.readline, Logger::ERROR, "Saw an exception during note: bar in bar: blah in foo")
+      @stream.eof?.should be_true
+    end
+
+    it 'should note errors when the level is set to WARN and an exception occurs' do
+      begin
+        @logger.operation(:foo) do
+          @logger.operation(:bar, "blah") do
+            raise "foo"
+          end
+        end
+      rescue
+      end
+      @stream.rewind
+      match_log_entry(@stream.readline, Logger::ERROR, "Saw foo during bar: blah in foo")
+      @stream.eof?.should be_true
+    end
+  end
+
+  context 'with the level set to DEBUG' do
+    before(:each) do
+      @logger.level = Logger::DEBUG
+    end
+
+    it 'should log begin/commit' do
+      @logger.operation(:foo) do
+        @logger.operation(:bar, "blah") do
+        end
+      end
+      @stream.rewind
+      match_log_entry(@stream.readline, Logger::DEBUG, "> begin foo")
+      match_log_entry(@stream.readline, Logger::DEBUG, ">> begin bar: blah")
+      match_log_entry(@stream.readline, Logger::DEBUG, ">> commit bar: blah")
+      match_log_entry(@stream.readline, Logger::DEBUG, "> commit foo")
+      @stream.eof?.should be_true
+    end
+
+    it 'should log begin/abort when an exception occurs' do
+      @logger.level = Logger::DEBUG
+      begin
+        @logger.operation(:foo) do
+          @logger.operation(:bar, "blah") do
+            raise "foo"
+          end
+        end
+      rescue
+      end
+      @stream.rewind
+      match_log_entry(@stream.readline, Logger::DEBUG, "> begin foo")
+      match_log_entry(@stream.readline, Logger::DEBUG, ">> begin bar: blah")
+      match_log_entry(@stream.readline, Logger::ERROR, "Saw foo during bar: blah in foo")
+      match_log_entry(@stream.readline, Logger::DEBUG, ">> abort bar: blah")
+      match_log_entry(@stream.readline, Logger::DEBUG, "> abort foo")
+      @stream.eof?.should be_true
+    end
+  end
+end