Merge pull request #11 from kvz/osxfailpasses

Fix Travis OSX testing

README.md

@@ -51,6 +51,7 @@ Although *3* introduces a node.js dependency, this does allow for easy version p
 
 ### master (Unreleased)
 
+- Fix Travis OSX testing (before, it would silently pass failures) (#10)
 - Enable dashes in long, GNU style options, as well as numbers (thanks @zbeekman)
 
 ### v2.0.0 (2016-02-17)

test/acceptance.sh

@@ -147,17 +147,14 @@ for scenario in $(echo ${scenarios}); do
         fi
       fi
 
-      diff \
-        --strip-trailing-cr \
-        "${__dir}/fixture/${scenario}.${typ}" \
-        "${curFile}" || ( \
-        echo -e "\n\n==> MISMATCH OF: ${typ}";
-        echo -e "\n\n==> EXPECTED STDIO: ";
-        cat "${__dir}/fixture/${scenario}.stdio";
-        echo -e "\n\n==> ACTUAL STDIO: ";
-        cat "${__accptstTmpDir}/${scenario}.stdio";
-        exit 1; \
-      )
+      if ! diff --strip-trailing-cr "${__dir}/fixture/${scenario}.${typ}" "${curFile}"; then
+        echo -e "\n\n==> MISMATCH OF: ${scenario}.${typ} ---^"
+        echo -e "\n\n==> EXPECTED STDIO: "
+        cat "${__dir}/fixture/${scenario}.stdio" || true
+        echo -e "\n\n==> ACTUAL STDIO: "
+        cat "${__accptstTmpDir}/${scenario}.stdio" || true
+        exit 1
+      fi
 
       echo "✓"
     done