sproutit-sproutcore 1.0.203 → 1.0.20090721145236

data/spec/lib/models/project/target_for_spec.rb

@@ -0,0 +1,33 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Project, 'target_for' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+  end
+  
+  it "should lookup absolute target names (/foo) from the top" do
+    target = @project.target_for('/sproutcore')
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/sproutcore')
+  end
+  
+  it "should lookup relative target names from the top (like absolute)" do
+    target = @project.target_for(:sproutcore)
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/sproutcore')
+  end
+  
+  it "should return nil if no matching target could be found" do
+    target = @project.target_for(:does_not_exist)
+    target.should be_nil
+  end
+  
+  it "should NOT call prepare! on new targets" do
+    target = @project.target_for(:contacts)
+    target.prepared?.should be_false
+  end
+  
+end

data/spec/lib/models/project/targets_spec.rb

@@ -0,0 +1,62 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Project, 'targets' do
+
+  include SC::SpecHelpers
+
+  it "should clone targets from a parent project" do
+    
+    # Generate a dummy class to manually add a target.  This way we isolate
+    # the tests for merging from tests of finding targets...
+    test_project = Class.new(SC::Project) do
+      def find_targets_for(root_path, root_name, config)
+        self.add_target :base1, :dummy_type, { :source_root => "foo" }
+      end
+    end
+    parent = test_project.new fixture_path('buildfiles', 'empty_project')
+    
+    # Now generate a real project with an empty path.  This should clone 
+    # targets from the parent...
+    project = SC::Project.new fixture_path('buildfiles', 'empty_project'), :parent => parent
+    project.targets.size.should eql(1)
+    
+    t = project.targets['base1']
+    t.target_name.should == :base1
+    t.target_type.should == :dummy_type
+    t.source_root.should  == "foo"
+  end
+  
+  # We want to test this because find_targets_for() is a callback we expect
+  # people who customize the build tools to override.
+  it "should invoke the find_targets_for method on itself" do
+    
+    # Generate dummy class with custom find_targets_for.
+    test_project = Class.new(SC::Project) do
+      
+      attr_reader :did_call_find_targets_for
+      attr_reader :passed_root_path
+      attr_reader :passed_root_name
+      attr_reader :passed_config
+      
+      def find_targets_for(root_path, root_name, config)
+        @did_call_find_targets_for = true
+        @passed_root_path = root_path
+        @passed_root_name = root_name
+        @passed_config = config
+        return self
+      end
+    end
+    
+    # create project & get targets
+    project = test_project.new fixture_path('buildfiles', 'empty_project')
+    project.targets.size.should eql(0)
+    
+    # verify that callback method was run
+    project.did_call_find_targets_for.should be_true
+    project.passed_root_path.should eql(project.project_root)
+    project.passed_root_name.should be_nil
+    project.passed_config.should eql(project.config)
+  end
+    
+end
+

data/spec/lib/models/target/compute_build_number_spec.rb

@@ -0,0 +1,125 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+require 'tempfile'
+require 'fileutils'
+
+describe SC::Target, 'compute_build_number' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = temp_project(:real_world)
+  end
+  
+  after do
+    @project.cleanup
+  end
+    
+  def add_dummyfile(project)
+    file = File.new(File.join(project.source_root, 'dummyfile'), 'w')
+    file.write "TEST!"
+    file.close
+  end
+    
+  it "uses the config.build_number if specified for the target" do
+    # add build number for target to general buildfile
+    @project.buildfile.define! do
+      config :sproutcore, :build_number => 'foo'
+    end
+    target = @project.target_for(:sproutcore)
+    target.compute_build_number.should eql('foo')
+  end
+  
+  it "uses the config.build_numbers.target_name if specified" do
+    # add a build_numbers hash
+    @project.buildfile.define! do
+      config :all, :build_numbers => { '/sproutcore' => 'foo' }
+    end
+    target = @project.target_for(:sproutcore)
+    target.compute_build_number.should eql('foo')
+  end
+  
+  it "uses config.build_numbers.target_name, even if specified with symbols" do
+    @project.buildfile.define! do
+      config :all, :build_numbers => { :'/sproutcore' => :foo }
+    end
+    target = @project.target_for(:sproutcore)
+    target.compute_build_number.should eql('foo')
+  end
+    
+  
+  describe "accurate method to compute build number" do
+    
+    before do
+      @target = @project.target_for(:sproutcore)
+      @target.config.build_numbers = nil #precondition
+      @target.config.build_number = nil  #precondition
+    end
+      
+    it "generates a unique build number based on content if nothing is explicitly set" do
+      @target.compute_build_number.should_not be_nil
+    end
+  
+    it "changes its generated build number if contents of source files change" do
+      old_build_number = @target.compute_build_number
+    
+      # write an extra file into target for testing
+      add_dummyfile(@target)
+    
+      # get new build number
+      new_build_number = @target.compute_build_number
+      new_build_number.should_not eql(old_build_number)
+    end
+  end
+
+  it "changes generated build number if build number for a required target changes" do
+    target = @project.target_for(:sproutcore)
+    target.should_not be_nil
+
+    required = target.target_for(:desktop)
+    required.should_not be_nil
+    required.config.build_numbers = nil #precondition
+    required.config.build_number = nil  #precondition
+  
+    target.config.build_numbers = nil #precondition
+    target.config.build_number = nil  #precondition
+    target.required_targets.should include(required) #precondition
+  
+    old_build_number = target.compute_build_number
+  
+    # write an extra file into required target for testing -- changes number
+    add_dummyfile(required)
+  
+    # get new build number
+    new_build_number = target.compute_build_number
+    new_build_number.should_not eql(old_build_number)
+  end
+
+  it "does not change generated build number if a nested target that is not required by target changes" do
+    target = @project.target_for(:sproutcore)
+    target.should_not be_nil
+
+    not_required = target.target_for(:mobile)
+    not_required.should_not be_nil
+
+    #precondition
+    target.expand_required_targets.should_not include(not_required) 
+  
+    target = @project.target_for(:sproutcore)
+    old_build_number = target.compute_build_number
+  
+    # write an extra file into required target for testing -- changes number
+    add_dummyfile(not_required)
+  
+    # get new build number
+    new_build_number = target.compute_build_number
+    new_build_number.should eql(old_build_number)
+  end
+  
+  it "recursively required project should not cause an error" do
+    recursive = fixture_project(:recursive_project)
+    target = recursive.target_for :sproutcore
+    lambda { target.compute_build_number }.should_not raise_error
+  end
+  
+end
+

data/spec/lib/models/target/config_spec.rb

@@ -0,0 +1,30 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'config' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+  end
+  
+  it "should pickup global config setting when target does not override" do
+    # required property is defined in real_world/Buildfile
+    @target = @project.target_for :contacts
+    @target.config.required.should eql(:sproutcore)
+  end
+  
+  it "should pickup config setting overidden in target's Buildfile config" do
+    # define in real_world/frameworks/sproutcore/Buildfile
+    @target = @project.target_for :sproutcore
+    @target.config.required.should eql(:desktop)
+    
+  end
+  
+  it "should pickup config from parent buildfiles" do
+    # define in real_world/frameworks/sproutcore/Buildfile
+    @target = @project.target_for 'sproutcore/foundation'
+    @target.config.required.should eql(:costello)
+  end
+    
+end

data/spec/lib/models/target/expand_required_targets_spec.rb

@@ -0,0 +1,48 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'expand_required_targets' do
+
+  include SC::SpecHelpers
+
+  it "should return all of targets required by a target in proper order" do
+    @project = fixture_project(:real_world)
+    target = @project.target_for :sproutcore
+    required = target.expand_required_targets.map { |x| x.target_name }
+    required.should eql([
+      :"/sproutcore/costello", 
+      :"/sproutcore/foundation", 
+      :"/sproutcore/application", 
+      :"/sproutcore/data_store", 
+      :"/sproutcore/desktop"])
+  end
+  
+  it "should still return a valid response even with a recursive project" do
+    @project = fixture_project(:recursive_project)
+    target = @project.target_for :sproutcore
+    
+    required = nil 
+    lambda { required = target.expand_required_targets }.should_not raise_error
+    
+    required.map! { |x| x.target_name }
+    required.should eql([:'/sproutcore/costello'])
+  end
+  
+  it "should include test or debug required if passed as options" do
+    @project = fixture_project(:real_world)
+    target = @project.target_for :sproutcore
+    target.config.debug_required = 'sproutcore/debug'
+    target.config.test_required = 'sproutcore/qunit'
+    
+    debug_expected = @project.target_for 'sproutcore/debug'
+    test_expected = @project.target_for 'sproutcore/qunit'
+    
+    target.expand_required_targets().should_not include(debug_expected)
+    target.expand_required_targets(:debug => false ).should_not include(debug_expected)
+    target.expand_required_targets(:debug => true).should include(debug_expected)
+
+    target.expand_required_targets().should_not include(test_expected)
+    target.expand_required_targets(:test => false ).should_not include(test_expected)
+    target.expand_required_targets(:test => true).should include(test_expected)
+  end
+  
+end

data/spec/lib/models/target/installed_languages_spec.rb

@@ -0,0 +1,47 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'installed_languages' do
+
+  include SC::SpecHelpers
+  
+  before do
+    @project = fixture_project 'languages'
+  end
+    
+  def langs(langs)
+    langs.map { |l| l.to_s }.sort.map { |l| l.to_sym }
+  end
+  
+  it "should detect all installed languages with short names" do
+    expected = langs %w(de en-CA en-GB en-US en es foo fr it ja)
+    @short_names = @project.target_for :short_names
+    @short_names.installed_languages.should eql(expected)
+  end
+  
+  it "should map long languages to short names, and pass through others" do
+    expected = langs %w(en fr de it ja es unknown)
+    @long_names = @project.target_for :long_names
+    @long_names.installed_languages.should eql(expected)
+  end
+
+  it "should always return the preferred language, even if no languages" do
+    @no_names  = @project.target_for :no_names  
+    @no_names.installed_languages.should eql([:en])
+  end
+  
+  it "should ignore case for long names and respect case for unknown" do
+    @long_names = @project.target_for :caps_long_names
+    @long_names.installed_languages.should eql([:en, :fr, :UnknOWN])
+  end
+  
+  # NOTE: This is the same test as above, but it is placed here to test 
+  # case-sensitivity explicitly.  On some platforms, sorting will differ
+  # depending on the case...
+  it "should always return names using case insensitive sort" do
+    @long_names = @project.target_for :caps_long_names
+    @long_names.installed_languages.should eql([:en, :fr, :UnknOWN])
+  end
+    
+  
+  
+end

data/spec/lib/models/target/lproj_for_spec.rb

@@ -0,0 +1,38 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'lproj_for' do
+
+  include SC::SpecHelpers
+  
+  before do
+    @project = fixture_project 'languages'
+    @long_names = @project.target_for :long_names
+    @short_names = @project.target_for :short_names    
+  end
+    
+
+  it "should return the long name of a language if installed" do
+    SC::Target::SHORT_LANGUAGE_MAP.each do |short_name, long_name|
+      @long_names.lproj_for(short_name.to_sym).should eql("#{long_name}.lproj")
+    end
+  end
+  
+  it "should return the short name of a language if installed" do
+    SC::Target::SHORT_LANGUAGE_MAP.each do |short_name, long_name|
+      @short_names.lproj_for(short_name.to_sym).should eql("#{short_name}.lproj")
+    end
+  end
+  
+  it "should return the language name if installed + no explicit mapping" do
+    @short_names.lproj_for('en-GB').should eql('en-GB.lproj')
+  end
+  
+  it "should return nil for a non-existant language" do
+    @short_names.lproj_for('imaginary').should be_nil
+  end
+  
+  it "should return long language name if no mapping is known" do
+    @long_names.lproj_for(:unknown).should eql('unknown.lproj')
+  end
+  
+end

data/spec/lib/models/target/manifest_for_spec.rb

@@ -0,0 +1,42 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'manifest_for' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+    @target = @project.target_for :sproutcore
+  end
+  
+  it "should return a new manifest instance for a new variation with variation properties set" do
+    manifest = @target.manifest_for :language => :en
+    manifest.should_not be_nil
+    manifest.language.should eql(:en)
+  end
+  
+  it "should return the same manifest instance for the same variation" do
+    manifest_en = @target.manifest_for :language => :en
+    manifest_fr = @target.manifest_for :language => :fr
+    
+    @target.manifest_for(:language => :en).should eql(manifest_en)
+    @target.manifest_for(:language => :fr).should eql(manifest_fr)
+  end
+  
+  it "should return first manifest matching variation if multiples match" do
+    manifest1 = @target.manifest_for :language => :en, :vers => 1
+    manifest2 = @target.manifest_for :language => :en, :vers => 2
+
+    found = @target.manifest_for(:language => :en)
+    found.vers.should eql(1)
+  end
+  
+  # IMPORTANT:  This condition is assumed by the manifest/prepare_spec.
+  it "should NOT call prepare! when creating a new manifest" do
+    manifest = @target.manifest_for :language => :en
+    manifest.prepared?.should be_false
+  end
+  
+end
+
+    

data/spec/lib/models/target/parent_target_spec.rb

@@ -0,0 +1,21 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'parent_target' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+  end
+
+  it "should return next direct parent when nested" do
+    target = @project.target_for('sproutcore/costello')
+    target.parent_target.should eql(@project.target_for(:sproutcore))
+  end
+  
+  it "should return project if top-level" do
+    target = @project.target_for(:sproutcore)
+    target.parent_target.should eql(@project)
+  end
+    
+end

data/spec/lib/models/target/prepare_spec.rb

@@ -0,0 +1,53 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'prepare!' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+
+    # get target from project manually since target_for() calls prepare!
+    @target = @project.targets['/contacts']
+    
+    @target.buildfile.define! do
+      replace_task 'target:prepare' do
+        TARGET.task_did_run = (TARGET.task_did_run || 0) + 1
+      end
+    end
+    
+  end
+
+  it "should return self" do
+    @target.prepare!.should eql(@target)
+  end
+  
+  it "should execute target:prepare if defined" do
+    @target.prepared?.should be_false # check precondition
+    @target.prepare!
+    @target.prepared?.should be_true
+    @target.task_did_run.should eql(1) # ran?
+  end
+
+  it "should do nothing if target:prepare is not defined" do
+
+    # get an empty project with no build tasks...
+    project = empty_project
+    project.add_target '/default', :default, :source_root => project.project_root
+    target = project.targets['/default']
+    target.buildfile.lookup('target:prepare').should be_nil
+    
+    lambda { target.prepare! }.should_not raise_error
+    
+  end
+  
+  it "should execute target:prepare only once" do
+    @target.prepared?.should be_false # check precondition
+    @target.prepare!.prepare!.prepare!
+    @target.prepared?.should be_true
+    @target.task_did_run.should eql(1) # ran only once?
+  end
+  
+end
+
+  

data/spec/lib/models/target/required_targets_spec.rb

@@ -0,0 +1,119 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'required_targets' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+  end
+    
+  it "should resolve references to child targets" do
+    target = @project.target_for :sproutcore
+    
+    # verify precondition 
+    # -- the required property should NOT be array ...
+    target.config.required.should eql(:desktop)
+  
+    expected = @project.target_for('sproutcore/desktop')
+    
+    target.required_targets.size.should eql(1)
+    target.required_targets.first.should eql(expected)
+  end
+  
+  it "should resolve references to siblings" do
+    target = @project.target_for 'sproutcore/application'
+  
+    # verify precondition
+    #  -- the required config should be an array
+    target.config.required.size.should eql(2)
+    
+    expected = @project.target_for('sproutcore/costello')
+    target.required_targets.size.should eql(2)
+    target.required_targets.should include(expected)
+  end
+  
+  it "should resolve references to other subtargets" do
+    target = @project.target_for :mobile_photos
+    target.required_targets.size.should eql(1)
+    
+    expected = @project.target_for('sproutcore/mobile')
+    target.required_targets.first.should eql(expected)
+  end
+  
+  it "should include any debug_required if passed :debug => true" do
+
+    expected = @project.target_for 'sproutcore/debug'
+    
+    # verify load_debug = false
+    target = @project.target_for 'sproutcore/desktop'
+    target.config.debug_required = 'sproutcore/debug'
+    target.required_targets().should_not include(expected)
+    target.required_targets(:debug => false).should_not include(expected)
+    target.required_targets(:debug => true).should include(expected)
+  end
+
+  it "should include any test_required if passed :test => true" do
+
+    expected = @project.target_for 'sproutcore/qunit'
+    
+    # verify load_debug = false
+    target = @project.target_for 'sproutcore/desktop'
+    target.config.test_required = 'sproutcore/qunit'
+    target.required_targets().should_not include(expected)
+    target.required_targets(:test => false).should_not include(expected)
+    target.required_targets(:test => true).should include(expected)
+  end
+  
+  it "should log a warning if a required test or debug target could not be found" do
+    target = @project.target_for :sproutcore
+    target.config.test_required = 'imaginary_foo'
+    target.config.debug_required = 'imaginary_bar'
+    
+    capture('stderr') { target.required_targets(:test => true) }.size.should_not == 0
+    capture('stderr') { target.required_targets(:debug => true) }.size.should_not == 0
+  end
+  
+  it "should include any CONFIG.theme if passed :theme => true && target_type == :app" do
+    expected = @project.target_for 'sproutcore/standard_theme'
+    
+    target = @project.target_for :contacts
+    target.target_type.should == :app # precondition
+    target.config.theme = 'sproutcore/standard_theme'
+    target.required_targets().should_not include(expected)
+    target.required_targets(:theme => false).should_not include(expected)
+    target.required_targets(:theme => true).should include(expected)
+
+    target = @project.target_for :sproutcore
+    target.target_type.should_not == :app # precondition
+    target.config.theme = 'sproutcore/standard_theme'
+    target.required_targets().should_not include(expected)
+    target.required_targets(:theme => false).should_not include(expected)
+    target.required_targets(:theme => true).should_not include(expected)
+  end  
+  
+  it "should only find targets with a target type of :theme for CONFIG.theme" do
+    # theme type
+    expected = @project.target_for 'sproutcore/standard_theme'
+    expected.target_type.should == :theme #precondition
+
+    target = @project.target_for :contacts
+    target.config.theme = 'sproutcore/standard_theme'
+    target.required_targets(:theme => true).should include(expected)
+    
+    # non-theme type
+    expected = @project.target_for 'sproutcore/costello'
+    expected.target_type.should_not == :theme #precondition
+
+    target = @project.target_for :calendar
+    target.config.theme = 'sproutcore/costello'
+    
+    # should warn!
+    result = nil
+    capture('stderr') { result = target.required_targets(:theme => true) }.size.should > 0
+    
+    # and should not include theme
+    result.should_not include(expected)
+  end
+  
+end

data/spec/lib/models/target/target_for_spec.rb

@@ -0,0 +1,56 @@
+require File.join(File.dirname(__FILE__), %w[.. .. .. spec_helper])
+
+describe SC::Target, 'target_for' do
+
+  include SC::SpecHelpers
+
+  before do
+    @project = fixture_project(:real_world)
+    @target = @project.target_for(:sproutcore)
+  end
+  
+  it "should lookup absolute target names (/foo) from the top" do
+    target = @target.target_for('/contacts')
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/contacts')
+  end
+  
+  it "should look for relative names in children first" do
+    # note: uploader should exist both in /apps and /sproutcore/frameworks
+    target = @target.target_for(:uploader)
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/sproutcore/uploader')
+  end
+
+  it "should look for relative names in sibling after children" do
+    target = @target.target_for(:core_photos)
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/core_photos')
+  end
+
+  it "should look for relative names in parents after siblings" do
+    @target = @project.target_for('sproutcore/costello')
+    target = @target.target_for(:contacts)
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/contacts')
+  end
+
+  it "should not find nested targets without naming parents" do
+    @target = @project.target_for(:contacts)
+    target = @target.target_for(:costello)
+    target.should be_nil
+  end
+
+  it "should be able to handle multiple/relative/names" do
+    @target = @project.target_for(:contacts)
+    target = @target.target_for('sproutcore/costello')
+    target.should_not be_nil
+    target.target_name.to_s.should eql('/sproutcore/costello')
+  end
+  
+  it "should return nil if no matching target could be found" do
+    target = @target.target_for(:does_not_exist)
+    target.should be_nil
+  end
+  
+end