From 35d86dd7982291f096686a0912da30e3b3bfd444 Mon Sep 17 00:00:00 2001
From: Brad Davis <bdavis@saintandreas.org>
Date: Thu, 23 Aug 2018 12:19:50 -0700
Subject: [PATCH] Fail unit test on non-compiling pipelines

---
 tests/shaders/src/ShaderTests.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/shaders/src/ShaderTests.cpp b/tests/shaders/src/ShaderTests.cpp
index 4dd15710f9..03dc034cd0 100644
--- a/tests/shaders/src/ShaderTests.cpp
+++ b/tests/shaders/src/ShaderTests.cpp
@@ -230,8 +230,8 @@ void ShaderTests::testShaderLoad() {
             auto glBackend = std::static_pointer_cast<gpu::gl::GLBackend>(_gpuContext->getBackend());
             auto glshader = gpu::gl::GLShader::sync(*glBackend, *program);
             if (!glshader) {
-                qDebug() << "Failed to compile or link vertex " << vertexId << " fragment " << fragmentId;
-                continue;
+                qWarning() << "Failed to compile or link vertex " << vertexId << " fragment " << fragmentId;
+                QFAIL("Program link error");
             }
 
             QVERIFY(glshader != nullptr);
@@ -301,7 +301,7 @@ void ShaderTests::testShaderLoad() {
         QFAIL(error.what());
     }
 
-    for (uint32_t i = 0; i <= maxShader; ++i) {
+    for (uint32_t i = 1; i <= maxShader; ++i) {
         auto used = usedShaders.count(i);
         if (0 != usedShaders.count(i)) {
             continue;