motiro 0.6.3

data/test/unit/runner_test.rb

@@ -0,0 +1,93 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'stubio'
+require 'ports/runner'
+
+class RunnerTest < Test::Unit::TestCase
+
+  def test_feeds_inner_process_given_input
+    FlexMock.use('creator') do |creator|
+      command = 'svn diff https://svn.sourceforge.net/svnroot/motiro/ -r363:364'
+      output = "diff\n"
+
+      pio = StubIO.new(output)
+      creator.should_receive(:popen).with(command, 'r+').once.
+        and_return(pio)
+
+      runner = Runner.new(creator)
+      assert_equal output, runner.run(command, "t\n")
+      assert_equal "t\n", pio.string
+    end
+  end
+  
+  def test_augments_and_cleans_environment
+    FlexMock.use('creator') do |creator|
+      command = 'svn log -v https://svn.sourceforge.net/svnroot/motiro/ -r363'
+      creator.should_receive(:popen).with(command, 'r+').once.
+        returns do
+          assert_equal 'test_value', ENV['TEST_VAR']
+          StringIO.new
+        end
+
+      Runner.new(creator).run(command, '', 'TEST_VAR' => 'test_value')
+      
+      assert_nil ENV['TEST_VAR']
+    end
+  end
+  
+  def test_cleans_overlapping_environment
+    FlexMock.use('creator') do |creator|
+      command = 'svn diff https://svn.sourceforge.net/svnroot/motiro/ -r363:364'
+      output = "diff\n"
+      creator.should_receive(:popen).with(command, 'r+').once.
+        and_return(StringIO.new)
+
+      Runner.new(creator).run(command, '', 'FIRST_TEST_VAR' => 'first_value',
+                                           'SECOND_TEST_VAR' => 'second_value')
+
+      assert_equal 'original_value', ENV['FIRST_TEST_VAR']
+      assert_nil ENV['SECOND_TEST_VAR']
+    end
+  end
+  
+  def test_recovers_from_broken_pipe_io_error
+    FlexMock.use('creator') do |creator|
+      command = 'svn diff https://svn.sourceforge.net/svnroot/motiro/ -r363:364'
+      output = "diff command output\n"
+      pio = StubIO.new(output)
+      def pio.<<(_); raise Errno::EPIPE; end
+
+      creator.should_receive(:popen).with(command, 'r+').
+        and_return(pio).
+        once
+
+      assert_equal output, Runner.new(creator).run(command)
+    end
+  end
+  
+  def setup
+    ENV['FIRST_TEST_VAR'] = 'original_value'
+  end
+  
+  def teardown
+    ENV.delete 'FIRST_TEST_VAR'
+  end
+  
+end

data/test/unit/settings_test.rb

@@ -0,0 +1,55 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'stringio'
+
+require 'core/settings'
+
+class SettingsProviderTest < Test::Unit::TestCase
+
+  def setup
+    @expected_file_name = File.expand_path(File.dirname(__FILE__) +
+                                           '../../../config/motiro.yml')
+    @opener = FlexMock.new 
+        
+    @opener.should_receive(:open).with(@expected_file_name).
+      once.
+      returns( StringIO.new( "package_size: 8\n" +
+                             "update_interval: 10\n" +
+                             "darcs:\n" +
+                             "  repo: http://eclipsefp.sf.net/repos/trunk\n"))
+
+    @provider = SubversionSettingsProvider.new(@opener)
+  end
+
+  def teardown
+    @opener.mock_verify
+  end
+
+  def test_active_reporters
+    assert_equal ['darcs'], @provider.active_reporter_ids
+  end
+
+  def test_fetches_package_size
+    assert_equal 8, @provider.package_size
+  end
+    
+  def test_fectches_update_interval
+    assert_equal 10, @provider.update_interval
+  end
+    
+  def test_dynamically_fetches_parameters
+    assert_equal 8, @provider.package_size
+    @opener.mock_verify
+        
+    @opener.should_receive(:open).with(@expected_file_name).
+      returns do
+        StringIO.new("package_size: 5\n" +
+                     "svn:\n" +
+                     "  repo: http://svn.berlios.de/svnroot/repos/motiro\n")
+      end 
+        
+    assert_equal 5, @provider.package_size
+    assert_equal ['svn'], @provider.active_reporter_ids
+  end
+
+end

data/test/unit/string_extensions_test.rb

@@ -0,0 +1,10 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class StringExtensionsTest < Test::Unit::TestCase
+
+  def test_medialize
+    str = "= Motiro =\n\nAnother paragraph"
+    assert_equal '<h1>Motiro</h1><p>Another paragraph</p>', str.medialize
+  end
+
+end

data/test/unit/svn_connection_test.rb

@@ -0,0 +1,183 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'test/unit'
+
+require 'reporters/svn_connection'
+
+require 'stubs/svn_settings'
+
+class SubversionConnectionTest < Test::Unit::TestCase
+  
+  def test_reads_repo_url
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new(
+                   :repo => 'svn://svn.berlios.de/motiro')
+      
+      runner.should_receive(:run).once.
+        with('svn log svn://svn.berlios.de/motiro -v -rHEAD:1 --limit=5',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.log
+    end
+  end
+
+  def test_issues_commands_with_credentials
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new(
+                   :repo => 'http://rapidsvn.tigris.org/svn/rapidsvn/trunk',
+                   :repo_user => 'guest',
+                   :repo_password => 'with spaces')
+      
+      runner.should_receive(:run).once.
+        with("svn --username='guest' --password='with spaces' log http://rapidsvn.tigris.org/svn/rapidsvn/trunk -v -rHEAD:1 --limit=5",
+             "t\n", 'LC_MESSAGES' => 'C')
+      runner.should_receive(:run).once.
+        with("svn --username='guest' --password='with spaces' info -r18 --xml http://rapidsvn.tigris.org/svn/rapidsvn/trunk/trunk/file_a.txt",
+             "t\n", 'LC_MESSAGES' => 'C')
+      runner.should_receive(:run).once.
+        with("svn --username='guest' --password='with spaces' diff http://rapidsvn.tigris.org/svn/rapidsvn/trunk -r17:18",
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.log
+      connection.info('/trunk/file_a.txt', 18)
+      connection.diff(18)
+    end
+  end
+
+  def test_limit_query_size
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new :package_size => 3
+      
+      runner.should_receive(:run).once.
+        with('svn log http://svn.fake.domain.org/fake_repo -v -rHEAD:1 --limit=3',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.log
+    end
+  end
+  
+  def test_limit_to_one_revision
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+      
+      runner.should_receive(:run).twice.
+        with('svn log http://svn.fake.domain.org/fake_repo -v -r7',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.log(:only => 7)
+      connection.log(:only => '7')
+    end
+  end
+  
+  def test_diff
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+      
+      runner.should_receive(:run).once.
+        with('svn diff http://svn.fake.domain.org/fake_repo -r14:15',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.diff(15)
+    end
+  end
+  
+  def test_cache_results_to_avoid_network_usage
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+      expected_diff_output = 'diff output'
+      
+      runner.should_receive(:run).with_any_args.
+        once.
+        returns(expected_diff_output)
+      
+      connection = SubversionConnection.new(settings, runner)
+      assert_equal expected_diff_output, connection.diff(18)
+      assert_equal expected_diff_output, connection.diff(18)
+    end
+  end
+  
+  def test_info
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+      
+      runner.should_receive(:run).once.
+        with('svn info -r18 --xml http://svn.fake.domain.org/fake_repo/trunk/file_a.txt',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.info('/trunk/file_a.txt', 18)
+    end
+  end
+  
+  def test_uses_english_locale_and_temporarily_accepts_ssl_certificate
+    FlexMock.use do |runner|
+      runner.should_receive(:run).once.
+        with('svn log http://svn.fake.domain.org/fake_repo -v -rHEAD:1 --limit=5',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(StubConnectionSettingsProvider.new,
+                     runner)
+      
+      connection.log
+    end
+  end
+  
+  def test_unlimited_logging
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+      
+      runner.should_receive(:run).once.
+        with('svn log http://svn.fake.domain.org/fake_repo -v',
+             "t\n", 'LC_MESSAGES' => 'C')
+      
+      connection = SubversionConnection.new(settings, runner)
+      
+      connection.log(:all)
+    end
+  end
+  
+  def test_log_history_from_some_revision
+    FlexMock.use do |runner|
+      settings = StubConnectionSettingsProvider.new
+
+      runner.should_receive(:run).once.
+        with('svn log http://svn.fake.domain.org/fake_repo -v -rHEAD:13 --limit=5',
+             "t\n", 'LC_MESSAGES' => 'C')
+
+      connection = SubversionConnection.new(settings, runner)
+
+      connection.log(:history_from => 'r12')
+    end
+  end
+  
+  # TODO what happens if we ask for an inexistent revision
+  
+end

data/test/unit/svn_reporter_interaction_test.rb

@@ -0,0 +1,38 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'reporters/subversion_reporter'
+
+class SubversionReporterInteractionTest < Test::Unit::TestCase
+
+  def test_asks_for_specific_revision
+    FlexMock.use do |conn|
+      conn.should_receive(:log).with(:only => '113').returns('').once
+      SubversionReporter.new(conn).headline('r113')
+    end
+  end
+  
+  def test_asks_for_non_cached_revisions_only
+    FlexMock.use do |conn|
+      conn.should_receive(:log).with(:history_from => 'r206').returns('').once
+      SubversionReporter.new(conn).latest_headlines('r206')
+    end
+  end
+
+end

data/test/unit/svn_reporter_test.rb

@@ -0,0 +1,286 @@
+#  Motiro - A project tracking tool
+#  Copyright (C) 2006-2007  Thiago Arrais
+#  
+#  This program is free software; you can redistribute it and/or modify
+#  it under the terms of the GNU General Public License as published by
+#  the Free Software Foundation; either version 2 of the License, or
+#  any later version.
+#  
+#  This program is distributed in the hope that it will be useful,
+#  but WITHOUT ANY WARRANTY; without even the implied warranty of
+#  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#  GNU General Public License for more details.
+#  
+#  You should have received a copy of the GNU General Public License
+#  along with this program; if not, write to the Free Software
+#  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'svn_excerpts'
+require 'reporters/subversion_reporter'
+
+class SubversionReporterTest < Test::Unit::TestCase
+
+    def setup
+        @svn_log  = "------------------------------------------------------------------------\n"
+        @svn_log += "r1 | thiagoarrais | 2006-02-14 15:45:13 -0400 (Ter, 14 Fev 2006) | 1 line\n"
+        @svn_log += "Caminhos mudados:\n"
+        @svn_log += "   A /trunk\n"
+        @svn_log += "\n"
+        @svn_log += "Criacao do trunk do projeto\n"
+        @svn_log += "------------------------------------------------------------------------\n"
+        @svn_diff = ''
+        @svn_info = ''
+        
+        @svn_connection = FlexMock.new('svn connection')
+        @svn_connection.mock_handle(:log) do
+            @svn_log
+        end
+
+        @svn_connection.mock_handle(:diff) do
+            @svn_diff
+        end
+        
+        @svn_connection.mock_handle(:info) do
+            @svn_info
+        end
+        @reporter = SubversionReporter.new(@svn_connection)
+    end
+
+    def test_one_revision
+        hl = @reporter.latest_headline
+        assert_equal 'thiagoarrais', hl.author
+        assert_equal Time.local(2006, 02, 14, 15, 45, 13), hl.happened_at
+        assert_equal 'Criacao do trunk do projeto', hl.title
+    end
+    
+    def test_more_revisions
+        @svn_log = R13 + @svn_log
+
+        hls = @reporter.latest_headlines
+
+        assert_equal 'thiagoarrais', hls[0].author
+        assert_equal Time.local(2006, 02, 19, 8, 50, 07), hls[0].happened_at
+        assert_equal 'Leitura de uma revisao SVN pronta', hls[0].title
+
+        assert_equal 'thiagoarrais', hls[1].author
+        assert_equal Time.local(2006, 02, 14, 15, 45, 13), hls[1].happened_at
+        assert_equal 'Criacao do trunk do projeto', hls[1].title
+    end
+    
+    def test_revision_with_empty_comment
+        @svn_log = R15 + @svn_log
+
+        hl = @reporter.latest_headline
+        
+        assert_equal 'thiagoarrais', hl.author
+        assert_equal Time.local(2006, 02, 19, 9, 13, 7), hl.happened_at
+        assert_equal '', hl.title
+    end
+    
+    def test_revision_with_multiline_comment
+        @svn_log = R7 + @svn_log
+        
+        hls = @reporter.latest_headlines
+        
+        hl7 = hls[0]
+        assert_equal 'gilbertogil', hl7.author
+        assert_equal Time.local(2006, 02, 17, 18, 7, 55), hl7.happened_at
+        assert_equal 'Correcao para a revisao anterior (r6)', hl7.title
+        
+        hl1 = hls[1]        
+        assert_equal 'thiagoarrais', hl1.author
+        assert_equal Time.local(2006, 02, 14, 15, 45, 13), hl1.happened_at
+        assert_equal 'Criacao do trunk do projeto', hl1.title
+    end
+    
+    def test_comment_with_dashes
+        @svn_log = R2 + @svn_log
+
+        hls = @reporter.latest_headlines
+        
+        hl2 = hls[0]
+        assert_equal 'gilbertogil', hl2.author
+        assert_equal Time.local(2006, 02, 17, 18, 7, 55), hl2.happened_at
+        assert_equal 'Estou tentando enganar o SubversionReporter', hl2.title
+        
+        hl1 = hls[1]        
+        assert_equal 'thiagoarrais', hl1.author
+        assert_equal Time.local(2006, 02, 14, 15, 45, 13), hl1.happened_at
+        assert_equal 'Criacao do trunk do projeto', hl1.title
+    end
+    
+    def test_collect_all_available_headlines
+        @svn_log = R13 + R7 + @svn_log
+        
+        hls = @reporter.latest_headlines
+        assert_equal 3, hls.size
+        
+        assert_equal 'Leitura de uma revisao SVN pronta', hls[0].title
+        assert_equal 'Correcao para a revisao anterior (r6)', hls[1].title
+        assert_equal 'Criacao do trunk do projeto', hls[2].title
+    end
+    
+    def test_record_full_comment
+        @svn_log = R7 + @svn_log
+
+        hls = @reporter.latest_headlines
+        
+        expectedComment  = "Correcao para a revisao anterior (r6)\n"
+        expectedComment += "\n"
+        expectedComment += "Esqueci de colocar o svn_reporter. Foi mal!\n"
+
+        assert_equal(expectedComment, hls[0].description)
+    end
+
+    def test_name
+        assert_equal 'subversion', @reporter.name
+    end
+    
+    def test_sets_reported_by_field
+        hls = @reporter.latest_headlines
+        headline = hls[0]
+        
+        assert_equal @reporter.name, headline.reported_by
+    end
+    
+    def test_retrieves_headline_from_revision_id
+        hls = @reporter.latest_headlines
+        headline = hls[0]
+
+        headline_from_rid = @reporter.headline(headline.rid)
+
+        assert_not_nil headline_from_rid
+        assert_equal 'Criacao do trunk do projeto', headline_from_rid.description
+    end
+    
+    def test_fills_headline_with_details
+        hls = @reporter.latest_headlines
+        headline = hls[0]
+        
+        changes = headline.changes
+        assert_not_nil changes
+        assert_equal 1, changes.size
+        
+        assert_equal 'Criacao do trunk do projeto', headline.description
+        assert_equal '   A /trunk', changes[0].summary
+    end
+    
+    def test_records_diff_output
+        @svn_log = R105 + @svn_log
+        @svn_diff = R105DIFF
+
+        expected_diff  = "@@ -0,0 +1,7 @@\n"
+        expected_diff += "+class Change < ActiveRecord::Base\n"
+        expected_diff += "+\n"
+        expected_diff += "+    def to_s\n"
+        expected_diff += "+        return summary\n"
+        expected_diff += "+    end\n"
+        expected_diff += "+\n"
+        expected_diff += "+end"
+
+        hls = @reporter.latest_headlines
+        change = hls[0].changes[0]
+
+        assert_equal '   A /trunk/app/models/change.rb', change.summary
+        assert_equal expected_diff, change.diff
+
+        # TODO directory nodes
+        # TODO only directory diff (no diff output)
+    end
+    
+    def test_records_multiple_files_diff
+        @svn_log = R6 + "-------------\n"
+        @svn_diff = R6DIFF
+        
+        expected_diff_for_connection  = R6C1DIFF
+
+        expected_diff_for_headline = R6C2DIFF
+               
+        hls = @reporter.latest_headlines
+        changes = hls[0].changes
+        
+        assert_equal '   A /trunk/src/test/stubs/svn_connection.rb', changes[0].summary
+        assert_equal expected_diff_for_connection, changes[0].diff
+        
+        assert_equal '   A /trunk/src/test/unit/headline_test.rb', changes[1].summary
+        assert_equal expected_diff_for_headline, changes[1].diff
+    end
+    
+    def test_method_headline_records_diff_output
+        @svn_log = R105
+        @svn_diff = R105DIFF
+
+        expected_diff  = "@@ -0,0 +1,7 @@\n"
+        expected_diff += "+class Change < ActiveRecord::Base\n"
+        expected_diff += "+\n"
+        expected_diff += "+    def to_s\n"
+        expected_diff += "+        return summary\n"
+        expected_diff += "+    end\n"
+        expected_diff += "+\n"
+        expected_diff += "+end"
+
+        headline = @reporter.headline('r105')
+        change = headline.changes[0]
+
+        assert_equal '   A /trunk/app/models/change.rb', change.summary
+        assert_equal expected_diff, change.diff
+    end
+    
+    def test_correctly_associates_resources_with_diff_when_moving
+        @svn_log = R124
+        @svn_diff = R124DIFF
+        
+        headline = @reporter.headline('r124')
+        change_for_html_fragment = find_in(headline.changes) do |change|
+            Regexp.new('A /trunk/app/views/report/html_fragment.rhtml').
+                match(change.summary)
+        end
+        
+        assert_not_nil change_for_html_fragment
+        
+        change_for_svn_html_fragment = find_in(headline.changes) do |change|
+            Regexp.new('D /trunk/app/views/report/subversion_html_fragment.rhtml').
+                match(change.summary)
+        end
+        
+        assert_not_nil change_for_svn_html_fragment
+        
+        assert_nil change_for_html_fragment.diff.match(/^-/)
+        assert_not_nil change_for_html_fragment.diff.match(/^\+/)
+
+        assert_nil change_for_svn_html_fragment.diff.match(/^\+/)
+        assert_not_nil change_for_svn_html_fragment.diff.match(/^-/)
+    end
+    
+    def test_records_node_type
+        @svn_log = R10
+        @svn_diff = '' # simulate a connection timeout
+
+        @svn_info = R10INFO
+        
+        headline = @reporter.headline('r10')
+        
+        change = headline.changes.first
+        assert_equal 'file', change.resource_kind
+    end
+
+    def teardown
+        @svn_connection.mock_verify
+    end
+ 
+private
+
+    def find_in(arr)
+        arr.each do |item| 
+            return item if yield(item)
+        end
+        
+        return nil
+    end
+    
+    #TODO test similar paths (see SubversionReporter#diff_for)
+    #TODO simulate a connection timeout on live and cached modes
+    
+end