xcoder 0.1.11 → 0.1.12

data/spec/TestProject/TestProject.xcodeproj/xcshareddata/xcschemes/TestProject.xcscheme

@@ -32,8 +32,18 @@
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "7165D46A146B4EA100DE2F0E"
-               BuildableName = "TestProjectTests.octest"
-               BlueprintName = "TestProjectTests"
+               BuildableName = "LogicTests.octest"
+               BlueprintName = "LogicTests"
+               ReferencedContainer = "container:TestProject.xcodeproj">
+            </BuildableReference>
+         </TestableReference>
+         <TestableReference
+            skipped = "NO">
+            <BuildableReference
+               BuildableIdentifier = "primary"
+               BlueprintIdentifier = "7147F906153E347700763224"
+               BuildableName = "ApplicationTests.octest"
+               BlueprintName = "ApplicationTests"
                ReferencedContainer = "container:TestProject.xcodeproj">
             </BuildableReference>
          </TestableReference>
@@ -54,6 +64,7 @@
       launchStyle = "0"
       useCustomWorkingDirectory = "NO"
       buildConfiguration = "Debug"
+      ignoresPersistentStateOnLaunch = "NO"
       debugDocumentVersioning = "YES"
       allowLocationSimulation = "YES">
       <BuildableProductRunnable>

data/spec/TestWorkspace2.xcworkspace/contents.xcworkspacedata

@@ -0,0 +1,7 @@
+<?xml version='1.0' encoding='UTF-8'?>
+<Workspace
+   version = '1.0'>
+   <FileRef
+      location = 'group:TestProject/TestProject.xcodeproj'>
+   </FileRef>
+</Workspace>

data/spec/build_phase_spec.rb

@@ -71,7 +71,7 @@ describe Xcode::BuildPhase do
         source_file = project.groups.create_file 'publicheader.h'
         subject.add_build_file_with_public_privacy source_file
         subject.build_file('publicheader.h').should_not be_nil
-        subject.file('publicheader.h').settings.should == { 'ATTRIBUTES' => "Public" }
+        subject.file('publicheader.h').settings.should == { 'ATTRIBUTES' => ["Public"] }
       end
     end
     
@@ -80,7 +80,7 @@ describe Xcode::BuildPhase do
         source_file = project.groups.create_file 'privateheader.h'
         subject.add_build_file_with_private_privacy source_file
         subject.build_file('privateheader.h').should_not be_nil
-        subject.file('privateheader.h').settings.should == { 'ATTRIBUTES' => "Private" }
+        subject.file('privateheader.h').settings.should == { 'ATTRIBUTES' => ["Private"] }
       end
     end
     

data/spec/builder_spec.rb

@@ -24,7 +24,14 @@ describe Xcode::Builder do
         Xcode::Shell.should_receive(:execute).with(default_build_parameters)
         subject.build
       end
-
+      
+      it "should allow the override of the sdk" do
+        expected = default_build_parameters
+        expected[1] = '-sdk macosx10.7'
+        Xcode::Shell.should_receive(:execute).with(expected)
+        subject.build :sdk => 'macosx10.7'
+      end
+      
     end
     
     describe "#testflight" do
@@ -59,10 +66,10 @@ describe Xcode::Builder do
     describe "#test" do
       
       let(:configuration) do
-        Xcode.project('TestProject').target('TestProjectTests').config('Debug')
+        Xcode.project('TestProject').target('LogicTests').config('Debug')
       end
       
-      let(:default_test_parameters) do
+      let(:iphonesimulator_test_parameters) do
         [ "xcodebuild", 
           "-sdk iphonesimulator", 
           "-project \"#{configuration.target.project.path}\"", 
@@ -70,12 +77,46 @@ describe Xcode::Builder do
           "-configuration \"#{configuration.name}\"", 
           "OBJROOT=\"#{File.dirname(configuration.target.project.path)}/build/\"", 
           "SYMROOT=\"#{File.dirname(configuration.target.project.path)}/build/\"",
+          "TEST_AFTER_BUILD=YES"
+          ]
+      end
+      
+      let(:macosx_test_parameters) do
+        [ "xcodebuild", 
+          "-sdk macosx10.7", 
+          "-project \"#{configuration.target.project.path}\"", 
+          "-target \"#{configuration.target.name}\"", 
+          "-configuration \"#{configuration.name}\"", 
+          "OBJROOT=\"#{File.dirname(configuration.target.project.path)}/build/\"", 
+          "SYMROOT=\"#{File.dirname(configuration.target.project.path)}/build/\"",
           "TEST_AFTER_BUILD=YES",
-          "TEST_HOST=''",]
+          ]
+      end
+      
+      
+      it "should be able to run the test target on iphonesimulator" do
+        Xcode::Shell.should_receive(:execute).with(iphonesimulator_test_parameters, false)
+        subject.test :sdk => 'iphonesimulator'
+      end
+
+      it "should be able to run the test target on macosx10.7" do
+        Xcode::Shell.should_receive(:execute).with(macosx_test_parameters, false)
+        subject.test :sdk => 'macosx10.7'
+      end
+      
+      it "should allow the override of the sdk" do
+        expected = macosx_test_parameters
+        expected[1] = '-sdk macosx10.7'
+        Xcode::Shell.should_receive(:execute).with(expected, false)
+        subject.test :sdk => 'macosx10.7'
       end
       
-      it "should be able to run the test target" do
-        Xcode::Shell.should_receive(:execute).with(default_test_parameters, false)
+      it "should not exit when test failed" do
+        Xcode::Shell.stub(:execute)
+        fake_parser = stub(:parser)
+        fake_parser.stub(:failed? => true)
+        fake_parser.stub(:flush)
+        Xcode::Test::Parsers::OCUnitParser.stub(:new => fake_parser)
         subject.test
       end
       
@@ -95,7 +136,7 @@ describe Xcode::Builder do
       end
 
 
-      it "should clean the project with the default parametesr" do
+      it "should clean the project with the default parameter" do
         Xcode::Shell.should_receive(:execute).with(default_clean_parameters)
         subject.clean
       end
@@ -116,7 +157,7 @@ describe Xcode::Builder do
         [ "xcodebuild",
           "-sdk iphoneos",
           "-project \"#{scheme.project.path}\"",
-          "-scheme #{scheme.name}",
+          "-scheme \"#{scheme.name}\"",
           "OBJROOT=\"#{File.dirname(scheme.project.path)}/build/\"", 
           "SYMROOT=\"#{File.dirname(scheme.project.path)}/build/\"" ]
       end
@@ -134,14 +175,14 @@ describe Xcode::Builder do
         [ "xcodebuild",
           "-project \"#{scheme.project.path}\"",
           "-sdk iphoneos",
-          "-scheme #{scheme.name}",
+          "-scheme \"#{scheme.name}\"",
           "OBJROOT=\"#{File.dirname(scheme.project.path)}/build/\"", 
           "SYMROOT=\"#{File.dirname(scheme.project.path)}/build/\"",
           "clean" ]
       end
 
 
-      it "should clean the project with the default parametesr" do
+      it "should clean the project with the default parameter" do
         Xcode::Shell.should_receive(:execute).with(default_clean_parameters)
         subject.clean
       end

data/spec/integration/builder_spec.rb

@@ -3,13 +3,11 @@ require_relative '../spec_helper'
 describe Xcode::Builder, :integration => true do
   
   context "when using a builder built from a configuration" do
-    
-    let(:configuration) { Xcode.project('TestProject').target('TestProject').config('Debug') }
-
-    let(:subject) { configuration.builder }
   
     describe "#build" do
     
+      let(:configuration) { Xcode.project('TestProject').target('TestProject').config('Debug') }
+      let(:subject) { configuration.builder }
     
       it "should be able to build" do
         subject.clean
@@ -28,6 +26,44 @@ describe Xcode::Builder, :integration => true do
   
     end
     
+    
+    describe "logic tests" do
+      
+      let(:configuration) { Xcode.project('TestProject').target('LogicTests').config('Debug') }
+      let(:subject) { configuration.builder }
+      
+      it "should be able to run unit tests" do
+        subject.clean
+        report = subject.test
+        report.suites.count.should == 2
+        
+        tests = report.suites[1].tests
+        tests.count.should==2
+        tests[0].should be_passed
+        tests[1].should be_failed
+
+        tests = report.suites[0].tests
+        tests.count.should==1
+        tests[0].should be_passed
+      end
+    
+    end
+    
+    
+    describe "application tests" do
+      
+      # FIXME: known issue with xcodebuild and iphonesimulator application tests (when TEST_HOST is set)
+      let(:configuration) { Xcode.project('TestProject').target('ApplicationTests').config('Debug') }
+      let(:subject) { configuration.builder }
+      
+      it "should be able to run unit tests" do
+        subject.clean
+        report = subject.test
+        report.should be_succeed
+      end
+    
+    end
+    
   end
   
   context "when using a builder built from a scheme" do
@@ -54,6 +90,26 @@ describe Xcode::Builder, :integration => true do
       end
       
     end
+    
+    describe "#test" do
+      
+      # FIXME: cant seem to run tests when part of a scheme
+      it "should be able to run unit tests" do
+        subject.clean
+        report = subject.test
+        report.suites.count.should == 2
+        
+        tests = report.suites[1].tests
+        tests.count.should==2
+        tests[0].should be_passed
+        tests[1].should be_failed
+
+        tests = report.suites[0].tests
+        tests.count.should==1
+        tests[0].should be_passed
+      end
+    
+    end
   
   end
   

data/spec/ocunit_parser_spec.rb

@@ -0,0 +1,164 @@
+require 'rspec'
+require 'xcoder'
+
+require 'xcode/test/parsers/ocunit_parser'
+
+describe Xcode::Test::Parsers::OCUnitParser do 
+  
+  let :parser do
+    Xcode::Test::Parsers::OCUnitParser.new
+  end
+  
+  let :example_report do
+    parser << "Run test suite AnExampleTestSuite"
+    parser << "Test Suite 'AnExampleTestSuite' started at 2012-02-10 00:37:04 +0000"
+    
+    parser << "Run test case anExampleTest1"
+    parser << "Test Case '-[AnExampleTestSuite anExampleTest1]' started."
+    parser << "Test Case '-[AnExampleTestSuite anExampleTest1]' passed (0.003 seconds)."
+
+    parser << "Run test case anExampleTest2"
+    parser << "Test Case '-[AnExampleTestSuite anExampleTest2]' started."
+    parser << "Test Case '-[AnExampleTestSuite anExampleTest2]' passed (0.003 seconds)."
+    
+    parser << "Test Suite 'AnExampleTestSuite' finished at 2012-02-10 00:37:04 +0000."
+    parser << "Executed 1 test, with 0 failures (0 unexpected) in 0.000 (0.000) seconds"
+    
+    parser.report
+  end
+  
+  let :example_failing_report do
+    parser << "Run test suite AnExampleTestSuite"
+    parser << "Test Suite 'AnExampleTestSuite' started at 2012-02-10 00:37:04 +0000"
+    parser << "Test Case '-[AnExampleTestSuite aFailingTest]' started."
+    
+    parser << '2012-02-17 15:03:06.521 otest[24979:7803] line1'
+    parser << '2012-02-17 15:03:06.521 otest[24979:7803] line2'
+    parser << '2012-02-17 15:03:06.521 otest[24979:7803] line3'
+    parser << '2012-02-17 15:03:06.521 otest[24979:7803] line4'
+    parser << '/Some/Path/To/Test.m:1234: error: -[AnExampleTestSuite aFailingTest] : This is an error message'
+    
+    parser << "Test Case '-[AnExampleTestSuite aFailingTest]' failed (2 seconds)."
+    parser << "Test Suite 'AnExampleTestSuite' finished at 2012-02-10 00:37:04 +0000."
+    parser << "Executed 1 test, with 0 failures (0 unexpected) in 0.000 (0.000) seconds"
+    
+    parser.report
+  end
+  
+  it "should start the report" do
+    parser.report.start_time.should be_nil
+    parser << "Test Suite '/Users/blake/Projects/RestKit/RestKit/Build/Products/Debug-iphonesimulator/RestKitTests.octest(Tests)' started at 2012-04-17 01:36:20 +0000\n"
+    parser.report.start_time.should_not be_nil
+  end
+  
+  it "should capture output for a test case" do
+    
+    failure = example_failing_report.suites.first.tests[0]
+    failure.passed?.should==false
+    failure.errors.count.should==1
+    failure.errors[0][:data].count.should==4
+    failure.errors[0][:data][0].should=~/line1/
+    failure.errors[0][:data][1].should=~/line2/
+    failure.errors[0][:data][2].should=~/line3/
+    failure.errors[0][:data][3].should=~/line4/
+  end
+  
+  it "should capture errors reported during a test" do
+    failure = example_failing_report.suites.first.tests[0]
+    failure.passed?.should==false
+    failure.errors.count.should==1
+    failure.errors[0][:message].should=='This is an error message'
+    failure.errors[0][:location].should=='/Some/Path/To/Test.m:1234'
+  end
+  
+  it "should create a test case" do 
+    t = example_report.suites
+    t.count.should==1
+    t.first.name.should=="AnExampleTestSuite"
+    t.first.start_time.should==Time.parse("2012-02-10 00:37:04 +0000")
+    t.first.end_time.should==Time.parse("2012-02-10 00:37:04 +0000")
+  end
+  
+  it "should detect a passing report" do
+    t = example_report
+    t.should be_succeed
+    t.should_not be_failed
+  end
+  
+  it "should detect a failing report" do 
+    t = example_failing_report
+    t.should_not be_succeed
+    t.should be_failed
+  end
+  
+  it "should record a failure" do
+    t = example_failing_report
+    t.suites.first.total_failed_tests.should==1
+  end
+  
+  it "should create a test case with some tests" do 
+    t = example_report.suites
+      
+    t.count.should==1
+    t.first.tests.count.should==2
+    t.first.tests[0].name.should=='anExampleTest1'
+    t.first.tests[0].time.should==0.003
+    t.first.tests[0].passed?.should==true
+    
+    t.first.tests[1].name.should=='anExampleTest2'
+    t.first.tests[1].time.should==0.003
+    t.first.tests[1].passed?.should==true
+  end
+  
+  it "should capture failed build" do
+    parser << "Run test suite AnExampleTestSuite"
+    parser << "Test Suite 'AnExampleTestSuite' started at 2012-02-10 00:37:04 +0000"
+    parser << "Run test case anExampleTest1"
+    parser << "Test Case '-[AnExampleTestSuite anExampleTest1]' started."
+    parser << "/Path/To/Project/Tests/YPKeywordSuggestHandlerTest.m:45: error: -[AnExampleTestSuite anExampleTest1] : 'An example test spec' [FAILED], mock received unexpected message -setSuspended: 1 "
+    parser << "/Developer/Tools/RunPlatformUnitTests.include: line 415: 32225 Bus error: 10           \"${THIN_TEST_RIG}\" \"${OTHER_TEST_FLAGS}\" \"${TEST_BUNDLE_PATH}\""
+    parser << "/Developer/Tools/RunPlatformUnitTests.include:451: error: Test rig '/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator5.0.sdk/Developer/usr/bin/otest' exited abnormally with code 138 (it may have crashed)."
+  
+    parser.flush
+    
+    report = parser.report
+    
+    report.failed?.should==true
+    report.finished?.should==true
+    failure = report.suites.first.tests[0]
+    failure.passed?.should==false
+    failure.data.count.should==2
+    failure.data[0].should=~/32225 Bus error: 10/
+    failure.data[1].should=~/Test rig/
+  end
+  
+  context "encoding" do
+    it "should not fail due to Unicode characters" do
+      expect do
+        string = "2012-04-09 16:56:32.682 otest[81203:7803] E restkit.object_mapping:RKObjectMappingOperation.m:248 Validation failed while mapping attribute at key path boolString to value FAIL. Error: The operation couldn\xE2\x80\x99t be completed. (org.restkit.RestKit.ErrorDomain error 1003.)"
+        string.force_encoding("US-ASCII")
+        parser << string
+      end.not_to raise_error
+    end
+  end
+
+  context "Junit output" do
+    
+    it "should write out reports in junit format" do 
+      report_dir = "#{File.dirname(__FILE__)}/test-reports"
+      FileUtils.rm_rf report_dir
+    
+      parser.report.add_formatter :junit, report_dir
+      
+      example_report
+    
+      files = Dir["#{report_dir}/*.xml"]
+      files.count.should==1
+      files.first.should=~/TEST-AnExampleTestSuite.xml$/
+    
+      # FIXME: parse the report
+    end
+    
+  end
+  
+end

data/spec/project_spec.rb

@@ -7,7 +7,7 @@ describe Xcode::Project do
   describe "Targets" do
     
     let(:expected_first_target) { "TestProject" }
-    let(:expected_second_target) { "TestProjectTests" }
+    let(:expected_second_target) { "LogicTests" }
     
     describe "#targets" do
 

data/spec/workspace_spec.rb

@@ -3,9 +3,9 @@ require_relative 'spec_helper'
 describe Xcode::Workspace do 
   it "should enumerate all workspaces in current directory" do 
     workspaces = Xcode.workspaces
-    workspaces.size.should==1
-    workspaces.first.name.should=="TestWorkspace"
-    workspaces.first.projects.size.should==1
+    workspaces.size.should == 2
+    workspaces.first.name.should == "TestWorkspace"
+    workspaces.first.projects.size.should == 1
   end
   
   it "should fetch workspace by name" do 
@@ -18,6 +18,11 @@ describe Xcode::Workspace do
     w.should_not be_nil
   end
   
+  it "should handle workspace that use ' in the XML" do 
+    w = Xcode.workspace "#{File.dirname(__FILE__)}/TestWorkspace2.xcworkspace"
+    w.should_not be_nil
+  end
+  
   it "should have many projects" do 
     w = Xcode.workspace "TestWorkspace"
     w.projects.size.should==1