Merge pull request #103 from ianfixes/2019-01-29_updates

Unit test additions (both Ruby and C++)

Author: ianfixes

CHANGELOG.md

@@ -10,10 +10,18 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 - `ArduinoInstallation::force_install` now optionally accepts a version string
 - `arduino_library_location.rb` script to print Arduino library location to stdout
 - `arduino_ci_remote.rb` now supports `--skip-unittests` and `--skip-compilation`.  If you skip both, only the `autolocate!` of the Arduino binary will be performed.
+- `keepachangelog_manager` gem to begin streamlining the release process
+- `unittest_setup()` and `unittest_teardown()` macros, my thanks to @hlovdal for contributing this code
+- Added rspec sensitivity to the environment variable `$ARDUINO_CI_SKIP_SPLASH_SCREEN_RSPEC_TESTS` (for `arduino_ci` gem hackers)
+- Added rspec sensitivity to the environment variable `$ARDUINO_CI_SKIP_RUBY_RSPEC_TESTS` (for `arduino_ci` gem hackers)
+- Added rspec sensitivity to the environment variable `$ARDUINO_CI_SKIP_CPP_RSPEC_TESTS` (for `arduino_ci` gem hackers)
+- `nullptr` definition in C++
+- `assertNull()` for unit tests
 
 ### Changed
 - Unit tests and examples are now executed alphabetically by filename
 - The `pgm_read_...` preprocessor macros in cpp/arduino/avr/pgmspace.h now expands to an expression with applicable type.
+- Unit tests for interrupts (`attachInterrupt` and `detachInterrupt`) get their own file
 
 ### Deprecated
 

CONTRIBUTING.md

@@ -32,8 +32,8 @@ See `SampleProjects/TestSomething/test/*.cpp` for the existing tests (run by rsp
 * Merge pull request with new features
 * `git stash save` (at least before the gem build step, but easiest here).
 * `git pull --rebase`
+* Update the sections of `CHANGELOG.md` by running `bundle exec keepachangelog_release.md --increment-patch`
 * Bump the version in lib/arduino_ci/version.rb and change it in README.md (since rubydoc.info doesn't always redirect to the latest version)
-* Update the sections of `CHANGELOG.md`
 * `git add README.md CHANGELOG.md lib/arduino_ci/version.rb`
 * `git commit -m "vVERSION bump"`
 * `git tag -a vVERSION -m "Released version VERSION"`

SampleProjects/TestSomething/test/godmode.cpp

@@ -34,17 +34,6 @@ unittest(random)
   assertEqual(state->seed, 4294967282);
 }
 
-void myInterruptHandler() {
-}
-
-unittest(interrupts)
-{
-  // these are meaningless for testing; just call the routine directly.
-  // make sure our mocks work though
-  attachInterrupt(2, myInterruptHandler, CHANGE);
-  detachInterrupt(2);
-}
-
 unittest(pins)
 {
   GodmodeState* state = GODMODE();
@@ -172,17 +161,6 @@ unittest(pin_write_history)
 
 }
 
-unittest(interrupt_attachment) {
-  GodmodeState *state = GODMODE();
-  state->reset();
-  assertFalse(state->interrupt[0].attached);
-  attachInterrupt(0, (void (*)(void))0, 3);
-  assertTrue(state->interrupt[0].attached);
-  assertEqual(state->interrupt[0].mode, 3);
-  detachInterrupt(0);
-  assertFalse(state->interrupt[0].attached);
-}
-
 unittest(spi) {
   GodmodeState *state = GODMODE();
   state->reset();

SampleProjects/TestSomething/test/interrupts.cpp

@@ -0,0 +1,28 @@
+#include <ArduinoUnitTests.h>
+#include <Arduino.h>
+
+void myInterruptHandler() {
+}
+
+unittest(interrupts)
+{
+  // these are meaningless for testing; just call the routine directly.
+  // make sure our mocks work though
+  attachInterrupt(2, myInterruptHandler, CHANGE);
+  detachInterrupt(2);
+}
+
+unittest(interrupt_attachment) {
+  GodmodeState *state = GODMODE();
+  state->reset();
+  assertFalse(state->interrupt[0].attached);
+  attachInterrupt(0, (void (*)(void))0, 3);
+  assertTrue(state->interrupt[0].attached);
+  assertEqual(state->interrupt[0].mode, 3);
+  detachInterrupt(0);
+  assertFalse(state->interrupt[0].attached);
+}
+
+
+
+unittest_main()

SampleProjects/TestSomething/test/null.cpp

@@ -19,4 +19,11 @@ unittest(nothing)
 {
 }
 
+unittest(nullpointer)
+{
+  int* myPointer = NULL;
+  assertNull(myPointer);
+  assertNull(nullptr);
+}
+
 unittest_main()

SampleProjects/TestSomething/test/setup_and_treardown.cpp renamed to SampleProjects/TestSomething/test/setup_and_teardown.cpp

@@ -8,50 +8,66 @@ class LcdInterface {
 
 class MockLcd : public LcdInterface {
   public:
-  void print(const char *) {}
+    String s;
+    void print(const char* c)
+    {
+      s = String(c);
+    }
 };
 
-LcdInterface *Lcd_p;
-
 class Calculator {
+  private:
+    LcdInterface *m_lcd;
+
   public:
-  int add(int a, int b) {
-    int result = a + b;
-    char buf[40];
-    sprintf(buf, "%d + %d = %d", a, b, result);
-    Lcd_p->print(buf);
-    return result;
+    Calculator(LcdInterface* lcd) {
+      m_lcd = lcd;
+    }
+
+    ~Calculator() {
+      m_lcd = 0;
+    }
+
+    int add(int a, int b)
+    {
+      int result = a + b;
+      char buf[40];
+      sprintf(buf, "%d + %d = %d", a, b, result);
+      m_lcd->print(buf);
+      return result;
   }
 };
 
+
+// This is a typical test where using setup (and teardown) would be useful
+// to set up the "external" lcd dependency that the calculator uses indirectly
+// but it is not something that is related to the functionality that is tested.
+
+MockLcd* lcd_p;
+Calculator* c;
+
 unittest_setup()
 {
-  Lcd_p = new MockLcd();
+  lcd_p = new MockLcd();
+  c = new Calculator(lcd_p);
 }
 
 unittest_teardown()
 {
-  delete Lcd_p;
+  delete c;
+  delete lcd_p;
 }
 
-// This is a typical test where using setup (and teardown) would be useful
-// to set up the "external" lcd dependency that the calculator uses indirectly
-// but it is not something that is related to the functionality that is tested.
 
 // When you want to test that the calculator actually prints the calculations,
 // then that should be done in the arrange part of the actual test (by setting
 // up an mock lcd class that keeps a list of all messages printed).
 
 unittest(add)
 {
-  // Arrange
-  Calculator c;
-
-  // Act
-  int result = c.add(11, 22);
-
-  // Assert
+  int result = c->add(11, 22);
   assertEqual(33, result);
+  assertEqual("11 + 22 = 33", lcd_p->s);
 }
 
 unittest_main()

arduino_ci.gemspec

@@ -29,6 +29,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "rubyzip", "~> 1.2"
 
   spec.add_development_dependency "bundler", "~> 1.15"
+  spec.add_development_dependency "keepachangelog_manager", "~> 0.0.1"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency 'rubocop', '~>0.59.0'
   spec.add_development_dependency 'yard', '~>0.9.11'

cpp/arduino/Arduino.h

@@ -72,5 +72,7 @@ inline unsigned int makeWord(unsigned char h, unsigned char l) { return (h << 8)
 #define word(...) makeWord(__VA_ARGS__)
 
 
+// Define C++11 nullptr
+#define nullptr (std::nullptr_t)NULL
 
 

cpp/unittest/ArduinoUnitTests.h

@@ -69,29 +69,29 @@ class Test
         }
 
         template <typename A, typename B> void onAssert(
-              const char* file,
-              int line,
-              const char* description,
-              bool pass,
-              const char* lhsRelevance,
-              const char* lhsLabel,
-              const A &lhs,
-              const char* opLabel,
-              const char* rhsRelevance,
-              const char* rhsLabel,
-              const B &rhs
-          ) {
-            cerr << "    " << (pass ? "" : "not ") << "ok " << ++mAssertCounter << " - ";
-            cerr << description << " " << lhsLabel << " " << opLabel << " " << rhsLabel << endl;
-            if (!pass) {
-              cerr << "      ---" << endl;
-              cerr << "      operator: " << opLabel << endl;
-              cerr << "      " << lhsRelevance << ": " << lhs << endl;
-              cerr << "      " << rhsRelevance << ": " << rhs << endl;
-              cerr << "      at:" << endl;
-              cerr << "        file: " << file << endl;
-              cerr << "        line: " << line << endl;
-              cerr << "      ..." << endl;
+          const char* file,
+          int line,
+          const char* description,
+          bool pass,
+          const char* lhsRelevance,
+          const char* lhsLabel,
+          const A &lhs,
+          const char* opLabel,
+          const char* rhsRelevance,
+          const char* rhsLabel,
+          const B &rhs
+        ) {
+          cerr << "    " << (pass ? "" : "not ") << "ok " << ++mAssertCounter << " - ";
+          cerr << description << " " << lhsLabel << " " << opLabel << " " << rhsLabel << endl;
+          if (!pass) {
+            cerr << "      ---" << endl;
+            cerr << "      operator: " << opLabel << endl;
+            cerr << "      " << lhsRelevance << ": " << lhs << endl;
+            cerr << "      " << rhsRelevance << ": " << rhs << endl;
+            cerr << "      at:" << endl;
+            cerr << "        file: " << file << endl;
+            cerr << "        line: " << line << endl;
+            cerr << "      ..." << endl;
           }
         }
     };

cpp/unittest/Assertion.h

@@ -38,6 +38,7 @@
 #define assertMoreOrEqual(arg1,arg2) assertOp("assertMoreOrEqual","upperBound",arg1,compareMoreOrEqual,">=","lowerBound",arg2)
 #define assertTrue(arg) assertEqual(true, arg)
 #define assertFalse(arg) assertEqual(false, arg)
+#define assertNull(arg) assertEqual((void*)NULL, (void*)arg)
 
 /** macro generates optional output and calls fail() followed by a return if false. */
 #define assureEqual(arg1,arg2)       assureOp("assureEqual","expected",arg1,compareEqual,"==","actual",arg2)
@@ -48,4 +49,5 @@
 #define assureMoreOrEqual(arg1,arg2) assureOp("assureMoreOrEqual","upperBound",arg1,compareMoreOrEqual,">=","lowerBound",arg2)
 #define assureTrue(arg) assureEqual(true, arg)
 #define assureFalse(arg) assureEqual(false, arg)
+#define assureNull(arg) assureEqual((void*)NULL, (void*)arg)