feat clients: add http-method convert from string

add clients::http::HttpMethod convert
commit_hash:7e42f12587c8a788d143a1141e1f84f81026c95c

Author: fdr400

.mapping.json

@@ -1097,6 +1097,7 @@
   "core/src/clients/http/error.cpp":"taxi/uservices/userver/core/src/clients/http/error.cpp",
   "core/src/clients/http/form.cpp":"taxi/uservices/userver/core/src/clients/http/form.cpp",
   "core/src/clients/http/form_test.cpp":"taxi/uservices/userver/core/src/clients/http/form_test.cpp",
+  "core/src/clients/http/http_method_test.cpp":"taxi/uservices/userver/core/src/clients/http/http_method_test.cpp",
   "core/src/clients/http/plugin.cpp":"taxi/uservices/userver/core/src/clients/http/plugin.cpp",
   "core/src/clients/http/plugins/headers_propagator/component.cpp":"taxi/uservices/userver/core/src/clients/http/plugins/headers_propagator/component.cpp",
   "core/src/clients/http/plugins/headers_propagator/plugin.cpp":"taxi/uservices/userver/core/src/clients/http/plugins/headers_propagator/plugin.cpp",

core/include/userver/clients/http/request.hpp

@@ -40,11 +40,15 @@ namespace impl {
 class EasyWrapper;
 }  // namespace impl
 
-/// HTTP request method
+/// @brief HTTP request method
 enum class HttpMethod { kGet, kPost, kHead, kPut, kDelete, kPatch, kOptions };
 
+/// @brief Convert HTTP method enum value to string
 std::string_view ToStringView(HttpMethod method);
 
+/// @brief Convert HTTP method string to enum value
+HttpMethod HttpMethodFromString(std::string_view method_str);
+
 using USERVER_NAMESPACE::http::HttpVersion;
 
 enum class HttpAuthType {

core/src/clients/http/http_method_test.cpp

@@ -0,0 +1,32 @@
+#include <userver/utest/using_namespace_userver.hpp>
+
+#include <userver/clients/http/request.hpp>
+#include <userver/utest/utest.hpp>
+
+USERVER_NAMESPACE_BEGIN
+
+namespace {
+namespace ch = clients::http;
+}
+
+UTEST(ClientHttpMethodTest, Convert) {
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kGet), "GET");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kHead), "HEAD");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kPost), "POST");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kPatch), "PATCH");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kPut), "PUT");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kDelete), "DELETE");
+    EXPECT_EQ(ch::ToStringView(ch::HttpMethod::kOptions), "OPTIONS");
+
+    EXPECT_EQ(ch::HttpMethodFromString("GET"), ch::HttpMethod::kGet);
+    EXPECT_EQ(ch::HttpMethodFromString("HEAD"), ch::HttpMethod::kHead);
+    EXPECT_EQ(ch::HttpMethodFromString("POST"), ch::HttpMethod::kPost);
+    EXPECT_EQ(ch::HttpMethodFromString("PATCH"), ch::HttpMethod::kPatch);
+    EXPECT_EQ(ch::HttpMethodFromString("PUT"), ch::HttpMethod::kPut);
+    EXPECT_EQ(ch::HttpMethodFromString("DELETE"), ch::HttpMethod::kDelete);
+    EXPECT_EQ(ch::HttpMethodFromString("OPTIONS"), ch::HttpMethod::kOptions);
+
+    UEXPECT_THROW(ch::HttpMethodFromString("123"), std::runtime_error);
+}
+
+USERVER_NAMESPACE_END

core/src/clients/http/request.cpp

@@ -1,8 +1,6 @@
 #include <userver/clients/http/request.hpp>
 
-#include <chrono>
 #include <cstdlib>
-#include <map>
 #include <string>
 #include <string_view>
 #include <system_error>
@@ -38,6 +36,17 @@ namespace {
 
 constexpr std::string_view kHeaderExpect = "Expect";
 
+static constexpr utils::TrivialBiMap kHttpMethodMap([](auto selector) {
+    return selector()
+        .Case(HttpMethod::kGet, "GET")
+        .Case(HttpMethod::kHead, "HEAD")
+        .Case(HttpMethod::kPost, "POST")
+        .Case(HttpMethod::kPut, "PUT")
+        .Case(HttpMethod::kPatch, "PATCH")
+        .Case(HttpMethod::kDelete, "DELETE")
+        .Case(HttpMethod::kOptions, "OPTIONS");
+});
+
 std::string ToString(HttpMethod method) { return std::string{ToStringView(method)}; }
 
 curl::easy::http_version_t ToNative(HttpVersion version) {
@@ -171,19 +180,16 @@ bool IsAllowedSchemaInUrl(std::string_view url) {
 
 }  // namespace
 
-std::string_view ToStringView(HttpMethod method) {
-    static constexpr utils::TrivialBiMap kMap([](auto selector) {
-        return selector()
-            .Case(HttpMethod::kDelete, "DELETE")
-            .Case(HttpMethod::kGet, "GET")
-            .Case(HttpMethod::kHead, "HEAD")
-            .Case(HttpMethod::kPost, "POST")
-            .Case(HttpMethod::kPut, "PUT")
-            .Case(HttpMethod::kPatch, "PATCH")
-            .Case(HttpMethod::kOptions, "OPTIONS");
-    });
-
-    return utils::impl::EnumToStringView(method, kMap);
+std::string_view ToStringView(HttpMethod method) { return utils::impl::EnumToStringView(method, kHttpMethodMap); }
+
+HttpMethod HttpMethodFromString(std::string_view method_str) {
+    if (auto method = kHttpMethodMap.TryFindBySecond(method_str)) {
+        return *method;
+    }
+
+    throw std::runtime_error(
+        fmt::format("Unsupported http method '{}' (must be one of {})", method_str, kHttpMethodMap.DescribeSecond())
+    );
 }
 
 ProxyAuthType ProxyAuthTypeFromString(const std::string& auth_name) {