arduino_ci 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 63c7d8aea63187c680fcac08e18a72469421596e1c4888e4c89bf011b93ef7ab
-  data.tar.gz: 4400ad0218b36e4d0721632674de6dc2cef88126890ca1095b195433e825af74
+  metadata.gz: a38212612227d3adec2b30ed2303de39f506f12a31c06b2205ee1e703dccfbb6
+  data.tar.gz: '033169c0b7531d90c089296cf3cd5b708995bb2e89db068813fa515ab0022b17'
 SHA512:
-  metadata.gz: c720975cb84375e4a10d3781db680a589af77d8af89f4704994180bbe24e2838fe922b13cad50eceb2063403b908c0cc77c7241f6f2692eb9d96443c7cfa7624
-  data.tar.gz: 4bc2267fe4be0b0790bbb31ff4cc339c3e231e723519a428bdf12fdd615f441097dbdd230ce95a47e627b195c0a8afd393a9ac88de04deccef0c73b86fd2b3b0
+  metadata.gz: 7def94f0fe1d00fc32ccac118766da3cc8272f69b19123eaca24b78592e171b99687cead667c7506acb4e62f7b0445f141ed6ca716053b9d78061d66b5fafd19
+  data.tar.gz: af306adce91faa99e433a8006de72aa0a04a0d5f06e47187672f7d6db07254e6850c374684753e45708ce3e53cee7dd51dfffa17e63c4f58d15a20ee1e72513c

data/README.md

@@ -1,6 +1,6 @@
 [![Gem Version](https://badge.fury.io/rb/arduino_ci.svg)](https://rubygems.org/gems/arduino_ci)
 [![Build Status](https://travis-ci.org/ifreecarve/arduino_ci.svg)](https://travis-ci.org/ifreecarve/arduino_ci)
-[![Documentation](http://img.shields.io/badge/docs-rdoc.info-blue.svg)](http://www.rubydoc.info/gems/arduino_ci/0.1.2)
+[![Documentation](http://img.shields.io/badge/docs-rdoc.info-blue.svg)](http://www.rubydoc.info/gems/arduino_ci/0.1.3)
 
 # ArduinoCI Ruby gem (`arduino_ci`)
 
@@ -43,12 +43,10 @@ unittest(your_test_name)
   assertEqual(4, doSomething());
 }
 
-int main(int argc, char *argv[]) {
-  return Test::run_and_report(argc, argv);
-}
+unittest_main()
 ```
 
-This test defines one `unittest` (a macro provided by `ArduionUnitTests.h`), called `your_test_name`, which makes some assertions on the target library.  The `int main` section is boilerplate.
+This test defines one `unittest` (a macro provided by `ArduionUnitTests.h`), called `your_test_name`, which makes some assertions on the target library.  The `unittest_main()` is a macro for the `int main()` boilerplate required for unit testing.
 
 
 ## More Documentation

data/cpp/unittest/ArduinoUnitTests.h

@@ -130,14 +130,16 @@ class Test
       Results results = {0, 0, 0};
 
       for (Test *p = sRoot; p; p = p->mNext) {
-        TestData td = {p->name(), p->result()};
+        p->prepare();
         p->mReporter = reporter;
-        if (reporter) reporter->onTestStart(td);
+        TestData t1 = {p->name(), p->result()};
+        if (reporter) reporter->onTestStart(t1);
         p->test();
         if (p->mResult == RESULT_PASS) ++results.passed;
         if (p->mResult == RESULT_FAIL) ++results.failed;
         if (p->mResult == RESULT_SKIP) ++results.skipped;
-        if (reporter) reporter->onTestEnd(td);
+        TestData t2 = {p->name(), p->result()};
+        if (reporter) reporter->onTestEnd(t2);
       }
 
       return results;
@@ -154,8 +156,12 @@ class Test
       return results.failed + results.skipped;
     }
 
+    void prepare() {
+      mResult = RESULT_PASS;  // not None, and not fail unless we hear otherwise
+    }
+
     void test() {
-      mResult = RESULT_PASS; // not None, and not fail unless we hear otherwise
+      // thin wrapper.  nothing to do here for now
       task();
     }
 
@@ -210,3 +216,9 @@ class Test
     void task();                   \
   } test_##name##_instance;        \
   void test_##name ::task()
+
+
+#define unittest_main()                      \
+  int main(int argc, char *argv[]) {         \
+    return Test::run_and_report(argc, argv); \
+  }

data/cpp/unittest/Assertion.h

@@ -1,21 +1,30 @@
 #pragma once
 #include "Compare.h"
 
-// helper define for the operators below
-#define assertOp(desc, relevance1, arg1, op, op_name, relevance2, arg2) \
-  do                                                               \
-  {                                                                \
-    if (!assertion<typeof(arg1), typeof(arg2)>(__FILE__, __LINE__, \
-                                               desc,               \
-                                               relevance1, #arg1, (arg1),      \
-                                               op_name, op,        \
-                                               relevance2, #arg2, (arg2)))     \
-    {                                                              \
-      return;                                                      \
-    }                                                              \
+#define testBehaviorOp(die, desc, rel1, arg1, op, op_name, rel2, arg2) \
+  do                                                                   \
+    {                                                                  \
+    if (!assertion<typeof(arg1), typeof(arg2)>(__FILE__, __LINE__,     \
+                                               desc,                   \
+                                               rel1, #arg1, (arg1),    \
+                                               op_name, op,            \
+                                               rel2, #arg2, (arg2)))   \
+    {                                                                  \
+      if (die) return;                                                 \
+    }                                                                  \
   } while (0)
 
-/** macro generates optional output and calls fail() followed by a return if false. */
+
+
+// helper define for the operators below
+#define assertOp(desc, rel1, arg1, op, op_name, rel2, arg2) \
+  testBehaviorOp(false, desc, rel1, arg1, op, op_name, rel2, arg2)
+
+#define assureOp(desc, rel1, arg1, op, op_name, rel2, arg2) \
+  testBehaviorOp(true, desc, rel1, arg1, op, op_name, rel2, arg2)
+
+
+/** macro generates optional output and calls fail() but does not return if false. */
 #define assertEqual(arg1,arg2)       assertOp("assertEqual","expected",arg1,compareEqual,"==","actual",arg2)
 #define assertNotEqual(arg1,arg2)    assertOp("assertNotEqual","unwanted",arg1,compareNotEqual,"!=","actual",arg2)
 #define assertLess(arg1,arg2)        assertOp("assertLess","lowerBound",arg1,compareLess,"<","upperBound",arg2)
@@ -25,3 +34,13 @@
 #define assertTrue(arg) assertEqual(arg,true)
 #define assertFalse(arg) assertEqual(arg,false)
 
+/** macro generates optional output and calls fail() followed by a return if false. */
+#define assureEqual(arg1,arg2)       assureOp("assureEqual","expected",arg1,compareEqual,"==","actual",arg2)
+#define assureNotEqual(arg1,arg2)    assureOp("assureNotEqual","unwanted",arg1,compareNotEqual,"!=","actual",arg2)
+#define assureLess(arg1,arg2)        assureOp("assureLess","lowerBound",arg1,compareLess,"<","upperBound",arg2)
+#define assureMore(arg1,arg2)        assureOp("assureMore","upperBound",arg1,compareMore,">","lowerBound",arg2)
+#define assureLessOrEqual(arg1,arg2) assureOp("assureLessOrEqual","lowerBound",arg1,compareLessOrEqual,"<=","upperBound",arg2)
+#define assureMoreOrEqual(arg1,arg2) assureOp("assureMoreOrEqual","upperBound",arg1,compareMoreOrEqual,">=","lowerBound",arg2)
+#define assureTrue(arg) assertEqual(arg,true)
+#define assureFalse(arg) assertEqual(arg,false)
+

data/exe/arduino_ci_remote.rb

@@ -72,8 +72,10 @@ end
 # do that, set the URLs, and download the packages
 all_packages = all_platforms.values.map { |v| v[:package] }.uniq.reject(&:nil?)
 all_urls = all_packages.map { |p| config.package_url(p) }.uniq.reject(&:nil?)
-assure("Setting board manager URLs") do
-  @arduino_cmd.set_pref("boardsmanager.additional.urls", all_urls.join(","))
+unless all_urls.empty?
+  assure("Setting board manager URLs") do
+    @arduino_cmd.set_pref("boardsmanager.additional.urls", all_urls.join(","))
+  end
 end
 
 all_packages.each do |p|
@@ -86,26 +88,6 @@ aux_libraries.each do |l|
   assure("Installing aux library '#{l}'") { @arduino_cmd.install_library(l) }
 end
 
-attempt("Setting compiler warning level") { @arduino_cmd.set_pref("compiler.warning_level", "all") }
-
-library_examples.each do |example_path|
-  ovr_config = config.from_example(example_path)
-  ovr_config.platforms_to_build.each do |p|
-    board = all_platforms[p][:board]
-    assure("Switching to board for #{p} (#{board})") { @arduino_cmd.use_board(board) }
-    example_name = File.basename(example_path)
-    attempt("Verifying #{example_name}") do
-      ret = @arduino_cmd.verify_sketch(example_path)
-      unless ret
-        puts
-        puts "Last command: #{@arduino_cmd.last_msg}"
-        puts @arduino_cmd.last_err
-      end
-      ret
-    end
-  end
-end
-
 config.platforms_to_unittest.each do |p|
   board = all_platforms[p][:board]
   assure("Switching to board for #{p} (#{board})") { @arduino_cmd.use_board(board) }
@@ -129,4 +111,24 @@ config.platforms_to_unittest.each do |p|
   end
 end
 
+attempt("Setting compiler warning level") { @arduino_cmd.set_pref("compiler.warning_level", "all") }
+
+library_examples.each do |example_path|
+  ovr_config = config.from_example(example_path)
+  ovr_config.platforms_to_build.each do |p|
+    board = all_platforms[p][:board]
+    assure("Switching to board for #{p} (#{board})") { @arduino_cmd.use_board(board) }
+    example_name = File.basename(example_path)
+    attempt("Verifying #{example_name}") do
+      ret = @arduino_cmd.verify_sketch(example_path)
+      unless ret
+        puts
+        puts "Last command: #{@arduino_cmd.last_msg}"
+        puts @arduino_cmd.last_err
+      end
+      ret
+    end
+  end
+end
+
 terminate(true)

data/lib/arduino_ci/arduino_cmd.rb

@@ -243,10 +243,6 @@ module ArduinoCI
         return false
       end
       ret = run_and_capture(flag_verify, path)
-      puts "=============="
-      puts ret[:out]
-      puts "--------------"
-      puts ret[:err]
       ret[:success]
     end
 

data/lib/arduino_ci/cpp_library.rb

@@ -35,12 +35,28 @@ module ArduinoCI
       @last_msg = ""
     end
 
+    # Guess whether a file is part of the vendor bundle (indicating we should ignore it).
+    #
+    # This assumes the vendor bundle will be at `vendor/bundle` and not some other location
+    # @param path [String] The path to check
+    # @return [Array<String>] The paths of the found files
+    def vendor_bundle?(path)
+      # TODO: look for Gemfile, look for .bundle/config and get BUNDLE_PATH from there?
+      base = File.join(@base_dir, "vendor")
+      real = File.join(File.realpath(@base_dir), "vendor")
+      return true if path.start_with?(base)
+      return true if path.start_with?(real)
+      false
+    end
+
     # Get a list of all CPP source files in a directory and its subdirectories
     # @param some_dir [String] The directory in which to begin the search
     # @return [Array<String>] The paths of the found files
     def cpp_files_in(some_dir)
       real = File.realpath(some_dir)
-      Find.find(real).select { |path| CPP_EXTENSIONS.include?(File.extname(path)) }
+      files = Find.find(real).reject { |path| File.directory?(path) }
+      ret = files.select { |path| CPP_EXTENSIONS.include?(File.extname(path)) }
+      ret
     end
 
     # CPP files that are part of the project library under test
@@ -50,6 +66,7 @@ module ArduinoCI
       cpp_files_in(@base_dir).reject do |p|
         next true if File.dirname(p).include?(tests_dir)
         next true if File.dirname(p).include?(real_tests_dir)
+        next true if vendor_bundle?(p)
       end
     end
 
@@ -80,8 +97,12 @@ module ArduinoCI
     # Find all directories in the project library that include C++ header files
     # @return [Array<String>]
     def header_dirs
-      files = Find.find(@base_dir).select { |path| HPP_EXTENSIONS.include?(File.extname(path)) }
-      files.map { |path| File.dirname(path) }.uniq
+      real = File.realpath(@base_dir)
+      all_files = Find.find(real).reject { |path| File.directory?(path) }
+      unbundled = all_files.reject { |path| vendor_bundle?(path) }
+      files = unbundled.select { |path| HPP_EXTENSIONS.include?(File.extname(path)) }
+      ret = files.map { |path| File.dirname(path) }.uniq
+      ret
     end
 
     # wrapper for the GCC command

data/lib/arduino_ci/version.rb

@@ -1,3 +1,3 @@
 module ArduinoCI
-  VERSION = "0.1.2".freeze
+  VERSION = "0.1.3".freeze
 end

data/misc/default.yaml

@@ -1,3 +1,5 @@
+# Note that ci_config_spec.rb has tests for this file's contents
+
 packages:
   esp8266:esp8266:
     url: http://arduino.esp8266.com/stable/package_esp8266com_index.json
@@ -22,7 +24,7 @@ platforms:
       warnings:
       flags:
   zero:
-    board: arduino:samd:zero
+    board: arduino:samd:arduino_zero_native
     package: arduino:samd
     gcc:
       features:
@@ -30,7 +32,7 @@ platforms:
       warnings:
       flags:
   esp8266:
-    board: esp8266:esp8266:huzzah
+    board: esp8266:esp8266:huzzah:FlashSize=4M3M,CpuFrequency=80
     package: esp8266:esp8266
     gcc:
       features:
@@ -67,6 +69,7 @@ compile:
   platforms:
     - uno
     - due
+    - zero
     - leonardo
 
 unittest:
@@ -74,4 +77,5 @@ unittest:
   platforms:
     - uno
     - due
+    - zero
     - leonardo

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: arduino_ci
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Ian Katz