From: Brad Hubbard <bhubbard@redhat.com>
Date: Fri, 9 Mar 2018 04:35:36 +0000 (+1000)
Subject: tests: Check for empty output in test_dump_pgstate_history
X-Git-Tag: v13.1.0~582^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=2c642e73c6f366df103e664db281f28abe61abb3;p=ceph.git

tests: Check for empty output in test_dump_pgstate_history

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
---

diff --git a/src/test/admin_socket_output_tests.cc b/src/test/admin_socket_output_tests.cc
index 72a573b780927..58bc036540805 100644
--- a/src/test/admin_socket_output_tests.cc
+++ b/src/test/admin_socket_output_tests.cc
@@ -36,6 +36,11 @@ bool test_dump_pgstate_history(std::string &output) {
   }
 
   JSONObjIter iter = parser.find_first();
+  if (iter.end()) { //Empty
+    std::cerr << "test_dump_pgstate_history: command output empty, failing"
+              << std::endl;
+    return false;
+  }
   for (; !iter.end(); ++iter) {
     if ((*iter)->get_name() == "pg") {
       ret = !(*iter)->get_data().empty();