Merge pull request #31 from ifreecarve/2018-04-12_ci

Hotfix for CI script problems

Author: ianfixes

CHANGELOG.md

@@ -6,6 +6,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 ### Added
+- Explicit tests of `.arduino-ci.yml` in `TestSomething` example
 
 ### Changed
 
@@ -14,6 +15,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Removed
 
 ### Fixed
+- Malformed YAML (duplicate unittests section) now has no duplicate section
+- arduino_ci_remote.rb script now has correct arguments in build_for_test_with_configuration
 
 ### Security
 

SampleProjects/TestSomething/.arduino-ci.yml

@@ -2,7 +2,5 @@ unittest:
   platforms:
     - uno
     - due
-
-unittest:
   compilers:
     - g++

SampleProjects/TestSomething/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: /Users/ikatz/Development/non-wayfair/arduino_ci
   specs:
-    arduino_ci (0.0.1)
+    arduino_ci (0.1.8)
       os (~> 1.0)
 
 GEM

exe/arduino_ci_remote.rb

@@ -117,7 +117,7 @@ def assure(message, &block)
 if cpp_library.test_files.empty?
   attempt("Skipping unit tests; no test files were found") { true }
 elsif config.platforms_to_unittest.empty?
-  attempt("Skipping unit tests; no platforms were requestsed") { true }
+  attempt("Skipping unit tests; no platforms were requested") { true }
 else
   config.platforms_to_unittest.each do |p|
     board = all_platforms[p][:board]
@@ -130,6 +130,7 @@ def assure(message, &block)
           exe = cpp_library.build_for_test_with_configuration(
             unittest_path,
             config.aux_libraries_for_unittest,
+            gcc_binary,
             config.gcc_config(p)
           )
           puts

spec/testsomething_unittests_spec.rb

@@ -30,6 +30,10 @@
       expect(config.compilers_to_use.length.zero?).to be(false)
     end
 
+    it "has at least one unit test platform defined" do
+      expect(config.platforms_to_unittest.length.zero?).to be(false)
+    end
+
     test_files = config.allowable_unittest_files(cpp_library.test_files)
     test_files.each do |path|
       tfn = File.basename(path)