motiro 0.6.3

data/test/unit/change_test.rb

@@ -0,0 +1,152 @@
+#  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'
+
+class ChangeTest < Test::Unit::TestCase
+  
+  def test_render_diff_with_unset_diff
+    change = Change.new(:summary => 'A /directory', :diff => nil)
+    
+    assert_equal '', change.render_diff
+  end
+  
+  def test_render_diff_with_empty_diff
+    change = Change.new(:summary => 'A /directory', :diff => '')
+    
+    assert_equal '', change.render_diff
+  end
+  
+  def test_render_diff_with_non_empty_diff
+    diff_output = "@@ -0,0 +1 @@\n" +
+                  "+These are the file_contents"
+    change = Change.new(:summary => 'A /a_file.txt', :diff => diff_output)
+    
+    actual_rendered_output = change.render_diff
+    
+    md = actual_rendered_output.match /\A<div id='((\w|\d|-)+)' class='diff-window'><center><h2>Changes to a_file.txt<\/h2>/
+    
+    assert_not_nil md
+    
+    remain = md.post_match
+    
+    md = remain.match /<\/div>\Z/
+    
+    assert_not_nil md
+  end
+  
+  def test_passes_lines_numbers_to_differ
+    FlexMock.use do |differ|
+      differ.should_receive(:start_line).once.with(22, 34)
+      differ.should_receive(:render_diff_table).once.
+      and_return('rendered table')
+      differ.should_ignore_missing
+      
+      change = Change.new(:summary => 'A /a_file.txt',
+                          :diff => "@@ -22,0 +34 @@\n" +
+                                   "+These are the file_contents")
+      
+      change.use_differ(differ)
+      assert_equal 'rendered table', change.render_diff_table
+    end
+  end
+  
+  def test_render_summary_with_unset_diff
+    change = Change.new(:summary => 'A /directory', :diff => nil)
+    
+    assert_equal 'A /directory', change.render_summary
+  end
+  
+  def test_render_summary_with_non_empty_diff
+    diff_output = "@@ -0,0 +1 @@\n" +
+                  "+These are the file_contents"
+    change = Change.new(:summary => 'A /a_file.txt', :diff => diff_output)
+    
+    actual_rendered_output = change.render_summary
+    
+    md = actual_rendered_output.match /\A<a href='\#' onClick="showOnly\('((\w|\d|-)+)'\)">A \/a_file.txt<\/a>\Z/
+    
+    assert_not_nil md
+  end
+  
+  def test_renders_summary_and_diff_using_the_same_ref
+    diff_output = "@@ -0,0 +1 @@\n" +
+                  "+These are the file_contents"
+    change = Change.new(:summary => 'A /a_file.txt', :diff => diff_output)
+    
+    md = change.render_summary.match /\A<a href='\#' onClick="showOnly\('((\w|\d|-)+)'\)/
+    summary_ref = md[1]
+    
+    md = change.render_diff.match /\A<div id='((\w|\d|-)+)'/
+    diff_ref = md[1]
+    
+    assert_equal summary_ref, diff_ref
+  end
+  
+  def test_simple_prefixed_qualified_resource_name
+    change = Change.new(:summary => 'A /a_file.txt')
+    assert_equal '/a_file.txt', change.qualified_resource_name
+  end
+  
+  def test_complex_prefixed_qualified_resource_name
+    change = Change.new(:summary => 'A /subdir/fileC.txt (from /fileA.txt:2)')
+    assert_equal '/subdir/fileC.txt', change.qualified_resource_name
+  end
+  
+  def test_simple_pure_qualified_resource_name
+    change = Change.new(:summary => '/directory/a_file.txt')
+    assert_equal '/directory/a_file.txt', change.qualified_resource_name
+  end
+
+  def test_complex_resource_name
+    change = Change.new(:summary => 'A /subdir/fileC.txt (from /fileA.txt:2)')
+    assert_equal 'fileC.txt', change.resource_name
+  end
+  
+  def test_file_not_filled
+    change = Change.new(:summary => 'A /subdir/fileC.txt',
+                        :resource_kind => 'file',
+                        :diff => nil)
+    
+    assert !change.filled?
+  end
+  
+  def test_file_filled
+    change = Change.new(:summary => 'A /subdir/fileC.txt',
+                        :resource_kind => 'file',
+                        :diff => '+file change')
+    
+    assert change.filled?
+  end
+  
+  def test_directory_always_filled
+    change = Change.new(:summary => 'A /subdir',
+                        :resource_kind => 'dir',
+                        :diff => nil)
+    
+    assert change.filled?
+  end
+  
+  def test_unknown_always_not_filled
+    change = Change.new(:summary => 'A /subdir/fileC.txt',
+                        :resource_kind => nil,
+                        :diff => '+file change')
+    
+    assert ! change.filled?
+  end
+  
+end

data/test/unit/chdir_runner_test.rb

@@ -0,0 +1,77 @@
+#  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 'ports/chdir_runner'
+
+class ChdirRunnerTest < Test::Unit::TestCase
+
+  def setup
+    @prevdir = Dir.pwd
+    assert_not_equal(TEMP_DIR, @prevdir)
+  end
+  
+  def test_changes_dir_before_command_and_rollsback_after
+    FlexMock.use do |runner|
+      runner.should_receive(:run).with('my_command', '', {}).once.
+        returns { assert_equal(e(TEMP_DIR), e(Dir.pwd)) }
+        
+      prevdir = Dir.pwd
+      
+      cdrunner = ChdirRunner.new(TEMP_DIR, runner)
+      cdrunner.run('my_command')
+      
+      assert_equal(prevdir, Dir.pwd)
+    end
+  end
+  
+  def test_rollsback_dir_even_on_error
+    FlexMock.use do |runner|
+      runner.should_receive(:run).with('my_command', '', {}).once.
+        returns { raise 'fatal error' }
+        
+      prevdir = Dir.pwd
+      
+      begin
+        cdrunner = ChdirRunner.new(TEMP_DIR, runner)
+        cdrunner.run('my_command')
+      rescue
+        #just ignore the expected error
+      end
+      
+      assert_equal(prevdir, Dir.pwd)
+    end
+  end
+  
+  def test_returns_inner_runner_result
+    FlexMock.use do |runner|
+      runner.should_receive(:run).with('my_command', '', {}).once.
+        returns('expectedd output')
+        
+      cdrunner = ChdirRunner.new(TEMP_DIR, runner)
+      assert_equal 'expectedd output', cdrunner.run('my_command')
+    end
+  end
+  
+  def teardown
+    Dir.chdir @prevdir
+  end
+  
+  def e(path); File.expand_path(path); end
+
+end

data/test/unit/chief_editor_test.rb

@@ -0,0 +1,234 @@
+#  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 'rubygems'
+
+require 'flexmock'
+
+require 'core/chief_editor'
+
+require 'stub_hash'
+require 'stubs/svn_settings'
+require 'mocks/headline'
+
+require 'mocks/svn_reporter'
+
+class Class
+  include Test::Unit::Assertions
+end
+
+class Headline
+  @@old_latest = self.method(:latest)
+  
+  def self.old_latest(num, reporter)
+    @@old_latest.call(num, reporter)        
+  end
+end                
+
+class ChiefEditorTest < Test::Unit::TestCase
+  
+  fixtures :headlines, :changes
+  
+  @@log = ''
+  
+  def self.append_to_log(txt)
+    @@log << txt
+  end
+  
+  def test_reads_package_size
+    def Headline.latest(num, reporter)
+      assert_equal 6, num
+      ChiefEditorTest.append_to_log('latest')
+      return Array.new.fill(MockHeadline.new, 6)
+    end
+
+    FlexMock.use('1', '2') do |reporter, fetcher|
+      fetcher.should_receive(:active_reporters).and_return([reporter]).once
+      reporter.should_receive(:cache?).and_return(true)
+      reporter.should_receive(:name).and_return('mock_subversion')
+      settings = StubConnectionSettingsProvider.new(:package_size => 6)
+      
+      ChiefEditor.new(settings, fetcher).latest_news_from 'mock_subversion'
+      
+      assert_equal 'latest', @@log
+    end
+    
+    def Headline.latest(num, reporter)
+      old_latest(num, reporter)
+    end
+  end
+  
+  def test_refetches_latest_news_on_every_call_when_in_development_mode
+    FlexMock.use do |reporter|
+      settings = StubConnectionSettingsProvider.new(:update_interval => 0)
+      fetcher = { :active_reporters => [reporter] }
+      svn_demo_headline = headlines('svn_demo_headline')
+      
+      reporter.should_receive(:name).returns('subversion')
+      reporter.should_receive(:latest_headlines).once.
+        returns([].fill(MockHeadline.new, 0, 3))
+      
+      ChiefEditor.new(settings, fetcher).latest_news_from('subversion')
+    end
+  end
+  
+  def test_refetches_news_on_every_call_when_in_development_mode
+    FlexMock.use do |reporter|
+      settings = StubConnectionSettingsProvider.new(:update_interval => 0)
+      fetcher = { :active_reporters => [reporter] }
+      svn_demo_headline = headlines('svn_demo_headline')
+      
+      reporter.should_receive(:name).returns('subversion')
+      reporter.should_receive(:headlines).once.
+        returns([svn_demo_headline])
+      
+      ChiefEditor.new(settings, fetcher).news_from('subversion')
+    end
+  end
+
+  def test_does_not_ask_reporter_in_production_mode
+    FlexMock.use('1', '2') do |fetcher, reporter|
+      reporter.should_receive(:cache?).and_return(true)
+      reporter.should_receive(:name).and_return('mock_subversion')
+      reporter.should_receive(:latest_headlines).never
+      fetcher.should_receive(:active_reporters).and_return([reporter]).once
+      
+      settings = StubConnectionSettingsProvider.new(:update_interval => 6)
+      
+      ChiefEditor.new(settings, fetcher).latest_news_from 'mock_subversion'
+    end
+  end
+  
+  def test_asks_reporter_for_headline_in_development_mode
+    FlexMock.use do |reporter|
+      settings = StubConnectionSettingsProvider.new(:update_interval => 0)
+      fetcher = { :active_reporters => [reporter] }
+      svn_demo_headline = headlines('svn_demo_headline')
+      
+      reporter.should_receive(:name).
+        returns('subversion')
+      
+      reporter.should_receive(:headline).
+        with(svn_demo_headline.rid).
+        returns(svn_demo_headline).
+        once
+      
+      chief_editor = ChiefEditor.new(settings, fetcher)
+      
+      chief_editor.headline_with(svn_demo_headline.reported_by,
+                                 svn_demo_headline.rid)
+    end
+  end
+  
+  def test_fetches_cached_headlines_in_production_mode
+    FlexMock.use('1', '2') do |reporter, fetcher|
+      fetcher.should_receive(:active_reporters).and_return([reporter])
+      reporter.should_receive(:cache?).returns(true)
+      reporter.should_receive(:name).returns('subversion')
+      reporter.should_receive(:headline).never
+
+
+      settings = StubConnectionSettingsProvider.new(:update_interval => 8)
+      
+      chief_editor = ChiefEditor.new(settings, fetcher)
+      
+      svn_demo_headline = headlines('svn_demo_headline')
+
+      headline = chief_editor.headline_with(svn_demo_headline.reported_by,
+                                            svn_demo_headline.rid)
+      
+      reporter.mock_verify
+      
+      assert_equal svn_demo_headline.description, headline.description
+    end
+  end
+  
+  def test_fetches_reporter_title
+    settings = StubConnectionSettingsProvider.new
+    title = 'Latest news from the mail list'
+    reporter = { :name => 'mail_list', :channel_title => title }
+    fetcher = { :active_reporters => [reporter]}
+      
+    chief_editor = ChiefEditor.new(settings, fetcher)
+      
+    assert_equal title, chief_editor.title_for('mail_list')
+  end
+  
+  def test_fetches_title_from_cached_reporter
+    settings = StubConnectionSettingsProvider.new
+    title = 'Latest news from the mail list'
+    reporter = { :name => 'mail_list',
+                 :channel_title => title,
+                 :cache? => true }
+    fetcher = { :active_reporters => [reporter]}
+      
+    chief_editor = ChiefEditor.new(settings, fetcher)
+      
+    assert_equal title, chief_editor.title_for('mail_list')
+  end
+
+  def test_retrieves_headlines_from_correct_reporter_on_cached_mode
+    FlexMock.use('1', '2', '3') do |events_reporter, svn_reporter, fetcher|
+      settings = StubConnectionSettingsProvider.new(:update_interval => 8)
+      
+      events_reporter.should_receive(:name).returns('events')
+      events_reporter.should_receive(:cache?).returns(true)
+      svn_reporter.should_receive(:name).returns('subversion')
+      svn_reporter.should_receive(:cache?).returns(true)
+      fetcher.should_receive(:active_reporters).
+        and_return([events_reporter, svn_reporter]).
+        once
+      
+      chief_editor = ChiefEditor.new(settings, fetcher)
+      
+      events_news = chief_editor.latest_news_from 'events'
+      
+      assert_equal 1, events_news.size
+      
+      subversion_news = chief_editor.latest_news_from 'subversion'
+      
+      assert_equal 3, subversion_news.size
+    end
+  end
+  
+  def test_asks_fetcher_for_reporters_to_employ
+    FlexMock.use('1', '2') do |fetcher, reporter|
+      settings = StubConnectionSettingsProvider.new
+      
+      fetcher.should_receive(:active_reporters).once.
+        returns([reporter, reporter])
+      
+      reporter.should_receive(:name).returns('fake_reporter').twice
+      reporter.should_receive(:cache?).returns(true).twice
+      
+      editor = ChiefEditor.new(settings, fetcher)
+    end
+  end
+  
+  def test_returns_reporter
+    settings = StubConnectionSettingsProvider.new
+    reporter = {:name => 'stub', :cache? => false}
+    fetcher = {:active_reporters => [reporter]}
+    
+    editor = ChiefEditor.new(settings, fetcher)
+    assert_equal reporter, editor.reporter_with('stub')
+  end
+  
+  #TODO should signal when the reporter is not registered
+end

data/test/unit/darcs_connection_test.rb

@@ -0,0 +1,109 @@
+#  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 'fileutils'
+
+require 'reporters/darcs_connection'
+
+class DarcsConnectionTest  < Test::Unit::TestCase
+
+  TMP = TEMP_DIR + '/tmprepo'
+  
+  def setup
+    FileUtils.mkdir_p(TMP)
+  end
+
+  def test_reads_settings
+    FlexMock.use('1', '2') do |runner, repo_dir|
+      settings = StubConnectionSettingsProvider.new(
+                 :repo => 'http://motiro.sf.net/darcsrepo/trunk',
+                 :package_size => 7)
+      repo_dir.should_receive(:path).once.
+        returns(TMP)
+      runner.should_receive(:run).once.
+        with('darcs changes --xml --last=7', '', {}).
+        returns('')
+        
+      connection = DarcsConnection.new(settings, runner, repo_dir)
+      
+      connection.changes
+    end
+  end
+  
+  def test_provides_full_change_log
+    FlexMock.use('1', '2') do |runner, repo_dir|
+      settings = StubConnectionSettingsProvider.new
+      repo_dir.should_receive(:path).once.returns(TMP)
+      runner.should_receive(:run).once.
+        with("darcs changes --xml", '', {}).returns('')
+        
+      DarcsConnection.new(settings, runner, repo_dir).changes(:all)
+    end
+  end
+
+  def test_changes_with_hashcode_parameter_asks_for_only_one_patch
+    FlexMock.use('1', '2') do |runner, repo_dir|
+      hashcode = '20060717200939-49d33-c04fbb63892ae64cd96d1ad8f1ad2dd0a6e8e7da.gz'
+      settings = StubConnectionSettingsProvider.new(
+                 :repo => 'http://motiro.sf.net/darcsrepo/trunk')
+      repo_dir.should_receive(:path).once.
+        returns(TMP)
+      runner.should_receive(:run).once.
+        with('darcs changes --xml' +
+                          " --from-match=\"hash #{hashcode}\"" +
+                          " --to-match=\"hash #{hashcode}\"", '', {}).
+        returns('')
+        
+      connection = DarcsConnection.new(settings, runner, repo_dir)
+      
+      connection.changes(hashcode)
+    end
+  end
+  
+  def test_pulling
+    FlexMock.use('1', '2') do |runner, repo_dir|
+      settings = StubConnectionSettingsProvider.new(
+                 :repo => 'http://motiro.sf.net/darcsrepo/trunk')
+      repo_dir.should_receive(:path).once.returns(TMP)
+      runner.should_receive(:run).once.
+        with('darcs pull -a http://motiro.sf.net/darcsrepo/trunk', '', {}).
+        returns('')
+        
+      DarcsConnection.new(settings, runner, repo_dir).pull
+    end
+  end
+  
+  def test_diffing
+    FlexMock.use('1', '2') do |runner, repo_dir|
+      hashcode = '20060717200939-49d33-c04fbb63892ae64cd96d1ad8f1ad2dd0a6e8e7da.gz'
+      settings = StubConnectionSettingsProvider.new
+      repo_dir.should_receive(:path).once.returns(TMP)
+      runner.should_receive(:run).once.
+        with("darcs diff -u --match \"hash #{hashcode}\"", '', {}).
+        returns('')
+        
+      DarcsConnection.new(settings, runner, repo_dir).diff(hashcode)
+    end
+  end
+  
+  def teardown
+    FileUtils.rm_rf(TMP)
+  end
+
+end