Implements comparison operator between HTTP requests + test showcasing how to build a http request in different phases

Overall I am happy with the design, only drawback is that when we do `.execute()` we still have to re-build the same request despite being 're-used'. Will look into this eventually

Author: ggcr

src/s3cpp/httpclient.cpp

@@ -7,6 +7,7 @@
 
 HttpResponse HttpRequest::execute() { return client_.execute(*this); }
 
+
 HttpResponse HttpClient::execute(HttpRequest &request) {
   if (!curl_handle) {
     throw std::runtime_error(

src/s3cpp/httpclient.h

@@ -22,6 +22,10 @@ class HttpResponse {
   bool is_client_error() const { return code_ >= 400 && code_ < 500; }
   bool is_server_error() const { return code_ >= 500 && code_ < 600; }
 
+  bool operator==(const HttpResponse &other) const {
+    return (code_ == other.code_ && body_ == other.body_);
+  }
+
 private:
   int code_;
   std::string body_;
@@ -45,7 +49,7 @@ class HttpRequest {
 
   HttpResponse execute();
 
-  const std::string& getURL() const { return URL_; }
+  const std::string &getURL() const { return URL_; }
   const int getTimeout() const { return timeout_; }
 
 private:

test/httpclient_test.cpp

@@ -66,4 +66,24 @@ TEST(HTTP, HTTPHandleTimeout) {
   }
 }
 
+TEST(HTTP, HTTPRequestInDifferentPhases) {
+  HttpClient client{};
+  HttpRequest req = client.get("https://postman-echo.com/get").timeout(1);
+	auto responses = std::vector<HttpResponse>{};
+  try {
+    HttpResponse resp1 = client.execute(req);
+		responses.push_back(resp1);
+  } catch (const std::exception &e) {
+    SUCCEED(); // libcurl error for request: Timeout was reached
+  }
+  try {
+		HttpResponse resp2 = client.execute(req);
+		responses.push_back(resp2);
+  } catch (const std::exception &e) {
+    SUCCEED(); // libcurl error for request: Timeout was reached
+  }
+  ASSERT_EQ(responses.size(), 2);
+  ASSERT_EQ(responses[0], responses[1]);
+}
+
 // TODO(cristian): Headers extensive tests