metior 0.1.4 → 0.2.0

data/test/fixtures.rb

@@ -3,7 +3,7 @@
 #
 # Copyright (c) 2011, Sebastian Staudt
 
-require 'rash'
+require 'hashie/mash'
 require 'time'
 
 module Fixtures
@@ -18,14 +18,21 @@ module Fixtures
       commits = []
 
       commit = {}
+      last_commit = nil
       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")
+        if line.match /^[A]\0/
+          commit[:added_files] = [] unless commit.key? :added_files
+          commit[:added_files] << line.split("\0").last.strip
+        elsif line.match /^[D]\0/
+          commit[:modified_files] = [] unless commit.key? :modified_files
+          commit[:modified_files] << line.split("\0").last.strip
+        elsif line.match /^[M]\0/
+          commit[:deleted_files] = [] unless commit.key? :deleted_files
+          commit[:deleted_files] << line.split("\0").last.strip
         elsif line.match /\d+\0\d+$/
           commit[:impact] = line.split("\0")
         else
@@ -45,7 +52,9 @@ module Fixtures
           end
 
           if commit != {}
+            last_commit[:parent] = commit unless last_commit.nil?
             commits << commit
+            last_commit = commit
             commit = {}
           end
         end
@@ -58,42 +67,48 @@ module Fixtures
   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
+    commits = commits(range).map do |commit|
+      parents = commit[:parent].nil? ? [] : [commit[:parent]]
+
+      diffs = []
+      commit[:added_files].each do |added_file|
+        diffs << { :new_file => true, :b_path => added_file }
+      end if commit.key? :added_files
+      commit[:deleted_files].each do |deleted_file|
+        diffs << { :deleted_file => true, :b_path => deleted_file }
+      end if commit.key? :deleted_files
+      commit[:modified_files].each do |modified_file|
+        diffs << { :b_path => modified_file }
+      end if commit.key? :modified_files
+
+      Hashie::Mash.new({
+        :author         => {
+          :email => commit[:info][5],
+          :name  => commit[:info][4]
+        },
+        :authored_data  => Time.at(commit[:info][6].to_i),
+        :committer      => {
+          :email => commit[:info][2],
+          :name  => commit[:info][1]
+        },
+        :committed_date => Time.at(commit[:info][3].to_i),
+        :diffs          => diffs,
+        :id             => commit[:ids].first,
+        :message        => commit[:info].first,
+        :parents        => parents.map { |p| { :id => p[:ids].first } },
+        :stats          => {
+          :additions => commit.key?(:impact) ? commit[:impact].first.to_i : 0,
+          :deletions => commit.key?(:impact) ? commit[:impact].last.to_i : 0
+        }
+      })
     end
+
+    HASH_CLASS[commits.map { |c| [c.id, c] }]
   end
 
   def self.commits_as_rashies(range)
-    commits = commits(range)
-
-    commits.map do |commit|
-      Hashie::Rash.new({
+    commits = commits(range).map do |commit|
+      Hashie::Mash.new({
         :author         => {
           :email => commit[:info][4],
           :login => commit[:info][5],
@@ -108,8 +123,11 @@ module Fixtures
         },
         :id             => commit[:ids].first,
         :message        => commit[:info].first,
+        :parents        => ([{ :id => commit[:parent][:ids].first }] rescue [])
       })
     end
+
+    HASH_CLASS[commits.map { |c| [c.id, c] }]
   end
 
 end

data/test/helper.rb

@@ -3,14 +3,21 @@
 #
 # Copyright (c) 2011, Sebastian Staudt
 
+require 'test/unit'
+
+require 'rubygems'
+require 'bundler'
+
+Bundler.setup
+
+require 'mocha'
+require 'shoulda'
+
 $LOAD_PATH.unshift File.join(File.dirname(__FILE__), '..', 'lib')
 $LOAD_PATH.unshift File.dirname(__FILE__)
 require 'metior'
 include Metior
 
-Bundler.setup
-Bundler.require :test
-
 # Extends TestCase functionality
 class Test::Unit::TestCase
 

data/test/{test_class_loading.rb → test_1st_class_loading.rb}

@@ -5,7 +5,7 @@
 
 require 'helper'
 
-class TestClassLoading < Test::Unit::TestCase
+class Test1stClassLoading < Test::Unit::TestCase
 
   context 'Metior' do
 

data/test/test_actor_colletion.rb

@@ -0,0 +1,78 @@
+# 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 'fixtures'
+require 'helper'
+require 'metior/collections/actor_collection'
+
+class TestActorCollection < Test::Unit::TestCase
+
+  context 'A collection of actors' do
+
+    setup do
+      repo = Metior::Git::Repository.new File.dirname(File.dirname(__FILE__))
+      @@grit_commits ||= Fixtures.commits_as_grit_commits(''..'master')
+      Grit::Repo.any_instance.stubs(:commits).returns @@grit_commits.values
+      @@grit_commits.each do |id, commit|
+        Grit::Repo.any_instance.stubs(:commit).with(id).returns commit
+      end
+      @authors = repo.authors
+    end
+
+    should 'be an instance of Collection' do
+      assert_kind_of Collection, @authors
+    end
+
+    should 'allow to get all the commits authored by those actors' do
+      commits = @authors.authored_commits
+      assert_instance_of CommitCollection, commits
+      assert_equal 460, commits.size
+      assert_equal '1b2fe77', commits.first.id
+      assert_equal '80f136f', commits.last.id
+    end
+
+    should 'allow to get the commits authored by a single of those actors' do
+      commits = @authors.authored_commits 'tom@mojombo.com'
+      assert_instance_of CommitCollection, commits
+      assert_equal 173, commits.size
+      assert_equal 'a3c5139', commits.first.id
+      assert_equal '634396b', commits.last.id
+    end
+
+    should 'allow to get all the commits committed by those actors' do
+      commits = @authors.committed_commits
+      assert_instance_of CommitCollection, commits
+      assert_equal 460, commits.size
+      assert_equal '1b2fe77', commits.first.id
+      assert_equal '80f136f', commits.last.id
+    end
+
+    should 'allow to get the commits committed by a single of those actors' do
+      commits = @authors.committed_commits 'technoweenie@gmail.com'
+      assert_instance_of CommitCollection, commits
+      assert_equal 55, commits.size
+      assert_equal 'ed1b3ae', commits.first.id
+      assert_equal 'c9cf68f', commits.last.id
+    end
+
+    should 'allow to get the most significant actors' do
+      authors = @authors.most_significant
+      assert_instance_of ActorCollection, authors
+      assert_equal 3, authors.size
+      assert_equal 'tom@mojombo.com', authors.first.id
+      assert_equal 'rsanheim@gmail.com', authors.last.id
+    end
+
+    should 'allow to get the top contributing actors' do
+      authors = @authors.top
+      assert_instance_of ActorCollection, authors
+      assert_equal 3, authors.size
+      assert_equal 'tom@mojombo.com', authors.first.id
+      assert_equal 'technoweenie@gmail.com', authors.last.id
+    end
+
+  end
+
+end

data/test/test_collection.rb

@@ -0,0 +1,61 @@
+# 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 'helper'
+require 'metior/collections/collection'
+
+class Dummy
+    def id
+      __id__
+    end
+end
+
+class TestCollections < Test::Unit::TestCase
+
+  context 'A collection of objects' do
+
+    setup do
+      @collection = Collection.new
+      @object1 = Dummy.new
+      @object2 = Dummy.new
+      @object3 = Dummy.new
+      @collection << @object1
+      @collection << @object2
+      @collection << @object3
+    end
+
+    should 'have a simple constructor' do
+      collection = Collection.new [@object1, @object2, @object3]
+      assert_equal [@object1.id, @object2.id, @object3.id], collection.keys
+      assert_equal @object1, collection[@object1.id]
+      assert_equal @object2, collection[@object2.id]
+      assert_equal @object3, collection[@object3.id]
+    end
+
+    should 'be a subclass of Hash' do
+      assert_kind_of Hash, @collection
+      assert_kind_of OrderedHash, @collection if RUBY_VERSION.match(/^1\.8/)
+    end
+
+    should 'have a working << operator' do
+      object = Dummy.new
+      @collection << object
+      assert_equal object, @collection[object.id]
+    end
+
+    should 'have a working #each method' do
+      objects = []
+      result = @collection.each { |obj| objects << obj }
+      assert_equal @collection, result
+      assert_equal [@object1, @object2, @object3], objects
+    end
+
+    should 'have a working #last method' do
+      assert_equal @object3, @collection.last
+    end
+
+  end
+
+end

data/test/test_commit_collection.rb

@@ -0,0 +1,139 @@
+# 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 'fixtures'
+require 'helper'
+require 'metior/collections/commit_collection'
+
+class TestCommitCollection < Test::Unit::TestCase
+
+  context 'A collection of commits' do
+
+    setup do
+      repo = Metior::Git::Repository.new File.dirname(File.dirname(__FILE__))
+      @@grit_commits ||= Fixtures.commits_as_grit_commits(''..'master')
+      Grit::Repo.any_instance.stubs(:commits).returns @@grit_commits.values
+      @@grit_commits.each do |id, commit|
+        Grit::Repo.any_instance.stubs(:commit).with(id).returns commit
+      end
+      @commits = repo.commits
+    end
+
+    should 'be an instance of Collection' do
+      assert_kind_of Collection, @commits
+    end
+
+    should 'allow to get all the authors of those commits' do
+      authors = @commits.authors
+      assert_instance_of ActorCollection, authors
+      assert_equal 37, authors.size
+      assert_equal 'rtomayko@gmail.com', authors.first.id
+      assert_equal 'tom@taco.(none)', authors.last.id
+    end
+
+    should 'allow to get the author of a single of those commits' do
+      authors = @commits.authors '4c592b4'
+      assert_instance_of ActorCollection, authors
+      assert_equal 1, authors.size
+      assert_equal 'bobbywilson0@gmail.com', authors.first.id
+    end
+
+    should 'allow to get all commits of a specific author' do
+      commits = @commits.by 'rtomayko@gmail.com'
+      assert_instance_of CommitCollection, commits
+      assert_equal 47, commits.size
+      assert_equal '1b2fe77', commits.first.id
+      assert_equal 'd731f85', commits.last.id
+    end
+
+    should 'allow to get all commits of a some authors' do
+      commits = @commits.by 'tom@mojombo.com', 'rtomayko@gmail.com'
+      assert_instance_of CommitCollection, commits
+      assert_equal 220, commits.size
+      assert_equal '1b2fe77', commits.first.id
+      assert_equal '634396b', commits.last.id
+    end
+
+    should 'allow to get all commits after a given date' do
+      commits = @commits.after '31-12-2010'
+      assert_instance_of CommitCollection, commits
+      assert_equal 29, commits.size
+      assert_equal '1b2fe77', commits.first.id
+      assert_equal '696761d', commits.last.id
+    end
+
+    should 'allow to get all commits before a given date' do
+      commits = @commits.before '31-12-2009'
+      assert_instance_of CommitCollection, commits
+      assert_equal 325, commits.size
+      assert_equal '2f1f63e', commits.first.id
+      assert_equal '634396b', commits.last.id
+    end
+
+    should 'allow to get all the committers of those commits' do
+      committers = @commits.committers
+      assert_instance_of ActorCollection, committers
+      assert_equal 29, committers.size
+      assert_equal 'rtomayko@gmail.com', committers.first.id
+      assert_equal 'tom@taco.(none)', committers.last.id
+    end
+
+    should 'allow to get the committers of a single of those commits' do
+      committers = @commits.committers '4c592b4'
+      assert_instance_of ActorCollection, committers
+      assert_equal 1, committers.size
+      assert_equal 'bobbywilson0@gmail.com', committers.first.id
+    end
+
+    should 'allow to get the activity statistics of those commits' do
+      activity = @commits.activity
+      assert_instance_of Hash, activity
+      assert_instance_of Hash, activity[:active_days]
+      assert_equal 157, activity[:active_days].size
+      assert activity[:active_days].keys.all? { |k| k.is_a? Time }
+      assert activity[:active_days].values.all? { |v| v.is_a? Fixnum }
+      assert_in_delta 2.9299, activity[:commits_per_active_day], 0.0001
+      assert_equal Time.at(1191997100), activity[:first_commit_date]
+      assert_equal Time.at(1306794294), activity[:last_commit_date]
+      assert_equal Time.parse('14-02-2009 00:00 +0000'), activity[:most_active_day]
+    end
+
+    should 'allow to get all commits changing specified files' do
+      commits = @commits.changing('lib/grit.rb')
+      assert_instance_of CommitCollection, commits
+      assert_equal 57, commits.size
+      assert_equal '18bfda9', commits.first.id
+      assert_equal '634396b', commits.last.id
+    end
+
+    should 'allow to get all commits with a minimum impact' do
+      commits = @commits.with_impact(100)
+      assert_instance_of CommitCollection, commits
+      assert_equal 55, commits.size
+      assert_equal 'ef2870b', commits.first.id
+      assert_equal 'a47fd41', commits.last.id
+    end
+
+    should 'allow to get the most significant commits' do
+      commits = @commits.most_significant(5)
+      assert_instance_of CommitCollection, commits
+      assert_equal 5, commits.size
+      assert_equal 'f91f3c8', commits.first.id
+      assert_equal '3c230a3', commits.last.id
+    end
+
+    should 'allow to get the line stats of the commits' do
+      assert_equal 25546, @commits.additions
+      assert_equal 6359, @commits.deletions
+    end
+
+    should 'lazy load the line stats of the commits' do
+      @commits.expects(:load_line_stats).once
+      @commits.additions
+    end
+
+  end
+
+end