buildr 1.3.5 → 1.4.0

data/spec/core/application_spec.rb

@@ -72,7 +72,7 @@ describe Buildr::Application do
       Buildr.application.environment.should eql('test')
       ENV['BUILDR_ENV'].should eql('test')
     end
-    
+
     it 'should be echoed to user' do
       write 'buildfile'
       ENV['BUILDR_ENV'] = 'spec'
@@ -80,12 +80,22 @@ describe Buildr::Application do
       lambda { Buildr.application.send :load_buildfile }.should show(%r{(in .*, spec)})
     end
   end
-  
+
   describe 'options' do
     it "should have 'tasks' as the sole default rakelib" do
       Buildr.application.send(:handle_options)
       Buildr.application.options.rakelib.should == ['tasks']
     end
+
+    it 'should show the version when prompted with -V' do
+      ARGV.push('-V')
+      test_exit(0) { Buildr.application.send(:handle_options) }.should show(/Buildr #{Buildr::VERSION}.*/)
+    end
+
+    it 'should show the version when prompted with --version' do
+      ARGV.push('--version')
+      test_exit(0) { Buildr.application.send(:handle_options) }.should show(/Buildr #{Buildr::VERSION}.*/)
+    end
   end
 
   describe 'gems' do
@@ -248,45 +258,45 @@ describe Buildr::Application do
       @original_loaded_features = $LOADED_FEATURES.dup
       Buildr.application.options.rakelib = ["tasks"]
     end
-    
+
     after do
       $taskfiles = nil
       ($LOADED_FEATURES - @original_loaded_features).each do |new_load|
         $LOADED_FEATURES.delete(new_load)
       end
     end
-    
+
     def write_task(filename)
       write filename, <<-RUBY
         $taskfiles ||= []
         $taskfiles << __FILE__
       RUBY
     end
-    
+
     def loaded_tasks
       @loaded ||= Buildr.application.load_tasks
       $taskfiles
     end
-    
+
     it "should load {options.rakelib}/foo.rake" do
       write_task 'tasks/foo.rake'
       loaded_tasks.should have(1).task
       loaded_tasks.first.should =~ %r{tasks/foo\.rake$}
     end
-    
+
     it 'should load all *.rake files from the rakelib' do
       write_task 'tasks/bar.rake'
       write_task 'tasks/quux.rake'
       loaded_tasks.should have(2).tasks
     end
-    
+
     it 'should not load files which do not have the .rake extension' do
       write_task 'tasks/foo.rb'
       write_task 'tasks/bar.rake'
       loaded_tasks.should have(1).task
       loaded_tasks.first.should =~ %r{tasks/bar\.rake$}
     end
-    
+
     it 'should load files only from the directory specified in the rakelib option' do
       Buildr.application.options.rakelib = ['extensions']
       write_task 'extensions/amp.rake'
@@ -297,7 +307,7 @@ describe Buildr::Application do
         loaded_tasks.select{|x| x =~ %r{extensions/#{filename}\.rake}}.should have(1).entry
       end
     end
-    
+
     it 'should load files from all the directories specified in the rakelib option' do
       Buildr.application.options.rakelib = ['ext', 'more', 'tasks']
       write_task 'ext/foo.rake'
@@ -306,17 +316,71 @@ describe Buildr::Application do
       write_task 'more/baz.rake'
       loaded_tasks.should have(4).tasks
     end
-    
+
     it 'should not load files from the rakelib more than once' do
       write_task 'tasks/new_one.rake'
       write_task 'tasks/already.rake'
       $LOADED_FEATURES << 'tasks/already.rake'
-      
+
       loaded_tasks.should have(1).task
       loaded_tasks.first.should =~ %r{tasks/new_one\.rake$}
     end
   end
 
+  describe 'exception handling' do
+
+    it 'should exit when given a SystemExit exception' do
+      test_exit(3) { Buildr.application.standard_exception_handling { raise SystemExit.new(3) } }
+    end
+
+    it 'should exit with status 1 when given an OptionParser::ParseError exception' do
+      test_exit(1) { Buildr.application.standard_exception_handling { raise OptionParser::ParseError.new() } }
+    end
+
+    it 'should exit with status 1 when given any other type of exception exception' do
+      test_exit(1) { Buildr.application.standard_exception_handling { raise Exception.new() } }
+    end
+
+    it 'should print the class name and the message when receiving an exception (except when the exception is named Exception)' do
+
+      # Our fake $stderr for the exercise. We could start it with a matcher instead
+      class FakeStdErr
+
+        attr_accessor :messages
+
+        def puts(*args)
+          @messages ||= []
+          @messages += args
+        end
+
+        alias :write :puts
+      end
+
+      # Save the old $stderr and make sure to restore it in the end.
+      old_stderr = $stderr
+      begin
+
+        $stderr = FakeStdErr.new
+        test_exit(1) {
+          Buildr.application.send :standard_exception_handling do
+            class MyOwnNicelyNamedException < Exception
+            end
+            raise MyOwnNicelyNamedException.new('My message')
+          end
+        }.call
+        $stderr.messages.select {|msg| msg =~ /.*MyOwnNicelyNamedException : My message.*/}.size.should == 1
+        $stderr.messages.clear
+        test_exit(1) {
+          Buildr.application.send :standard_exception_handling do
+            raise Exception.new('My message')
+          end
+        }.call
+        $stderr.messages.select {|msg| msg =~ /.*My message.*/ && !(msg =~ /Exception/)}.size.should == 1
+      end
+      $stderr = old_stderr
+    end
+  end
+
 end
 
 
@@ -439,48 +503,55 @@ describe Buildr, 'settings' do
       @buildfile_time = Time.now - 10
       write 'buildfile'; File.utime(@buildfile_time, @buildfile_time, 'buildfile')
     end
-    
+
     it 'should point to the buildfile' do
       Buildr.application.buildfile.should point_to_path('buildfile')
     end
-    
+
     it 'should be a defined task' do
       Buildr.application.buildfile.should == file(File.expand_path('buildfile'))
     end
-    
+
     it 'should ignore any rake namespace' do
       namespace 'dummy_ns' do
         Buildr.application.buildfile.should point_to_path('buildfile')
       end
     end
-    
+
     it 'should have the same timestamp as the buildfile' do
       Buildr.application.buildfile.timestamp.should be_close(@buildfile_time, 1)
     end
-    
+
     it 'should have the same timestamp as build.yaml if the latter is newer' do
       write 'build.yaml'; File.utime(@buildfile_time + 5, @buildfile_time + 5, 'build.yaml')
       Buildr.application.run
       Buildr.application.buildfile.timestamp.should be_close(@buildfile_time + 5, 1)
     end
-    
+
     it 'should have the same timestamp as the buildfile if build.yaml is older' do
       write 'build.yaml'; File.utime(@buildfile_time - 5, @buildfile_time - 5, 'build.yaml')
       Buildr.application.run
       Buildr.application.buildfile.timestamp.should be_close(@buildfile_time, 1)
     end
-    
-    it 'should have the same timestamp as build.rb in home dir if the latter is newer' do
+
+    it 'should have the same timestamp as build.rb in home dir if the latter is newer (until version 1.6)' do
+      Buildr::VERSION.should < '1.6'
       write 'home/buildr.rb'; File.utime(@buildfile_time + 5, @buildfile_time + 5, 'home/buildr.rb')
       Buildr.application.send :load_tasks
       Buildr.application.buildfile.timestamp.should be_close(@buildfile_time + 5, 1)
     end
+
+    it 'should have the same timestamp as build.rb in home dir if the latter is newer' do
+      write 'home/.buildr/buildr.rb'; File.utime(@buildfile_time + 5, @buildfile_time + 5, 'home/.buildr/buildr.rb')
+      Buildr.application.send :load_tasks
+      Buildr.application.buildfile.timestamp.should be_close(@buildfile_time + 5, 1)
+    end
   end
 end
 
 
 describe Buildr do
-  
+
   describe 'environment' do
     it 'should be same as Buildr.application.environment' do
       Buildr.environment.should eql(Buildr.application.environment)
@@ -498,5 +569,24 @@ describe Buildr do
       Buildr.settings.should == Buildr.application.settings
     end
   end
+
+end
+
+describe Rake do
+  describe 'define_task' do
+   it 'should restore call chain when invoke is called' do
+     task1 = Rake::Task.define_task('task1') do
+     end
+
+     task2 = Rake::Task.define_task('task2') do
+       chain1 = Thread.current[:rake_chain]
+       task1.invoke
+       chain2 = Thread.current[:rake_chain]
+       chain2.should == chain1
+     end
+
+     task2.invoke
+   end
+ end
 end
 

data/spec/core/build_spec.rb

@@ -169,7 +169,7 @@ describe Project, '#target' do
     @project.layout[:target] = 'baz'
     @project.target.should eql('baz')
   end
-  
+
   it 'should be removed in version 1.5 since it was deprecated in version 1.3' do
     Buildr::VERSION.should < '1.5'
   end
@@ -194,7 +194,7 @@ describe Project, '#reports' do
     @project.layout[:reports] = 'baz'
     @project.reports.should eql('baz')
   end
-  
+
   it 'should be removed in version 1.5 since it was deprecated in version 1.3' do
     Buildr::VERSION.should < '1.5'
   end
@@ -221,7 +221,7 @@ nothing to commit (working directory clean)
 #
 #       modified:   lib/buildr.rb
 #       modified:   spec/buildr_spec.rb
-# 
+#
 # Untracked files:
 #   (use "git add <file>..." to include in what will be committed)
 #
@@ -232,7 +232,7 @@ nothing to commit (working directory clean)
 
     it 'should reject a dirty repository, Git 1.4.3 or higher' do
       Git.should_receive(:`).with('git status').and_return <<-EOF
-# On branch master 
+# On branch master
 # Changed but not updated:
 #   (use "git add <file>..." to update what will be committed)
 #
@@ -381,7 +381,7 @@ describe 'a release process', :shared=>true do
       @release.should_receive(:ruby).with('-S', 'buildr', "_#{Buildr::VERSION}_", '--buildfile', File.expand_path('buildfile.next'),
                                           '--environment', 'development', 'clean', 'upload', 'DEBUG=no')
     end
-    
+
     it 'should tag a release with the release version' do
       @release.stub!(:update_version_to_next)
       @release.should_receive(:tag_release).with('1.0.0')
@@ -412,7 +412,7 @@ describe 'a release process', :shared=>true do
       @release.stub!(:tag_release)
       @release.make
       file('buildfile').should contain('VERSION_NUMBER = "1.0.1-SNAPSHOT"')
-    end    
+    end
   end
 
   describe '#resolve_tag' do
@@ -421,15 +421,15 @@ describe 'a release process', :shared=>true do
     end
 
     it 'should return tag specified by tag_name' do
-      @release.tag_name  = 'first'
+      Release.tag_name  = 'first'
       @release.send(:resolve_tag).should == 'first'
     end
-    
+
     it 'should use tag returned by tag_name if tag_name is a proc' do
-      @release.tag_name  = lambda { |version| "buildr-#{version}" }
+      Release.tag_name  = lambda { |version| "buildr-#{version}" }
       @release.send(:resolve_tag).should == 'buildr-1.0.0'
     end
-    after { @release.tag_name = nil }
+    after { Release.tag_name = nil }
   end
 
   describe '#tag_release' do
@@ -501,13 +501,13 @@ describe 'a release process', :shared=>true do
     end
 
     it 'should use the commit message specified by commit_message' do
-      @release.commit_message  = 'Here is my custom message'
+      Release.commit_message  = 'Here is my custom message'
       @release.should_receive(:message).and_return('Here is my custom message')
       @release.update_version_to_next
     end
-    
+
     it 'should use the commit message returned by commit_message if commit_message is a proc' do
-      @release.commit_message  = lambda { |new_version| 
+      Release.commit_message  = lambda { |new_version|
         new_version.should == '1.0.1-SNAPSHOT'
         "increment version number to #{new_version}"
       }
@@ -518,6 +518,7 @@ describe 'a release process', :shared=>true do
     it 'should inform the user of the new version' do
       lambda { @release.update_version_to_next }.should show_info('Current version is now 1.0.1-SNAPSHOT')
     end
+    after { Release.commit_message = nil }
   end
 
 end
@@ -542,7 +543,7 @@ describe GitRelease do
     it 'should accept a git repo' do
       FileUtils.mkdir '.git'
       FileUtils.touch File.join('.git', 'config')
-      GitRelease.applies_to?.should be_true 
+      GitRelease.applies_to?.should be_true
     end
   end
 
@@ -636,7 +637,7 @@ describe SvnRelease do
 
     it 'should accept a git repo' do
       FileUtils.touch '.svn'
-      SvnRelease.applies_to?.should be_true 
+      SvnRelease.applies_to?.should be_true
     end
   end
 

data/spec/core/cc_spec.rb

@@ -0,0 +1,174 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with this
+# work for additional information regarding copyright ownership.  The ASF
+# licenses this file to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+# License for the specific language governing permissions and limitations under
+# the License.
+
+
+require File.join(File.dirname(__FILE__), '../spec_helpers')
+
+module CCHelper
+  def setup_cc
+    project = define('foo')
+
+    cc = project.cc
+    project.stub!(:task).with(:cc).and_return(cc)
+
+    compile = mock 'compile'
+    project.stub!(:task).with(:compile).and_return(compile)
+
+    test_compile = mock 'test:compile'
+    project.stub!(:task).with('test:compile').and_return(test_compile)
+
+    filter = mock('resources').tap do |resources|
+      project.stub!(:task).with(:resources).and_return(resources)
+
+      back = mock 'filter'
+      resources.stub!(:filter).and_return(back)
+
+      back
+    end
+
+    sources
+    tests
+    resources
+
+    [ project, compile, test_compile, filter ]
+  end
+
+  def sources
+    @sources ||= ['Test1.java', 'Test2.java'].map { |f| File.join('src/main/java/thepackage', f) }.
+      each { |src| write src, "package thepackage; class #{src.pathmap('%n')} {}" }
+  end
+
+  def tests
+    @tests ||= ['Test1.java', 'Test2.java'].map { |f| File.join('src/test/java/thepackage', f) }.
+      each { |src| write src, "package thepackage; class #{src.pathmap('%n')} {}" }
+  end
+
+  def resources
+    @resources ||= ['Test1.html', 'Test2.html'].map { |f| File.join('src/main/resources', f) }.
+      each { |src| write src, '<html></html>' }
+  end
+end
+
+describe Buildr::CCTask do
+  include CCHelper
+
+  it 'should default to a delay of 0.2' do
+    define('foo').cc.delay.should == 0.2
+  end
+
+  it 'should compile and test:compile on initial start' do
+    project, compile, test_compile, filter = setup_cc
+
+    compile.should_receive :invoke
+    test_compile.should_receive :invoke
+    filter.should_not_receive :run
+
+    thread = Thread.new do
+      project.cc.invoke
+    end
+
+    sleep 0.5
+
+    thread.exit
+  end
+
+  it 'should detect a file change' do |spec|
+    
+    write 'src/main/java/Example.java', "public class Example {}"
+    write 'src/test/java/ExampleTest.java', "public class ExampleTest {}"
+    
+    project = define("foo")
+    cc = project.cc
+    
+    compile = project.compile
+    
+
+    test_compile = project.test.compile
+
+    filter = project.resources
+    
+    # After first period:
+    compile.should_receive :invoke
+    test_compile.should_receive :invoke
+    filter.should_not_receive :run
+    
+    thread = Thread.new do
+      project.cc.invoke
+    end
+    
+    sleep 1
+    
+    compile.should_receive :reenable
+    compile.should_receive :invoke
+
+    test_compile.should_receive :reenable
+    test_compile.should_receive :invoke
+
+    filter.should_not_receive :run
+    
+    sleep 1 # Wait one sec as the timestamp needs to be different.
+    touch File.join(Dir.pwd, 'src/main/java/Example.java')
+    sleep 0.2# Wait one standard delay
+    
+    thread.exit
+  end
+  
+  
+  it 'should support subprojects' do |spec|
+    
+    write 'foo/src/main/java/Example.java', "public class Example {}"
+    write 'foo/src/test/java/ExampleTest.java', "public class ExampleTest {}"
+    
+    define 'container' do
+      define('foo')
+    end
+    
+    project = project("container:foo")
+    cc = project.cc
+    
+    compile = project.compile
+
+    test_compile = project.test.compile
+
+    filter = project.resources
+    
+    
+    # After first period:
+    compile.should_receive :invoke
+    test_compile.should_receive :invoke
+    filter.should_not_receive :run
+    
+    thread = Thread.new do
+      project("container").cc.invoke
+    end
+    
+    sleep 0.5
+    
+    # After we changed the file:
+    compile.should_receive :reenable
+    compile.should_receive :invoke
+
+    test_compile.should_receive :reenable
+    test_compile.should_receive :invoke
+    
+    filter.should_not_receive :run
+    
+    sleep 1 # Wait one sec as the timestamp needs to be different.
+    touch File.join(Dir.pwd, 'foo/src/main/java/Example.java')
+    sleep 0.2 # Wait one standard delay
+    
+    thread.exit
+  end
+end