git_stats 1.0.0

data/spec/git_data/generator_spec.rb

@@ -0,0 +1,46 @@
+# -*- encoding : utf-8 -*-
+require 'spec_helper'
+
+describe GitStats::Generator do
+  let(:repo_path) { 'repo_path' }
+  let(:out_path) { 'out_path' }
+  let(:generator) { GitStats::Generator.new(repo_path, out_path) }
+
+  before { Dir.stub!(:exists? => true) }
+
+  it 'should raise exception if given repo path is not a git repository' do
+    Dir.should_receive(:exists?).with("#{repo_path}/.git").and_return(false)
+    expect { generator }.to raise_error(ArgumentError)
+  end
+
+  it "should raise exception if given out directory doesn't exist" do
+    Dir.should_receive(:exists?).with(out_path).and_return(false)
+    expect { generator }.to raise_error(ArgumentError)
+  end
+
+  it 'should pass command observer to repo' do
+    repo = double('repo')
+    GitStats::GitData::Repo.should_receive(:new).with(path: repo_path).and_return(repo)
+
+    generator = GitStats::Generator.new(repo_path, out_path)
+
+    observer = double('observer')
+    repo.should_receive(:add_command_observer).with(observer)
+
+    generator.add_command_observer observer
+  end
+
+  it 'should render all templates with view data for this repo' do
+    repo = double('repo')
+    GitStats::GitData::Repo.should_receive(:new).with(path: repo_path).and_return(repo)
+
+    view_data = double('view_data')
+    GitStats::StatsView::ViewData.should_receive(:new).with(repo).and_return(view_data)
+
+    view = double('view')
+    GitStats::StatsView::View.should_receive(:new).with(view_data, out_path).and_return(view)
+    view.should_receive(:render_all)
+
+    generator.render_all
+  end
+end

data/spec/git_data/repo_spec.rb

@@ -0,0 +1,46 @@
+# -*- encoding : utf-8 -*-
+require 'spec_helper'
+
+describe GitStats::GitData::Repo do
+  let(:repo) { build(:repo) }
+
+  describe 'git output parsing' do
+    context 'invoking authors command' do
+      before do
+        repo.should_receive(:run).with('git shortlog -se HEAD').and_return("   156	John Doe <john.doe@gmail.com>
+    53	Joe Doe <joe.doe@gmail.com>
+")
+      end
+      it 'should parse git shortlog output to authors hash' do
+        repo.authors.should == [
+            build(:author, repo: repo, name: "John Doe", email: "john.doe@gmail.com"),
+            build(:author, repo: repo, name: "Joe Doe", email: "joe.doe@gmail.com")
+        ]
+      end
+
+      it 'should parse git revlist output to date sorted commits array' do
+        repo.should_receive(:run).with("git rev-list --pretty=format:'%h|%at|%ai|%aE' HEAD | grep -v commit").and_return(
+            "e4412c3|1348603824|2012-09-25 22:10:24 +0200|john.doe@gmail.com
+ce34874|1347482927|2012-09-12 22:48:47 +0200|joe.doe@gmail.com
+5eab339|1345835073|2012-08-24 21:04:33 +0200|john.doe@gmail.com
+")
+
+        repo.commits.should == [
+            GitStats::GitData::Commit.new(
+                repo: repo, sha: "5eab339", stamp: "1345835073", date: DateTime.parse("2012-08-24 21:04:33 +0200"),
+                author: repo.authors.by_email("john.doe@gmail.com")),
+            GitStats::GitData::Commit.new(
+                repo: repo, sha: "ce34874", stamp: "1347482927", date: DateTime.parse("2012-09-12 22:48:47 +0200"),
+                author: repo.authors.by_email("joe.doe@gmail.com")),
+            GitStats::GitData::Commit.new(
+                repo: repo, sha: "e4412c3", stamp: "1348603824", date: DateTime.parse("2012-09-25 22:10:24 +0200"),
+                author: repo.authors.by_email("john.doe@gmail.com"))
+        ]
+      end
+    end
+    it 'should parse git rev-parse command to project version' do
+      repo.should_receive(:run).with('git rev-parse --short HEAD').and_return('xyz')
+      repo.project_version.should == 'xyz'
+    end
+  end
+end

data/spec/git_data/short_stat_spec.rb

@@ -0,0 +1,28 @@
+# -*- encoding : utf-8 -*-
+require 'spec_helper'
+
+describe GitStats::GitData::ShortStat do
+  let(:commit) { build(:commit, sha: 'abc') }
+
+  describe 'git output parsing' do
+    context 'parsing git show output' do
+      [
+          {content: '2 files changed, 17 insertions(+), 3 deletions(-)', expect: [2, 17, 3]},
+          {content: '1 file changed, 1 insertion(+), 1 deletion(-)', expect: [1, 1, 1]},
+          {content: '2 files changed, 3 deletions(-)', expect: [2, 0, 3]},
+          {content: '2 files changed, 5 insertions(+)', expect: [2, 5, 0]},
+          {content: '', expect: [0, 0, 0]},
+      ].each do |test|
+        it "#{test[:content]} parsing" do
+          commit.repo.should_receive(:run).with("git show --shortstat --oneline abc").and_return("abc some commit\n#{test[:content]}")
+
+
+          commit.short_stat.should be_a(GitStats::GitData::ShortStat)
+          commit.short_stat.files_changed.should == test[:expect][0]
+          commit.short_stat.insertions.should == test[:expect][1]
+          commit.short_stat.deletions.should == test[:expect][2]
+        end
+      end
+    end
+  end
+end

data/spec/hash_extension_spec.rb

@@ -0,0 +1,26 @@
+# -*- encoding : utf-8 -*-
+require 'spec_helper'
+
+describe Hash do
+  context 'to_key_indexed_array' do
+    it 'should convert hash to array using keys as indexes' do
+      hash = {1 => 'x', 2 => 1, 5 => 'a'}
+      hash.to_key_indexed_array.should == [nil, 'x', 1, nil, nil, 'a']
+    end
+
+    it 'should throw exception if not all of the keys are numbers' do
+      hash = {1 => 'x', 'b' => 2}
+      expect { hash.to_key_indexed_array }.to raise_error(ArgumentError)
+    end
+
+    context 'should take optional min_size and default parameters' do
+      let(:hash) { {1 => 'x', 2 => 1, 5 => 'a'} }
+      it 'should fill array with defaults up to min_size' do
+        hash.to_key_indexed_array(min_size: 8, default: 0).should == [0, 'x', 1, 0, 0, 'a', 0, 0]
+      end
+      it 'should use default value where key is not in hash' do
+        hash.to_key_indexed_array(min_size: 2, default: 0).should == [0, 'x', 1, 0, 0, 'a']
+      end
+    end
+  end
+end

data/spec/integration/activity_spec.rb

@@ -0,0 +1,33 @@
+# -*- encoding : utf-8 -*-
+require 'integration/shared'
+
+describe GitStats::GitData::Activity do
+  include_context "shared"
+
+  let(:activity) { repo.activity }
+
+  it 'should count commits by hour' do
+    activity.by_hour.should == {10 => 4, 12 => 3, 13 => 1, 15 => 1, 17 => 1}
+  end
+
+  it 'should count commits by day of week' do
+    activity.by_wday.should == {0 => 3, 3 => 1, 5 => 5, 6 => 1}
+  end
+
+  it 'should count commits by day of week and hour' do
+    activity.by_wday_hour.should == {0 => {12 => 2, 13 => 1}, 3 => {15 => 1}, 5 => {10 => 4, 17 => 1}, 6 => {12 => 1}}
+  end
+
+  it 'should count commits by month' do
+    activity.by_month.should == {10 => 10}
+  end
+
+  it 'should count commits by year' do
+    activity.by_year.should == {2012 => 10}
+  end
+
+  it 'should count commits by year and month' do
+    activity.by_year_month.should == {2012 => {10 => 10}}
+  end
+
+end

data/spec/integration/author_spec.rb

@@ -0,0 +1,67 @@
+# -*- encoding : utf-8 -*-
+require 'integration/shared'
+
+describe GitStats::GitData::Activity do
+  include_context "shared"
+
+  let(:tg) { repo.authors.by_email('tomasz.gieniusz@gmail.com') }
+  let(:jd) { repo.authors.by_email('john.doe@gmail.com') }
+
+  it 'should filter commits to author' do
+    tg.commits.map(&:sha).should =~ %w(b3b4f81 d60b5ec ab47ef8 2c11f5e c87ecf9 b621a5d 4e7d0e9 872955c)
+    jd.commits.map(&:sha).should =~ %w(fd66657 81e8bef)
+  end
+
+  context 'activity' do
+    it 'should count commits by hour' do
+      tg.activity.by_hour.should == {10 => 4, 12 => 2, 13 => 1, 17 => 1}
+      jd.activity.by_hour.should == {12 => 1, 15 => 1}
+    end
+
+    it 'should count commits by day of week' do
+      tg.activity.by_wday.should == {0 => 2, 5 => 5, 6 => 1}
+      jd.activity.by_wday.should == {0 => 1, 3 => 1}
+    end
+
+    it 'should count commits by day of week and hour' do
+      tg.activity.by_wday_hour.should == {0 => {12 => 1, 13 => 1}, 5 => {10 => 4, 17 => 1}, 6 => {12 => 1}}
+      jd.activity.by_wday_hour.should == {0 => {12 => 1}, 3 => {15 => 1}}
+    end
+
+    it 'should count commits by month' do
+      tg.activity.by_month.should == {10 => 8}
+      jd.activity.by_month.should == {10 => 2}
+    end
+
+    it 'should count commits by year' do
+      tg.activity.by_year.should == {2012 => 8}
+      jd.activity.by_year.should == {2012 => 2}
+    end
+
+    it 'should count commits by year and month' do
+      tg.activity.by_year_month.should == {2012 => {10 => 8}}
+      jd.activity.by_year_month.should == {2012 => {10 => 2}}
+    end
+
+    it 'should count commits_sum_by_date' do
+      tg.commits_sum_by_date.map { |d, s| d }.should == tg_commit_dates
+      tg.commits_sum_by_date.map { |d, s| s }.should == [1, 2, 3, 4, 5, 6, 7, 8]
+      jd.commits_sum_by_date.map { |d, s| d }.should == jd_commit_dates
+      jd.commits_sum_by_date.map { |d, s| s }.should == [1, 2]
+    end
+
+    it 'should count insertions_by_date' do
+      tg.insertions_by_date.map { |d, s| d }.should == tg_commit_dates
+      tg.insertions_by_date.map { |d, s| s }.should == [4, 9, 14, 15, 20, 1020, 1021, 1021]
+      jd.insertions_by_date.map { |d, s| d }.should == jd_commit_dates
+      jd.insertions_by_date.map { |d, s| s }.should == [3, 103]
+    end
+
+    it 'should count deletions_by_date' do
+      tg.deletions_by_date.map { |d, s| d }.should == tg_commit_dates
+      tg.deletions_by_date.map { |d, s| s }.should == [0, 0, 4, 4, 9, 9, 10, 10]
+      jd.deletions_by_date.map { |d, s| d }.should == jd_commit_dates
+      jd.deletions_by_date.map { |d, s| s }.should == [0, 0]
+    end
+  end
+end

data/spec/integration/file_spec.rb

@@ -0,0 +1,31 @@
+# -*- encoding : utf-8 -*-
+require 'integration/shared'
+
+describe GitStats::GitData::Repo do
+  include_context "shared"
+
+  it 'should gather all files in repo' do
+    repo.files.map(&:filename).should =~ %w(long_second.haml  long.txt  second.txt  test2.rb  test.rb  test.txt)
+  end
+
+  it 'should retrieve correct file content for old file' do
+    repo.commits.by_sha('c87ecf9').files.by_filename('test.txt').content.should == "bb
+
+
+
+test
+"
+  end
+
+  it 'should retrieve correct file content for the newest file' do
+    file = repo.files.by_filename('test.txt')
+    file.content.should == "bb
+
+testtest
+
+test
+"
+  end
+
+
+end

data/spec/integration/repo_spec.rb

@@ -0,0 +1,68 @@
+# -*- encoding : utf-8 -*-
+require 'integration/shared'
+
+describe GitStats::GitData::Repo do
+  include_context "shared"
+
+  it 'should gather all authors' do
+    repo.authors.should =~ expected_authors
+  end
+
+  it 'should calculate correct commits period' do
+    repo.commits_period.should == [DateTime.parse('2012-10-19 10:44:34 +0200'), DateTime.parse('2012-10-26 17:05:25 +0200')]
+  end
+
+  it 'should gather all commits sorted by date' do
+    repo.commits.map(&:sha).should =~ %w(b3b4f81 d60b5ec ab47ef8 2c11f5e c87ecf9 b621a5d fd66657 81e8bef 4e7d0e9 872955c)
+  end
+
+  it 'should return project name from dir' do
+    repo.project_name.should == 'test_repo'
+  end
+
+  it 'should return project version as last commit hash' do
+    repo.project_version.should == '872955c'
+  end
+
+  it 'should count files in repo' do
+    repo.files_count.should == 6
+  end
+
+  it 'should count files by date' do
+    repo.files_count_by_date.keys.should == commit_dates
+    repo.files_count_by_date.values.should == [1, 2, 2, 3, 3, 4, 5, 5, 6, 6]
+  end
+
+  it 'should count lines by date' do
+    repo.lines_count_by_date.keys.should == commit_dates
+    repo.files_count_by_date.values.should == [1, 2, 2, 3, 3, 4, 5, 5, 6, 6]
+  end
+
+  it 'should count all lines in repo' do
+    repo.lines_count.should == 1114
+  end
+
+  it 'should count files by extension in repo' do
+    repo.files_by_extension_count.should == {'.haml' => 1, '.txt' => 3, '.rb' => 2}
+  end
+
+  it 'should count lines by extension in repo' do
+    repo.lines_by_extension.should == {'.haml' => 100, '.txt' => 1008, '.rb' => 6}
+  end
+
+  it 'should count commits_count_by_author' do
+    repo.commits_count_by_author.keys.should == expected_authors
+    repo.commits_count_by_author.values.should == [8, 2]
+  end
+
+  it 'should count lines_added_by_author' do
+    repo.insertions_by_author.keys.should == expected_authors
+    repo.insertions_by_author.values.should == [1021, 103]
+  end
+
+  it 'should count lines_deleted_by_author' do
+    repo.deletions_by_author.keys.should == expected_authors
+    repo.deletions_by_author.values.should == [10, 0]
+  end
+
+end

data/spec/integration/shared.rb

@@ -0,0 +1,37 @@
+# -*- encoding : utf-8 -*-
+require 'spec_helper'
+
+shared_context "shared" do
+  let(:repo) { build(:test_repo, last_commit_sha: '872955c') }
+  let(:commit_dates) { [
+      DateTime.parse('2012-10-19 10:44:34 +0200'),
+      DateTime.parse('2012-10-19 10:46:10 +0200'),
+      DateTime.parse('2012-10-19 10:46:56 +0200'),
+      DateTime.parse('2012-10-19 10:47:35 +0200'),
+      DateTime.parse('2012-10-20 12:49:02 +0200'),
+      DateTime.parse('2012-10-21 12:49:02 +0200'),
+      DateTime.parse('2012-10-21 12:54:02 +0200'),
+      DateTime.parse('2012-10-21 13:20:00 +0200'),
+      DateTime.parse('2012-10-24 15:49:02 +0200'),
+      DateTime.parse('2012-10-26 17:05:25 +0200'),
+  ] }
+  let(:tg_commit_dates) { [
+      DateTime.parse('2012-10-19 10:44:34 +0200'),
+      DateTime.parse('2012-10-19 10:46:10 +0200'),
+      DateTime.parse('2012-10-19 10:46:56 +0200'),
+      DateTime.parse('2012-10-19 10:47:35 +0200'),
+      DateTime.parse('2012-10-20 12:49:02 +0200'),
+      DateTime.parse('2012-10-21 12:49:02 +0200'),
+      DateTime.parse('2012-10-21 13:20:00 +0200'),
+      DateTime.parse('2012-10-26 17:05:25 +0200'),
+  ] }
+  let(:jd_commit_dates) { [
+      DateTime.parse('2012-10-21 12:54:02 +0200'),
+      DateTime.parse('2012-10-24 15:49:02 +0200'),
+  ] }
+
+  let(:expected_authors) { [
+      build(:author, repo: repo, name: "Tomasz Gieniusz", email: "tomasz.gieniusz@gmail.com"),
+      build(:author, repo: repo, name: "John Doe", email: "john.doe@gmail.com"),
+  ] }
+end

data/spec/spec_helper.rb

@@ -0,0 +1,13 @@
+# -*- encoding : utf-8 -*-
+require 'simplecov'
+SimpleCov.start
+
+require 'git_stats'
+
+require 'factory_girl'
+FactoryGirl.find_definitions
+
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+end
+

data/templates/activity/_activity.haml

@@ -0,0 +1,101 @@
+.tabbable.tabs-left
+  %ul.nav.nav-tabs
+    %li{class: page == :activity_by_date ? "active" : "" }
+      %a{href: "by_date.html"}= :activity_by_date.t
+    %li{class: page == :hour_of_day ? "active" : "" }
+      %a{href: "hour_of_day.html"}= :hour_of_day.t
+    %li{class: page == :day_of_week ? "active" : "" }
+      %a{href: "day_of_week.html"}= :day_of_week.t
+    %li{class: page == :hour_of_week ? "active" : "" }
+      %a{href: "hour_of_week.html"}= :hour_of_week.t
+    %li{class: page == :month_of_year ? "active" : "" }
+      %a{href: "month_of_year.html"}= :month_of_year.t
+    %li{class: page == :year ? "active" : "" }
+      %a{href: "year.html"}= :year.t
+    %li{class: page == :year_month ? "active" : "" }
+      %a{href: "year_month.html"}= :year_month.t
+
+  .tab-content
+    .tab-pane.active
+      .page-header.pagination-centered
+        %h1= page.t
+        %h3= "#{author.respond_to?(:name) ? author.name : ""} #{author.respond_to?(:email) ? "<#{author.email}>" : ""}"
+    - if page == :activity_by_date
+      = high_stock("charts.activity_by_date", charts.activity_by_date(author))
+
+    - elsif page == :hour_of_day
+      %table{:class => "table table-bordered table-condensed"}
+        %tr
+          %th= :hour.t
+          - (0..23).each do |h|
+            %th= h
+        %tr
+          %th= :commits.t
+          - author.activity.by_hour_array.each do |commits|
+            %td= commits
+        %tr
+          %th= :percentage.t
+          - author.activity.by_hour_array.each do |commits|
+            %td= (commits * 100.0 / author.activity.by_hour_array.sum).round(1)
+      = high_chart("charts.activity_by_hour", charts.activity_by_hour(author))
+
+    - elsif page == :day_of_week
+      %table{:class => "table table-bordered table-condensed"}
+        %tr
+          %th= :day.t
+          - I18n.t('date.abbr_day_names').each do |d|
+            %th= d
+        %tr
+          %th= :commits.t
+          - author.activity.by_wday_array.each do |commits|
+            %td= commits
+        %tr
+          %th= :percentage.t
+          - author.activity.by_wday_array.each do |commits|
+            %td= (commits * 100.0 / author.activity.by_wday_array.sum).round(1)
+      = high_chart("charts.activity_by_wday", charts.activity_by_wday(author))
+
+    - elsif page == :hour_of_week
+      %table{:class => "table table-bordered table-condensed"}
+        %tr
+          %th
+          - (0..23).each do |h|
+            %th= h
+        - max = author.activity.by_wday_hour.values.map {|h| h.values.empty? ? 0 : h.values.max }.max
+        - (0..6).each do |day|
+          %tr
+            %th= I18n.t('date.abbr_day_names')[day]
+            - (0..23).each do |hour|
+              - color = max ? "%02x" % (255 - author.activity.by_wday_hour[day][hour] * 100 / max) : "FF"
+              %td{style: "background-color: ##{color}#{color}#{color};"}= author.activity.by_wday_hour[day][hour]
+
+    - elsif page == :month_of_year
+      %table{:class => "table table-bordered table-condensed"}
+        %tr
+          %th= :month.t
+          - I18n.t('date.abbr_month_names')[1..-1].each do |m|
+            %th= m
+        %tr
+          %th= :commits.t
+          - author.activity.by_month_array.each do |commits|
+            %td= commits
+        %tr
+          %th= :percentage.t
+          - author.activity.by_month_array.each do |commits|
+            %td= (commits * 100.0 / author.activity.by_month_array.sum).round(1)
+      = high_chart("charts.activity_by_month", charts.activity_by_month(author))
+
+    - elsif page == :year
+      = high_chart("charts.activity_by_year", charts.activity_by_year(author))
+
+    - elsif page == :year_month
+      %table{:class => "table table-bordered table-condensed"}
+        %tr
+          %th
+          - I18n.t('date.abbr_month_names')[1..-1].each do |h|
+            %th= h
+        - author.activity.by_year_month.each do |year, months|
+          %tr
+            %th= year
+            - (1..12).each do |month|
+              %td= months[month]