controller: db: fix function return value on success to true

As part of testing the full flows of the persistent Db we encountered
an issue where a successful removal of a client entry from the
persistent DB resulted in a false-positive error.
The cause of this error is that remove_client_entry_and_update_counter()
in the db class returned false both for success and for failure.

Changed the successful return value of the above-mentioned function
from false to true.

Signed-off-by: Adam Dov <[email protected]>

controller/src/beerocks/master/db/db.cpp

@@ -4415,7 +4415,7 @@ bool db::remove_client_entry_and_update_counter(const std::string &entry_name)
     }
     --m_persistent_db_clients_count;
 
-    return false;
+    return true;
 }
 
 bool db::remove_candidate_client()