Do not compile test/* in included libraries

rosogon · rosogon · commit 640201afb14b · 2020-10-03T23:00:42.000+01:00
If libB depends on libA, both with arduino_ci tests, trying to run the unit
tests on libB will raise the error "multiple definition of main". This happens
because cpp_files_libraries returns the whole tree of the referenced library
(libA in the example), not removing the files in test/.
diff --git a/lib/arduino_ci/cpp_library.rb b/lib/arduino_ci/cpp_library.rb
@@ -186,7 +186,8 @@ def cpp_files_unittest
     # @param [Array<String>] aux_libraries
     # @return [Array<Pathname>]
     def cpp_files_libraries(aux_libraries)
-      arduino_library_src_dirs(aux_libraries).map { |d| cpp_files_in(d) }.flatten.uniq
+      l = arduino_library_src_dirs(aux_libraries).map { |d| cpp_files_in(d) }.flatten.uniq
+      l = l.reject { |p| p.dirname.to_s.end_with?("/test") }
     end
 
     # Returns the Pathnames for all paths to exclude from testing and compilation
