Added the new method Rset::isEmpty() - I am guessing that the migration work...

Added the new method Rset::isEmpty() - I am guessing that the migration work of Michael would have loved to have had this method

rdbms/Rset.cpp

@@ -176,6 +176,14 @@ bool Rset::next() {
   return aRowHasBeenRetrieved;
 }
 
+//------------------------------------------------------------------------------
+// isEmpty
+//------------------------------------------------------------------------------
+bool Rset::isEmpty() const
+{
+  return nullptr == m_impl;
+}
+
 //------------------------------------------------------------------------------
 // columnIsNull
 //------------------------------------------------------------------------------

rdbms/Rset.hpp

@@ -91,6 +91,12 @@ public:
    */
   bool next();
 
+  /**
+   * Returns true if the result set does not contain any more rows.
+   * @return True if the result set does not contain any more rows.
+   */
+  bool isEmpty() const;
+
   /**
    * Returns true if the specified column contains a null value.
    *

rdbms/RsetTest.cpp

@@ -36,6 +36,14 @@ protected:
   }
 };
 
+TEST_F(cta_rdbms_RsetTest, constructor) {
+  using namespace cta::rdbms;
+
+  Rset rset;
+
+  ASSERT_TRUE(rset.isEmpty());
+}
+
 TEST_F(cta_rdbms_RsetTest, next) {
   using namespace cta::rdbms;
 
@@ -60,12 +68,15 @@ TEST_F(cta_rdbms_RsetTest, next) {
     Stmt stmt = pool.getStmt(*conn, sql);
     auto rset = stmt.executeQuery();
 
+    ASSERT_FALSE(rset.isEmpty());
     ASSERT_TRUE(rset.next());
     ASSERT_EQ(1, rset.columnUint64("ID"));
 
     ASSERT_FALSE(rset.next());
 
     ASSERT_THROW(rset.next(), InvalidResultSet);
+
+    ASSERT_TRUE(rset.isEmpty());
   }
 }