refactor(eventsub): conditionally use Howard's date lib (#6048)

Co-authored-by: Nerixyz <nerixdev@outlook.de>

lib/twitch-eventsub-ws/include/twitch-eventsub-ws/chrono.hpp

@@ -3,7 +3,6 @@
 #include <boost/json.hpp>
 
 #include <chrono>
-#include <sstream>
 
 namespace chatterino::eventsub::lib {
 

lib/twitch-eventsub-ws/include/twitch-eventsub-ws/errors.hpp

@@ -15,6 +15,7 @@ enum class Kind : int {
     InnerRootMissing,
     NoMessageHandler,
     UnknownVariant,
+    BadTimeFormat,
 };
 
 class ApplicationErrorCategory final : public boost::system::error_category

lib/twitch-eventsub-ws/src/CMakeLists.txt

@@ -48,8 +48,12 @@ add_library(${PROJECT_NAME} STATIC ${SOURCE_FILES})
 # Generate source groups for use in IDEs
 # source_group(TREE ${CMAKE_SOURCE_DIR} FILES ${SOURCE_FILES})
 
-target_include_directories(${PROJECT_NAME} PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/../include")
-
+target_include_directories(${PROJECT_NAME}
+    PUBLIC
+        "${CMAKE_CURRENT_SOURCE_DIR}/../include"
+    PRIVATE
+        "${CMAKE_CURRENT_SOURCE_DIR}/../lib/date/include"
+)
 
 target_link_libraries(${PROJECT_NAME}
         PUBLIC

lib/twitch-eventsub-ws/src/chrono.cpp

@@ -1,16 +1,22 @@
 #include "twitch-eventsub-ws/chrono.hpp"
 
-#include "twitch-eventsub-ws/date.h"
+#include "twitch-eventsub-ws/detail/errors.hpp"
 
-#include <sstream>
+#if __cpp_lib_chrono >= 201907L
+#    include <chrono>
+using namespace std::chrono;
+#else
+#    include <date/date.h>
+using namespace date;
+#endif
 
 namespace chatterino::eventsub::lib {
 
 boost::json::result_for<std::chrono::system_clock::time_point,
                         boost::json::value>::type
-    tag_invoke(
-        boost::json::try_value_to_tag<std::chrono::system_clock::time_point>,
-        const boost::json::value &jvRoot, const AsISO8601 &)
+    tag_invoke(boost::json::try_value_to_tag<
+                   std::chrono::system_clock::time_point> /*tag*/,
+               const boost::json::value &jvRoot, const AsISO8601 & /*tag*/)
 {
     const auto raw = boost::json::try_value_to<std::string>(jvRoot);
     if (raw.has_error())
@@ -18,10 +24,18 @@ boost::json::result_for<std::chrono::system_clock::time_point,
         return raw.error();
     }
 
-    std::chrono::system_clock::time_point tp;
+    std::chrono::time_point<std::chrono::system_clock, std::chrono::nanoseconds>
+        tp;
     std::istringstream in{*raw};
-    in >> date::parse("%FT%H:%M:%12SZ", tp);
+    in >> parse("%FT%H:%M:%12SZ", tp);
 
-    return tp;
+    if (!in.good())
+    {
+        EVENTSUB_BAIL_HERE(error::Kind::BadTimeFormat);
+    }
+
+    return std::chrono::time_point_cast<std::chrono::system_clock::duration>(
+        tp);
 }
+
 }  // namespace chatterino::eventsub::lib

lib/twitch-eventsub-ws/src/errors.cpp

@@ -27,6 +27,8 @@ std::string ApplicationErrorCategory::message(int ev) const
             return "No message handler found"s;
         case Kind::UnknownVariant:
             return "Unknown variant value"s;
+        case Kind::BadTimeFormat:
+            return "Bad time format"s;
     }
 
     assert(false);

lib/twitch-eventsub-ws/tests/CMakeLists.txt

@@ -3,6 +3,7 @@ project(twitch-eventsub-ws-test)
 set(SOURCES
     src/parse.cpp
     src/string.cpp
+    src/chrono.cpp
 )
 
 add_executable(${PROJECT_NAME} ${SOURCES})

lib/twitch-eventsub-ws/tests/src/chrono.cpp

@@ -0,0 +1,54 @@
+#include "twitch-eventsub-ws/errors.hpp"
+
+#include <gtest/gtest.h>
+#include <twitch-eventsub-ws/chrono.hpp>
+
+using namespace chatterino::eventsub;
+
+namespace {
+
+using namespace std::chrono;
+using namespace std::chrono_literals;
+
+constexpr auto MAY14 =
+    std::chrono::sys_days{May / 14 / 2024} + 12h + 31min + 47s;
+constexpr auto MAY14_PRECISE = MAY14 + 123ms + 456us;
+
+boost::json::result_for<std::chrono::system_clock::time_point,
+                        boost::json::value>::type
+    buildTP(std::string_view sv)
+{
+    auto xd = boost::json::string(sv);
+
+    return boost::json::try_value_to<std::chrono::system_clock::time_point>(
+        xd, lib::AsISO8601());
+}
+
+}  // namespace
+
+TEST(Chrono, Success)
+{
+    {
+        auto oTp = buildTP("2024-05-14T12:31:47Z");
+        ASSERT_FALSE(oTp.has_error());
+        ASSERT_EQ(oTp.value(), MAY14);
+    }
+    {
+        std::chrono::system_clock::time_point tp;
+        auto oTp = buildTP("2024-05-14T12:31:47.123456789Z");
+        ASSERT_FALSE(oTp.has_error());
+        ASSERT_EQ(std::chrono::time_point_cast<std::chrono::microseconds>(
+                      oTp.value()),
+                  MAY14_PRECISE);
+    }
+}
+
+TEST(Chrono, Fail)
+{
+    {
+        auto oTp = buildTP("2024-05-14T12:31:47.123456789123456789Z");
+        ASSERT_TRUE(oTp.has_error());
+        ASSERT_EQ(static_cast<lib::error::Kind>(oTp.error().value()),
+                  lib::error::Kind::BadTimeFormat);
+    }
+}