slather 0.0.235 → 1.0.0

data/spec/slather/coverage_service/simple_output_spec.rb

@@ -0,0 +1,26 @@
+require File.join(File.dirname(__FILE__), '..', '..', 'spec_helper')
+
+describe Slather::CoverageService::SimpleOutput do
+
+  let(:fixtures_project) do
+    proj = Slather::Project.open(FIXTURES_PROJECT_PATH)
+    proj.extend(Slather::CoverageService::SimpleOutput)
+  end
+
+  describe '#coverage_file_class' do
+    it "should return CoverallsCoverageFile" do
+      expect(fixtures_project.send(:coverage_file_class)).to eq(Slather::CoverallsCoverageFile)
+    end
+  end
+
+  describe '#post' do
+    it "should print out the coverage for each file, and then total coverage" do
+      expect(fixtures_project).to receive(:puts).with("spec/fixtures/fixtures/fixtures.m: 2 of 4 lines (50.00%)")
+      expect(fixtures_project).to receive(:puts).with("spec/fixtures/fixtures/more_files/peekaview.m: 0 of 6 lines (0.00%)")
+      expect(fixtures_project).to receive(:puts).with("spec/fixtures/fixturesTests/fixturesTests.m: 7 of 7 lines (100.00%)")
+      expect(fixtures_project).to receive(:puts).with("spec/fixtures/fixturesTests/peekaviewTests.m: 6 of 6 lines (100.00%)")
+      expect(fixtures_project).to receive(:puts).with("Test Coverage: 65.22%")
+      fixtures_project.post
+    end
+  end
+end

data/spec/slather/project_spec.rb

@@ -0,0 +1,219 @@
+require File.join(File.dirname(__FILE__), '..', 'spec_helper')
+
+describe Slather::Project do
+
+  let(:fixtures_project) do
+    Slather::Project.any_instance.stub(:configure_from_yml)
+    Slather::Project.open(FIXTURES_PROJECT_PATH)
+  end
+
+  describe "::open" do
+    it "should return a project instance that has been configured from yml" do
+      expect_any_instance_of(Slather::Project).to receive(:configure_from_yml)
+      expect(fixtures_project).not_to be_nil
+    end
+  end
+
+  describe "#derived_data_dir" do
+    it "should return the system's derived data directory" do
+      expect(fixtures_project.send(:derived_data_dir)).to eq(File.expand_path('~') + "/Library/Developer/Xcode/DerivedData/")
+    end
+  end
+
+  describe "#build_directory" do
+    it "should return the build_directory property, if it has been explicitly set" do
+      build_directory_mock = double(String)
+      fixtures_project.build_directory = build_directory_mock
+      expect(fixtures_project.build_directory).to eq(build_directory_mock)
+    end
+
+    it "should return the derived_data_dir if no build_directory has been set" do
+      derived_data_dir_mock = double(String)
+      fixtures_project.stub(:derived_data_dir).and_return(derived_data_dir_mock)
+      expect(fixtures_project.build_directory).to eq(derived_data_dir_mock)
+    end
+  end
+
+  describe "::yml" do
+    before(:each) { Slather::Project.instance_variable_set("@yml", nil) }
+
+    context ".slather.yml file exists" do
+      before(:all) { File.open(".slather.yml", "w") { |f| f.write("two: 2") } }
+      after(:all) { File.delete(".slather.yml") }
+
+      it "should load and return .slather.yml, if it exists" do
+        expect(Slather::Project.yml).to eq({"two" => 2})
+      end
+    end
+
+    context ".slather.yml file doesn't exist" do
+      it "should return an empy hash" do
+        expect(Slather::Project.yml).to eq({})
+      end
+    end
+  end
+
+  describe "#coverage_files" do
+    class SpecCoverageFile < Slather::CoverageFile
+    end
+
+    before(:each) do
+      Dir.stub(:[]).and_call_original
+      Dir.stub(:[]).with("#{fixtures_project.build_directory}/**/*.gcno").and_return(["/some/path/fixtures.gcno",
+                                                                                  "/some/path/peekaview.gcno",
+                                                                                  "/some/path/fixturesTests.gcno",
+                                                                                  "/some/path/peekaviewTests.gcno",
+                                                                                  "/some/path/NotInProject.gcno",
+                                                                                  "/some/path/NSRange.gcno"])
+      fixtures_project.stub(:coverage_file_class).and_return(SpecCoverageFile)
+    end
+
+    it "should return coverage file objects of type coverage_file_class for unignored project files" do
+      fixtures_project.ignore_list = ["*fixturesTests*"]
+      coverage_files = fixtures_project.send(:coverage_files)
+      coverage_files.each { |cf| expect(cf.kind_of?(SpecCoverageFile)).to be_truthy }
+      expect(coverage_files.map { |cf| cf.source_file_pathname.basename.to_s }).to eq(["fixtures.m", "peekaview.m"])
+    end
+
+    it "should raise an exception if no unignored project coverage file files were found" do
+      fixtures_project.ignore_list = ["*fixturesTests*", "*fixtures*"]
+      expect {fixtures_project.send(:coverage_files)}.to raise_error(StandardError)
+    end
+  end
+
+  describe "#configure_from_yml" do
+    it "should configure all properties from the yml" do
+      unstubbed_project = Slather::Project.open(FIXTURES_PROJECT_PATH)
+      expect(unstubbed_project).to receive(:configure_build_directory_from_yml)
+      expect(unstubbed_project).to receive(:configure_ignore_list_from_yml)
+      expect(unstubbed_project).to receive(:configure_ci_service_from_yml)
+      expect(unstubbed_project).to receive(:configure_coverage_service_from_yml)
+      unstubbed_project.configure_from_yml
+    end
+  end
+
+  describe "#configure_ignore_list_from_yml" do
+    it "should set the ignore_list if it has been provided in the yml and has not already been set" do
+      Slather::Project.stub(:yml).and_return({"ignore" => ["test", "ing"] })
+      fixtures_project.configure_ignore_list_from_yml
+      expect(fixtures_project.ignore_list).to eq(["test", "ing"])
+    end
+
+    it "should force the ignore_list into an array" do
+      Slather::Project.stub(:yml).and_return({"ignore" => "test" })
+      fixtures_project.configure_ignore_list_from_yml
+      expect(fixtures_project.ignore_list).to eq(["test"])
+    end
+
+    it "should not set the ignore_list if it has already been set" do
+      Slather::Project.stub(:yml).and_return({"ignore" => ["test", "ing"] })
+      fixtures_project.ignore_list = ["already", "set"]
+      fixtures_project.configure_ignore_list_from_yml
+      expect(fixtures_project.ignore_list).to eq(["already", "set"])
+    end
+
+    it "should default the ignore_list to an empty array if nothing is provided in the yml" do
+      Slather::Project.stub(:yml).and_return({})
+      fixtures_project.configure_ignore_list_from_yml
+      expect(fixtures_project.ignore_list).to eq([])
+    end
+  end
+
+  describe "#configure_build_directory_from_yml" do
+    it "should set the build_directory if it has been provided in the yml and has not already been set" do
+      Slather::Project.stub(:yml).and_return({"build_directory" => "/some/path"})
+      fixtures_project.configure_build_directory_from_yml
+      expect(fixtures_project.build_directory).to eq("/some/path")
+    end
+
+    it "should not set the build_directory if it has already been set" do
+      Slather::Project.stub(:yml).and_return({"build_directory" => "/some/path"})
+      fixtures_project.build_directory = "/already/set"
+      fixtures_project.configure_build_directory_from_yml
+      expect(fixtures_project.build_directory).to eq("/already/set")
+    end
+
+    it "should default the build_directory to derived data if nothing is provided in the yml" do
+      Slather::Project.stub(:yml).and_return({})
+      fixtures_project.configure_build_directory_from_yml
+      expect(fixtures_project.build_directory).to eq(fixtures_project.send(:derived_data_dir))
+    end
+  end
+
+  describe "#configure_ci_service_from_yml" do
+    it "should set the ci_service if it has been provided in the yml and has not already been set" do
+      Slather::Project.stub(:yml).and_return({"ci_service" => "some_service"})
+      fixtures_project.configure_ci_service_from_yml
+      expect(fixtures_project.ci_service).to eq(:some_service)
+    end
+
+    it "should not set the ci_service if it has already been set" do
+      Slather::Project.stub(:yml).and_return({"ci_service" => "some service"})
+      fixtures_project.ci_service = "already_set"
+      fixtures_project.configure_ci_service_from_yml
+      expect(fixtures_project.ci_service).to eq(:already_set)
+    end
+
+    it "should default the ci_service to :travis_ci if nothing is provided in the yml" do
+      Slather::Project.stub(:yml).and_return({})
+      fixtures_project.configure_ci_service_from_yml
+      expect(fixtures_project.ci_service).to eq(:travis_ci)
+    end
+  end
+
+  describe "#ci_service=" do
+    it "should set the ci_service as a symbol" do
+      fixtures_project.ci_service = "foobar"
+      expect(fixtures_project.ci_service).to eq(:foobar)
+    end
+  end
+
+  describe "#configure_coverage_service_from_yml" do
+    it "should set the coverage_service if it has been provided by the yml" do
+      Slather::Project.stub(:yml).and_return({"coverage_service" => "some_service"})
+      expect(fixtures_project).to receive(:coverage_service=).with("some_service")
+      fixtures_project.configure_coverage_service_from_yml
+    end
+
+    it "should default the coverage_service to :terminal if nothing is provided in the yml" do
+      Slather::Project.stub(:yml).and_return({})
+      expect(fixtures_project).to receive(:coverage_service=).with(:terminal)
+      fixtures_project.configure_coverage_service_from_yml
+    end
+
+    it "should not set the coverage_service if it has already been set" do
+      Slather::Project.stub(:yml).and_return({"coverage_service" => "some_service" })
+      fixtures_project.stub(:coverage_service).and_return("already set")
+      expect(fixtures_project).to_not receive(:coverage_service=)
+      fixtures_project.configure_coverage_service_from_yml
+    end
+  end
+
+  describe "#coverage_service=" do
+    it "should extend Slather::CoverageService::Coveralls and set coverage_service = :coveralls if given coveralls" do
+      expect(fixtures_project).to receive(:extend).with(Slather::CoverageService::Coveralls)
+      fixtures_project.coverage_service = "coveralls"
+      expect(fixtures_project.coverage_service).to eq(:coveralls)
+    end
+
+    it "should extend Slather::CoverageService::SimpleOutput and set coverage_service = :terminal if given terminal" do
+      expect(fixtures_project).to receive(:extend).with(Slather::CoverageService::SimpleOutput)
+      fixtures_project.coverage_service = "terminal"
+      expect(fixtures_project.coverage_service).to eq(:terminal)
+    end
+
+    it "should raise an exception if it does not recognize the coverage service" do
+      expect { fixtures_project.coverage_service = "xcode bots, lol" }.to raise_error(StandardError)
+    end
+  end
+
+  describe "#setup_for_coverage" do
+    it "should enable the correct flags to generate test coverage on all of the build_configurations build settings" do
+      fixtures_project.setup_for_coverage
+      fixtures_project.build_configurations.each do |build_configuration|
+        expect(build_configuration.build_settings["GCC_INSTRUMENT_PROGRAM_FLOW_ARCS"]).to eq("YES")
+        expect(build_configuration.build_settings["GCC_GENERATE_TEST_COVERAGE_FILES"]).to eq("YES")
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,16 @@
+require 'bundler/setup'
+Bundler.setup
+
+require 'slather'
+require 'pry'
+require 'coveralls'
+
+Coveralls.wear!
+
+FIXTURES_PROJECT_PATH = File.join(File.dirname(__FILE__), 'fixtures/fixtures.xcodeproj')
+
+RSpec.configure do |config|
+  config.after(:suite) do
+    FileUtils.rm_rf(Dir[File.expand_path('~') + "/Library/Developer/Xcode/DerivedData/fixture*"].first)
+  end
+end

metadata

@@ -1,97 +1,111 @@
 --- !ruby/object:Gem::Specification
 name: slather
 version: !ruby/object:Gem::Version
-  version: 0.0.235
+  version: 1.0.0
 platform: ruby
 authors:
 - Mark Larsen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-21 00:00:00.000000000 Z
+date: 2014-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.3'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.14'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.9'
 - !ruby/object:Gem::Dependency
   name: clamp
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.6'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.6'
 - !ruby/object:Gem::Dependency
   name: xcodeproj
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.17'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.17'
 description: 
@@ -102,12 +116,15 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
+- .coveralls.yml
+- .gitignore
+- .travis.yml
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - bin/slather
+- docs/logo.jpg
 - lib/slather.rb
 - lib/slather/coverage_file.rb
 - lib/slather/coverage_service/coveralls.rb
@@ -115,7 +132,26 @@ files:
 - lib/slather/coveralls_coverage_file.rb
 - lib/slather/project.rb
 - lib/slather/version.rb
+- scripts/travis.sh
 - slather.gemspec
+- spec/fixtures/fixtures.xcodeproj/project.pbxproj
+- spec/fixtures/fixtures.xcodeproj/project.xcworkspace/contents.xcworkspacedata
+- spec/fixtures/fixtures.xcodeproj/xcshareddata/xcschemes/fixtures.xcscheme
+- spec/fixtures/fixtures/Supporting Files/fixtures-Prefix.pch
+- spec/fixtures/fixtures/fixtures.h
+- spec/fixtures/fixtures/fixtures.m
+- spec/fixtures/fixtures/more_files/peekaview.h
+- spec/fixtures/fixtures/more_files/peekaview.m
+- spec/fixtures/fixturesTests/Supporting Files/en.lproj/InfoPlist.strings
+- spec/fixtures/fixturesTests/Supporting Files/fixturesTests-Info.plist
+- spec/fixtures/fixturesTests/fixturesTests.m
+- spec/fixtures/fixturesTests/peekaviewTests.m
+- spec/slather/coverage_file_spec.rb
+- spec/slather/coverage_service/coveralls_spec.rb
+- spec/slather/coverage_service/simple_output_spec.rb
+- spec/slather/fixtures.gcno
+- spec/slather/project_spec.rb
+- spec/spec_helper.rb
 homepage: https://github.com/marklarr/slather
 licenses:
 - MIT
@@ -126,12 +162,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
@@ -140,4 +176,23 @@ rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Test coverage reports for Xcode projects
-test_files: []
+test_files:
+- spec/fixtures/fixtures.xcodeproj/project.pbxproj
+- spec/fixtures/fixtures.xcodeproj/project.xcworkspace/contents.xcworkspacedata
+- spec/fixtures/fixtures.xcodeproj/xcshareddata/xcschemes/fixtures.xcscheme
+- spec/fixtures/fixtures/Supporting Files/fixtures-Prefix.pch
+- spec/fixtures/fixtures/fixtures.h
+- spec/fixtures/fixtures/fixtures.m
+- spec/fixtures/fixtures/more_files/peekaview.h
+- spec/fixtures/fixtures/more_files/peekaview.m
+- spec/fixtures/fixturesTests/Supporting Files/en.lproj/InfoPlist.strings
+- spec/fixtures/fixturesTests/Supporting Files/fixturesTests-Info.plist
+- spec/fixtures/fixturesTests/fixturesTests.m
+- spec/fixtures/fixturesTests/peekaviewTests.m
+- spec/slather/coverage_file_spec.rb
+- spec/slather/coverage_service/coveralls_spec.rb
+- spec/slather/coverage_service/simple_output_spec.rb
+- spec/slather/fixtures.gcno
+- spec/slather/project_spec.rb
+- spec/spec_helper.rb
+has_rdoc: