ceedling 0.15.5 → 0.15.6

data/lib/ceedling/defaults.rb

@@ -42,11 +42,8 @@ DEFAULT_TEST_FIXTURE_TOOL = {
   :arguments => [].freeze
   }
 
-
-
 DEFAULT_TEST_INCLUDES_PREPROCESSOR_TOOL = {
-  # :executable => FilePathUtils.os_executable_ext('cpp').freeze, # disabled for OSX clang
-  :executable => FilePathUtils.os_executable_ext('gcc').freeze, # OSX clang
+  :executable => FilePathUtils.os_executable_ext('gcc').freeze,
   :name => 'default_test_includes_preprocessor'.freeze,
   :stderr_redirect => StdErrRedirect::NONE.freeze,
   :background_exec => BackgroundExec::NONE.freeze,
@@ -62,10 +59,9 @@ DEFAULT_TEST_INCLUDES_PREPROCESSOR_TOOL = {
     {"-I$" => 'COLLECTION_PATHS_TEST_TOOLCHAIN_INCLUDE'}.freeze,
     {"-D$" => 'COLLECTION_DEFINES_TEST_AND_VENDOR'}.freeze,
     {"-D$" => 'DEFINES_TEST_PREPROCESS'}.freeze,
-    # "-DGNU_PREPROCESSOR".freeze,
     "-DGNU_COMPILER".freeze, # OSX clang
     '-w'.freeze,
-    # '-nostdinc'.freeze,
+    # '-nostdinc'.freeze, # disabled temporarily due to stdio access violations on OSX
     "\"${1}\"".freeze
     ].freeze
   }
@@ -83,12 +79,19 @@ DEFAULT_TEST_FILE_PREPROCESSOR_TOOL = {
     {"-D$" => 'COLLECTION_DEFINES_TEST_AND_VENDOR'}.freeze,
     {"-D$" => 'DEFINES_TEST_PREPROCESS'}.freeze,
     "-DGNU_COMPILER".freeze,
-    # '-nostdinc'.freeze,
+    # '-nostdinc'.freeze, # disabled temporarily due to stdio access violations on OSX
     "\"${1}\"".freeze,
     "-o \"${2}\"".freeze
     ].freeze
   }
 
+# Disable the -MD flag for OSX LLVM Clang, since unsupported
+if RUBY_PLATFORM =~ /darwin/ && `gcc --version 2> /dev/null` =~ /Apple LLVM version .* \(clang/m # OSX w/LLVM Clang
+  MD_FLAG = '' # Clang doesn't support the -MD flag
+else
+  MD_FLAG = '-MD'
+end
+
 DEFAULT_TEST_DEPENDENCIES_GENERATOR_TOOL = {
   :executable => FilePathUtils.os_executable_ext('gcc').freeze,
   :name => 'default_test_dependencies_generator'.freeze,
@@ -104,7 +107,7 @@ DEFAULT_TEST_DEPENDENCIES_GENERATOR_TOOL = {
     "-DGNU_COMPILER".freeze,
     "-MT \"${3}\"".freeze,
     '-MM'.freeze,
-    # '-MD'.freeze, # OSX clang
+    MD_FLAG.freeze, 
     '-MG'.freeze,
     "-MF \"${2}\"".freeze,
     "-c \"${1}\"".freeze,
@@ -127,7 +130,7 @@ DEFAULT_RELEASE_DEPENDENCIES_GENERATOR_TOOL = {
     "-DGNU_COMPILER".freeze,
     "-MT \"${3}\"".freeze,
     '-MM'.freeze,
-    # '-MD'.freeze, # OSX clang
+    MD_FLAG.freeze, 
     '-MG'.freeze,
     "-MF \"${2}\"".freeze,
     "-c \"${1}\"".freeze,

data/lib/ceedling/dependinator.rb

@@ -4,7 +4,7 @@ class Dependinator
   constructor :configurator, :project_config_manager, :test_includes_extractor, :file_path_utils, :rake_wrapper, :file_wrapper
 
   def touch_force_rebuild_files
-    @file_wrapper.touch( @configurator.project_test_force_rebuild_filepath ) 
+    @file_wrapper.touch( @configurator.project_test_force_rebuild_filepath )
     @file_wrapper.touch( @configurator.project_release_force_rebuild_filepath ) if (@configurator.project_release_build)
   end
 
@@ -16,9 +16,8 @@ class Dependinator
 
 
   def enhance_release_file_dependencies(files)
-    files.each do |filepath| 
+    files.each do |filepath|
       @rake_wrapper[filepath].enhance( [@configurator.project_release_force_rebuild_filepath] ) if (@project_config_manager.release_config_changed)
-      @rake_wrapper[filepath].enhance( [@configurator.ceedling_build_info_filepath] )
     end
   end
 
@@ -32,14 +31,12 @@ class Dependinator
 
   def enhance_runner_dependencies(runner_filepath)
     @rake_wrapper[runner_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-    @rake_wrapper[runner_filepath].enhance( [@configurator.ceedling_build_info_filepath] )    
   end
-  
+
 
   def enhance_shallow_include_lists_dependencies(include_lists)
     include_lists.each do |include_list_filepath|
       @rake_wrapper[include_list_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-      @rake_wrapper[include_list_filepath].enhance( [@configurator.ceedling_build_info_filepath] )
     end
   end
 
@@ -47,7 +44,6 @@ class Dependinator
   def enhance_preprocesed_file_dependencies(files)
     files.each do |filepath|
       @rake_wrapper[filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-      @rake_wrapper[filepath].enhance( [@configurator.ceedling_build_info_filepath] )
     end
   end
 
@@ -57,7 +53,6 @@ class Dependinator
     mocks_list.each do |mock_filepath|
       @rake_wrapper[mock_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
       @rake_wrapper[mock_filepath].enhance( [@configurator.cmock_unity_helper] )                  if (@configurator.cmock_unity_helper)
-      @rake_wrapper[mock_filepath].enhance( [@configurator.ceedling_build_info_filepath] )
       @rake_wrapper[mock_filepath].enhance( [@configurator.cmock_build_info_filepath] )
     end
   end
@@ -66,7 +61,6 @@ class Dependinator
   def enhance_dependencies_dependencies(dependencies)
     dependencies.each do |dependencies_filepath|
       @rake_wrapper[dependencies_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-      @rake_wrapper[dependencies_filepath].enhance( [@configurator.ceedling_build_info_filepath] )
     end
   end
 
@@ -74,14 +68,12 @@ class Dependinator
   def enhance_test_build_object_dependencies(objects)
     objects.each do |object_filepath|
       @rake_wrapper[object_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-      @rake_wrapper[object_filepath].enhance( [@configurator.ceedling_build_info_filepath] )
     end
   end
-  
+
 
   def enhance_results_dependencies(result_filepath)
     @rake_wrapper[result_filepath].enhance( [@configurator.project_test_force_rebuild_filepath] ) if (@project_config_manager.test_config_changed)
-    @rake_wrapper[result_filepath].enhance( [@configurator.ceedling_build_info_filepath] )    
   end
 
 

data/lib/ceedling/generator.rb

@@ -1,6 +1,5 @@
 require 'ceedling/constants'
 
-
 class Generator
 
   constructor :configurator,
@@ -66,9 +65,11 @@ class Generator
 
     @streaminator.stdout_puts("Generating runner for #{module_name}...", Verbosity::NORMAL)
 
+    test_file_includes = [] # Empty list for now, since apparently unused
+
     # build runner file
     begin
-      @generator_test_runner.generate(module_name, runner_filepath, test_cases, mock_list, [])
+      @generator_test_runner.generate(module_name, runner_filepath, test_cases, mock_list, test_file_includes)
     rescue
       raise
     ensure

data/lib/ceedling/generator_test_runner.rb

@@ -52,13 +52,13 @@ class GeneratorTestRunner
     return tests_and_line_numbers
   end
 
-  def generate(module_name, runner_filepath, test_cases, mock_list, mock_includes)
+  def generate(module_name, runner_filepath, test_cases, mock_list, test_file_includes=[])
     require 'generate_test_runner.rb'
     @test_runner_generator ||= UnityTestRunnerGenerator.new( @configurator.get_runner_config )
     @test_runner_generator.generate( module_name,
                                      runner_filepath,
                                      test_cases,
                                      mock_list,
-                                     mock_includes)
+                                     test_file_includes)
   end
 end

data/lib/ceedling/tasks_base.rake

@@ -1,38 +1,36 @@
 require 'ceedling/constants'
 require 'ceedling/file_path_utils'
-
+require 'ceedling/version'
 
 desc "Display build environment version info."
 task :version do
-  tools = [
-      ['  Ceedling', CEEDLING_ROOT],
+  puts "  Ceedling:: #{Ceedling::VERSION::Ceedling}"
+
+  [
       ['CException', File.join( CEEDLING_VENDOR, CEXCEPTION_ROOT_PATH)],
       ['     CMock', File.join( CEEDLING_VENDOR, CMOCK_ROOT_PATH)],
       ['     Unity', File.join( CEEDLING_VENDOR, UNITY_ROOT_PATH)],
-    ]
-  
-  tools.each do |tool|
+  ].each do |tool|
     name      = tool[0]
     base_path = tool[1]
-    
+
     version_string = @ceedling[:file_wrapper].read( File.join(base_path, 'release', 'version.info') ).strip
     build_string   = @ceedling[:file_wrapper].read( File.join(base_path, 'release', 'build.info') ).strip
     puts "#{name}:: #{version_string.empty? ? '#.#.' : (version_string + '.')}#{build_string.empty? ? '?' : build_string}"
   end
 end
 
-
 desc "Set verbose output (silent:[#{Verbosity::SILENT}] - obnoxious:[#{Verbosity::OBNOXIOUS}])."
 task :verbosity, :level do |t, args|
   verbosity_level = args.level.to_i
-  
+
   if (PROJECT_USE_MOCKS)
     # don't store verbosity level in setupinator's config hash, use a copy;
     # otherwise, the input configuration will change and trigger entire project rebuilds
     hash = @ceedling[:setupinator].config_hash[:cmock].clone
     hash[:verbosity] = verbosity_level
 
-    @ceedling[:cmock_builder].manufacture( hash )  
+    @ceedling[:cmock_builder].manufacture( hash )
   end
 
   @ceedling[:configurator].project_verbosity = verbosity_level
@@ -41,13 +39,11 @@ task :verbosity, :level do |t, args|
   verbose( ((verbosity_level >= Verbosity::OBNOXIOUS) ? true : false) )
 end
 
-
 desc "Enable logging"
 task :logging do
   @ceedling[:configurator].project_logging = true
 end
 
-
 # non advertised debug task
 task :debug do
   Rake::Task[:verbosity].invoke(Verbosity::DEBUG)
@@ -55,14 +51,12 @@ task :debug do
   @ceedling[:configurator].project_debug = true
 end
 
-
 # non advertised sanity checking task
 task :sanity_checks, :level do |t, args|
   check_level = args.level.to_i
   @ceedling[:configurator].sanity_checks = check_level
 end
 
-
 # list expanded environment variables
 if (not ENVIRONMENT.empty?)
 desc "List all configured environment variables."
@@ -72,11 +66,10 @@ task :environment do
       name = key.to_s.upcase
       puts " - #{name}: \"#{env[key]}\""
     end
-  end  
+  end
 end
 end
 
-
 namespace :options do
 
   COLLECTION_PROJECT_OPTIONS.each do |option_path|

data/lib/ceedling/version.rb

@@ -2,7 +2,7 @@
 module Ceedling
   module Version
     # @private
-    GEM = "0.15.5"
+    GEM = "0.15.6"
 
     # @private
     CEEDLING = GEM

data/lib/ceedling/version.rb.erb

@@ -2,7 +2,7 @@
 module Ceedling
   module Version
     # @private
-    GEM = "0.15.5"
+    GEM = "0.15.6"
 
     # @private
     CEEDLING = "<%= versions["CEEDLING"] %>"

data/spec/ceedling_spec.rb

@@ -10,7 +10,7 @@ describe 'Ceedling' do
       # execute method
       location = Ceedling.location
       # validate results
-      location.should == ceedling_path
+      expect(location).to eq(ceedling_path)
     end
   end
 
@@ -23,7 +23,7 @@ describe 'Ceedling' do
       # execute method
       location = Ceedling.load_path
       # validate results
-      location.should == load_path
+      expect(location).to eq(load_path)
     end
   end
 
@@ -36,7 +36,7 @@ describe 'Ceedling' do
       # execute method
       location = Ceedling.rakefile
       # validate results
-      location.should == rakefile_path
+      expect(location).to eq(rakefile_path)
     end
   end
 
@@ -47,11 +47,11 @@ describe 'Ceedling' do
       rakefile_path = File.join(File.dirname(__FILE__), '..').gsub('spec','lib')
       rakefile_path = File.join( rakefile_path, 'lib', 'ceedling', 'rakefile.rb' )
       # mocks/stubs/expected calls
-      Ceedling.should_receive(:load).with(rakefile_path)
+      expect(Ceedling).to receive(:load).with(rakefile_path)
       # execute method
       Ceedling.load_project
       # validate results
-      ENV['CEEDLING_MAIN_PROJECT_FILE'].should == './project.yml'
+      expect(ENV['CEEDLING_MAIN_PROJECT_FILE']).to eq('./project.yml')
     end
 
     it 'should load the project with the specified yaml file' do
@@ -60,11 +60,11 @@ describe 'Ceedling' do
       rakefile_path = File.join(File.dirname(__FILE__), '..').gsub('spec','lib')
       rakefile_path = File.join( rakefile_path, 'lib', 'ceedling', 'rakefile.rb' )
       # mocks/stubs/expected calls
-      Ceedling.should_receive(:load).with(rakefile_path)
+      expect(Ceedling).to receive(:load).with(rakefile_path)
       # execute method
       Ceedling.load_project(config: './foo.yml')
       # validate results
-      ENV['CEEDLING_MAIN_PROJECT_FILE'].should == './foo.yml'
+      expect(ENV['CEEDLING_MAIN_PROJECT_FILE']).to eq('./foo.yml')
     end
 
     it 'should load the project with the yaml file specified by the existing environment variable' do
@@ -73,11 +73,11 @@ describe 'Ceedling' do
       rakefile_path = File.join(File.dirname(__FILE__), '..').gsub('spec','lib')
       rakefile_path = File.join( rakefile_path, 'lib', 'ceedling', 'rakefile.rb' )
       # mocks/stubs/expected calls
-      Ceedling.should_receive(:load).with(rakefile_path)
+      expect(Ceedling).to receive(:load).with(rakefile_path)
       # execute method
       Ceedling.load_project
       # validate results
-      ENV['CEEDLING_MAIN_PROJECT_FILE'].should == './bar.yml'
+      expect(ENV['CEEDLING_MAIN_PROJECT_FILE']).to eq('./bar.yml')
     end
 
     it 'should load the project with the specified plugins enabled' do
@@ -88,15 +88,15 @@ describe 'Ceedling' do
       rakefile_path = File.join(File.dirname(__FILE__), '..').gsub('spec','lib')
       rakefile_path = File.join( rakefile_path, 'lib', 'ceedling', 'rakefile.rb' )
       # mocks/stubs/expected calls
-      Gem::Specification.should_receive(:find_by_name).with('ceedling-foo').and_return(spec_double)
-      spec_double.should_receive(:gem_dir).and_return('dummy/path')
-      Ceedling.should_receive(:require).with('ceedling/defaults')
-      Ceedling.should_receive(:load).with(rakefile_path)
+      expect(Gem::Specification).to receive(:find_by_name).with('ceedling-foo').and_return(spec_double)
+      expect(spec_double).to receive(:gem_dir).and_return('dummy/path')
+      expect(Ceedling).to receive(:require).with('ceedling/defaults')
+      expect(Ceedling).to receive(:load).with(rakefile_path)
       # execute method
       Ceedling.load_project( config:  './foo.yml',
                              plugins: ['foo'])
       # validate results
-      ENV['CEEDLING_MAIN_PROJECT_FILE'].should == './foo.yml'
+      expect(ENV['CEEDLING_MAIN_PROJECT_FILE']).to eq('./foo.yml')
     end
 
     it 'should set the project root if the root key is provided' do
@@ -107,13 +107,13 @@ describe 'Ceedling' do
       rakefile_path = File.join(File.dirname(__FILE__), '..').gsub('spec','lib')
       rakefile_path = File.join( rakefile_path, 'lib', 'ceedling', 'rakefile.rb' )
       # mocks/stubs/expected calls
-      Ceedling.should_receive(:load).with(rakefile_path)
+      expect(Ceedling).to receive(:load).with(rakefile_path)
       # execute method
       Ceedling.load_project( config:  './foo.yml',
                              root:    './')
       # validate results
-      ENV['CEEDLING_MAIN_PROJECT_FILE'].should == './foo.yml'
-      PROJECT_ROOT.should == './'
+      expect(ENV['CEEDLING_MAIN_PROJECT_FILE']).to eq('./foo.yml')
+      expect(PROJECT_ROOT).to eq('./')
     end
   end
 
@@ -124,14 +124,14 @@ describe 'Ceedling' do
       DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths].clear()
       spec_double = double('spec-double')
       # mocks/stubs/expected calls
-      Gem::Specification.should_receive(:find_by_name).with('ceedling-foo').and_return(spec_double)
-      spec_double.should_receive(:gem_dir).and_return('dummy/path')
-      Ceedling.should_receive(:require).with('ceedling/defaults')
+      expect(Gem::Specification).to receive(:find_by_name).with('ceedling-foo').and_return(spec_double)
+      expect(spec_double).to receive(:gem_dir).and_return('dummy/path')
+      expect(Ceedling).to receive(:require).with('ceedling/defaults')
       # execute method
       Ceedling.register_plugin('foo')
       # validate results
-      DEFAULT_CEEDLING_CONFIG[:plugins][:enabled].should    == ["foo"]
-      DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths].should == ["dummy/path"]
+      expect(DEFAULT_CEEDLING_CONFIG[:plugins][:enabled]).to eq ["foo"]
+      expect(DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths]).to eq(["dummy/path"])
     end
 
     it 'should register a plugin with an alternative prefix' do
@@ -140,14 +140,14 @@ describe 'Ceedling' do
       DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths].clear()
       spec_double = double('spec-double')
       # mocks/stubs/expected calls
-      Gem::Specification.should_receive(:find_by_name).with('prefix-foo').and_return(spec_double)
-      spec_double.should_receive(:gem_dir).and_return('dummy/path')
-      Ceedling.should_receive(:require).with('ceedling/defaults')
+      expect(Gem::Specification).to receive(:find_by_name).with('prefix-foo').and_return(spec_double)
+      expect(spec_double).to receive(:gem_dir).and_return('dummy/path')
+      expect(Ceedling).to receive(:require).with('ceedling/defaults')
       # execute method
       Ceedling.register_plugin('foo','prefix-')
       # validate results
-      DEFAULT_CEEDLING_CONFIG[:plugins][:enabled].should    == ["foo"]
-      DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths].should == ["dummy/path"]
+      expect(DEFAULT_CEEDLING_CONFIG[:plugins][:enabled]).to eq(["foo"])
+      expect(DEFAULT_CEEDLING_CONFIG[:plugins][:load_paths]).to eq(["dummy/path"])
     end
   end
 end

data/spec/par_map_spec.rb

@@ -1,6 +1,6 @@
 require 'ceedling/par_map'
 
-def short_task_should(n) 
+def short_task_should(n)
   done_count = 0
   par_map(n, [0.01] * (n - 1) + [0] * 10) do |seconds|
     sleep(seconds)
@@ -18,14 +18,14 @@ describe "par_map" do
   end
 
   it "should run shorter tasks while larger tasks are blocking (with 3 threads)" do
-    short_task_should(3) 
+    short_task_should(3)
   end
 
   it "should run shorter tasks while larger tasks are blocking (with 4 threads)" do
     short_task_should(4)
-  end 
+  end
 
-  #the following two tests are still slightly nondeterministic and may occasionally 
+  #the following two tests are still slightly nondeterministic and may occasionally
   #  show false positives (though we think we've gotten it pretty stable)
   it "should collide if multiple threads are used" do
     is_running = false
@@ -40,18 +40,18 @@ describe "par_map" do
       sleep(0.01 * x)
       is_running = false
     end
-    collision.should be true
+    expect(collision).to eq true
   end
 
   it "should be serial if only one thread is used" do
     is_running = false
     par_map(1, (1..5).to_a) do |x|
-      is_running.should be false
+      expect(is_running).to eq false
       is_running = true
       sleep(0.01 * x)
       is_running = false
     end
   end
 
-  
-end
+
+end