buildr 1.2.10 → 1.3.0

data/spec/java_compilers_spec.rb

@@ -0,0 +1,446 @@
+# 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')
+
+
+describe 'javac compiler' do
+  it 'should identify itself from source directories' do
+    write 'src/main/java/com/example/Test.java', 'package com.example; class Test {}' 
+    define('foo').compile.compiler.should eql(:javac)
+  end
+
+  it 'should identify from source directories using custom layout' do
+    write 'src/com/example/Code.java', 'package com.example; class Code {}' 
+    write 'testing/com/example/Test.java', 'package com.example; class Test {}' 
+    custom = Layout.new
+    custom[:source, :main, :java] = 'src'
+    custom[:source, :test, :java] = 'testing'
+    define 'foo', :layout=>custom do
+      compile.compiler.should eql(:javac)
+      test.compile.compiler.should eql(:javac)
+    end
+  end
+
+  it 'should identify from compile source directories' do
+    write 'src/com/example/Code.java', 'package com.example; class Code {}' 
+    write 'testing/com/example/Test.java', 'package com.example; class Test {}' 
+    define 'foo' do
+      lambda { compile.from 'src' }.should change { compile.compiler }.to(:javac)
+      lambda { test.compile.from 'testing' }.should change { test.compile.compiler }.to(:javac)
+    end
+  end
+
+  it 'should report the language as :java' do
+    define('foo').compile.using(:javac).language.should eql(:java)
+  end
+
+  it 'should set the target directory to target/classes' do
+    define 'foo' do
+      lambda { compile.using(:javac) }.should change { compile.target.to_s }.to(File.expand_path('target/classes'))
+    end
+  end
+
+  it 'should not override existing target directory' do
+    define 'foo' do
+      compile.into('classes')
+      lambda { compile.using(:javac) }.should_not change { compile.target }
+    end
+  end
+
+  it 'should not change existing list of sources' do
+    define 'foo' do
+      compile.from('sources')
+      lambda { compile.using(:javac) }.should_not change { compile.sources }
+    end
+  end
+
+  it 'should include classpath dependencies' do
+    write 'src/dependency/Dependency.java', 'class Dependency {}'
+    define 'dependency', :version=>'1.0' do
+      compile.from('src/dependency').into('target/dependency')
+      package(:jar)
+    end
+    write 'src/test/DependencyTest.java', 'class DependencyTest { Dependency _var; }'
+    define('foo').compile.from('src/test').with(project('dependency')).invoke
+    file('target/classes/DependencyTest.class').should exist
+  end
+
+  it 'should include tools.jar dependency' do
+    write 'src/main/java/UseApt.java', <<-JAVA
+      import com.sun.mirror.apt.AnnotationProcessor;
+      public class UseApt { }
+    JAVA
+    define('foo').compile.invoke
+    file('target/classes/UseApt.class').should exist
+  end
+end
+
+
+describe 'javac compiler options' do
+  def compile_task
+    @compile_task ||= define('foo').compile.using(:javac)
+  end
+
+  def javac_args
+    compile_task.instance_eval { @compiler }.send(:javac_args)
+  end
+
+  it 'should set warnings option to false by default' do
+    compile_task.options.warnings.should be_false
+  end
+
+  it 'should set wranings option to true when running with --verbose option' do
+    verbose true
+    compile_task.options.warnings.should be_true
+  end
+
+  it 'should use -nowarn argument when warnings is false' do
+    compile_task.using(:warnings=>false)
+    javac_args.should include('-nowarn') 
+  end
+
+  it 'should not use -nowarn argument when warnings is true' do
+    compile_task.using(:warnings=>true)
+    javac_args.should_not include('-nowarn') 
+  end
+
+  it 'should not use -verbose argument by default' do
+    javac_args.should_not include('-verbose') 
+  end
+
+  it 'should use -verbose argument when running with --trace option' do
+    trace true
+    javac_args.should include('-verbose') 
+  end
+
+  it 'should set debug option to true by default' do
+    compile_task.options.debug.should be_true
+  end
+
+  it 'should set debug option to false based on Buildr.options' do
+    Buildr.options.debug = false
+    compile_task.options.debug.should be_false
+  end
+
+  it 'should set debug option to false based on debug environment variable' do
+    ENV['debug'] = 'no'
+    compile_task.options.debug.should be_false
+  end
+
+  it 'should set debug option to false based on DEBUG environment variable' do
+    ENV['DEBUG'] = 'no'
+    compile_task.options.debug.should be_false
+  end
+
+  it 'should use -g argument when debug option is true' do
+    compile_task.using(:debug=>true)
+    javac_args.should include('-g')
+  end
+
+  it 'should not use -g argument when debug option is false' do
+    compile_task.using(:debug=>false)
+    javac_args.should_not include('-g')
+  end
+
+  it 'should set deprecation option to false by default' do
+    compile_task.options.deprecation.should be_false
+  end
+
+  it 'should use -deprecation argument when deprecation is true' do
+    compile_task.using(:deprecation=>true)
+    javac_args.should include('-deprecation')
+  end
+
+  it 'should not use -deprecation argument when deprecation is false' do
+    compile_task.using(:deprecation=>false)
+    javac_args.should_not include('-deprecation')
+  end
+
+  it 'should not set source option by default' do
+    compile_task.options.source.should be_nil
+    javac_args.should_not include('-source')
+  end
+
+  it 'should not set target option by default' do
+    compile_task.options.target.should be_nil
+    javac_args.should_not include('-target')
+  end
+
+  it 'should use -source nn argument if source option set' do
+    compile_task.using(:source=>'1.5')
+    javac_args.should include('-source', '1.5')
+  end
+
+  it 'should use -target nn argument if target option set' do
+    compile_task.using(:target=>'1.5')
+    javac_args.should include('-target', '1.5')
+  end
+
+  it 'should set lint option to false by default' do
+    compile_task.options.lint.should be_false
+  end
+
+  it 'should use -lint argument if lint option is true' do
+    compile_task.using(:lint=>true)
+    javac_args.should include('-Xlint')
+  end
+
+  it 'should use -lint argument with value of option' do
+    compile_task.using(:lint=>'all')
+    javac_args.should include('-Xlint:all')
+  end
+
+  it 'should use -lint argument with value of option as array' do
+    compile_task.using(:lint=>['path', 'serial'])
+    javac_args.should include('-Xlint:path,serial')
+  end
+
+  it 'should not set other option by default' do
+    compile_task.options.other.should be_nil
+  end
+
+  it 'should pass other argument if other option is string' do
+    compile_task.using(:other=>'-Xprint')
+    javac_args.should include('-Xprint')
+  end
+
+  it 'should pass other argument if other option is array' do
+    compile_task.using(:other=>['-Xstdout', 'msgs'])
+    javac_args.should include('-Xstdout', 'msgs')
+  end
+
+  it 'should complain about options it doesn\'t know' do
+    write 'source/Test.java', 'class Test {}'
+    compile_task.using(:unknown=>'option')
+    lambda { compile_task.from('source').invoke }.should raise_error(ArgumentError, /no such option/i)
+  end
+
+  it 'should inherit options from parent' do
+    define 'foo' do
+      compile.using(:warnings=>true, :debug=>true, :deprecation=>true, :source=>'1.5', :target=>'1.4')
+      define 'bar' do
+        compile.using(:javac)
+        compile.options.warnings.should be_true
+        compile.options.debug.should be_true
+        compile.options.deprecation.should be_true
+        compile.options.source.should eql('1.5')
+        compile.options.target.should eql('1.4')
+      end
+    end
+  end
+
+  after do
+    Buildr.options.debug = nil
+    ENV.delete "debug"
+    ENV.delete "DEBUG"
+  end
+end
+
+
+describe Project, '#javadoc' do
+  def sources
+    @sources ||= (1..3).map { |i| "Test#{i}" }.
+      each { |name| write "src/main/java/foo/#{name}.java", "package foo; public class #{name}{}" }.
+      map { |name| "src/main/java/foo/#{name}.java" }
+  end
+
+  it 'should return the project\'s Javadoc task' do
+    define('foo') { compile.using(:javac) }
+    project('foo').javadoc.name.should eql('foo:javadoc')
+  end
+
+  it 'should return a Javadoc task' do
+    define('foo') { compile.using(:javac) }
+    project('foo').javadoc.should be_kind_of(Javadoc::JavadocTask)
+  end
+
+  it 'should set target directory to target/javadoc' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.target.to_s.should point_to_path('target/javadoc')
+    end
+  end
+
+  it 'should create file task for target directory' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.should_receive(:invoke_prerequisites)
+    end
+    project('foo').file('target/javadoc').invoke
+  end
+
+  it 'should respond to into() and return self' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.into('docs').should be(javadoc)
+    end
+  end
+
+  it 'should respond to into() and change target directory' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.into('docs')
+      javadoc.should_receive(:invoke_prerequisites)
+    end
+    file('docs').invoke
+  end
+
+  it 'should respond to from() and return self' do
+    task = nil
+    define('foo') { task = javadoc.from('srcs') }
+    task.should be(project('foo').javadoc)
+  end
+
+  it 'should respond to from() and add sources' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.from('srcs').should be(javadoc)
+    end
+  end
+
+  it 'should respond to from() and add file task' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.from('srcs').should be(javadoc)
+    end
+    project('foo').javadoc.source_files.first.should point_to_path('srcs')
+  end
+
+  it 'should respond to from() and add project\'s sources and dependencies' do
+    write 'bar/src/main/java/Test.java'
+    define 'foo' do
+      compile.using(:javac)
+      define('bar') { compile.using(:javac).with 'group:id:jar:1.0' }
+      javadoc.from project('foo:bar')
+    end
+    project('foo').javadoc.source_files.first.should point_to_path('bar/src/main/java/Test.java')
+    project('foo').javadoc.classpath.map(&:to_spec).should include('group:id:jar:1.0')
+  end
+
+  it 'should generate javadocs from project' do
+    sources
+    define('foo') { compile.using(:javac) }
+    project('foo').javadoc.source_files.sort.should == sources.sort.map { |f| File.expand_path(f) }
+  end
+
+  it 'should include compile dependencies' do
+    define('foo') { compile.using(:javac).with 'group:id:jar:1.0' }
+    project('foo').javadoc.classpath.map(&:to_spec).should include('group:id:jar:1.0')
+  end
+
+  it 'should respond to include() and return self' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.include('srcs').should be(javadoc)
+    end
+  end
+
+  it 'should respond to include() and add files' do
+    included = sources.first
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.include included
+    end
+    project('foo').javadoc.source_files.should include(included)
+  end
+
+  it 'should respond to exclude() and return self' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.exclude('srcs').should be(javadoc)
+    end
+  end
+
+  it 'should respond to exclude() and ignore files' do
+    excluded = sources.first
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.exclude excluded
+    end
+    sources
+    project('foo').javadoc.source_files.sort.should == sources[1..-1].map { |f| File.expand_path(f) }
+  end
+
+  it 'should respond to using() and return self' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.using(:windowtitle=>'Fooing').should be(javadoc)
+    end
+  end
+
+  it 'should respond to using() and accept options' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.using :windowtitle=>'Fooing'
+    end
+    project('foo').javadoc.options[:windowtitle].should eql('Fooing')
+  end
+
+  it 'should pick -windowtitle from project name' do
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.options[:windowtitle].should eql('foo')
+
+      define 'bar' do
+        compile.using(:javac)
+        javadoc.options[:windowtitle].should eql('foo:bar')
+      end
+    end
+  end
+
+  it 'should pick -windowtitle from project description' do
+    desc 'My App'
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.options[:windowtitle].should eql('My App')
+    end
+  end
+
+  it 'should produce documentation' do
+    sources
+    define('foo') { compile.using(:javac) }
+    project('foo').javadoc.invoke
+    (1..3).map { |i| "target/javadoc/foo/Test#{i}.html" }.each { |f| file(f).should exist }
+  end
+
+  it 'should fail on error' do
+    write 'Test.java', 'class Test {}'
+    define 'foo' do
+      compile.using(:javac)
+      javadoc.include 'Test.java'
+    end
+    lambda { project('foo').javadoc.invoke }.should raise_error(RuntimeError, /Failed to generate Javadocs/)
+  end
+
+  it 'should be local task' do
+    define 'foo' do
+      define('bar') { compile.using(:javac) }
+    end
+    project('foo:bar').javadoc.should_receive(:invoke_prerequisites)
+    in_original_dir(project('foo:bar').base_dir) { task('javadoc').invoke }
+  end
+
+  it 'should not recurse' do
+    define 'foo' do
+      compile.using(:javac)
+      define('bar') { compile.using(:javac) }
+    end
+    project('foo:bar').javadoc.should_not_receive(:invoke_prerequisites)
+    project('foo').javadoc.invoke
+  end
+end
+

data/spec/java_packaging_spec.rb

@@ -0,0 +1,1042 @@
+# 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')
+require File.join(File.dirname(__FILE__), 'packaging_helper')
+
+
+describe Project, '#manifest' do
+  it 'should include user name' do
+    ENV['USER'] = 'MysteriousJoe'
+    define('foo').manifest['Build-By'].should eql('MysteriousJoe')
+  end
+
+  it 'should include JDK version' do
+    define('foo').manifest['Build-Jdk'].should =~ /^1\.\d+\.\w+$/
+  end
+
+  it 'should include project comment' do
+    desc 'My Project'
+    define('foo').manifest['Implementation-Title'].should eql('My Project')
+  end
+
+  it 'should include project name if no comment' do
+    define('foo').manifest['Implementation-Title'].should eql('foo')
+  end
+
+  it 'should include project version' do
+    define('foo', :version=>'2.1').manifest['Implementation-Version'].should eql('2.1')
+  end
+
+  it 'should not include project version unless specified' do
+    define('foo').manifest['Implementation-Version'].should be_nil
+  end
+
+  it 'should inherit from parent project' do
+    define('foo', :version=>'2.1') { define 'bar' }
+    project('foo:bar').manifest['Implementation-Version'].should eql('2.1')
+  end
+
+end
+
+
+shared_examples_for 'package with manifest' do
+  before do
+    @long_line = 'No line may be longer than 72 bytes (not characters), in its UTF8-encoded form. If a value would make the initial line longer than this, it should be continued on extra lines (each starting with a single SPACE).'
+  end
+
+  def package_with_manifest(manifest = nil)
+    packaging = @packaging
+    @project = define('foo', :version=>'1.2') do
+      package packaging
+      package(packaging).with(:manifest=>manifest) unless manifest.nil?
+    end
+  end
+
+  def inspect_manifest
+    package = project('foo').package(@packaging)
+    package.invoke
+    yield Buildr::Packaging::Java::Manifest.from_zip(package)
+  end
+
+  it 'should include default header when no options specified' do
+    ENV['USER'] = 'MysteriousJoe'
+    package_with_manifest # Nothing for default.
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(1)
+      manifest.main.should == {
+        'Manifest-Version'        =>'1.0',
+        'Created-By'              =>'Buildr',
+        'Implementation-Title'    =>@project.name,
+        'Implementation-Version'  =>'1.2',
+        'Build-Jdk'               =>ENV_JAVA['java.version'],
+        'Build-By'                =>'MysteriousJoe'
+      }
+    end
+  end
+
+  it 'should not exist when manifest=false' do
+    package_with_manifest false
+    @project.package(@packaging).invoke
+    Zip::ZipFile.open(@project.package(@packaging).to_s) do |zip|
+      zip.file.exist?('META-INF/MANIFEST.MF').should be_false
+    end
+  end
+
+  it 'should map manifest from hash' do
+    package_with_manifest 'Foo'=>1, :bar=>'Bar'
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(1)
+      manifest.main['Manifest-Version'].should eql('1.0')
+      manifest.main['Created-By'].should eql('Buildr')
+      manifest.main['Foo'].should eql('1')
+      manifest.main['bar'].should eql('Bar')
+    end
+  end
+
+  it 'should end hash manifest with EOL' do
+    package_with_manifest 'Foo'=>1, :bar=>'Bar'
+    package = project('foo').package(@packaging)
+    package.invoke
+    Zip::ZipFile.open(package.to_s) { |zip| zip.file.read('META-INF/MANIFEST.MF').should =~ /#{Buildr::Packaging::Java::Manifest::LINE_SEPARATOR}$/ }
+  end
+
+  it 'should break hash manifest lines longer than 72 characters using continuations' do
+    package_with_manifest 'foo'=>@long_line
+    package = project('foo').package(@packaging)
+    inspect_manifest do |manifest|
+      manifest.main['foo'].should == @long_line
+    end
+  end
+
+  it 'should map manifest from array' do
+    package_with_manifest [ { :foo=>'first' }, { :bar=>'second' } ]
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(2)
+      manifest.main['Manifest-Version'].should eql('1.0')
+      manifest.main['foo'].should eql('first')
+      manifest.sections.last['bar'].should eql('second')
+    end
+  end
+
+  it 'should end array manifest with EOL' do
+    package_with_manifest [ { :foo=>'first' }, { :bar=>'second' } ]
+    package = project('foo').package(@packaging)
+    package.invoke
+    Zip::ZipFile.open(package.to_s) { |zip| zip.file.read('META-INF/MANIFEST.MF')[-1].should == ?\n }
+  end
+
+  it 'should break array manifest lines longer than 72 characters using continuations' do
+    package_with_manifest ['foo'=>@long_line]
+    package = project('foo').package(@packaging)
+    inspect_manifest do |manifest|
+      manifest.main['foo'].should == @long_line
+    end
+  end
+
+  it 'should put Name: at beginning of section' do
+    package_with_manifest [ {}, { 'Name'=>'first', :Foo=>'first', :bar=>'second' } ]
+    package = project('foo').package(@packaging)
+    package.invoke
+    inspect_manifest do |manifest|
+      manifest.sections[1]["Name"].should == "first"
+    end
+  end
+
+  it 'should create manifest from proc' do
+    package_with_manifest lambda { 'Meta: data' }
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(1)
+      manifest.main['Manifest-Version'].should eql('1.0')
+      manifest.main['Meta'].should eql('data')
+    end
+  end
+
+  it 'should create manifest from file' do
+    write 'MANIFEST.MF', 'Meta: data'
+    package_with_manifest 'MANIFEST.MF'
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(1)
+      manifest.main['Manifest-Version'].should eql('1.0')
+      manifest.main['Meta'].should eql('data')
+    end
+  end
+
+  it 'should create manifest from task' do
+    file 'MANIFEST.MF' do |task|
+      write task.to_s, 'Meta: data'
+    end
+    package_with_manifest 'MANIFEST.MF'
+    inspect_manifest do |manifest|
+      manifest.sections.size.should be(1)
+      manifest.main['Manifest-Version'].should eql('1.0')
+      manifest.main['Meta'].should eql('data')
+    end
+  end
+
+  it 'should respond to with() and accept manifest' do
+    write 'DISCLAIMER'
+    mkpath 'target/classes'
+    packaging = @packaging
+    define('foo', :version=>'1.0') { package(packaging).with :manifest=>{'Foo'=>'data'} }
+    inspect_manifest { |manifest| manifest.main['Foo'].should eql('data') }
+  end
+
+  it 'should include META-INF directory' do
+    packaging = @packaging
+    package = define('foo', :version=>'1.0') { package(packaging) }.packages.first
+    package.invoke
+    Zip::ZipFile.open(package.to_s) do |zip|
+      zip.entries.map(&:to_s).should include('META-INF/')
+    end
+  end
+end
+
+
+describe Project, '#meta_inf' do
+  it 'should by an array' do
+    define('foo').meta_inf.should be_kind_of(Array)
+  end
+
+  it 'should include LICENSE file if found' do
+    write 'LICENSE'
+    define('foo').meta_inf.first.should point_to_path('LICENSE')
+  end
+
+  it 'should be empty unless LICENSE exists' do
+    define('foo').meta_inf.should be_empty
+  end
+
+  it 'should inherit from parent project' do
+    write 'LICENSE'
+    define('foo') { define 'bar' }
+    project('foo:bar').meta_inf.first.should point_to_path('LICENSE')
+  end
+
+  it 'should expect LICENSE file parent project' do
+    write 'bar/LICENSE'
+    define('foo') { define 'bar' }
+    project('foo:bar').meta_inf.should be_empty
+  end
+end
+
+
+describe 'package with meta_inf', :shared=>true do
+
+  def package_with_meta_inf(meta_inf = nil)
+    packaging = @packaging
+    @project = Buildr.define('foo', :version=>'1.2') do
+      package packaging
+      package(packaging).with(:meta_inf=>meta_inf) if meta_inf
+    end
+  end
+
+  def inspect_meta_inf
+    package = project('foo').package(@packaging)
+    package.invoke
+    assumed = Array(@meta_inf_ignore)
+    Zip::ZipFile.open(package.to_s) do |zip|
+      entries = zip.entries.map(&:to_s).select { |f| File.dirname(f) == 'META-INF' }.map { |f| File.basename(f) }
+      assumed.each { |f| entries.should include(f) }
+      yield entries - assumed if block_given?
+    end
+  end
+
+  it 'should default to LICENSE file' do
+    write 'LICENSE'
+    package_with_meta_inf
+    inspect_meta_inf { |files| files.should eql(['LICENSE']) }
+  end
+
+  it 'should be empty if no LICENSE file' do
+    package_with_meta_inf
+    inspect_meta_inf { |files| files.should be_empty }
+  end
+
+  it 'should include file specified by :meta_inf option' do
+    write 'README'
+    package_with_meta_inf 'README'
+    inspect_meta_inf { |files| files.should eql(['README']) }
+  end
+
+  it 'should include files specified by :meta_inf option' do
+    files = ['README', 'DISCLAIMER'].each { |file| write file }
+    package_with_meta_inf files
+    inspect_meta_inf { |files| files.should eql(files) }
+  end
+
+  it 'should include file task specified by :meta_inf option' do
+    file('README') { |task| write task.to_s }
+    package_with_meta_inf file('README')
+    inspect_meta_inf { |files| files.should eql(['README']) }
+  end
+
+  it 'should include file tasks specified by :meta_inf option' do
+    files = ['README', 'DISCLAIMER'].each { |file| file(file) { |task| write task.to_s } }
+    package_with_meta_inf files.map { |f| file(f) }
+    inspect_meta_inf { |files| files.should eql(files) }
+  end
+
+  it 'should complain if cannot find file' do
+    package_with_meta_inf 'README'
+    lambda { inspect_meta_inf }.should raise_error(RuntimeError, /README/)
+  end
+
+  it 'should complain if cannot build task' do
+    file('README')  { fail 'Failed' }
+    package_with_meta_inf 'README'
+    lambda { inspect_meta_inf }.should raise_error(RuntimeError, /Failed/)
+  end
+
+  it 'should respond to with() and accept manifest and meta_inf' do
+    write 'DISCLAIMER'
+    mkpath 'target/classes'
+    packaging = @packaging
+    define('foo', :version=>'1.0') { package(packaging).with :meta_inf=>'DISCLAIMER' }
+    inspect_meta_inf { |files| files.should eql(['DISCLAIMER']) }
+  end
+end
+
+
+describe Packaging, 'jar' do
+  it_should_behave_like 'packaging'
+  before { @packaging = :jar }
+  it_should_behave_like 'package with manifest'
+  it_should_behave_like 'package with meta_inf'
+  before { @meta_inf_ignore = 'MANIFEST.MF' }
+
+  it 'should use files from compile directory if nothing included' do
+    write 'src/main/java/Test.java', 'class Test {}'
+    define('foo', :version=>'1.0') { package(:jar) }
+    project('foo').package(:jar).invoke
+    Zip::ZipFile.open(project('foo').package(:jar).to_s) do |jar|
+      jar.entries.map(&:to_s).sort.should include('META-INF/MANIFEST.MF', 'Test.class')
+    end
+  end
+
+  it 'should use files from resources directory if nothing included' do
+    write 'src/main/resources/test/important.properties'
+    define('foo', :version=>'1.0') { package(:jar) }
+    project('foo').package(:jar).invoke
+    Zip::ZipFile.open(project('foo').package(:jar).to_s) do |jar|
+      jar.entries.map(&:to_s).sort.should include('test/important.properties')
+    end
+  end
+
+  it 'should include class directories' do
+    write 'src/main/java/code/Test.java', 'package code ; class Test {}'
+    define('foo', :version=>'1.0') { package(:jar) }
+    project('foo').package(:jar).invoke
+    Zip::ZipFile.open(project('foo').package(:jar).to_s) do |jar|
+      jar.entries.map(&:to_s).sort.should include('code/')
+    end
+  end
+
+  it 'should include resource files starting with dot' do
+    write 'src/main/resources/test/.config'
+    define('foo', :version=>'1.0') { package(:jar) }
+    project('foo').package(:jar).invoke
+    Zip::ZipFile.open(project('foo').package(:jar).to_s) do |jar|
+      jar.entries.map(&:to_s).sort.should include('test/.config')
+    end
+  end
+
+  it 'should include empty resource directories' do
+    mkpath 'src/main/resources/empty'
+    define('foo', :version=>'1.0') { package(:jar) }
+    project('foo').package(:jar).invoke
+    Zip::ZipFile.open(project('foo').package(:jar).to_s) do |jar|
+      jar.entries.map(&:to_s).sort.should include('empty/')
+    end
+  end
+end
+
+
+describe Packaging, 'war' do
+  it_should_behave_like 'packaging'
+  before { @packaging = :war }
+  it_should_behave_like 'package with manifest'
+  it_should_behave_like 'package with meta_inf'
+  before { @meta_inf_ignore = 'MANIFEST.MF' }
+
+  def make_jars
+    artifact('group:id:jar:1.0') { |t| write t.to_s }
+    artifact('group:id:jar:2.0') { |t| write t.to_s }
+  end
+
+  def inspect_war
+    project('foo').package(:war).invoke
+    Zip::ZipFile.open(project('foo').package(:war).to_s) do |war|
+      yield war.entries.map(&:to_s).sort
+    end
+  end
+
+  it 'should use files from webapp directory if nothing included' do
+    write 'src/main/webapp/test.html'
+    define('foo', :version=>'1.0') { package(:war) }
+    inspect_war { |files| files.should include('test.html') }
+  end
+
+  it 'should accept files from :classes option' do
+    write 'src/main/java/Test.java', 'class Test {}'
+    write 'classes/test'
+    define('foo', :version=>'1.0') { package(:war).with(:classes=>'classes') }
+    inspect_war { |files| files.should include('WEB-INF/classes/test') }
+  end
+
+  it 'should use files from compile directory if nothing included' do
+    write 'src/main/java/Test.java', 'class Test {}'
+    define('foo', :version=>'1.0') { package(:war) }
+    inspect_war { |files| files.should include('WEB-INF/classes/Test.class') }
+  end
+
+  it 'should ignore compile directory if no source files to compile' do
+    define('foo', :version=>'1.0') { package(:war) }
+    inspect_war { |files| files.should_not include('target/classes') }
+  end
+
+  it 'should include only specified classes directories' do
+    write 'src/main/java'
+    define('foo', :version=>'1.0') { package(:war).with :classes=>_('additional') }
+    project('foo').package(:war).classes.should_not include(project('foo').file('target/classes'))
+    project('foo').package(:war).classes.should include(project('foo').file('additional'))
+  end
+
+  it 'should use files from resources directory if nothing included' do
+    write 'src/main/resources/test/important.properties'
+    define('foo', :version=>'1.0') { package(:war) }
+    inspect_war { |files| files.should include('WEB-INF/classes/test/important.properties') }
+  end
+
+  it 'should include empty resource directories' do
+    mkpath 'src/main/resources/empty'
+    define('foo', :version=>'1.0') { package(:war) }
+    inspect_war { |files| files.should include('WEB-INF/classes/empty/') }
+  end
+
+  it 'should accept file from :libs option' do
+    make_jars
+    define('foo', :version=>'1.0') { package(:war).with(:libs=>'group:id:jar:1.0') }
+    inspect_war { |files| files.should include('META-INF/MANIFEST.MF', 'WEB-INF/lib/id-1.0.jar') }
+  end
+
+  it 'should accept file from :libs option' do
+    make_jars
+    define('foo', :version=>'1.0') { package(:war).with(:libs=>['group:id:jar:1.0', 'group:id:jar:2.0']) }
+    inspect_war { |files| files.should include('META-INF/MANIFEST.MF', 'WEB-INF/lib/id-1.0.jar', 'WEB-INF/lib/id-2.0.jar') }
+  end
+
+  it 'should use artifacts from compile classpath if no libs specified' do
+    make_jars
+    define('foo', :version=>'1.0') { compile.with 'group:id:jar:1.0', 'group:id:jar:2.0' ; package(:war) }
+    inspect_war { |files| files.should include('META-INF/MANIFEST.MF', 'WEB-INF/lib/id-1.0.jar', 'WEB-INF/lib/id-2.0.jar') }
+  end
+
+  it 'should include only specified libraries' do
+    define 'foo', :version=>'1.0' do
+      compile.with 'group:id:jar:1.0'
+      package(:war).with :libs=>'additional:id:jar:1.0'
+    end
+    project('foo').package(:war).libs.should_not include(artifact('group:id:jar:1.0'))
+    project('foo').package(:war).libs.should include(artifact('additional:id:jar:1.0'))
+  end
+
+end
+
+
+describe Packaging, 'aar' do
+  it_should_behave_like 'packaging'
+  before { @packaging = :aar }
+  it_should_behave_like 'package with manifest'
+  it_should_behave_like 'package with meta_inf'
+  before do
+    write 'src/main/axis2/services.xml'
+    @meta_inf_ignore = ['MANIFEST.MF', 'services.xml']
+  end
+
+  def make_jars
+    artifact('group:id:jar:1.0') { |t| write t.to_s }
+    artifact('group:id:jar:2.0') { |t| write t.to_s }
+  end
+
+  def inspect_aar
+    project('foo').package(:aar).invoke
+    Zip::ZipFile.open(project('foo').package(:aar).to_s) do |aar|
+      yield aar.entries.map(&:to_s).sort
+    end
+  end
+
+  it 'should automatically include services.xml and any *.wsdl files under src/main/axis2' do
+    write 'src/main/axis2/my-service.wsdl'
+    define('foo', :version=>'1.0') { package(:aar) }
+    inspect_aar { |files| files.should include('META-INF/MANIFEST.MF', 'META-INF/services.xml', 'META-INF/my-service.wsdl') }
+  end
+
+  it 'should accept files from :include option' do
+    write 'test'
+    define('foo', :version=>'1.0') { package(:aar).include 'test' }
+    inspect_aar { |files| files.should include('META-INF/MANIFEST.MF', 'test') }
+  end
+
+  it 'should use files from compile directory if nothing included' do
+    write 'src/main/java/Test.java', 'class Test {}'
+    define('foo', :version=>'1.0') { package(:aar) }
+    inspect_aar { |files| files.should include('Test.class') }
+  end
+
+  it 'should use files from resources directory if nothing included' do
+    write 'src/main/resources/test/important.properties'
+    define('foo', :version=>'1.0') { package(:aar) }
+    inspect_aar { |files| files.should include('test/important.properties') }
+  end
+
+  it 'should include empty resource directories' do
+    mkpath 'src/main/resources/empty'
+    define('foo', :version=>'1.0') { package(:aar) }
+    inspect_aar { |files| files.should include('empty/') }
+  end
+
+  it 'should accept file from :libs option' do
+    make_jars
+    define('foo', :version=>'1.0') { package(:aar).with :libs=>'group:id:jar:1.0' }
+    inspect_aar { |files| files.should include('META-INF/MANIFEST.MF', 'lib/id-1.0.jar') }
+  end
+
+  it 'should accept file from :libs option' do
+    make_jars
+    define('foo', :version=>'1.0') { package(:aar).with :libs=>['group:id:jar:1.0', 'group:id:jar:2.0'] }
+    inspect_aar { |files| files.should include('META-INF/MANIFEST.MF', 'lib/id-1.0.jar', 'lib/id-2.0.jar') }
+  end
+
+  it 'should NOT use artifacts from compile classpath if no libs specified' do
+    make_jars
+    define('foo', :version=>'1.0') { compile.with 'group:id:jar:1.0', 'group:id:jar:2.0' ; package(:aar) }
+    inspect_aar { |files| files.should include('META-INF/MANIFEST.MF') }
+  end
+
+  it 'should return all libraries from libs attribute' do
+    define 'foo', :version=>'1.0' do
+      compile.with 'group:id:jar:1.0'
+      package(:aar).with :libs=>'additional:id:jar:1.0'
+    end
+    project('foo').package(:aar).libs.should_not include(artifact('group:id:jar:1.0'))
+    project('foo').package(:aar).libs.should include(artifact('additional:id:jar:1.0'))
+  end
+
+end
+
+
+describe Packaging, 'ear' do
+  it_should_behave_like 'packaging'
+  before { @packaging = :ear }
+  it_should_behave_like 'package with manifest'
+  it_should_behave_like 'package with meta_inf'
+  before { @meta_inf_ignore = ['MANIFEST.MF', 'application.xml'] }
+
+  def inspect_ear
+    project('foo').package(:ear).invoke
+    Zip::ZipFile.open(project('foo').package(:ear).to_s) do |ear|
+      yield ear.entries.map(&:to_s).sort
+    end
+  end
+
+  def inspect_application_xml
+    project('foo').package(:ear).invoke
+    Zip::ZipFile.open(project('foo').package(:ear).to_s) do |ear|
+      yield REXML::Document.new(ear.read('META-INF/application.xml')).root
+    end
+  end
+
+  def inspect_classpath(package)
+    project('foo').package(:ear).invoke
+    Zip::ZipFile.open(project('foo').package(:ear).to_s) do |ear|
+      File.open('tmp.zip', 'wb') do |tmp|
+        tmp.write ear.file.read(package)
+      end
+      manifest = Buildr::Packaging::Java::Manifest.from_zip('tmp.zip')
+      yield manifest.main['Class-Path'].to_s.split(' ')
+    end
+  end
+
+  it 'should set display name from project id' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).display_name.should eql('foo')
+      define 'bar' do
+        package(:ear).display_name.should eql('foo-bar')
+      end
+    end
+  end
+
+  it 'should set display name in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear)
+    end
+    inspect_application_xml { |xml| xml.get_text('/application/display-name').should == 'foo' }
+  end
+
+  it 'should accept different display name' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).display_name = 'bar'
+    end
+    inspect_application_xml { |xml| xml.get_text('/application/display-name').should == 'bar' }
+  end
+
+  it 'should map WARs to /war directory' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:war)
+    end
+    inspect_ear { |files| files.should include('war/foo-1.0.war') }
+  end
+
+  it 'should map EJBs to /ejb directory' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add :ejb=>package(:jar)
+    end
+    inspect_ear { |files| files.should include('ejb/foo-1.0.jar') }
+  end
+
+  it 'should not modify original artifact for its components' do
+    define 'one', :version => '1.0' do 
+      write 'src/main/resources/one.txt', '1'
+      package(:jar)
+    end
+    
+    define 'two', :version => '1.0' do 
+      write 'src/main/resources/two.txt', '2'
+      package(:jar)
+    end
+    
+    define 'foo', :version => '1.0' do 
+      package(:ear).add project(:one).package(:jar)
+      package(:ear).add :ejb => project(:two).package(:jar)
+    end
+
+    inspect_ear { |files| files.should include('lib/one-1.0.jar', 'ejb/two-1.0.jar') }
+    
+    Buildr::Packaging::Java::Manifest.from_zip(project('one').package(:jar)).main['Class-Path'].should be_nil
+    Buildr::Packaging::Java::Manifest.from_zip(project('two').package(:jar)).main['Class-Path'].should be_nil
+    
+    inspect_classpath 'ejb/two-1.0.jar' do |classpath|
+      classpath.should include('../lib/one-1.0.jar')
+    end
+  end
+  
+  it 'should map JARs to /lib directory' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:jar)
+    end
+    inspect_ear { |files| files.should include('lib/foo-1.0.jar') }
+  end
+
+  it 'should accept component type with :type option' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add package(:jar), :type=>:ejb
+    end
+    inspect_ear { |files| files.should include('ejb/foo-1.0.jar') }
+  end
+
+  it 'should accept component and its type as type=>artiract' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add :ejb=>package(:jar)
+    end
+    inspect_ear { |files| files.should include('ejb/foo-1.0.jar') }
+  end
+
+  it 'should map typed JARs to /jar directory' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add :jar=>package(:jar)
+    end
+    inspect_ear { |files| files.should include('jar/foo-1.0.jar') }
+  end
+
+  it 'should add multiple components at a time using the type=>component style' do
+    define 'bar', :version => '1.5' do
+      package(:war) # must be added as a webapp
+      package(:jar) # must be added as a shared lib
+      package(:zip) # this one should be excluded
+    end
+    define 'baz', :version => '1.5' do
+      package(:jar, :id => 'one')
+      package(:jar, :id => 'two')
+    end
+    define 'foo', :version => '1.0' do 
+      package(:ear).add :lib => project('baz'),
+                        :war => project('bar').package(:war),
+                        :ejb => project('bar').package(:jar)
+    end
+    inspect_ear do |files| 
+      files.should include(*%w{ lib/one-1.5.jar lib/two-1.5.jar war/bar-1.5.war ejb/bar-1.5.jar  })
+      files.should_not satisfy { files.any? { |f| f =~ /\.zip$/ } }
+    end
+  end
+
+  it 'should add all EAR supported packages when given a project argument' do
+    define 'bar', :version => '1.5' do
+      package(:war) # must be added as a webapp
+      package(:jar) # must be added as a shared lib
+      package(:zip) # this one should be excluded
+    end
+    define 'baz', :version => '1.5' do
+      package(:war)
+      package(:jar)
+    end
+    define 'foo', :version => '1.0' do 
+      package(:ear).add projects(:bar, :baz)
+    end
+    inspect_ear do |files| 
+      files.should include('war/bar-1.5.war', 'lib/bar-1.5.jar', 'lib/baz-1.5.jar', 'war/baz-1.5.war')
+      files.should_not satisfy { files.any? { |f| f =~ /\.zip$/ } }
+    end
+  end
+
+  it 'should complain about unknown component type' do
+    define 'foo', :version=>'1.0' do
+      lambda { package(:ear).add package(:zip) }.should raise_error(RuntimeError, /ear component/i)
+    end
+  end
+
+  it 'should allow unknown component types with explicit type' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add :lib=>package(:zip)
+    end
+    inspect_ear { |files| files.should include('lib/foo-1.0.zip') }
+  end
+
+  it 'should accept alternative directory name' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add package(:jar), :path=>'trash'
+    end
+    inspect_ear { |files| files.should include('trash/foo-1.0.jar') }
+  end
+
+  it 'should accept customization of directory map' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).dirs[:jar] = 'jarred'
+      package(:ear).add :jar=>package(:jar)
+    end
+    inspect_ear { |files| files.should include('jarred/foo-1.0.jar') }
+  end
+
+  it 'should accept customization of directory map with nil paths in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).dirs[:war] = nil
+      package(:ear).add :war=>package(:war)
+      package(:ear).add package(:jar)
+    end
+    inspect_ear { |files| files.should include('foo-1.0.war') }
+    inspect_application_xml do |xml|
+      xml.get_text("/application/module[@id='foo']/web/web-uri").to_s.should eql('foo-1.0.war')
+    end
+  end
+
+  it 'should accept customization of directory map with nil paths in the classpath' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).dirs[:lib] = nil
+      package(:ear).add :war=>package(:war)
+      package(:ear) << package(:jar)
+    end
+    inspect_classpath 'war/foo-1.0.war' do |classpath|
+      classpath.should include('../foo-1.0.jar')
+    end
+  end
+
+  it 'should list WAR components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:war) << package(:war, :id=>'bar')
+    end
+    inspect_application_xml do |xml|
+      xml.get_elements("/application/module[@id='foo'][web]").should_not be_empty
+      xml.get_elements("/application/module[@id='bar'][web]").should_not be_empty
+    end
+  end
+
+  it 'should specify web-uri for WAR components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:war)
+      package(:ear).add package(:war, :id=>'bar'), :path=>'ws'
+    end
+    inspect_application_xml do |xml|
+      xml.get_text("/application/module[@id='foo']/web/web-uri").to_s.should eql('war/foo-1.0.war')
+      xml.get_text("/application/module[@id='bar']/web/web-uri").to_s.should eql('ws/bar-1.0.war')
+    end
+  end
+
+  it 'should specify context-root for WAR components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:war)
+      package(:ear).add package(:war, :id=>'bar')
+    end
+    inspect_application_xml do |xml|
+      xml.get_text("/application/module[@id='foo']/web/context-root").to_s.should eql('/foo')
+      xml.get_text("/application/module[@id='bar']/web/context-root").to_s.should eql('/bar')
+    end
+  end
+
+  it 'should accept context-root for WAR components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add package(:war), :context_root=>'rooted'
+    end
+    inspect_application_xml do |xml|
+      xml.get_text("/application/module[@id='foo']/web/context-root").to_s.should eql('/rooted')
+    end
+  end
+
+  it 'should allow disabling the context root' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add package(:war), :context_root=>false
+    end
+    inspect_application_xml do |xml|
+      xml.get_elements("/application/module[@id='foo']/web/context-root").should be_empty
+    end
+  end
+
+  it 'should list EJB components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear).add :ejb=>package(:jar)
+      package(:ear).add :ejb=>package(:jar, :id=>'bar')
+    end
+    inspect_application_xml do |xml|
+      xml.get_text("/application/module[@id='foo']/ejb").to_s.should eql('ejb/foo-1.0.jar')
+      xml.get_text("/application/module[@id='bar']/ejb").to_s.should eql('ejb/bar-1.0.jar')
+    end
+  end
+
+  it 'should list JAR components in application.xml' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << { :jar=>package(:jar) } << { :jar=>package(:jar, :id=>'bar') }
+    end
+    inspect_application_xml do |xml|
+      jars = xml.get_elements('/application/jar').map(&:texts).map(&:to_s)
+      jars.should include('jar/foo-1.0.jar', 'jar/bar-1.0.jar')
+    end
+  end
+
+  it 'should update WAR component classpath to include libraries' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:jar, :id=>'lib1') << package(:jar, :id=>'lib2')
+      package(:ear).add package(:war)
+    end
+    inspect_classpath 'war/foo-1.0.war' do |classpath|
+      classpath.should include('../lib/lib1-1.0.jar', '../lib/lib2-1.0.jar')
+    end
+  end
+
+  it 'should update WAR component classpath but skip internal libraries' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:jar, :id=>'lib1') << package(:jar, :id=>'lib2')
+      package(:war).with(:libs=>package(:jar, :id=>'lib1'))
+      package(:ear).add package(:war)
+    end
+    inspect_classpath 'war/foo-1.0.war' do |classpath|
+      classpath.should_not include('../lib/lib1-1.0.jar')
+      classpath.should include('../lib/lib2-1.0.jar')
+    end
+  end
+
+  it 'should update EJB component classpath to include libraries' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:jar, :id=>'lib1') << package(:jar, :id=>'lib2')
+      package(:ear).add :ejb=>package(:jar)
+    end
+    inspect_classpath 'ejb/foo-1.0.jar' do |classpath|
+      classpath.should include('../lib/lib1-1.0.jar', '../lib/lib2-1.0.jar')
+    end
+  end
+
+  it 'should update JAR component classpath to include libraries' do
+    define 'foo', :version=>'1.0' do
+      package(:ear) << package(:jar, :id=>'lib1') << package(:jar, :id=>'lib2')
+      package(:ear).add :jar=>package(:jar)
+    end
+    inspect_classpath 'jar/foo-1.0.jar' do |classpath|
+      classpath.should include('../lib/lib1-1.0.jar', '../lib/lib2-1.0.jar')
+    end
+  end
+
+  it 'should deal with very long classpaths' do
+    define 'foo', :version=>'1.0' do
+      20.times { |i| package(:ear) << package(:jar, :id=>"lib#{i}") }
+      package(:ear).add :jar=>package(:jar)
+    end
+    inspect_classpath 'jar/foo-1.0.jar' do |classpath|
+      classpath.should include('../lib/lib1-1.0.jar', '../lib/lib2-1.0.jar')
+    end
+  end
+
+
+  it 'should generate relative classpaths for top level EJB' do 
+    define 'foo', :version => '1.0' do
+      package(:ear).add package(:jar, :id => 'one'), :path => '.'
+      package(:ear).add package(:jar, :id => 'two'), :path => 'dos'
+      package(:ear).add package(:jar, :id => 'three'), :path => 'tres'
+      package(:ear).add :ejb => package(:jar, :id => 'ejb1'), :path => '.'
+    end
+    inspect_classpath 'ejb1-1.0.jar' do |classpath|
+      classpath.should include(*%w{ one-1.0.jar dos/two-1.0.jar tres/three-1.0.jar })
+    end
+  end
+
+  it 'should generate relative classpaths for second level EJB' do 
+    define 'foo', :version => '1.0' do
+      package(:ear).add package(:jar, :id => 'one'), :path => '.'
+      package(:ear).add package(:jar, :id => 'two'), :path => 'dos'
+      package(:ear).add package(:jar, :id => 'three'), :path => 'tres'
+      package(:ear).add :ejb => package(:jar, :id => 'ejb2'), :path => 'dos'
+    end
+    inspect_classpath 'dos/ejb2-1.0.jar' do |classpath|
+      classpath.should include(*%w{ ../one-1.0.jar two-1.0.jar ../tres/three-1.0.jar })
+    end
+  end
+
+  it 'should generate relative classpaths for nested EJB' do 
+    define 'foo', :version => '1.0' do
+      package(:ear).add package(:jar, :id => 'one'), :path => '.'
+      package(:ear).add package(:jar, :id => 'two'), :path => 'dos'
+      package(:ear).add package(:jar, :id => 'three'), :path => 'dos/tres'
+      package(:ear).add package(:jar, :id => 'four'), :path => 'dos/cuatro'
+      package(:ear).add :ejb => package(:jar, :id => 'ejb4'), :path => 'dos/cuatro'
+    end
+    inspect_classpath 'dos/cuatro/ejb4-1.0.jar' do |classpath|
+      classpath.should include(*%w{ ../../one-1.0.jar ../two-1.0.jar ../tres/three-1.0.jar four-1.0.jar })
+    end
+  end
+
+end
+
+
+describe Packaging, 'sources' do
+  it_should_behave_like 'packaging'
+  before { @packaging, @package_type = :sources, :zip }
+
+  it 'should create package of type :zip and classifier \'sources\'' do
+    define 'foo', :version=>'1.0' do
+      package(:sources).type.should eql(:zip)
+      package(:sources).classifier.should eql('sources')
+      package(:sources).name.should match(/foo-1.0-sources.zip$/)
+    end
+  end
+
+  it 'should contain source files' do
+    write 'src/main/java/Source.java'
+    define('foo', :version=>'1.0') { package(:sources) }
+    project('foo').task('package').invoke
+    project('foo').packages.first.should contain('Source.java')
+  end
+
+  it 'should be a ZipTask' do
+    define 'foo', :version=>'1.0' do
+      package(:javadoc).should be_kind_of(ZipTask)
+    end
+  end
+end
+
+
+describe Packaging, 'javadoc' do
+  it_should_behave_like 'packaging'
+  before { @packaging, @package_type = :javadoc, :zip }
+
+  it 'should create package of type :zip and classifier \'javadoc\'' do
+    define 'foo', :version=>'1.0' do
+      package(:javadoc).type.should eql(:zip)
+      package(:javadoc).classifier.should eql('javadoc')
+      package(:javadoc).name.pathmap('%f').should eql('foo-1.0-javadoc.zip')
+    end
+  end
+
+  it 'should contain Javadocs' do
+    write 'src/main/java/Source.java', 'public class Source {}'
+    define('foo', :version=>'1.0') { package(:javadoc) }
+    project('foo').task('package').invoke
+    project('foo').packages.first.should contain('Source.html', 'index.html')
+  end
+
+  it 'should use project description in window title' do
+    write 'src/main/java/Source.java', 'public class Source {}'
+    desc 'My Project'
+    define('foo', :version=>'1.0') { package(:javadoc) }
+    project('foo').task('package').invoke
+    project('foo').packages.first.entry('index.html').should contain('My Project')
+  end
+
+  it 'should be a ZipTask' do
+    define 'foo', :version=>'1.0' do
+      package(:javadoc).should be_kind_of(ZipTask)
+    end
+  end
+end
+
+
+shared_examples_for 'package_with_' do
+
+  def prepare(options = {})
+    packager = "package_with_#{@packaging}" 
+    write 'src/main/java/Source.java'
+    write 'baz/src/main/java/Source.java'
+    define 'foo', :version=>'1.0' do
+      send packager, options
+      define 'bar' ; define 'baz'
+    end
+  end
+
+  def applied_to
+    projects.select { |project| project.packages.first }.map(&:name)
+  end
+
+  it 'should create package of type zip with classifier' do
+    prepare
+    project('foo').packages.first.to_s.should =~ /foo-1.0-#{@packaging}.zip/
+  end
+
+  it 'should create package for projects that have source files' do
+    prepare
+    applied_to.should include('foo', 'foo:baz')
+  end
+
+  it 'should not create package for projects that have no source files' do
+    prepare
+    applied_to.should_not include('foo:bar')
+  end
+
+  it 'should limit to projects specified by :only' do
+    prepare :only=>'baz'
+    applied_to.should eql(['foo:baz'])
+  end
+
+  it 'should limit to projects specified by :only array' do
+    prepare :only=>['baz']
+    applied_to.should eql(['foo:baz'])
+  end
+
+  it 'should ignore project specified by :except' do
+    prepare :except=>'baz'
+    applied_to.should eql(['foo'])
+  end
+
+  it 'should ignore projects specified by :except array' do
+    prepare :except=>['baz']
+    applied_to.should eql(['foo'])
+  end
+end
+
+describe 'package_with_sources' do
+  it_should_behave_like 'package_with_'
+  before { @packaging = :sources }
+end
+
+describe 'package_with_javadoc' do
+  it_should_behave_like 'package_with_'
+  before { @packaging = :javadoc }
+end