fix automated headless testing on travis

.gitignore

@@ -1,6 +1,7 @@
 .DS_Store
 *.xcodeproj
 *.o
+*.actual.png
 /node_modules
 /mapnik-packaging
 /macosx/build

.travis.yml

@@ -16,7 +16,8 @@ env:
    - secure: "Cbvap9ubVKgjPe3hUhI6JGeDZzBXHpOG9RaYKh+SdoIPhKnlJiNOYm1egomi+e4uqJInlFKuVHTw7Ng9Cun6Zm0jIxpkSchv1GpsR7hmB3UGnGed19Dw8121FwuUaktN+4YnbVlsyd+u8EHD3+h58t4eELrLrZolM4rS7DL6caA="
 
 before_install:
-- source ./scripts/setup_travis.sh
+- ./scripts/setup_travis.sh
+- source ./scripts/flags.sh
 
 install:
 - make setup
@@ -25,8 +26,12 @@ before_script:
 - rm -rf mapnik-packaging/out/packages
 
 script:
-- make linux BUILDTYPE=${BUILDTYPE}
-- make run-tests BUILDTYPE=${BUILDTYPE}
+- make linux -j4 BUILDTYPE=${BUILDTYPE}
+- make test -j4 BUILDTYPE=${BUILDTYPE}
+- ./scripts/run_tests.sh
+
+after_script:
+- ./scripts/deploy_results.sh
 
 notifications:
   hipchat:

Makefile

@@ -27,10 +27,6 @@ build/test/Makefile: src common config.gypi test/test.gyp
 test: build/test/Makefile
 	$(MAKE) -C build/test BUILDTYPE=$(BUILDTYPE) V=$(V) test
 
-# Runs all test cases
-run-tests: test
-	BUILDTYPE=$(BUILDTYPE) ./scripts/run_tests.sh
-
 test/%: build/test/Makefile
 	$(MAKE) -C build/test BUILDTYPE=$(BUILDTYPE) V=$(V) $*
 	(cd build/$(BUILDTYPE) && exec ./test_$*)

bin/build-shaders.js

@@ -117,12 +117,14 @@ module.exports = function(shader_type, prefix, suffix) {
     code += '\n};\n';
     code += suffix + '\n';
 
-    var file_path = path.join(process.argv[2], 'include/llmr/shader/shaders.hpp');
-    mkdirp.sync(path.dirname(file_path));
-    fs.writeFileSync(file_path, header);
-    var header_path = path.join(process.argv[2], 'src/shader/shaders_' + shader_type + '.cpp');
+    var header_path = path.join(process.argv[2], 'include/llmr/shader/shaders.hpp');
     mkdirp.sync(path.dirname(header_path));
-    fs.writeFileSync(header_path, code);
+    fs.writeFileSync(header_path, header);
+    console.warn('wrote file ' + header_path);
+    var file_path = path.join(process.argv[2], 'src/shader/shaders_' + shader_type + '.cpp');
+    mkdirp.sync(path.dirname(file_path));
+    fs.writeFileSync(file_path, code);
+    console.warn('wrote file ' + file_path);
 
 };
 

llmr.gyp

@@ -15,9 +15,9 @@
             '<!@(find src -name "*.glsl")'
           ],
           'outputs': [
-            'include/llmr/shader/shaders.hpp',
-            'src/shader/shaders_gl.cpp',
-            'src/shader/shaders_gles2.cpp',
+            '<(SHARED_INTERMEDIATE_DIR)/include/llmr/shader/shaders.hpp',
+            '<(SHARED_INTERMEDIATE_DIR)/src/shader/shaders_gl.cpp',
+            '<(SHARED_INTERMEDIATE_DIR)/src/shader/shaders_gles2.cpp',
           ],
           'action': ['<@(node)', 'bin/build-shaders.js', '<(SHARED_INTERMEDIATE_DIR)'],
         }
@@ -84,6 +84,7 @@
       'target_name': 'llmr-x86',
       'product_name': 'llmr-x86',
       'type': 'static_library',
+      'hard_dependency': 1,
       'dependencies': [
           'build_stylesheet',
           'shaders',
@@ -151,6 +152,7 @@
       'target_name': 'llmr-ios',
       'product_name': 'llmr-ios',
       'type': 'static_library',
+      'hard_dependency': 1,
       'dependencies': [
           'build_stylesheet',
           'shaders',

scripts/deploy_results.sh

@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+
+set -e
+set -o pipefail
+
+aws s3 cp test/fixtures/styles/ s3://mapbox-gl-testing/headless/$TRAVIS_BUILD_NUMBER/$TRAVIS_JOB_NUMBER/ --acl public-read --recursive --exclude "*" --include "*.png"
+
+cd test/fixtures/styles
+for FILE in *.png ; do
+    echo http://mapbox-gl-testing.s3.amazonaws.com/headless/$TRAVIS_BUILD_NUMBER/$TRAVIS_JOB_NUMBER/$FILE
+done
+cd ../../..

scripts/flags.sh

@@ -0,0 +1,31 @@
+#!/usr/bin/env bash
+
+if [[ ${TRAVIS_OS_NAME} == "linux" ]]; then
+    export DISPLAY=:99.0
+    sh -e /etc/init.d/xvfb start
+
+    # use g++ that supports c++11
+    if [[ ${CXX} == "g++" ]]; then
+        export CXX="g++-4.8" ;
+    fi
+
+    # If building in debug then turn on sanitizers.
+    # They should both work with latest g++/clang++
+    # but it's not worth a standalone matrix so here
+    # we just test address->gcc and thread->clang for
+    # some limited coverage
+    if [[ ${BUILDTYPE} == "Debug" ]]; then
+        if [[ ${CXX} == "g++" ]]; then
+            export CXXFLAGS="-fsanitize=address ${CXXFLAGS}"
+            export CFLAGS="-fsanitize=address ${CFLAGS}"
+            export LDFLAGS="-fsanitize=address  ${LDFLAGS}"
+        elif [[ ${CXX} == "clang++" ]]; then
+            export CXXFLAGS="-fsanitize=thread -fPIC ${CXXFLAGS}"
+            export CFLAGS="-fsanitize=thread ${CFLAGS}"
+            export LDFLAGS="-fsanitize=thread -pie ${LDFLAGS}"
+        fi
+    fi
+fi
+
+# turn off a few warnings
+export CXXFLAGS="-Wno-unknown-warning-option -Wno-unused-local-typedefs -Wno-unknown-pragmas ${CXXFLAGS}";

scripts/run_tests.sh

@@ -3,8 +3,10 @@
 set -e
 set -o pipefail
 
-cd build/$BUILDTYPE
+BUILDTYPE="${BUILDTYPE:-Release}"
 
-for TEST in test_* ; do
-    ./$TEST
+cd build/${BUILDTYPE}
+
+for TEST in ./test_* ; do
+    ${TEST}
 done

scripts/setup_travis.sh

@@ -3,15 +3,6 @@
 set -e
 set -o pipefail
 
-# enable testing locally (not on travis)
-if [[ "${TRAVIS_OS_NAME:-false}" == false ]]; then
-    if [[ $(uname -s) == "Darwin" ]]; then
-        TRAVIS_OS_NAME="osx"
-    elif [[ $(uname -s) == "Linux" ]]; then
-        TRAVIS_OS_NAME="linux"
-    fi
-fi
-
 if [[ ${TRAVIS_OS_NAME} == "linux" ]]; then
 
     if [[ `lsb_release -r` =~ "12.04" ]]; then
@@ -19,36 +10,10 @@ if [[ ${TRAVIS_OS_NAME} == "linux" ]]; then
     fi
     sudo apt-get update -y
     sudo apt-get -y install gcc-4.8 g++-4.8
-    sudo apt-get -y install git build-essential zlib1g-dev automake libtool xutils-dev make cmake pkg-config
+    sudo apt-get -y install git build-essential zlib1g-dev automake libtool xutils-dev make cmake pkg-config python-pip
     sudo apt-get -y install libxi-dev libglu1-mesa-dev x11proto-randr-dev x11proto-xext-dev libxrandr-dev x11proto-xf86vidmode-dev libxxf86vm-dev libxcursor-dev
 
-    export DISPLAY=:99.0
-    sh -e /etc/init.d/xvfb start
-
-    # use g++ that supports c++11
-    if [[ ${CXX} == "g++" ]]; then
-        export CXX="g++-4.8" ;
-    fi
-
-    # If building in debug then turn on sanitizers.
-    # They should both work with latest g++/clang++
-    # but it's not worth a standalone matrix so here
-    # we just test address->gcc and thread->clang for
-    # some limited coverage
-    if [[ ${BUILDTYPE} == "Debug" ]]; then
-        if [[ ${CXX} == "g++" ]]; then
-            export CXXFLAGS="-fsanitize=address ${CXXFLAGS}"
-            export CFLAGS="-fsanitize=address ${CFLAGS}"
-            export LDFLAGS="-fsanitize=address  ${LDFLAGS}"
-        elif [[ ${CXX} == "clang++" ]]; then
-            export CXXFLAGS="-fsanitize=thread -fPIC ${CXXFLAGS}"
-            export CFLAGS="-fsanitize=thread ${CFLAGS}"
-            export LDFLAGS="-fsanitize=thread -pie ${LDFLAGS}"
-        fi
-    fi
+    sudo pip install awscli
 elif [[ ${TRAVIS_OS_NAME} == "osx" ]]; then
     brew install autoconf automake libtool makedepend cmake pkg-config node git
 fi
-
-# turn off a few warnings
-export CXXFLAGS="-Wno-unknown-warning-option -Wno-unused-local-typedefs -Wno-unknown-pragmas ${CXXFLAGS}";

setup-libraries.sh

@@ -96,6 +96,13 @@ cd ../../
 
 elif [ ${UNAME} = 'Linux' ]; then
 
+if [ ! -z "${TRAVIS:-}" ]; then
+    if aws s3 cp s3://mapbox-gl-testing/dependencies/build-cpp11-libstdcpp-gcc-x86_64-linux.tar.gz ./out/ ; then
+        rm -rf out/build-cpp11-libstdcpp-gcc-x86_64-linux
+        tar -xzf out/build-cpp11-libstdcpp-gcc-x86_64-linux.tar.gz
+    fi
+fi
+
 source Linux.sh
     if [ ! -f out/build-cpp11-libstdcpp-gcc-x86_64-linux/lib/libglfw3.a ] ; then ./scripts/build_glfw.sh ; fi
     if [ ! -f out/build-cpp11-libstdcpp-gcc-x86_64-linux/lib/libpng.a ] ; then ./scripts/build_png.sh ; fi
@@ -104,6 +111,13 @@ source Linux.sh
     if [ ! -f out/build-cpp11-libstdcpp-gcc-x86_64-linux/lib/libcurl.a ] ; then ./scripts/build_curl.sh ; fi
     if [ ! -f out/build-cpp11-libstdcpp-gcc-x86_64-linux/lib/libboost_regex.a ] ; then ./scripts/build_boost.sh --with-regex ; fi
 
+if [ ! -z "${TRAVIS:-}" ]; then
+    if ! tar --compare -zf out/build-cpp11-libstdcpp-gcc-x86_64-linux.tar.gz ; then
+        tar -zcf out/build-cpp11-libstdcpp-gcc-x86_64-linux.tar.gz out/build-cpp11-libstdcpp-gcc-x86_64-linux
+        aws s3 cp out/build-cpp11-libstdcpp-gcc-x86_64-linux.tar.gz s3://mapbox-gl-testing/dependencies/
+    fi
+fi
+
 cd ../../
 ./configure \
 --pkg-config-root=`pwd`/mapnik-packaging/osx/out/build-cpp11-libstdcpp-gcc-x86_64-linux/lib/pkgconfig \

src/map/map.cpp

@@ -491,15 +491,6 @@ void Map::render() {
 
     painter.drawClippingMasks(style->getActiveSources());
 
-#if defined(DEBUG)
-    // Generate debug information
-    size_t source_id = 0;
-    for (const std::shared_ptr<Source> &source : style->getActiveSources()) {
-        size_t count = source->getTileCount();
-        debug.emplace_back(util::sprintf<100>("source %d: %d\n", source_id, count));
-        source_id++;
-    }
-#endif
     // Actually render the layers
     if (debug::renderTree) { std::cout << "{" << std::endl; indent++; }
     renderLayers(style->layers);
@@ -515,10 +506,6 @@ void Map::render() {
 
     painter.renderMatte();
 
-#if defined(DEBUG)
-    painter.renderDebugText(debug);
-#endif
-
     // Schedule another rerender when we definitely need a next frame.
     if (transform.needsTransition() || style->hasTransitions()) {
         update();