git-commit-notifier 0.8.0 → 0.8.1

data/test/unit/test_commit_hook.rb

@@ -1,43 +0,0 @@
-require 'rubygems'
-require 'test/unit'
-require 'mocha'
-
-require File.dirname(__FILE__) + '/../../lib/commit_hook'
-require File.dirname(__FILE__) + '/../../lib/git'
-
-class CommitHookTest < Test::Unit::TestCase
-
-  def test_hook_ignore_merge
-    # 4 commits, one email for each of them, without merge
-    run_with_config('test/fixtures/git-notifier-ignore-merge.yml', 4)
-  end
-
-  def test_hook_with_merge
-    # 5 commits, one email for each of them, with merge mail
-    run_with_config('test/fixtures/git-notifier-with-merge.yml', 5)
-  end
-
-  def test_hook_group_email_by_push
-    # 1 commit for the push, all commits in the one message
-    run_with_config('test/fixtures/git-notifier-group-email-by-push.yml', 1)
-  end
-
-  def run_with_config(config, times)
-    expect_repository_access
-
-    emailer = mock('Emailer')
-    Emailer.expects(:new).times(times).returns(emailer)
-    emailer.expects(:send).times(times)
-    CommitHook.run config, REVISIONS.first, REVISIONS.last, 'refs/heads/master'
-  end
-
-  def expect_repository_access
-    path = File.dirname(__FILE__) + '/../fixtures/'
-    Git.expects(:log).with(REVISIONS.first, REVISIONS.last).returns(read_file(path + 'git_log'))
-    Git.expects(:mailing_list_address).returns('recipient@test.com')
-    REVISIONS.each do |rev|
-      Git.expects(:show).with(rev).returns(read_file(path + "git_show_#{rev}"))
-    end
-  end
-
-end

data/test/unit/test_diff_to_html.rb

@@ -1,160 +0,0 @@
-require 'rubygems'
-require 'mocha'
-require 'cgi'
-require 'hpricot'
-
-require File.dirname(__FILE__) + '/../test_helper'
-
-require File.dirname(__FILE__) + '/../../lib/diff_to_html'
-require File.dirname(__FILE__) + '/../../lib/git'
-
-class DiffToHtmlTest < Test::Unit::TestCase
-
-  def test_multiple_commits
-    path = File.dirname(__FILE__) + '/../fixtures/'
-    Git.expects(:log).with(REVISIONS.first, REVISIONS.last).returns(read_file(path + 'git_log'))
-    REVISIONS.each do |rev|
-      Git.expects(:show).with(rev).returns(read_file(path + 'git_show_' + rev))
-    end
-
-    diff = DiffToHtml.new
-    diff.diff_between_revisions REVISIONS.first, REVISIONS.last, 'testproject', 'master'
-    assert_equal 5, diff.result.size # one result for each of the commits
-
-    diff.result.each do |html|
-      assert !html.include?('@@') # diff correctly processed
-    end
-
-    # first commit
-    hp = Hpricot diff.result.first[:html_content]
-    assert_equal 2, (hp/"table").size # 8 files updated - one table for each of the files
-    (hp/"table/tr/").each do |td|
-      if td.inner_html == "require&nbsp;'iconv'"
-        # first added line in changeset a4629e707d80a5769f7a71ca6ed9471015e14dc9
-        assert_equal '', td.parent.search('td')[0].inner_text # left
-        assert_equal '2', td.parent.search('td')[1].inner_text # right
-        assert_equal "require&nbsp;'iconv'", td.parent.search('td')[2].inner_html # change
-      end
-    end
-
-    # second commit
-    hp = Hpricot diff.result[1][:html_content]
-    assert_equal 1, (hp/"table").size # 1 file updated
-
-    # third commit - dce6ade4cdc2833b53bd600ef10f9bce83c7102d
-    hp = Hpricot diff.result[2][:html_content]
-    assert_equal 6, (hp/"table").size # 6 files updated
-    assert_equal 'Added binary file railties/doc/guides/source/images/icons/callouts/11.png', (hp/"h2")[1].inner_text
-    assert_equal 'Deleted binary file railties/doc/guides/source/icons/up.png', (hp/"h2")[2].inner_text
-    assert_equal 'Deleted file railties/doc/guides/source/icons/README', (hp/"h2")[3].inner_text
-    assert_equal 'Added file railties/doc/guides/source/images/icons/README', (hp/"h2")[4].inner_text
-
-    # fourth commit - 51b986619d88f7ba98be7d271188785cbbb541a0
-    hp = Hpricot diff.result[3][:html_content]
-    assert_equal 3, (hp/"table").size # 3 files updated
-    (hp/"table/tr/").each do |td|
-      if td.inner_html =~ /create_btn/
-        cols = td.parent.search('td')
-        ['405', '408', ''].include? cols[0].inner_text # line 405 changed
-      end
-    end
-  end
-
-  def test_single_commit
-    path = File.dirname(__FILE__) + '/../fixtures/'
-    Git.expects(:log).never
-    Git.expects(:show).with(REVISIONS.first).returns(read_file(path + 'git_show_' + REVISIONS.first))
-
-    diff = DiffToHtml.new
-    diff.diff_between_revisions REVISIONS.first, REVISIONS.first, 'testproject', 'master'
-    assert_equal 1, diff.result.size # single result for a single commit
-    assert_equal 'Allow use of :path_prefix and :name_prefix outside of namespaced routes. [#1188 state:resolved]', diff.result.first[:commit_info][:message]
-    assert_equal 'Tom Stuart', diff.result.first[:commit_info][:author]
-    assert_equal 'tom@experthuman.com', diff.result.first[:commit_info][:email]
-
-    hp = Hpricot(diff.result.first[:html_content])
-    assert !diff.result.first[:html_content].include?('@@')
-    assert_equal 2, (hp/"table").size # 2 files updated
-    (hp/"table/tr/").each do |td|
-      if td.inner_html == "require&nbsp;'iconv'"
-        # first added line in changeset a4629e707d80a5769f7a71ca6ed9471015e14dc9
-        assert_equal '', td.parent.search('td')[0].inner_text # left
-        assert_equal '2', td.parent.search('td')[1].inner_text # right
-        assert_equal "require&nbsp;'iconv'", td.parent.search('td')[2].inner_html # change
-      end
-    end
-  end
-
-  def test_tokenize
-    s = "keys = I18n.send :normalize_translation_keys, locale, key, scope"
-    diff = DiffToHtml.new
-    tokens = diff.tokenize_string(s)
-
-    assert_equal ['keys', ' ', '=', ' ', 'I18n', '.', 'send',' ',':','normalize','_','translation','_','keys',',',' ','locale',',',' ',
-              'key',',',' ','scope'], tokens
-  end
-
-  def test_gitweb_operation_description
-    diff = DiffToHtml.new
-    diff.current_file_name = "file/to/test.yml"
-    assert_equal "<h2>Changed file file/to/test.yml</h2>\n", diff.operation_description
-
-    config = {
-      "link_files" => "gitweb",
-      "gitweb" => {
-        "path" => "http://developerserver/path_to_gitweb", 
-        "project" => "test.git"
-      }
-    }
-
-    diff = DiffToHtml.new(nil, config)
-    diff.current_file_name = "file/to/test.yml"
-    assert_equal "<h2>Changed file <a href='http://developerserver/path_to_gitweb?p=test.git;f=file/to/test.yml;hb=HEAD'>file/to/test.yml</a></h2>\n", diff.operation_description
-  end
-
-  def test_gitorious_operation_description
-    diff = DiffToHtml.new
-    diff.current_file_name = "file/to/test.yml"
-    assert_equal "<h2>Changed file file/to/test.yml</h2>\n", diff.operation_description
-
-    config = {
-      "link_files" => "gitorious",
-      "gitorious" => {
-        "path" => "http://example.com/gitorious", 
-        "project" => "tests",
-        "repository" => "test",
-      }
-    }
-
-    diff = DiffToHtml.new(nil, config)
-    diff.current_file_name = "file/to/test.yml"
-    assert_equal "<h2>Changed file <a href='http://example.com/gitorious/tests/test/blobs/HEAD/file/to/test.yml'>file/to/test.yml</a></h2>\n", diff.operation_description
-  end
-
-  def test_should_correctly_set_line_numbers_on_single_line_add_to_new_file
-    diff = DiffToHtml.new
-    diff.expects(:add_line_to_result).with() { |line, escape| line[:added] == 1 }
-    content = IO.read('test/fixtures/new_file_one_line.txt')
-    result = diff.diff_for_revision(content)
-  end
-
-  def test_should_correctly_set_line_numbers_on_single_line_add_to_existing_file
-    diff = DiffToHtml.new
-    diff.expects(:add_line_to_result).at_least_once
-    diff.expects(:add_line_to_result).at_least_once.with() { |line, escape| line[:added] == 5 && line[:removed] == nil }
-    content = IO.read('test/fixtures/existing_file_one_line.txt')
-    result = diff.diff_for_revision(content)
-  end
-
-  def test_message_map_should_correctly_rework_message
-    diff = DiffToHtml.new(nil, 'message_map' => {
-      '\brefs\s*\#(\d+)' => 'http://redmine.example.com/issues/show/\1'
-    }, 'message_integration' => {
-      'bugzilla' => 'http://bugzilla.example.com'
-    })
-    assert_equal(
-      '<a href="http://bugzilla.example.com/show_bug.cgi?id=15">BUG 15</a>. Draft design <a href="http://redmine.example.com/issues/show/5654">refs #5654</a>', 
-      diff.message_map('BUG 15. Draft design refs #5654')
-    )
-  end
-end

data/test/unit/test_result_processor.rb

@@ -1,95 +0,0 @@
-require 'test/unit'
-require 'jcode'
-require File.dirname(__FILE__) + '/../../lib/result_processor'
-require File.dirname(__FILE__) + '/../../lib/diff_to_html'
-
-class ResultProcessorTest < Test::Unit::TestCase
-  # button_to_remote 'create_btn'
-  # submit_to_remote 'create_btn'
-
-  def setup
-    create_test_input
-  end
-
-  def test_processor
-    proc = ResultProcessor.new(@diff)
-    removal, addition = proc.results
-    assert_equal 1, removal.size
-
-    assert removal[0].include?('&nbsp;&nbsp;<span class="rr">b</span>')
-    assert removal[0].include?('<span class="rr">ton</span>')
-
-    assert_equal 1, removal[0].split('<span>').size # one occurrence (beginning of string)
-    assert_equal 1, addition.size
-    assert addition[0].include?('&nbsp;&nbsp;<span class="aa">s</span>')
-    assert addition[0].include?('<span class="aa">bmi</span>')
-    assert_equal 1, addition[0].split('<span>').size
-  end
-
-  def test_processor_with_almost_no_common_part
-    @diff = [
-      { :action => :match, :token => ' '},
-      { :action => :match, :token => ' '},
-      { :action => :discard_a, :token => 'button'},
-      {:action => :discard_b, :token => 'submit'},
-      { :action => :match, :token => 'x'}]
-
-    proc = ResultProcessor.new(@diff)
-    removal, addition = proc.results
-
-    assert_equal 1, removal.size
-    assert_equal '&nbsp;&nbsp;buttonx', removal[0] # no highlight
-    assert_equal 1, addition.size
-    assert_equal '&nbsp;&nbsp;submitx', addition[0] # no highlight
-  end
-
-  def test_close_span_tag_when_having_difference_at_the_end
-    diff = []
-    s1 = "  submit_to_remote 'create_btn', 'Create', :url => { :action => 'cre"
-    s2 = "  submit_to_remote 'create_btn', 'Create', :url => { :action => 'sub"
-
-    s1[0..s1.size-6].each_char do |c|
-      diff << { :action => :match, :token => c}
-    end
-    diff << { :action => :discard_a, :token => 'c'}
-    diff << { :action => :discard_a, :token => 'r'}
-    diff << { :action => :discard_a, :token => 'e'}
-    diff << { :action => :discard_b, :token => 's'}
-    diff << { :action => :discard_b, :token => 'u'}
-    diff << { :action => :discard_b, :token => 'b'}
-
-    proc = ResultProcessor.new(diff)
-    removal, addition = proc.results
-    assert_equal 1, removal.size
-    assert removal[0].include?('action&nbsp;=&gt;<span class="rr">cre</span>')
-
-    assert_equal 1, addition.size
-    assert addition[0].include?('action&nbsp;=&gt;<span class="aa">sub</span>')
-  end
-
-  def create_test_input
-    @diff = []
-    s1 = "  button_to_remote 'create_btn', 'Create', :url => { :action => 'create' }"
-    s2 = "  submit_to_remote 'create_btn', 'Create', :url => { :action => 'create' }"
-
-    @diff = [
-     [:match,    ' '],
-     [:match,    ' '],
-     [:discard_a,'b'],
-     [:discard_b,'s'],
-     [:match,    'u'],
-     [:discard_b,'b'],
-     [:discard_b,'m'],
-     [:discard_b,'i'],
-     [:match,    't'],
-     [:discard_a,'t'],
-     [:discard_a,'o'],
-     [:discard_a,'n']]
-    @diff = @diff.collect { |d| { :action => d.first, :token => d.last}}
-
-    s1[@diff.size..-1].each_char do |c|
-      @diff << { :action => :match, :token => c}
-    end
-  end
-
-end