metior 0.1.2 → 0.1.3

data/Gemfile

@@ -1,7 +1,5 @@
 source :rubygems
 
-gem 'bundler', '~> 1.0.14'
-
 group :git do
   gem 'grit', '~> 2.4.1'
 end

data/Gemfile.lock

@@ -35,7 +35,6 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  bundler (~> 1.0.14)
   grit (~> 2.4.1)
   mocha (~> 0.9.12)
   octokit (~> 0.6.3)

data/lib/metior.rb

@@ -5,12 +5,6 @@
 
 require 'bundler'
 
-libdir = File.dirname(__FILE__)
-Dir.chdir libdir do
-  Bundler.setup
-end
-$LOAD_PATH.unshift(libdir) unless $LOAD_PATH.include?(libdir)
-
 require 'core_ext/object'
 require 'metior/git'
 require 'metior/github'
@@ -22,6 +16,14 @@ require 'metior/version'
 # @author Sebastian Staudt
 module Metior
 
+  # Returns the Bundler runtime to access Metior's dependencies
+  #
+  # @return [Bundler::Runtime] The Bundler runtime instance
+  def self.runtime
+    @@runtime ||= Bundler::Runtime.new File.dirname(__FILE__),
+                                       Bundler.definition
+  end
+
   # Creates a new repository for the given repository type and path
   #
   # @param [Symbol] type The type of the repository, e.g. `:git`
@@ -35,19 +37,23 @@ module Metior
   # Calculates simplistic stats for the given repository and branch
   #
   # @param [Symbol] type The type of the repository, e.g. `:git`
-  # @param [String] path The file system path of the repository
-  # @param [String] branch The repository's 'branch to analyze. `nil` will use
-  #        the VCS's default branch
+  # @param [Array<Object>] options The options for the repository
+  # @param [String, Range] range The range of commits for which the commits
+  #        should be loaded. This may be given as a string
+  #        (`'master..development'`), a range (`'master'..'development'`)
+  #        or as a single ref (`'master'`). A single ref name means all
+  #        commits reachable from that ref.
   # @return [Hash] The calculated stats for the given repository and branch
-  def self.simple_stats(type, path, branch = nil)
-    repo = repository type, path
-    branch ||= vcs(type)::DEFAULT_BRANCH
+  def self.simple_stats(type, *options)
+    arity  = vcs(type)::Repository.instance_method(:initialize).arity
+    branch = options.delete_at arity
+    branch = vcs(type)::DEFAULT_BRANCH if branch.nil?
+    repo   = repository type, *options
 
     {
-      :authors        => repo.authors(branch).values,
-      :commit_count   => repo.commits(branch).size,
-      :top_committers => repo.top_contributors(branch, 3)
-    }
+      :commit_count     => repo.commits(branch).size,
+      :top_contributors => repo.top_contributors(branch, 5),
+    }.merge Commit.activity(repo.commits(branch))
   end
 
 end

data/lib/metior/github/commit.rb

@@ -29,20 +29,20 @@ module Metior
 
         @added_files    = []
         @additions      = 0
-        @authored_date  = commit.authored_date
-        @committed_date = commit.committed_date
+        @authored_date  = Time.parse commit.authored_date
+        @committed_date = Time.parse commit.committed_date
         @deleted_files  = []
         @deletions      = 0
         @id             = commit.id
         @message        = commit.message
         @modified_files = []
 
-        authors = repo.authors(branch)
+        authors = repo.authors range
         @author = authors[Actor.id_for commit.author]
         @author = Actor.new repo, commit.author if author.nil?
         @author.add_commit self
 
-        committers = repo.committers branch
+        committers = repo.committers range
         @committer = committers[Actor.id_for commit.committer]
         @committer = Actor.new repo, commit.committer if @committer.nil?
         @committer.add_commit self

data/lib/metior/github/repository.rb

@@ -57,37 +57,39 @@ module Metior
       #        or as a single ref (`'master'`). A single ref name means all
       #        commits reachable from that ref.
       # @return [Array<Commit>] All commits in the given commit range
-      # @see #load_branch_commits
+      # @see #load_ref_commits
       def load_commits(range)
-        commits      = load_branch_commits(range.last, range)
-        base_commits = load_branch_commits(range.first, range).map! do |commit|
-          commit.id
+        commits      = load_ref_commits(range.last)
+        if range.first == ''
+          base_commits = []
+        else
+          base_commits = load_ref_commits(range.first).map! do |commit|
+            commit.id
+          end
         end
         commits.reject { |commit| base_commits.include? commit.id }
       end
 
-      # This method uses Octokit to load all commits from the given branch
+      # This method uses Octokit to load all commits from the given ref
       #
       # Because of GitHub API limitations, the commits have to be loaded in
       # batches.
       #
       # @note GitHub API is currently limited to 60 calls a minute, so you
-      #       won't be able to query branches with more than 2100 commits
-      #       (35 commits per call).
-      # @param [String] branch The branch to load commits from
-      # @param [String, Range] range The range of commits to which the loaded
-      #        commits should be assigned
-      # @return [Array<Commit>] All commits from the given branch
+      #       won't be able to query refs with more than 2100 commits (35
+      #       commits per call).
+      # @param [String] ref The ref to load commits from
+      # @return [Array<Commit>] All commits from the given ref
       # @see Octokit::Commits#commits
-      def load_branch_commits(branch, range)
+      def load_ref_commits(ref)
         commits = []
         page = 1
         begin
-          begin
-            commits += Octokit.commits(@path, range, :page => page)
+          loop do
+            commits += Octokit.commits(@path, ref, :page => page)
             page += 1
-          end while true
-        rescue Octokit::NotFound
+          end
+        rescue Faraday::Error::ResourceNotFound
         end
         commits
       end

data/lib/metior/vcs.rb

@@ -64,7 +64,9 @@ module Metior
       #
       # @see Bundler.require
       def init
-        Bundler.require self::NAME
+        Metior.runtime.dependencies_for(self::NAME).each do |dep|
+          gem dep.name, dep.requirement
+        end
 
         path = self::NAME.to_s
         require "metior/#{path}/actor"

data/lib/metior/version.rb

@@ -6,6 +6,6 @@
 module Metior
 
   # The current version of the Metior gem
-  VERSION = '0.1.2'
+  VERSION = '0.1.3'
 
 end

data/metior.gemspec

@@ -12,6 +12,7 @@ Gem::Specification.new do |s|
   s.summary     = 'A source code history analyzer API'
   s.description = 'Metior is a source code history analyzer that provides various statistics about a source code repository and its change over time.'
 
+  s.add_dependency 'bundler', '~> 1.0.14'
   Bundler.definition.dependencies.each do |dep|
     if dep.groups.include?(:development) || dep.groups.include?(:test)
       s.add_development_dependency(dep.name, dep.requirement.to_s)

data/test/fixtures.rb

@@ -0,0 +1,115 @@
+# This code is free software; you can redistribute it and/or modify it under
+# the terms of the new BSD License.
+#
+# Copyright (c) 2011, Sebastian Staudt
+
+require 'rash'
+require 'time'
+
+module Fixtures
+
+  @@commits = {}
+
+  def self.commits(range)
+    unless @@commits.key? range
+      file = File.open(File.expand_path("#{File.dirname(__FILE__)}/fixtures/mojombo-grit-master-1b2fe77.txt"), 'r')
+      file.set_encoding 'utf-8' if file.respond_to? :set_encoding
+
+      commits = []
+
+      commit = {}
+      file.lines.each do |line|
+        line.strip!
+
+        next if line.empty?
+
+        if line.match /^[AMD]\0/
+          commit[:files] = [] unless commit.key? :files
+          commit[:files] << line.split("\0")
+        elsif line.match /\d+\0\d+$/
+          commit[:impact] = line.split("\0")
+        else
+          line = line.split("\0")
+          ids = [line[0]]
+          ids += line[1][2..-2].split(', ') unless line[1].empty?
+          commit[:ids]  = ids
+          commit[:info] = line[2..-1]
+
+          if commits.empty?
+            if !commit[:ids].include?(range.last)
+              commit = {}
+              next
+            end
+          elsif commit[:ids].include? range.first
+            break
+          end
+
+          if commit != {}
+            commits << commit
+            commit = {}
+          end
+        end
+      end
+
+      @@commits[range] = commits
+    end
+
+    @@commits[range]
+  end
+
+  def self.commits_as_grit_commits(range)
+    commits = commits(range)
+
+    commits.map do |commit|
+      grit_commit = Grit::Commit.new(
+        nil,
+        commit[:ids].first,
+        [],
+        nil,
+        Grit::Actor.new(commit[:info][4], commit[:info][5]),
+        Time.at(commit[:info][6].to_i),
+        Grit::Actor.new(commit[:info][1], commit[:info][2]),
+        Time.at(commit[:info][3].to_i),
+        commit[:info].first.lines.to_a
+      )
+
+      grit_commit.stubs(:diffs).returns []
+
+      stats = Hashie::Rash.new
+      if commit.key? :impact
+        stats.additions = commit[:impact].first.to_i
+        stats.deletions = commit[:impact].last.to_i
+      else
+        stats.additions = 0
+        stats.deletions = 0
+      end
+      grit_commit.stubs(:stats).returns stats
+
+      grit_commit
+    end
+  end
+
+  def self.commits_as_rashies(range)
+    commits = commits(range)
+
+    commits.map do |commit|
+      Hashie::Rash.new({
+        :author         => {
+          :email => commit[:info][4],
+          :login => commit[:info][5],
+          :name  => commit[:info][5]
+        },
+        :authored_date  => Time.at(commit[:info][6].to_i).to_s,
+        :committed_date => Time.at(commit[:info][3].to_i).to_s,
+        :committer      => {
+          :email => commit[:info][1],
+          :login => commit[:info][2],
+          :name  => commit[:info][2]
+        },
+        :id             => commit[:ids].first,
+        :message        => commit[:info].first,
+      })
+    end
+  end
+
+end

data/test/helper.rb

@@ -8,6 +8,7 @@ $LOAD_PATH.unshift File.dirname(__FILE__)
 require 'metior'
 include Metior
 
+Bundler.setup
 Bundler.require :test
 
 # Extends TestCase functionality

data/test/test_git.rb

@@ -3,6 +3,7 @@
 #
 # Copyright (c) 2011, Sebastian Staudt
 
+require 'fixtures'
 require 'helper'
 
 class TestGit < Test::Unit::TestCase
@@ -19,4 +20,97 @@ class TestGit < Test::Unit::TestCase
 
   end
 
+  context 'A Git repository' do
+
+    setup do
+      @repo = Metior::Git::Repository.new File.dirname(File.dirname(__FILE__))
+      @@grit_commits ||= Fixtures.commits_as_grit_commits(''..'master')
+      @commits_stub = Grit::Repo.any_instance.stubs :commits
+      @commits_stub.returns @@grit_commits
+    end
+
+    should 'be able to load all commits from the repository\'s default branch' do
+      commits = @repo.commits
+      assert_equal 460, commits.size
+      assert commits.all? { |commit| commit.is_a? Metior::Git::Commit }
+
+      head = commits.first
+      assert_equal '1b2fe77', head.id
+    end
+
+    should 'be able to load a range of commits from the repository' do
+      @commits_stub = Grit::Repo.any_instance.stubs :commits
+      api_response = Fixtures.commits_as_grit_commits('ef2870b'..'4c592b4')
+      @commits_stub.returns api_response
+
+      commits = @repo.commits 'ef2870b'..'4c592b4'
+      assert_equal 6, commits.size
+      assert commits.all? { |commit| commit.is_a? Metior::Git::Commit }
+      assert_equal '4c592b4', commits.first.id
+      assert_equal 'f0cc7f7', commits.last.id
+    end
+
+    should 'know the authors of the repository' do
+      authors = @repo.authors
+      assert_equal 37, authors.size
+      assert authors.values.all? { |author| author.is_a? Metior::Git::Actor }
+
+      assert_equal %w{
+        adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
+        bobbywilson0@gmail.com bryce@worldmedia.net cehoffman@gmail.com
+        chapados@sciencegeeks.org cho45@lowreal.net chris@ozmm.org
+        davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
+        engel@engel.uk.to evil@che.lu gram.gibson@uky.edu
+        hiroshi3110@gmail.com igor@wiedler.ch johan@johansorensen.com
+        jos@catnook.com jpriddle@nevercraft.net kamal.fariz@gmail.com
+        koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
+        paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
+        rtomayko@gmail.com schacon@gmail.com scott@railsnewbie.com
+        technoweenie@gmail.com tim@dysinger.net tim@spork.in
+        tom@mojombo.com tom@taco.(none) voker57@gmail.com wayne@larsen.st
+      }, authors.keys.sort
+    end
+
+    should 'know the committers of the repository' do
+      committers = @repo.committers
+      assert_equal 29, committers.size
+      assert committers.values.all? { |committer| committer.is_a? Metior::Git::Actor }
+
+      assert_equal %w{
+         adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
+         bobbywilson0@gmail.com bryce@worldmedia.net chris@ozmm.org
+         davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
+         engel@engel.uk.to evil@che.lu hiroshi3110@gmail.com
+         johan@johansorensen.com jos@catnook.com kamal.fariz@gmail.com
+         koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
+         paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
+         rtomayko@gmail.com schacon@gmail.com technoweenie@gmail.com
+         tim@dysinger.net tim@spork.in tom@mojombo.com tom@taco.(none)
+         voker57@gmail.com
+      }, committers.keys.sort
+    end
+
+    should 'know the top authors of the repository' do
+      authors = @repo.top_authors
+      assert_equal 3, authors.size
+      assert authors.all? { |author| author.is_a? Metior::Git::Actor }
+
+      assert_equal [
+        "tom@mojombo.com", "schacon@gmail.com", "technoweenie@gmail.com"
+      ], authors.collect { |author| author.id }
+    end
+
+    should 'provide easy access to simple repository statistics' do
+      stats = Metior.simple_stats :git, File.dirname(File.dirname(__FILE__))
+
+      assert_equal 157, stats[:active_days].size
+      assert_equal 460, stats[:commit_count]
+      assert_in_delta 2.92993630573248, stats[:commits_per_active_day], 0.0001
+      assert_equal Time.at(1191997100), stats[:first_commit_date]
+      assert_equal Time.at(1306794294), stats[:last_commit_date]
+      assert_equal 5, stats[:top_contributors].size
+    end
+
+  end
+
 end

data/test/test_github.rb

@@ -3,6 +3,7 @@
 #
 # Copyright (c) 2011, Sebastian Staudt
 
+require 'fixtures'
 require 'helper'
 
 class TestGitHub < Test::Unit::TestCase
@@ -17,27 +18,123 @@ class TestGitHub < Test::Unit::TestCase
       assert_not Metior::GitHub.supports? :line_stats
     end
 
+  end
+
+  context 'A GitHub repository' do
+
+    setup do
+      @repo = Metior::GitHub::Repository.new 'mojombo', 'grit'
+
+      api_response = Fixtures.commits_as_rashies(''..'master')
+      @commits_stub = Octokit.stubs :commits
+      14.times { @commits_stub.returns api_response.shift(35) }
+      @commits_stub.then.raises Faraday::Error::ResourceNotFound.new(nil)
+    end
+
+    should 'be able to load all commits from the repository\'s default branch' do
+      commits = @repo.commits
+      assert_equal 460, commits.size
+      assert commits.all? { |commit| commit.is_a? Metior::GitHub::Commit }
+
+      head = commits.first
+      assert_equal '1b2fe77', head.id
+    end
+
+    should 'be able to load a range of commits from the repository' do
+      @commits_stub = Octokit.stubs :commits
+      api_response = Fixtures.commits_as_rashies(''..'4c592b4')
+      14.times { @commits_stub.returns api_response.shift(35) }
+      @commits_stub.raises Faraday::Error::ResourceNotFound.new(nil)
+      api_response = Fixtures.commits_as_rashies(''..'ef2870b')
+      13.times { @commits_stub.returns api_response.shift(35) }
+      @commits_stub.then.raises Faraday::Error::ResourceNotFound.new(nil)
+
+      commits = @repo.commits 'ef2870b'..'4c592b4'
+      assert_equal 6, commits.size
+      assert commits.all? { |commit| commit.is_a? Metior::GitHub::Commit }
+      assert_equal '4c592b4', commits.first.id
+      assert_equal 'f0cc7f7', commits.last.id
+    end
+
+    should 'know the authors of the repository' do
+      authors = @repo.authors
+      assert_equal 37, authors.size
+      assert authors.values.all? { |author| author.is_a? Metior::GitHub::Actor }
+
+      assert_equal %w{
+        adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
+        bobbywilson0@gmail.com bryce@worldmedia.net cehoffman@gmail.com
+        chapados@sciencegeeks.org cho45@lowreal.net chris@ozmm.org
+        davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
+        engel@engel.uk.to evil@che.lu gram.gibson@uky.edu
+        hiroshi3110@gmail.com igor@wiedler.ch johan@johansorensen.com
+        jos@catnook.com jpriddle@nevercraft.net kamal.fariz@gmail.com
+        koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
+        paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
+        rtomayko@gmail.com schacon@gmail.com scott@railsnewbie.com
+        technoweenie@gmail.com tim@dysinger.net tim@spork.in
+        tom@mojombo.com tom@taco.(none) voker57@gmail.com wayne@larsen.st
+      }, authors.keys.sort
+    end
+
+    should 'know the committers of the repository' do
+      committers = @repo.committers
+      assert_equal 29, committers.size
+      assert committers.values.all? { |committer| committer.is_a? Metior::GitHub::Actor }
+
+      assert_equal %w{
+         adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
+         bobbywilson0@gmail.com bryce@worldmedia.net chris@ozmm.org
+         davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
+         engel@engel.uk.to evil@che.lu hiroshi3110@gmail.com
+         johan@johansorensen.com jos@catnook.com kamal.fariz@gmail.com
+         koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
+         paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
+         rtomayko@gmail.com schacon@gmail.com technoweenie@gmail.com
+         tim@dysinger.net tim@spork.in tom@mojombo.com tom@taco.(none)
+         voker57@gmail.com
+      }, committers.keys.sort
+    end
+
+    should 'know the top authors of the repository' do
+      authors = @repo.top_authors
+      assert_equal 3, authors.size
+      assert authors.all? { |author| author.is_a? Metior::GitHub::Actor }
+
+      assert_equal [
+        "tom@mojombo.com", "schacon@gmail.com", "technoweenie@gmail.com"
+      ], authors.collect { |author| author.id }
+    end
+
     should 'not be able to get file stats of a repository' do
-      repo = Metior::GitHub::Repository.new 'koraktor', 'rubikon'
       assert_raises UnsupportedError do
-        repo.file_stats
+        @repo.file_stats
       end
     end
 
     should 'not be able to get the most significant authors of a repository' do
-      repo = Metior::GitHub::Repository.new 'koraktor', 'rubikon'
       assert_raises UnsupportedError do
-        repo.significant_authors
+        @repo.significant_authors
       end
     end
 
     should 'not be able to get the most significant commits of a repository' do
-      repo = Metior::GitHub::Repository.new 'koraktor', 'rubikon'
       assert_raises UnsupportedError do
-        repo.significant_commits
+        @repo.significant_commits
       end
     end
 
+    should 'provide easy access to simple repository statistics' do
+      stats = Metior.simple_stats :github, 'mojombo', 'grit'
+
+      assert_equal 157, stats[:active_days].size
+      assert_equal 460, stats[:commit_count]
+      assert_in_delta 2.92993630573248, stats[:commits_per_active_day], 0.0001
+      assert_equal Time.at(1191997100), stats[:first_commit_date]
+      assert_equal Time.at(1306794294), stats[:last_commit_date]
+      assert_equal 5, stats[:top_contributors].size
+    end
+
   end
 
 end

data/test/test_repository.rb

@@ -43,108 +43,4 @@ class TestRepository < Test::Unit::TestCase
 
   end
 
-  context 'A repository implementation' do
-
-    setup do
-      require 'mock_vcs/repository'
-
-      repo_file = "#{File.dirname(__FILE__)}/fixtures/mojombo-grit-master-1b2fe77.txt"
-      @repo = MockVCS::Repository.new repo_file
-    end
-
-    should 'be able to load all commits from the repository' do
-      commits = @repo.commits
-      assert_equal 415, commits.size
-      assert commits.all? { |commit| commit.is_a? Metior::Commit }
-
-      head = commits.first
-      assert_equal '4c592b4', head.id
-    end
-
-    should 'be able to load a range of commits from the repository' do
-      commits = @repo.commits 'ef2870b'..'4c592b4'
-      assert_equal 5, commits.size
-      assert commits.all? { |commit| commit.is_a? Metior::Commit }
-      assert_equal '4c592b4', commits.first.id
-      assert_equal 'f0cc7f7', commits.last.id
-    end
-
-    should 'know the authors of the repository' do
-      authors = @repo.authors
-      assert_equal 37, authors.size
-      assert authors.values.all? { |author| author.is_a? Metior::Actor }
-
-      assert_equal %w{
-        adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
-        bobbywilson0@gmail.com bryce@worldmedia.net cehoffman@gmail.com
-        chapados@sciencegeeks.org cho45@lowreal.net chris@ozmm.org
-        davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
-        engel@engel.uk.to evil@che.lu gram.gibson@uky.edu
-        hiroshi3110@gmail.com igor@wiedler.ch johan@johansorensen.com
-        jos@catnook.com jpriddle@nevercraft.net kamal.fariz@gmail.com
-        koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
-        paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
-        rtomayko@gmail.com schacon@gmail.com scott@railsnewbie.com
-        technoweenie@gmail.com tim@dysinger.net tim@spork.in
-        tom@mojombo.com tom@taco.(none) voker57@gmail.com wayne@larsen.st
-      }, authors.keys.sort
-    end
-
-    should 'know the committers of the repository' do
-      committers = @repo.committers
-      assert_equal 29, committers.size
-      assert committers.values.all? { |committer| committer.is_a? Metior::Actor }
-
-      assert_equal %w{
-         adam@therealadam.com aman@tmm1.net antonin@hildebrand.cz
-         bobbywilson0@gmail.com bryce@worldmedia.net chris@ozmm.org
-         davetron5000@gmail.com david.kowis@rackspace.com dustin@spy.net
-         engel@engel.uk.to evil@che.lu hiroshi3110@gmail.com
-         johan@johansorensen.com jos@catnook.com kamal.fariz@gmail.com
-         koraktor@gmail.com mtraverso@acm.org ohnobinki@ohnopublishing.net
-         paul+git@mjr.org pjhyett@gmail.com rsanheim@gmail.com
-         rtomayko@gmail.com schacon@gmail.com technoweenie@gmail.com
-         tim@dysinger.net tim@spork.in tom@mojombo.com tom@taco.(none)
-         voker57@gmail.com
-      }, committers.keys.sort
-    end
-
-    should 'know the most significant authors of the repository' do
-      authors = @repo.significant_authors
-      assert_equal 3, authors.size
-      assert authors.all? { |author| author.is_a? Metior::Actor }
-
-      assert_equal [
-        "tom@mojombo.com", "schacon@gmail.com", "rtomayko@gmail.com"
-      ], authors.collect { |author| author.id }
-    end
-
-    should 'know the most significant commits of the repository' do
-      commits = @repo.significant_commits('master', 8)
-      assert_equal 8, commits.size
-      assert commits.all? { |commit| commit.is_a? Metior::Commit }
-
-      assert_equal %w{
-        c0f0b4f 47ab25c f3a24ae 18ec70e 242253b c87612b 6bb41e4 4d9c7be
-      }, commits.collect { |commit| commit.id }
-
-      modifications = commits.first.modifications
-      commits[1..-1].each do |commit|
-        assert modifications >= commit.modifications
-        modifications = commit.modifications
-      end
-    end
-
-    should 'know the top authors of the repository' do
-      authors = @repo.top_authors
-      assert_equal 3, authors.size
-      assert authors.all? { |author| author.is_a? Metior::Actor }
-
-      assert_equal [
-        "tom@mojombo.com", "schacon@gmail.com", "technoweenie@gmail.com"
-      ], authors.collect { |author| author.id }
-    end
-
-  end
-
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: metior
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: 
   segments: 
   - 0
   - 1
-  - 2
-  version: 0.1.2
+  - 3
+  version: 0.1.3
 platform: ruby
 authors: 
 - Sebastian Staudt
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-09 00:00:00 Z
+date: 2011-06-13 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   type: :runtime
@@ -165,11 +165,9 @@ files:
 - lib/metior/vcs.rb
 - lib/metior/version.rb
 - metior.gemspec
+- test/fixtures.rb
 - test/fixtures/mojombo-grit-master-1b2fe77.txt
 - test/helper.rb
-- test/mock_vcs/actor.rb
-- test/mock_vcs/commit.rb
-- test/mock_vcs/repository.rb
 - test/test_class_loading.rb
 - test/test_git.rb
 - test/test_github.rb
@@ -210,11 +208,9 @@ signing_key:
 specification_version: 3
 summary: A source code history analyzer API
 test_files: 
+- test/fixtures.rb
 - test/fixtures/mojombo-grit-master-1b2fe77.txt
 - test/helper.rb
-- test/mock_vcs/actor.rb
-- test/mock_vcs/commit.rb
-- test/mock_vcs/repository.rb
 - test/test_class_loading.rb
 - test/test_git.rb
 - test/test_github.rb

data/test/mock_vcs/actor.rb

@@ -1,28 +0,0 @@
-# This code is free software; you can redistribute it and/or modify it under
-# the terms of the new BSD License.
-#
-# Copyright (c) 2011, Sebastian Staudt
-
-require 'metior/actor'
-
-module Metior
-
-  module MockVCS
-
-    class Actor < Metior::Actor
-
-      def self.id_for(actor)
-        actor.last
-      end
-
-      def initialize(repo, actor)
-        super repo
-
-        @id   = actor.last
-        @name = actor.first
-      end
-    end
-
-  end
-
-end

data/test/mock_vcs/commit.rb

@@ -1,53 +0,0 @@
-# This code is free software; you can redistribute it and/or modify it under
-# the terms of the new BSD License.
-#
-# Copyright (c) 2011, Sebastian Staudt
-
-require 'mock_vcs/actor'
-require 'metior/commit'
-
-module Metior
-
-  module MockVCS
-
-    class Commit < Metior::Commit
-
-      def initialize(repo, range, commit)
-        super repo, range
-
-        @additions      = commit[:impact].first.to_i
-        @authored_date  = Time.at commit[:info][6].to_i
-        @committed_date = Time.at commit[:info][3].to_i
-        @deletions      = commit[:impact].last.to_i
-        @id             = commit[:ids].first
-        @message        = commit[:info].first
-
-        authors = repo.authors range
-        @author = authors[Actor.id_for commit[:info][4..5]]
-        @author = Actor.new repo, commit[:info][4..5] if author.nil?
-        @author.add_commit self
-
-        committers = repo.committers range
-        @committer = committers[Actor.id_for commit[:info][1..2]]
-        @committer = Actor.new repo, commit[:info][1..2] if @committer.nil?
-        @committer.add_commit self
-
-        @added_files    = []
-        @modified_files = []
-        @deleted_files  = []
-        commit[:files].each do |file|
-          if file.first == 'A'
-            @added_files    << file.last
-          elsif file.first == 'A'
-            @deleted_files  << file.last
-          else
-            @modified_files << file.last
-          end
-        end
-      end
-
-    end
-
-  end
-
-end

data/test/mock_vcs/repository.rb

@@ -1,72 +0,0 @@
-# This code is free software; you can redistribute it and/or modify it under
-# the terms of the new BSD License.
-#
-# Copyright (c) 2011, Sebastian Staudt
-
-require 'metior/repository'
-require 'mock_vcs/commit'
-
-module Metior
-
-  module MockVCS
-
-    NAME = :mock
-
-    include VCS
-
-    class Repository < Metior::Repository
-
-      DEFAULT_BRANCH = 'master'
-
-      include Metior::MockVCS
-
-      def initialize(path)
-        super path
-
-        @file = File.open(File.expand_path(path), 'r')
-        @file.set_encoding 'utf-8' if @file.respond_to? :set_encoding
-      end
-
-      def load_commits(range)
-        commits = []
-
-        commit = {}
-        @file.lines.each do |line|
-          line.strip!
-
-          next if line.empty?
-
-          if line.match /^[AMD]\0/
-            commit[:files] = [] unless commit.key? :files
-            commit[:files] << line.split("\0")
-          elsif line.match /\d+\0\d+$/
-            commit[:impact] = line.split("\0")
-
-            if commits.empty?
-              if range.first != '' && !commit[:ids].include?(range.last)
-                commit = {}
-                next
-              end
-            elsif commit[:ids].include? range.first
-              break
-            end
-
-            commits << commit
-            commit = {}
-          else
-            line = line.split("\0")
-            ids = [line[0]]
-            ids += line[1][2..-2].split(', ') unless line[1].empty?
-            commit[:ids]  = ids
-            commit[:info] = line[2..-1]
-          end
-        end
-
-        commits
-      end
-
-    end
-
-  end
-
-end