git-commit-notifier 0.1.0

data/test/fixtures/git_show_dce6ade4cdc2833b53bd600ef10f9bce83c7102d

@@ -0,0 +1,83 @@
+Commit dce6ade4cdc2833b53bd600ef10f9bce83c7102d
+Author: Andrew Kaspick <andrew@redlinesoftware.com>
+Date:   Tue Sep 30 14:15:36 2008 -0500
+
+    Ensure select_tag#name attribute uses [] when :multiple is true. [#1146 state:resolved]
+
+    Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
+
+diff --git a/actionpack/lib/action_view/helpers/form_tag_helper.rb b/actionpack/lib/action_view/helpers/form_tag_helper.rb
+index 208bf91..7492348 100644
+--- a/actionpack/lib/action_view/helpers/form_tag_helper.rb
++++ b/actionpack/lib/action_view/helpers/form_tag_helper.rb
+@@ -62,7 +62,7 @@ module ActionView
+       #   #    <option>3</option><option>4</option></select>
+       #
+       #   select_tag "colors", "<option>Red</option><option>Green</option><option>Blue</option>", :multiple => true
+-      #   # => <select id="colors" multiple="multiple" name="colors"><option>Red</option>
++      #   # => <select id="colors" multiple="multiple" name="colors[]"><option>Red</option>
+       #   #    <option>Green</option><option>Blue</option></select>
+       #
+       #   select_tag "locations", "<option>Home</option><option selected="selected">Work</option><option>Out</option>"
+@@ -70,14 +70,15 @@ module ActionView
+       #   #    <option>Out</option></select>
+       #
+       #   select_tag "access", "<option>Read</option><option>Write</option>", :multiple => true, :class => 'form_input'
+-      #   # => <select class="form_input" id="access" multiple="multiple" name="access"><option>Read</option>
++      #   # => <select class="form_input" id="access" multiple="multiple" name="access[]"><option>Read</option>
+       #   #    <option>Write</option></select>
+       #
+       #   select_tag "destination", "<option>NYC</option><option>Paris</option><option>Rome</option>", :disabled => true
+       #   # => <select disabled="disabled" id="destination" name="destination"><option>NYC</option>
+       #   #    <option>Paris</option><option>Rome</option></select>
+       def select_tag(name, option_tags = nil, options = {})
+-        content_tag :select, option_tags, { "name" => name, "id" => name }.update(options.stringify_keys)
++        html_name = (options[:multiple] == true && !name.to_s.ends_with?("[]")) ? "#{name}[]" : name
++        content_tag :select, option_tags, { "name" => html_name, "id" => name }.update(options.stringify_keys)
+       end
+
+       # Creates a standard text field; use these text fields to input smaller chunks of text like a username
+diff --git a/railties/doc/guides/source/images/icons/callouts/11.png b/railties/doc/guides/source/images/icons/callouts/11.png
+new file mode 100644
+index 0000000..ce47dac
+Binary files /dev/null and b/railties/doc/guides/source/images/icons/callouts/11.png differ
+diff --git a/railties/doc/guides/source/icons/up.png b/railties/doc/guides/source/icons/up.png
+deleted file mode 100644
+index 2db1ce6..0000000
+Binary files a/railties/doc/guides/source/icons/up.png and /dev/null differ
+diff --git a/railties/doc/guides/source/icons/README b/railties/doc/guides/source/icons/README
+deleted file mode 100644
+index f12b2a7..0000000
+--- a/railties/doc/guides/source/icons/README
++++ /dev/null
+@@ -1,5 +0,0 @@
+-Replaced the plain DocBook XSL admonition icons with Jimmac's DocBook
+-icons (http://jimmac.musichall.cz/ikony.php3). I dropped transparency
+-from the Jimmac icons to get round MS IE and FOP PNG incompatibilies.
+-
+-Stuart Rackham
+diff --git a/railties/doc/guides/source/images/icons/README b/railties/doc/guides/source/images/icons/README
+new file mode 100644
+index 0000000..f12b2a7
+--- /dev/null
++++ b/railties/doc/guides/source/images/icons/README
+@@ -0,0 +1,5 @@
++Replaced the plain DocBook XSL admonition icons with Jimmac's DocBook
++icons (http://jimmac.musichall.cz/ikony.php3). I dropped transparency
++from the Jimmac icons to get round MS IE and FOP PNG incompatibilies.
++
++Stuart Rackham
+diff --git a/actionpack/test/template/form_tag_helper_test.rb b/actionpack/test/template/form_tag_helper_test.rb
+index ad8baef..1849a61 100644
+--- a/actionpack/test/template/form_tag_helper_test.rb
++++ b/actionpack/test/template/form_tag_helper_test.rb
+@@ -211,7 +211,8 @@ class FormTagHelperTest < ActionView::TestCase
+   def test_boolean_optios
+     assert_dom_equal %(<input checked="checked" disabled="disabled" id="admin" name="admin" readonly="readonly" type="checkbox" value="1" />), check_box_tag("admin", 1, true, 'disabled' => true, :readonly => "yes")
+     assert_dom_equal %(<input checked="checked" id="admin" name="admin" type="checkbox" value="1" />), check_box_tag("admin", 1, true, :disabled => false, :readonly => nil)
+-    assert_dom_equal %(<select id="people" multiple="multiple" name="people"><option>david</option></select>), select_tag("people", "<option>david</option>", :multiple => true)
++    assert_dom_equal %(<select id="people" multiple="multiple" name="people[]"><option>david</option></select>), select_tag("people", "<option>david</option>", :multiple => true)
++    assert_dom_equal %(<select id="people[]" multiple="multiple" name="people[]"><option>david</option></select>), select_tag("people[]", "<option>david</option>", :multiple => true)
+     assert_dom_equal %(<select id="people" name="people"><option>david</option></select>), select_tag("people", "<option>david</option>", :multiple => nil)
+   end
+

data/test/fixtures/git_show_e28ad77bba0574241e6eb64dfd0c1291b221effe

@@ -0,0 +1,76 @@
+commit e28ad77bba0574241e6eb64dfd0c1291b221effe
+Author: Tom Stuart <tom@experthuman.com>
+Date:   Wed Oct 8 09:31:00 2008 +0100
+
+    Allow use of :path_prefix and :name_prefix outside of namespaced routes. [#1188 state:resolved]
+
+    Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
+
+diff --git a/actionpack/lib/action_controller/routing/builder.rb b/actionpack/lib/action_controller/routing/builder.rb
+index 5704d9d..7b888fa 100644
+--- a/actionpack/lib/action_controller/routing/builder.rb
++++ b/actionpack/lib/action_controller/routing/builder.rb
+@@ -60,12 +60,10 @@ module ActionController
+       # segments are passed alongside in order to distinguish between default values
+       # and requirements.
+       def divide_route_options(segments, options)
+-        options = options.dup
++        options = options.except(:path_prefix, :name_prefix)
+
+         if options[:namespace]
+           options[:controller] = "#{options.delete(:namespace).sub(/\/$/, '')}/#{options[:controller]}"
+-          options.delete(:path_prefix)
+-          options.delete(:name_prefix)
+         end
+
+         requirements = (options.delete(:requirements) || {}).dup
+@@ -68,7 +73,9 @@ class Client < ActiveRecord::Base
+   end
+
+   def removable?
+-    self.projects.find(:first, :select => 'id').nil? && self.invoices.find(:first, :select => 'id').nil?
++    self.projects.find(:first, :select => 'id').nil? &&
++      self.invoices.find(:first, :select => 'id').nil? &&
++      self.recurring_invoices.find(:first, :select => 'id').nil?
+   end
+
+   # some comment
+diff --git a/actionpack/test/controller/routing_test.rb b/actionpack/test/controller/routing_test.rb
+index 1eb26a7..9699a04 100644
+--- a/actionpack/test/controller/routing_test.rb
++++ b/actionpack/test/controller/routing_test.rb
+@@ -924,6 +924,20 @@ uses_mocha 'LegacyRouteSet, Route, RouteSet and RouteLoading' do
+
+     end
+
++    def test_named_route_with_name_prefix
++      rs.add_named_route :page, 'page', :controller => 'content', :action => 'show_page', :name_prefix => 'my_'
++      x = setup_for_named_route
++      assert_equal("http://named.route.test/page",
++                   x.send(:my_page_url))
++    end
++
++    def test_named_route_with_path_prefix
++      rs.add_named_route :page, 'page', :controller => 'content', :action => 'show_page', :path_prefix => 'my'
++      x = setup_for_named_route
++      assert_equal("http://named.route.test/my/page",
++                   x.send(:page_url))
++    end
++
+     def test_named_route_with_nested_controller
+       rs.add_named_route :users, 'admin/user', :controller => 'admin/user', :action => 'index'
+       x = setup_for_named_route
+@@ -2147,6 +2161,13 @@ uses_mocha 'LegacyRouteSet, Route, RouteSet and RouteLoading' do
+       assert_equal [:x], set.extra_keys(args)
+     end
+
++    def test_generate_with_path_prefix
++      set.draw { |map| map.connect ':controller/:action/:id', :path_prefix => 'my' }
++
++      args = { :controller => "foo", :action => "bar", :id => "7", :x => "y" }
++      assert_equal "/my/foo/bar/7?x=y", set.generate(args)
++    end
++
+     def test_named_routes_are_never_relative_to_modules
+       set.draw do |map|
+         map.connect "/connection/manage/:action", :controller => 'connection/manage'

data/test/test_helper.rb

@@ -0,0 +1,21 @@
+require 'test/unit'
+
+unless defined? REVISIONS
+  REVISIONS = ['e28ad77bba0574241e6eb64dfd0c1291b221effe', # 2 files updated
+             'a4629e707d80a5769f7a71ca6ed9471015e14dc9', # 1 file updated
+             'dce6ade4cdc2833b53bd600ef10f9bce83c7102d', # 6 files updated
+             '51b986619d88f7ba98be7d271188785cbbb541a0'] # 3 files updated
+
+end
+
+class Test::Unit::TestCase
+
+  def read_file(name)
+    out = ''
+    File.open(name).each { |line|
+      out += line
+    }
+    out
+  end
+
+end

data/test/unit/test_commit_hook.rb

@@ -0,0 +1,22 @@
+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
+    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
+    emailer = mock('Emailer')
+    Emailer.expects(:new).times(4).returns(emailer) # 4 commit, one email for each of them
+    emailer.expects(:send).times(4)
+    CommitHook.run 'config/email.yml.sample', REVISIONS.first, REVISIONS.last, 'refs/heads/master'
+  end
+end

data/test/unit/test_diff_to_html.rb

@@ -0,0 +1,97 @@
+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 4, 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
+
+end

data/test/unit/test_result_processor.rb

@@ -0,0 +1,95 @@
+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

metadata

@@ -0,0 +1,125 @@
+--- !ruby/object:Gem::Specification 
+name: git-commit-notifier
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 1
+  - 0
+  version: 0.1.0
+platform: ruby
+authors: 
+- Bodo Tasche
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2010-03-24 00:00:00 +01:00
+default_executable: git-commit-notifier
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: diff-lcs
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: mocha
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  name: hpricot
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id003
+description: This git commit notifier sends html mails with nice diffs for every changed file.
+email: bodo@wannawork.de
+executables: 
+- git-commit-notifier
+extensions: []
+
+extra_rdoc_files: 
+- LICENSE
+- README.textile
+files: 
+- .document
+- .gitignore
+- LICENSE
+- README.textile
+- Rakefile
+- VERSION
+- bin/git-commit-notifier
+- config/git-notifier-configl.yml.sample
+- git-commit-notifier.gemspec
+- lib/commit_hook.rb
+- lib/diff_to_html.rb
+- lib/emailer.rb
+- lib/git.rb
+- lib/result_processor.rb
+- template/email.html.erb
+- template/styles.css
+- test/fixtures/git_log
+- test/fixtures/git_show_51b986619d88f7ba98be7d271188785cbbb541a0
+- test/fixtures/git_show_a4629e707d80a5769f7a71ca6ed9471015e14dc9
+- test/fixtures/git_show_dce6ade4cdc2833b53bd600ef10f9bce83c7102d
+- test/fixtures/git_show_e28ad77bba0574241e6eb64dfd0c1291b221effe
+- test/test_helper.rb
+- test/unit/test_commit_hook.rb
+- test/unit/test_diff_to_html.rb
+- test/unit/test_result_processor.rb
+has_rdoc: true
+homepage: http://github.com/bodo/git-commit-notifier
+licenses: []
+
+post_install_message: 
+rdoc_options: 
+- --charset=UTF-8
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.3.6
+signing_key: 
+specification_version: 3
+summary: Sends git commit messages with diffs
+test_files: 
+- test/test_helper.rb
+- test/unit/test_commit_hook.rb
+- test/unit/test_diff_to_html.rb
+- test/unit/test_result_processor.rb