From: Jim Morris <morris@wolfman.com>
Date: Sun, 13 Sep 2015 06:07:12 +0000 (-0700)
Subject: Merge remote-tracking branch 'upstream/development/test-framework' into development... 
X-Git-Url: http://git.hcoop.net/clinton/Smoothieware.git/commitdiff_plain/a3cb0c4cacfa85ea0cbe17a192f4819e38d65537?hp=2097978fc1a2d73e6a968da33e9ad284ccacdf2c

Merge remote-tracking branch 'upstream/development/test-framework' into development/test-framework

Conflicts:
	Rakefile
	src/testframework/Test_kernel.cpp
---

diff --git a/Rakefile b/Rakefile
index 410941fb..5213d6b0 100644
--- a/Rakefile
+++ b/Rakefile
@@ -119,6 +119,7 @@ if TESTING
   TESTMODULES= %w(tools/temperatureswitch) unless defined? EXCLUDE_MODULES
   puts "Modules under test: #{TESTMODULES}"
   excludes << %w(Kernel.cpp main.cpp) # we replace these with mock versions in testframework
+
   frameworkfiles= FileList['src/testframework/*.{c,cpp}', 'src/testframework/easyunit/*.{c,cpp}']
   extrafiles= FileList['src/modules/communication/SerialConsole.cpp', 'src/modules/communication/utils/Gcode.cpp', 'src/modules/robot/Conveyor.cpp', 'src/modules/robot/Block.cpp']
   testmodules= FileList['src/libs/**/*.{c,cpp}'].include(TESTMODULES.collect { |e| "src/modules/#{e}/**/*.{c,cpp}"}).include(TESTMODULES.collect { |e| "src/testframework/unittests/#{e}/*.{c,cpp}"}).exclude(/#{excludes.join('|')}/)
diff --git a/src/testframework/Test_kernel.cpp b/src/testframework/Test_kernel.cpp
index ab36bdc4..f9ee86c0 100644
--- a/src/testframework/Test_kernel.cpp
+++ b/src/testframework/Test_kernel.cpp
@@ -40,13 +40,6 @@ This is aprt of the Smoothie test framework, it generates a Mockable Kernl so ke
 
 Kernel* Kernel::instance;
 
-
-// define some dummies here to satisfy linker
-// Conveyor::Conveyor(){ /*dummy*/ }
-// void Conveyor::wait_for_empty_queue(){ /*dummy*/ }
-//template class HeapRing<Block>;
-
-
 // The kernel is the central point in Smoothie : it stores modules, and handles event calls
 Kernel::Kernel(){
     instance= this; // setup the Singleton instance of the kernel
@@ -66,7 +59,7 @@ Kernel::Kernel(){
 
     this->slow_ticker = new SlowTicker();
 
-    // dummies (woul dbe noce to refactor to not have to create a conveyor)
+    // dummies (would be nice to refactor to not have to create a conveyor)
     this->conveyor= new Conveyor();
 
     // Configure UART depending on MRI config