blastr 0.2.0 → 0.2.1

data/History.txt

@@ -1,3 +1,7 @@
+== 0.2.1 2012-12-17
+
+* Fix a bug where an empty revision log would cause Blastr to throw up on the console
+
 == 0.2.0 2012-12-17
 
 * Migrate from Hoe to a .gemspec file

data/Rakefile

@@ -9,4 +9,23 @@ Rake::TestTask.new do |t|
   t.libs << "test"
   t.test_files = FileList['test/**/test*.rb']
   t.verbose = false
+end
+
+ENCODING_HEADER = "# encoding: UTF-8"
+desc "Add UTF-8 encoding to source files"
+task :encoding do
+  FileList['test/**/*.rb', 'lib/**/*.rb'].each do |filename|
+    open(filename, 'r+') do |f|
+      beginning_of_file = f.pos
+      first_line = f.readline
+      f.seek(beginning_of_file)
+      original_content = f.read
+      f.seek(beginning_of_file)
+      unless first_line.strip == ENCODING_HEADER.strip
+        puts "Adding '#{ENCODING_HEADER}' to #{filename}"
+        f.write("#{ENCODING_HEADER}\n")
+        f.write(original_content)
+      end
+    end
+  end
 end

data/lib/blastr.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 $LOAD_PATH.unshift(File.dirname(__FILE__)) unless
   $LOAD_PATH.include?(File.dirname(__FILE__)) or $LOAD_PATH.include?(File.expand_path(File.dirname(__FILE__)))
 

data/lib/error.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 module Blastr
 
   class UsageError < ArgumentError

data/lib/filesystem.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'fileutils'
 
 module Blastr::FileSystem

data/lib/people/people.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'yaml'
 
 module Blastr

data/lib/scm/git.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'git'
 require File.expand_path(File.join(File.dirname(__FILE__), 'scm.rb'))
 
@@ -33,6 +34,11 @@ module Blastr::SourceControl
     def self.head
       GitRevision.new("HEAD", nil)
     end
+    
+    def ==(other)
+      return false unless other.is_a? GitRevision
+      other.name == @name and other.date == @date
+    end
   end
     
   class Git

data/lib/scm/hg.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'fileutils'
 require File.expand_path(File.join(File.dirname(__FILE__), 'scm.rb'))
 

data/lib/scm/scm.rb

@@ -1,15 +1,19 @@
+# encoding: UTF-8
 module Blastr::SourceControl
   
   class LogEntry
     attr_accessor :revision, :author, :comment
+
     def initialize(revision, author, comment)
       @revision = revision
       @author = author
       @comment = comment
     end
+
     def ==(other)
       @revision == other.revision and @author == other.author and @comment == other.comment
     end
+
     def to_s
       "revision #{@revision} by #{@author}: #{@comment}"
     end

data/lib/scm/svn.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'fileutils'
 require File.expand_path(File.join(File.dirname(__FILE__), 'scm.rb'))
 
@@ -48,7 +49,7 @@ module Blastr::SourceControl
     def latest_revision
       entries = commits_since(as_revision("HEAD"))
       return entries.first.revision unless entries.empty?
-      1
+      SubversionRevision.new("1")
     end
 
     def commits_since(since_revision = 1)

data/lib/tts/tts.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 module Blastr
   module TTS
     

data/lib/version.rb

@@ -1,4 +1,5 @@
+# encoding: UTF-8
 module Blastr
-  VERSION = '0.2.0'
+  VERSION = '0.2.1'
   COPYRIGHT = "Copyright (c) 2009-#{Time.now.year}, Lasse Koskela. All Rights Reserved."
 end

data/test/people/test_people.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 require 'fileutils'
 

data/test/scm/abstract_scm_testcase.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 class AbstractScmTestCase < Test::Unit::TestCase  
   abstract
 

data/test/scm/test_git.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestGit < AbstractScmTestCase
@@ -51,6 +52,27 @@ class TestGit < AbstractScmTestCase
     parent_of_git_repo = File.dirname(@local_repo)
     assert_urls_are_not_understood([ parent_of_git_repo ])
   end
+  
+  def test_detects_latest_revision_from_commit_log
+    latest_commit_sha = "LaT35tR3v1510N"
+    head_minus_one = revision("HEAD~1")
+    repo = scm.new("file:///fakerepo")
+    repo.expects(:as_revision).with("HEAD~1").returns(head_minus_one)
+    repo.expects(:commits_since).with(head_minus_one).returns([
+      logentry(commit(latest_commit_sha))
+    ])
+    assert_equal revision(latest_commit_sha), repo.latest_revision
+  end
+  
+  def test_latest_revision_for_empty_log_defaults_to_revision_1
+    head = revision("HEAD")
+    head_minus_one = revision("HEAD~1")
+    repo = scm.new("file:///fakerepo")
+    repo.expects(:as_revision).with("HEAD~1").returns(head_minus_one)
+    repo.expects(:as_revision).with("HEAD").returns(head)
+    repo.expects(:commits_since).with(head_minus_one).returns([])
+    assert_equal head, repo.latest_revision
+  end
 
   private
   
@@ -60,6 +82,20 @@ class TestGit < AbstractScmTestCase
     assert File.directory? dir
     dir
   end
+  
+  def commit(sha, options = {})
+    defaults = { :date => "FAKE DATE", :author => stub(:name => "author"), :message => "message" }
+    commit_data = defaults.merge(options.merge(:sha => sha))
+    stub(commit_data)
+  end
+  
+  def logentry(commit)
+    Blastr::SourceControl::GitLogEntry.new(commit)
+  end
+  
+  def revision(sha, date = "FAKE DATE")
+    Blastr::SourceControl::GitRevision.new(sha, date)
+  end
 
   def scm
     Blastr::SourceControl::Git

data/test/scm/test_git_log_entry.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestGitLogEntry < Test::Unit::TestCase

data/test/scm/test_git_revision.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestGitRevision < Test::Unit::TestCase

data/test/scm/test_log_entry.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestLogEntry < Test::Unit::TestCase  

data/test/scm/test_mercurial.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestMercurial < AbstractScmTestCase
@@ -16,8 +17,32 @@ class TestMercurial < AbstractScmTestCase
     assert_urls_are_not_understood(["http://acme.com/foo"])
   end
   
+  def test_detects_latest_revision_from_commit_log
+    repo = scm.new("hg:/fakerepo")
+    repo.expects(:commits_since).with(revision("tip")).returns([
+      logentry(revision("15"), "author", "comment")
+    ])
+    assert_equal revision("15"), repo.latest_revision
+  end
+  
+  def test_latest_revision_for_empty_log_defaults_to_revision_1
+    repo = scm.new("hg:/fakerepo")
+    repo.expects(:commits_since).with(revision("tip")).returns([])
+    assert_equal revision("tip"), repo.latest_revision
+  end
+  
+  private
+
   def scm
     Blastr::SourceControl::Mercurial
   end
+  
+  def logentry(revision, author = "johndoe", comment = "Small fix")
+    Blastr::SourceControl::LogEntry.new(revision, author, comment)
+  end
+  
+  def revision(value)
+    Blastr::SourceControl::MercurialRevision.new(value)
+  end
 
 end

data/test/scm/test_mercurial_revision.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestMercurialRevision < Test::Unit::TestCase

data/test/scm/test_svn.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestSubversion < AbstractScmTestCase
@@ -40,9 +41,23 @@ class TestSubversion < AbstractScmTestCase
     assert_urls_are_not_understood([ parent_of_svn_repo ])
   end
   
-  def test_subversion
+  def test_subversion_urls_are_understood
     assert_urls_are_understood(SVN_URLS)
   end
+  
+  def test_detects_latest_revision_from_commit_log
+    repo = scm.new("/fakerepo")
+    repo.expects(:commits_since).with(revision("HEAD")).returns([
+      logentry(revision("15"), "author", "comment")
+    ])
+    assert_equal revision("15"), repo.latest_revision
+  end
+  
+  def test_latest_revision_for_empty_log_defaults_to_revision_1
+    repo = scm.new("/fakerepo")
+    repo.expects(:commits_since).with(revision("HEAD")).returns([])
+    assert_equal revision("1"), repo.latest_revision
+  end
 
   private
   
@@ -50,6 +65,14 @@ class TestSubversion < AbstractScmTestCase
     Blastr::SourceControl::Subversion
   end
   
+  def logentry(revision, author = "johndoe", comment = "Small fix")
+    Blastr::SourceControl::LogEntry.new(revision, author, comment)
+  end
+  
+  def revision(value)
+    Blastr::SourceControl::SubversionRevision.new(value)
+  end
+  
   def create_local_repo
     dir = File.join(Blastr::FileSystem.temp_dir, 'svn_repo')
     %x[svnadmin create #{dir}]

data/test/scm/test_svn_revision.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 
 class TestSubversionRevision < Test::Unit::TestCase

data/test/test_blastr.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), 'test_helper.rb'))
 require 'mocha/setup'
 

data/test/test_filesystem.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), 'test_helper.rb'))
 
 module Blastr::FileSystem

data/test/test_helper.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require 'stringio'
 require 'test/unit'
 require File.expand_path(File.join(File.dirname(__FILE__), '../lib/blastr'))

data/test/tts/test_tts.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 require File.expand_path(File.join(File.dirname(__FILE__), '../test_helper.rb'))
 require 'mocha/setup'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: blastr
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-12-17 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &70176230716280 !ruby/object:Gem::Requirement
+  requirement: &70357899010860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70176230716280
+  version_requirements: *70357899010860
 - !ruby/object:Gem::Dependency
   name: git
-  requirement: &70176230715780 !ruby/object:Gem::Requirement
+  requirement: &70357899010420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70176230715780
+  version_requirements: *70357899010420
 description: Blastr observes a version control repository for commits and makes audible
   announcements out of the commit messages.
 email: