fix(reply_builder): split SendError(ErrorReply) into SendError(status) and SendError(str, type) methods

fixes #3190

Signed-off-by: Stepan Bagritsevich <[email protected]>

Author: BagritsevichStepan

src/facade/reply_builder.h

@@ -79,8 +79,8 @@ class SinkReplyBuilder {
   }
 
   virtual void SendError(std::string_view str, std::string_view type = {}) = 0;  // MC and Redis
-  virtual void SendError(ErrorReply error);
   virtual void SendError(OpStatus status);
+  void SendError(ErrorReply error);
 
   virtual void SendStored() = 0;  // Reply for set commands.
   virtual void SendSetSkipped() = 0;

src/facade/reply_capture.cc

@@ -21,11 +21,6 @@ void CapturingReplyBuilder::SendError(std::string_view str, std::string_view typ
   Capture(Error{str, type});
 }
 
-void CapturingReplyBuilder::SendError(ErrorReply error) {
-  SKIP_LESS(ReplyMode::ONLY_ERR);
-  Capture(Error{error.ToSv(), error.kind});
-}
-
 void CapturingReplyBuilder::SendMGetResponse(MGetResponse resp) {
   SKIP_LESS(ReplyMode::FULL);
   Capture(std::move(resp));

src/facade/reply_capture.h

@@ -24,12 +24,12 @@ class CapturingReplyBuilder : public RedisReplyBuilder {
 
  public:
   void SendError(std::string_view str, std::string_view type = {}) override;
-  void SendError(ErrorReply error) override;
   void SendMGetResponse(MGetResponse resp) override;
 
   // SendStored -> SendSimpleString("OK")
   // SendSetSkipped -> SendNull()
   void SendError(OpStatus status) override;
+  using RedisReplyBuilder::SendError;
 
   void SendNullArray() override;
   void SendEmptyArray() override;