ohloh_scm 2.5.1 → 3.0.1

data/spec/ohloh_scm/git/scm_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe 'Git::Scm' do
+  it 'must pull git repository' do
+    with_git_repository('git') do |src_core|
+      tmpdir do |dest_dir|
+        core = OhlohScm::Factory.get_core(scm_type: :git, url: dest_dir)
+        refute core.status.exist?
+
+        core.scm.pull(src_core.scm, TestCallback.new)
+        assert core.status.exist?
+      end
+    end
+  end
+
+  it 'must pull git repository with multiple branches' do
+    # This should not change current/default branch(e.g. master) to point to the branch commit being pulled
+    # In this case master should not point to test branch commit
+    with_git_repository('git_with_multiple_branch', 'test') do |src_core|
+      tmpdir do |dest_dir|
+        core = OhlohScm::Factory.get_core(scm_type: :git, url: dest_dir, branch_name: 'test')
+        refute core.status.exist?
+        core.scm.pull(src_core.scm, TestCallback.new)
+
+        remote_master_branch_sha = `cd #{dest_dir} && git rev-parse origin/master`
+        master_branch_sha = `cd #{dest_dir} && git rev-parse master`
+        test_branch_sha = `cd #{dest_dir} && git rev-parse test`
+
+        master_branch_sha.wont_equal test_branch_sha
+        master_branch_sha.must_equal remote_master_branch_sha
+      end
+    end
+  end
+
+  it 'must test the basic conversion to git' do
+    with_cvs_repository('cvs', 'simple') do |src_core|
+      tmpdir do |dest_dir|
+        core = OhlohScm::Factory.get_core(scm_type: :git, url: dest_dir)
+        refute core.status.exist?
+        core.scm.pull(src_core.scm, TestCallback.new)
+        assert core.status.exist?
+
+        dest_commits = core.activity.commits
+        src_core.activity.commits.each_with_index do |c, i|
+          # Because CVS does not track authors (only committers),
+          # the CVS committer becomes the Git author.
+          c.committer_date.must_equal dest_commits[i].author_date
+          c.committer_name.must_equal dest_commits[i].author_name
+
+          # Depending upon version of Git used, we may or may not have a trailing \n.
+          # We don't really care, so just compare the stripped versions.
+          c.message.strip.must_equal dest_commits[i].message.strip
+        end
+      end
+    end
+  end
+end

data/spec/ohloh_scm/git/status_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe 'Git::Status' do
+  it 'branch?' do
+    with_git_repository('git') do |git|
+      git.activity.send(:branches).must_equal %w[develop master]
+      assert git.status.branch? # checks master.
+      assert git.status.branch?('develop')
+    end
+  end
+end

data/spec/ohloh_scm/git/validation_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe 'Git::Validation' do
+  it 'must handle non existent remote source' do
+    core = OhlohScm::Factory.get_core(scm_type: :git, url: 'https://github.com/Person/foobar')
+    core.validate
+    core.errors.wont_be :empty?
+  end
+
+  it 'wont have errors for valid url' do
+    core = OhlohScm::Factory.get_core(scm_type: :git, url: 'https://github.com/ruby/ruby')
+    core.validation.send(:validate_attributes)
+    core.errors.must_be :empty?
+  end
+
+  it 'must have errors for invalid branch_name' do
+    get_core(:git, branch_name: 'x' * 81).validation.send(:branch_name_errors).wont_be :empty?
+    get_core(:git, branch_name: 'foo@bar').validation.send(:branch_name_errors).wont_be :empty?
+  end
+
+  it 'must have errors for invalid username' do
+    get_core(:git, username: 'x' * 33).validation.send(:username_errors).wont_be :empty?
+    get_core(:git, username: 'foo@bar').validation.send(:username_errors).wont_be :empty?
+  end
+
+  it 'must have errors for invalid password' do
+    get_core(:git, password: 'x' * 33).validation.send(:password_errors).wont_be :empty?
+    get_core(:git, password: 'escape').validation.send(:password_errors).wont_be :empty?
+  end
+
+  describe 'validate url' do
+    it 'must have errors for invalid urls' do
+      assert_url_error(:git, nil, '', 'foo', 'http:/', 'http:://', 'http://', 'http://a')
+      assert_url_error(:git, 'kernel.org/linux/linux.git') # missing a protocol prefix
+      assert_url_error(:git, 'http://kernel.org/linux/lin%32ux.git') # no encoded strings allowed
+      assert_url_error(:git, 'http://kernel.org/linux/linux.git malicious code') # no spaces allowed
+      assert_url_error(:git, 'svn://svn.mythtv.org/svn/trunk') # svn protocol is not allowed
+      assert_url_error(:git, '/home/robin/cvs') # local file paths not allowed
+      assert_url_error(:git, 'file:///home/robin/cvs') # file protocol is not allowed
+      # pserver is just wrong
+      assert_url_error(:git, ':pserver:anonymous:@juicereceiver.cvs.sourceforge.net:/cvsroot/juicereceiver')
+    end
+
+    it 'wont have errors for valid urls' do
+      assert_url_valid(:git, 'http://kernel.org/pub/scm/git/git.git')
+      assert_url_valid(:git, 'git://kernel.org/pub/scm/git/git.git')
+      assert_url_valid(:git, 'https://kernel.org/pub/scm/git/git.git')
+      assert_url_valid(:git, 'https://kernel.org:8080/pub/scm/git/git.git')
+      assert_url_valid(:git, 'git://kernel.org/~foo/git.git')
+      assert_url_valid(:git, 'http://git.onerussian.com/pub/deb/impose+.git')
+      assert_url_valid(:git, 'https://Person@github.com/Person/some_repo.git')
+      assert_url_valid(:git, 'http://Person@github.com/Person/some_repo.git')
+      assert_url_valid(:git, 'https://github.com/Person/some_repo')
+      assert_url_valid(:git, 'http://github.com/Person/some_repo')
+    end
+  end
+end

data/spec/ohloh_scm/git_svn/activity_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+require 'mocha'
+
+describe 'GitSvn::Activity' do
+  it 'must return all commit tokens' do
+    with_git_svn_repository('git_svn') do |git_svn|
+      git_svn.activity.commit_tokens.must_equal [1, 2, 3, 5]
+      git_svn.activity.commit_tokens(after: 2).must_equal [3, 5]
+    end
+  end
+
+  it 'must return commits' do
+    with_git_svn_repository('git_svn') do |git_svn|
+      git_svn.activity.commits.map(&:token).must_equal [1, 2, 3, 5]
+      git_svn.activity.commits(after: 2).map(&:token).must_equal [3, 5]
+      git_svn.activity.commits(after: 7).map(&:token).must_equal []
+    end
+  end
+
+  it 'must iterate each commit' do
+    with_git_svn_repository('git_svn') do |git_svn|
+      commits = []
+      git_svn.activity.each_commit { |c| commits << c }
+      git_svn.activity.commits.map(&:token).must_equal [1, 2, 3, 5]
+    end
+  end
+
+  it 'must return total commit count' do
+    with_git_svn_repository('git_svn') do |git_svn|
+      git_svn.activity.commit_count.must_equal 4
+      git_svn.activity.commit_count(after: 2).must_equal 2
+    end
+  end
+
+  describe 'cat' do
+    let(:commit_1) { OhlohScm::Commit.new(token: 1) }
+    let(:hello_diff) { OhlohScm::Diff.new(path: 'helloworld.c') }
+
+    it 'cat_file' do
+      with_git_svn_repository('git_svn') do |git_svn|
+        expected = <<-EXPECTED.gsub(/^\s+/, '')
+          /* Hello, World! */
+          #include <stdio.h>
+          main()
+          {
+            printf("Hello, World!\\n");
+          }
+        EXPECTED
+
+        git_svn.activity.cat_file(commit_1, hello_diff)
+               .delete("\t").strip.must_equal expected.strip
+      end
+    end
+
+    it 'cat_file_with_non_existent_token' do
+      with_git_svn_repository('git_svn') do |git_svn|
+        assert git_svn.activity.cat_file(OhlohScm::Commit.new(token: 999), hello_diff)
+      end
+    end
+
+    it 'cat_file_with_invalid_filename' do
+      with_git_svn_repository('git_svn') do |git_svn|
+        -> { git_svn.activity.cat_file(commit_1, OhlohScm::Diff.new(path: 'invalid')) }.must_raise(RuntimeError)
+      end
+    end
+
+    it 'cat_file_parent' do
+      with_git_svn_repository('git_svn') do |git_svn|
+        expected = <<-EXPECTED.gsub(/^\s+/, '')
+          /* Hello, World! */
+          #include <stdio.h>
+          main()
+          {
+            printf("Hello, World!\\n");
+          }
+        EXPECTED
+
+        commit = OhlohScm::Commit.new(token: 2)
+        git_svn.activity.cat_file_parent(commit, hello_diff).delete("\t").must_equal expected.strip
+      end
+    end
+
+    it 'cat_file_parent_with_first_token' do
+      with_git_svn_repository('git_svn') do |git_svn|
+        assert git_svn.activity.cat_file(commit_1, hello_diff)
+      end
+    end
+  end
+end

data/spec/ohloh_scm/git_svn/scm_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe 'GitSvn::Scm' do
+  it 'must convert to git Repository' do
+    with_svn_repository('svn') do |src|
+      tmpdir do |local_dir|
+        git_svn = get_core(:git_svn, url: local_dir)
+        git_svn.scm.pull(src.scm, TestCallback.new)
+        assert git_svn.status.exist?
+
+        git_svn.activity.commit_count.must_equal 5
+      end
+    end
+  end
+
+  it 'must fetch the repo' do
+    OhlohScm::GitSvn::Scm.any_instance.expects(:run).times(3)
+    with_git_svn_repository('git_svn') do |git_svn|
+      git_svn.scm.pull(git_svn.scm, TestCallback.new)
+    end
+  end
+end

data/spec/ohloh_scm/hg/activity_spec.rb

@@ -0,0 +1,323 @@
+require 'spec_helper'
+
+describe 'Hg::Activity' do
+  it 'must fetch tags' do
+    with_hg_repository('hg') do |hg|
+      time = Time.parse('Fri Jul 22 18:00:18 2016 +0530')
+      hg.activity.tags.first.must_equal ['tip', '5', time]
+      hg.activity.tags.last.first(2).must_equal ['tagname with space', '2']
+    end
+  end
+
+  it 'must export repo data' do
+    with_hg_repository('hg') do |hg|
+      Dir.mktmpdir do |dir|
+        hg.activity.export(dir)
+        Dir.entries(dir).sort.must_equal ['.', '..', 'README', 'makefile', 'two']
+      end
+    end
+  end
+
+  describe 'commits' do
+    it 'commit_count' do
+      with_hg_repository('hg') do |hg|
+        hg.activity.commit_count.must_equal 5
+        hg.activity.commit_count(after: 'b14fa4692f949940bd1e28da6fb4617de2615484').must_equal 3
+        hg.activity.commit_count(after: '655f04cf6ad708ab58c7b941672dce09dd369a18').must_equal 0
+      end
+    end
+
+    it 'commit_count_with_empty_branch' do
+      with_hg_repository('hg', '') do |hg|
+        hg.scm.branch_name.must_be_nil
+        hg.activity.commit_count.must_equal 5
+        hg.activity.commit_count(after: 'b14fa4692f949940bd1e28da6fb4617de2615484').must_equal 3
+        hg.activity.commit_count(after: '655f04cf6ad708ab58c7b941672dce09dd369a18').must_equal 0
+      end
+    end
+
+    it 'commits' do
+      with_hg_repository('hg') do |hg|
+        hg.activity.commits.map(&:token).must_equal(%w[01101d8ef3cea7da9ac6e9a226d645f4418f05c9
+                                                       b14fa4692f949940bd1e28da6fb4617de2615484
+                                                       468336c6671cbc58237a259d1b7326866afc2817
+                                                       75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                                       655f04cf6ad708ab58c7b941672dce09dd369a18])
+
+        after = '75532c1e1f1de55c2271f6fd29d98efbe35397c4'
+        hg.activity.commits(after: after).map(&:token).must_equal ['655f04cf6ad708ab58c7b941672dce09dd369a18']
+
+        # Check that the diffs are not populated
+        hg.activity.commits(after: '75532c1e1f1de55c2271f6fd29d98efbe35397c4').first.diffs.must_be :empty?
+
+        hg.activity.commits(after: '655f04cf6ad708ab58c7b941672dce09dd369a18').must_be :empty?
+      end
+    end
+
+    it 'commits_with_branch' do
+      with_hg_repository('hg', 'develop') do |hg|
+        hg.activity.commits.map(&:token).must_equal(%w[01101d8ef3cea7da9ac6e9a226d645f4418f05c9
+                                                       b14fa4692f949940bd1e28da6fb4617de2615484
+                                                       468336c6671cbc58237a259d1b7326866afc2817
+                                                       75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                                       4d54c3f0526a1ec89214a70615a6b1c6129c665c])
+
+        after = '75532c1e1f1de55c2271f6fd29d98efbe35397c4'
+        hg.activity.commits(after: after).map(&:token).must_equal(['4d54c3f0526a1ec89214a70615a6b1c6129c665c'])
+
+        # Check that the diffs are not populated
+        hg.activity.commits(after: '75532c1e1f1de55c2271f6fd29d98efbe35397c4').first.diffs.must_be :empty?
+
+        hg.activity.commits(after: '4d54c3f0526a1ec89214a70615a6b1c6129c665c').must_be :empty?
+      end
+    end
+
+    it 'trunk_only_commit_count' do
+      with_hg_repository('hg_dupe_delete') do |hg|
+        hg.activity.commit_count(trunk_only: false).must_equal 4
+        hg.activity.commit_count(trunk_only: true).must_equal 3
+      end
+    end
+
+    it 'trunk_only_commits' do
+      with_hg_repository('hg_dupe_delete') do |hg|
+        hg.activity.commits(trunk_only: true)
+          .map(&:token).must_equal(['73e93f57224e3fd828cf014644db8eec5013cd6b',
+                                    '732345b1d5f4076498132fd4b965b1fec0108a50',
+                                    # '525de321d8085bc1d4a3c7608fda6b4020027985', # branch
+                                    '72fe74d643bdcb30b00da3b58796c50f221017d0'])
+      end
+    end
+
+    it 'each_commit' do
+      commits = []
+      with_hg_repository('hg') do |hg|
+        hg.activity.each_commit do |c|
+          assert c.token.length == 40
+          assert c.committer_name
+          assert c.committer_date.is_a?(Time)
+          refute c.message.empty?
+          assert c.diffs.any?
+          # Check that the diffs are populated
+          c.diffs.each do |d|
+            assert d.action =~ /^[MAD]$/
+            refute d.path.empty?
+          end
+          commits << c
+        end
+
+        refute File.exist?(hg.activity.send(:log_filename)) # Make sure we cleaned up after ourselves
+
+        # Verify that we got the commits in forward chronological order
+        commits.map(&:token).must_equal(%w[01101d8ef3cea7da9ac6e9a226d645f4418f05c9
+                                           b14fa4692f949940bd1e28da6fb4617de2615484
+                                           468336c6671cbc58237a259d1b7326866afc2817
+                                           75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                           655f04cf6ad708ab58c7b941672dce09dd369a18])
+      end
+    end
+
+    it 'each_commit_for_branch' do
+      commits = []
+
+      with_hg_repository('hg', 'develop') do |hg|
+        commits = hg.activity.each_commit
+      end
+
+      commits.map(&:token).must_equal(%w[01101d8ef3cea7da9ac6e9a226d645f4418f05c9
+                                         b14fa4692f949940bd1e28da6fb4617de2615484
+                                         468336c6671cbc58237a259d1b7326866afc2817
+                                         75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                         4d54c3f0526a1ec89214a70615a6b1c6129c665c])
+    end
+
+    it 'each_commit_after' do
+      commits = []
+      with_hg_repository('hg') do |hg|
+        hg.activity.each_commit(after: '468336c6671cbc58237a259d1b7326866afc2817') do |c|
+          commits << c
+        end
+        commits.map(&:token).must_equal(%w[75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                           655f04cf6ad708ab58c7b941672dce09dd369a18])
+      end
+    end
+
+    it 'open_log_file_encoding' do
+      with_hg_repository('hg_with_invalid_encoding') do |hg|
+        hg.activity.send(:open_log_file) do |io|
+          io.read.valid_encoding?.must_equal true
+        end
+      end
+    end
+
+    it 'commits_encoding' do
+      with_hg_repository('hg_with_invalid_encoding') do |hg|
+        hg.activity.commits
+      end
+    end
+
+    it 'verbose_commit_encoding' do
+      with_hg_repository('hg_with_invalid_encoding') do |hg|
+        hg.activity.verbose_commit('51ea5277ca27')
+      end
+    end
+  end
+
+  describe 'head' do
+    it 'hg_head_and_parents' do
+      with_hg_repository('hg') do |hg|
+        hg.activity.head_token.must_equal '655f04cf6ad708ab58c7b941672dce09dd369a18'
+        hg.activity.head.token.must_equal '655f04cf6ad708ab58c7b941672dce09dd369a18'
+        assert hg.activity.head.diffs.any? # diffs should be populated
+      end
+    end
+
+    it 'head_with_branch' do
+      with_hg_repository('hg', 'develop') do |hg|
+        hg.activity.head.token.must_equal '4d54c3f0526a1ec89214a70615a6b1c6129c665c'
+        assert hg.activity.head.diffs.any?
+      end
+    end
+  end
+
+  describe 'commit_tokens' do
+    it 'must work with after argument' do
+      with_hg_repository('hg') do |hg|
+        hg.activity.commit_tokens.must_equal(%w[01101d8ef3cea7da9ac6e9a226d645f4418f05c9
+                                                b14fa4692f949940bd1e28da6fb4617de2615484
+                                                468336c6671cbc58237a259d1b7326866afc2817
+                                                75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                                655f04cf6ad708ab58c7b941672dce09dd369a18])
+
+        after = '01101d8ef3cea7da9ac6e9a226d645f4418f05c9'
+        hg.activity.commit_tokens(after: after).must_equal(%w[b14fa4692f949940bd1e28da6fb4617de2615484
+                                                              468336c6671cbc58237a259d1b7326866afc2817
+                                                              75532c1e1f1de55c2271f6fd29d98efbe35397c4
+                                                              655f04cf6ad708ab58c7b941672dce09dd369a18])
+
+        after = '75532c1e1f1de55c2271f6fd29d98efbe35397c4'
+        hg.activity.commit_tokens(after: after).must_equal ['655f04cf6ad708ab58c7b941672dce09dd369a18']
+
+        hg.activity.commit_tokens(after: '655f04cf6ad708ab58c7b941672dce09dd369a18').must_be :empty?
+      end
+    end
+
+    it 'must work with trunk_only argument' do
+      with_hg_repository('hg_dupe_delete') do |hg|
+        hg.activity.commit_tokens(trunk_only: false).must_equal(%w[73e93f57224e3fd828cf014644db8eec5013cd6b
+                                                                   732345b1d5f4076498132fd4b965b1fec0108a50
+                                                                   525de321d8085bc1d4a3c7608fda6b4020027985
+                                                                   72fe74d643bdcb30b00da3b58796c50f221017d0])
+
+        hg.activity.commit_tokens(trunk_only: true).must_equal(['73e93f57224e3fd828cf014644db8eec5013cd6b',
+                                                                '732345b1d5f4076498132fd4b965b1fec0108a50',
+                                                                # '525de321d8085bc1d4a3c7608fda6b4020027985', # branch
+                                                                '72fe74d643bdcb30b00da3b58796c50f221017d0'])
+      end
+    end
+
+    it 'must work with trunk_only and after arguments' do
+      with_hg_repository('hg_dupe_delete') do |hg|
+        opts = { after: '73e93f57224e3fd828cf014644db8eec5013cd6b', trunk_only: false }
+        hg.activity.commit_tokens(opts).must_equal(%w[732345b1d5f4076498132fd4b965b1fec0108a50
+                                                      525de321d8085bc1d4a3c7608fda6b4020027985
+                                                      72fe74d643bdcb30b00da3b58796c50f221017d0])
+
+        opts = { after: '73e93f57224e3fd828cf014644db8eec5013cd6b', trunk_only: true }
+        hg.activity.commit_tokens(opts).must_equal(['732345b1d5f4076498132fd4b965b1fec0108a50',
+                                                    # '525de321d8085bc1d4a3c7608fda6b4020027985', # On branch
+                                                    '72fe74d643bdcb30b00da3b58796c50f221017d0'])
+
+        hg.activity.commit_tokens(after: '72fe74d643bdcb30b00da3b58796c50f221017d0', trunk_only: true).must_be :empty?
+      end
+    end
+
+    it 'must work with after and upto arguments' do
+      with_hg_repository('hg_walk') do |hg|
+        commit_tokens = CommitTokensHelper.new(hg, commit_labels)
+        # Full history to a commit
+        commit_tokens.between(nil, :A).must_equal %i[A]
+        commit_tokens.between(nil, :B).must_equal %i[A B]
+        commit_tokens.between(nil, :C).must_equal %i[A B G H C]
+        commit_tokens.between(nil, :D).must_equal %i[A B G H C I D]
+        commit_tokens.between(nil, :G).must_equal %i[A B G]
+        commit_tokens.between(nil, :H).must_equal %i[A B G H]
+        commit_tokens.between(nil, :I).must_equal %i[A B G H C I]
+
+        # Limited history from one commit to another
+        commit_tokens.between(:A, :A).must_be :empty?
+        commit_tokens.between(:A, :B).must_equal %i[B]
+        commit_tokens.between(:A, :C).must_equal %i[B G H C]
+        commit_tokens.between(:A, :D).must_equal %i[B G H C I D]
+        commit_tokens.between(:B, :D).must_equal %i[G H C I D]
+        commit_tokens.between(:C, :D).must_equal %i[I D]
+      end
+    end
+
+    def commit_labels
+      { A: '4bfbf836feeebb236492199fbb0d1474e26f69d9',
+        B: '23edb79d0d06c8c315d8b9e7456098823335377d',
+        C: '7e33b9fde56a6e3576753868d08fa143e4e8a9cf',
+        D: '8daa1aefa228d3ee5f9a0f685d696826e88266fb',
+        G: 'e43cf1bb4b80d8ae70a695ec070ce017fdc529f3',
+        H: 'dca215d8a3e4dd3e472379932f1dd9c909230331',
+        I: '3a1495175e40b1c983441d6a8e8e627d2bd672b6' }
+    end
+  end
+
+  describe 'cat' do
+    it 'must get file contents by current or parent commit' do
+      with_hg_repository('hg') do |hg|
+        expected = <<-EXPECTED.gsub(/ {10}/, '')
+          /* Hello, World! */
+
+          /*
+           * This file is not covered by any license, especially not
+           * the GNU General Public License (GPL). Have fun!
+           */
+
+          #include <stdio.h>
+          main()
+          {
+          	printf("Hello, World!\\n");
+          }
+        EXPECTED
+
+        diff = OhlohScm::Diff.new(path: 'helloworld.c')
+        commit = OhlohScm::Commit.new(token: '75532c1e1f1d')
+        # The file was deleted in revision 468336c6671c. Check that it does not exist now, but existed in parent.
+        hg.activity.cat_file(commit, diff).must_be_nil
+        hg.activity.cat_file_parent(commit, diff).must_equal expected
+        hg.activity.cat_file(OhlohScm::Commit.new(token: '468336c6671c'), diff).must_equal expected
+      end
+    end
+
+    # Ensure that we escape bash-significant characters like ' and & when they appear in the filename
+    it 'must handle funny file names' do
+      tmpdir do |dir|
+        # Make a file with a problematic filename
+        funny_name = '#|file_name` $(&\'")#'
+        file_content = 'foobar'
+        File.open(File.join(dir, funny_name), 'w') { |f| f.write file_content }
+
+        # Add it to an hg repository
+        `cd #{dir} && hg init && hg add * 2> /dev/null && hg commit -u tester -m test`
+
+        # Confirm that we can read the file back
+        hg = OhlohScm::Factory.get_core(scm_type: :hg, url: dir)
+        diff = OhlohScm::Diff.new(path: funny_name)
+        hg.activity.cat_file(hg.activity.head, diff).must_equal file_content
+      end
+    end
+  end
+
+  describe 'cleanup' do
+    it 'must call shutdown hg_client' do
+      activity = OhlohScm::Factory.get_core(scm_type: :hg, url: 'foobar').activity
+      hg_client = Struct.new(:foo)
+      activity.stubs(:hg_client).returns(hg_client)
+      hg_client.expects(:shutdown)
+      activity.cleanup
+    end
+  end
+end