right_scraper 3.2.6 → 5.0.1

data/spec/git/retriever_spec_helper.rb

@@ -1,112 +0,0 @@
-#--
-# 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'))
-require File.expand_path(File.join(File.dirname(__FILE__), '..', 'retriever_spec_helper'))
-require File.expand_path(File.join(File.dirname(__FILE__), '..', 'scraper_helper'))
-require 'git'
-
-module Git
-  class Lib
-    def public_command(*args, &block)
-      command(*args, &block)
-    end
-  end
-end
-module RightScraper
-
-  # Git implementation of scraper spec helper
-  # See parent class for methods headers comments
-  class GitRetrieverSpecHelper < RetrieverSpecHelper
-    def initialize
-      super()
-      FileUtils.mkdir(scraper_path)
-      @git = Git.init(repo_path)
-    end
-
-    def repo
-      RightScraper::Repositories::Base.from_hash(:display_name => "test git repo",
-                                                 :repo_type => :git,
-                                                 :url => repo_path)
-    end
-
-    def setup_cookbooks
-      create_cookbook(repo_path, repo_content)
-      commit_content(repo_path)
-    end
-
-    def setup_workflows
-      create_workflow(repo_path, 'workflow', "sequence\na\nb", {})
-      commit_content(repo_path)
-    end
-
-    def scraper_path
-      File.join(@tmpdir, "scraper")
-    end
-
-    def commit_content(commit_message='commit')
-      @git.add('.')
-      @git.commit_all(commit_message)
-    end
-
-    def setup_branch(branch, new_content=nil)
-      @git.branch(branch).checkout
-      unless new_content.nil?
-        create_file_layout(repo_path, new_content)
-        @repo_content += new_content
-        File.open(File.join(repo_path, 'metadata.json'), 'w') { |f|
-          f.puts @repo_content.to_json
-        }
-        commit_content("Branch #{branch}")
-      end
-    end
-
-    def setup_tag(tag)
-      @git.add_tag(tag)
-    end
-
-    def delete_tag(tag)
-      @git.lib.tag(['-d', tag])
-    end
-
-    def branch?(branch)
-      Git.open(RightScraper::Retrievers::Base.repo_dir(scraper_path, repo)).branches.find {|b| b.name == branch}
-    end
-
-    def tag?(tag)
-      Git.open(RightScraper::Retrievers::Base.repo_dir(scraper_path, repo)).tags.find {|t| t.name == tag}
-    end
-
-    def delete_branch(branch)
-      @git.branch(branch).delete
-    end
-
-    def force_rebase(upstream, newbase)
-      @git.lib.public_command("rebase", ["--onto", newbase, upstream])
-    end
-
-    def commit_id(index_from_last=0)
-      @git.log.skip(1).first.sha
-    end
-  end
-end

data/spec/git/scraper_spec.rb

@@ -1,151 +0,0 @@
-#--
-# 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
-    # ulimit is a /usr/bin file on the Mac but only a shell command in Linux.
-    # invoking `sh -c ulimit -a` only returns the last token of output, in fact
-    # invoking 'ulimit -a' programmatically seems to omit the 'open files' field
-    # in which we are actually interested (whereas it is visible when run
-    # manually in bash). the more specific 'ulimit -n' appears to work as
-    # expected on mac and linux platforms *but* calling `sh -c ulimit -n`
-    # returns a nonsensical 'unlimited' token in ruby so we have to jump the
-    # additional hurdle of outputing to a temp file to get the right answer :@
-    @tmpdir = Dir.mktmpdir
-    output_file_path = File.join(@tmpdir, "ulimit_n.txt")
-    system('sh', '-c', "ulimit -n>#{output_file_path}")
-    ulimit = File.read(output_file_path).to_i
-    File.delete(output_file_path)
-    if ulimit < 512
-      raise "Cannot run this spec because ulimit -n is only #{ulimit}; need 512 minimum!"
-    end
-    @stream = StringIO.new()
-    @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

@@ -1,174 +0,0 @@
-#--
-# 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 c7:66:87:fc:17:b5:2f:32:f2:c1:ed:40:a6:8d:17:44 #{demofile} (RSA)\n"
-      `ssh-add -L`.should == <<FULLOUTPUT
-ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQC18gNzaOwgJkHhUgPPNtCNp0H08ywzH1AquwSca19ZOedGnRWa673t4sXW0BeBn1wd8v0EulHRNwIyn0xJLsEStMOpo4A0qN+B2sM9gjBcY8nMOyUqy5s32pXncGfEwiRuiAxqz45VJqvL3CD8X5WxG300u/DNUbLZN0IT1aPn52Bo6gcGleZklxF1cccrbMelWfXE7KYKjD3/TfLdJeOlP9PJM8ijFHCsyWcKt5AH8bFkQ/xETPfqPzUIwvLDU7HpVDLZZ6JBi2rxhAAG+NhE3GMmY5i+vMB+g7CCfY200SFxzyjIcag05MGGko8Rv9bHnE3AYj9cxCULyDJyZm/T #{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
-        pid.should_not == 0
-        demofile = File.expand_path(File.join(File.dirname(__FILE__), 'password_key'))
-        File.chmod(0600, demofile)
-        agent.add_keyfile(demofile)  # will fail due to missing password
-      end
-    }.should raise_exception(::RightScraper::Processes::SSHAgent::SSHAgentError, /Attempted to use credentials that require passwords; bailing/)
-
-    # the .with statement must ensure that the ssh-agent process terminates.
-    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 c7:66:87:fc:17:b5:2f:32:f2:c1:ed:40:a6:8d:17:44 .*? \(RSA\)\n$/
-      expected_pubkey = 'ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQC18gNzaOwgJkHhUgPPNtCNp0H08ywzH1AquwSca19ZOedGnRWa673t4sXW0BeBn1wd8v0EulHRNwIyn0xJLsEStMOpo4A0qN+B2sM9gjBcY8nMOyUqy5s32pXncGfEwiRuiAxqz45VJqvL3CD8X5WxG300u/DNUbLZN0IT1aPn52Bo6gcGleZklxF1cccrbMelWfXE7KYKjD3/TfLdJeOlP9PJM8ijFHCsyWcKt5AH8bFkQ/xETPfqPzUIwvLDU7HpVDLZZ6JBi2rxhAAG+NhE3GMmY5i+vMB+g7CCfY200SFxzyjIcag05MGGko8Rv9bHnE3AYj9cxCULyDJyZm/T'
-      `ssh-add -L`.should =~ %r{^#{Regexp.escape(expected_pubkey)} .*\n$}
-    end
-  end
-end

data/spec/git/url_spec.rb

@@ -1,103 +0,0 @@
-#--
-# 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