Don't define ARDUINO=100 if already defined as something else.

Author: James Foster

CHANGELOG.md

@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 - Don't define `ostream& operator<<(nullptr_t)` if already defined by Apple
+- Don't define `ARDUINO=100` if already defined as something else
 
 ### Security
 

lib/arduino_ci/cpp_library.rb

@@ -324,7 +324,7 @@ def build_for_test_with_configuration(test_file, aux_libraries, gcc_binary, ci_g
       executable = Pathname.new("unittest_#{base}.bin").expand_path
       File.delete(executable) if File.exist?(executable)
       arg_sets = []
-      arg_sets << ["-std=c++0x", "-o", executable.to_s, "-DARDUINO=100"]
+      arg_sets << ["-std=c++0x", "-o", executable.to_s]
       if libasan?(gcc_binary)
         arg_sets << [ # Stuff to help with dynamic memory mishandling
           "-g", "-O1",
@@ -334,6 +334,9 @@ def build_for_test_with_configuration(test_file, aux_libraries, gcc_binary, ci_g
         ]
       end
       arg_sets << test_args(aux_libraries, ci_gcc_config)
+      if (arg_sets.flatten(1).index{|s| s.include?("-DARDUINO=")} == nil) 
+        arg_sets << ["-DARDUINO=100"]
+      end
       arg_sets << cpp_files_libraries(aux_libraries).map(&:to_s)
       arg_sets << [test_file.to_s]
       args = arg_sets.flatten(1)