Add tracking and HttpResponse overload to FakeHttpClient

Adds RequestMade(string_view) to inspect requested URLs and
SetResponse(HttpResponse<string>) overload.

Assisted-by: Jetski:Gemini Next

Author: Databean

base/cvd/cuttlefish/host/libs/web/http_client/fake_http_client.cc

@@ -31,10 +31,27 @@ void FakeHttpClient::SetResponse(std::string data, std::string url) {
       .http_code = 200,
       .headers = {},
   };
-  auto handler = [res = std::move(res)](const HttpRequest&) { return res; };
+  SetResponse(std::move(res), std::move(url));
+}
+
+void FakeHttpClient::SetResponse(HttpResponse<std::string> response,
+                                 std::string url) {
+  auto handler = [res = std::move(response)](const HttpRequest&) {
+    return res;
+  };
   SetResponse(std::move(handler), std::move(url));
 }
 
+bool FakeHttpClient::RequestMade(std::string_view url) const {
+  std::lock_guard lock(mutex_);
+  for (const auto& requested : requested_urls_) {
+    if (requested.find(url) != std::string_view::npos) {
+      return true;
+    }
+  }
+  return false;
+}
+
 void FakeHttpClient::SetResponse(FakeHttpClient::Handler handler,
                                  std::string url) {
   std::lock_guard lock(mutex_);
@@ -60,6 +77,7 @@ const FakeHttpClient::Handler* FakeHttpClient::FindHandler(
 Result<HttpResponse<void>> FakeHttpClient::DownloadToCallback(
     HttpRequest request, HttpClient::DataCallback callback) {
   std::lock_guard lock(mutex_);
+  requested_urls_.push_back(request.url);
   CF_EXPECT(callback(nullptr, 0));
   const Handler* handler = FindHandler(request.url);
   HttpResponse<void> response;

base/cvd/cuttlefish/host/libs/web/http_client/fake_http_client.h

@@ -20,6 +20,7 @@
 #include <string>
 #include <string_view>
 #include <unordered_map>
+#include <vector>
 
 #include "cuttlefish/host/libs/web/http_client/http_client.h"
 #include "cuttlefish/result/result.h"
@@ -33,16 +34,19 @@ class FakeHttpClient : public HttpClient {
   // The longest string that is a complete substring of `url` is used to match
   // requests.
   void SetResponse(std::string data, std::string url = "");
+  void SetResponse(HttpResponse<std::string> response, std::string url = "");
   void SetResponse(Handler handler, std::string url = "");
+  bool RequestMade(std::string_view url) const;
   // Returns response's status code.
   Result<HttpResponse<void>> DownloadToCallback(
       HttpRequest request, HttpClient::DataCallback callback) override;
 
  private:
   const Handler* FindHandler(std::string_view url) const;
 
-  std::mutex mutex_;
+  mutable std::mutex mutex_;
   std::unordered_map<std::string, Handler> responses_;
+  std::vector<std::string> requested_urls_;
 };
 
 }  // namespace cuttlefish