CI wasn't running elasticurl tests. (#171)

Also suppress output unless test fails

Author: graebm

builder.json

@@ -1,10 +1,26 @@
 {
     "name": "aws-c-http",
+    "hosts": {
+        "al2012": {
+            "enabled": false,
+            "_comment": "al2012 can't currently pull builder.py from github. SSL: CERTIFICATE_VERIFY_FAILED"
+        }
+    },
+    "targets": {
+        "android": {
+            "enabled": false,
+            "_comment": "disabled until we need to support it. LibCrypto needs to be configured on build machine."
+        }
+    },
     "upstream": [
         { "name": "aws-c-io" },
         { "name": "aws-c-compression" }
     ],
     "downstream": [
         { "name": "aws-c-auth" }
+    ],
+    "test": [
+        ["ctest", ".", "--output-on-failure"],
+        ["{python}", "{source_dir}/integration-testing/http_client_test.py", "{install_dir}/bin/elasticurl{exe}"]
     ]
 }

integration-testing/http_client_test.py

@@ -25,11 +25,18 @@
 # Remove args from sys.argv so that unittest doesn't also try to parse them.
 sys.argv = sys.argv[:1]
 
-shell = sys.platform.startswith('win')
+def run_command(args):
+    # gather all stderr and stdout to a single string that we print only if things go wrong
+    process = subprocess.Popen(args, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+    output = process.communicate()[0]
 
+    if process.returncode != 0:
+        args_str = subprocess.list2cmdline(args)
+        print(args_str)
+        for line in output.splitlines():
+            print(line.decode())
 
-def run_command(args):
-    subprocess.check_call(args, shell=shell)
+        raise RuntimeError("Return code {code} from: {cmd}".format(code=process.returncode, cmd=args_str))
 
 def compare_files(filename_expected, filename_other):
     if not filecmp.cmp(filename_expected, filename_other, shallow=False):