commit 59b42286edafdb9a509ad5aac84377dce88eb05b
author Tony Barbour <tony@LunarG.com> Thu Nov 03 13:31:28 2016 -0600
committer Tony Barbour <tony@LunarG.com> Thu Nov 03 14:02:59 2016 -0600
tree 5146c12692c187dc9d612747cd896e0dac0246e6
parent 97dc68d2e1d565ae9f7f5c69de551b4ad73e568a

tests: Check for expected ids as well as messages

Also don't print other error message if there weren't other errors
Change-Id: Ia303370d1ce7ae329f6305ad3e66266c8c64ff40

tests/layer_validation_tests.cpp

diff --git a/tests/layer_validation_tests.cpp b/tests/layer_validation_tests.cpp
index ae357f1..55203fb 100644
--- a/tests/layer_validation_tests.cpp
+++ b/tests/layer_validation_tests.cpp
@@ -222,9 +222,11 @@
 
     void DumpFailureMsgs(void) {
         vector<string> otherMsgs = GetOtherFailureMsgs();
-        cout << "Other error messages logged for this test were:" << endl;
-        for (auto iter = otherMsgs.begin(); iter != otherMsgs.end(); iter++) {
-            cout << "     " << *iter << endl;
+        if (otherMsgs.size()) {
+            cout << "Other error messages logged for this test were:" << endl;
+            for (auto iter = otherMsgs.begin(); iter != otherMsgs.end(); iter++) {
+                cout << "     " << *iter << endl;
+            }
         }
     }
 
@@ -244,6 +246,9 @@
             for (auto desired_msg : m_desired_message_strings) {
                 FAIL() << "Did not receive expected error '" << desired_msg << "'";
             }
+            for (auto desired_id : m_desired_message_ids) {
+                FAIL() << "Did not receive expected error '" << desired_id << "'";
+            }
         }
     }