Remove unused clearResult method, and use std::lock_guard

1 files changed, 1 insertions, 4 deletions

diff --git a/storage/src/tests/persistence/common/persistenceproviderwrapper.h b/storage/src/tests/persistence/common/persistenceproviderwrapper.h
index 059bb4dea00..21e5d8016aa 100644
--- a/storage/src/tests/persistence/common/persistenceproviderwrapper.h
+++ b/storage/src/tests/persistence/common/persistenceproviderwrapper.h
@@ -54,7 +54,7 @@ private:
     mutable std::mutex  _lock;
     mutable std::vector<std::string> _log;
     uint32_t _failureMask;
-    using Guard = std::unique_lock<std::mutex>;
+    using Guard = std::lock_guard<std::mutex>;
 public:
     PersistenceProviderWrapper(spi::PersistenceProvider& spi);
     ~PersistenceProviderWrapper() override;
@@ -67,9 +67,6 @@ public:
         Guard guard(_lock);
         _result = result;
     }
-    void clearResult() {
-        _result = spi::Result(spi::Result::ErrorType::NONE, "");
-    }
     spi::Result getResult() const {
         Guard guard(_lock);
         return _result;