Update CI configuration and enhance websocket test callbacks with override specifier

Author: kzhdev

.github/workflows/ci.yml

@@ -86,4 +86,4 @@ jobs:
         run: cmake --build build --config ${{ matrix.build_type }} -j 4
 
       - name: Test
-        run: ctest --test-dir build -C ${{ matrix.build_type }} -E "(Order|Fill)" --output-on-failure
+        run: ctest --test-dir build -C ${{ matrix.build_type }} -E "(Order|Fill|Portfolio)" --output-on-failure

tests/rest_awaitable_tests.cpp

@@ -54,7 +54,7 @@ class CoinbaseAwaitableTest : public ::testing::Test {
 
     // Helper to run an async coroutine and wait for result
     template<typename Func>
-    auto run_async(Func&& coro) {
+    typename std::invoke_result_t<Func>::value_type run_async(Func&& coro) {
         using awaitable_type = std::invoke_result_t<Func>;
         using result_type = typename awaitable_type::value_type;
 

tests/websocket_tests.cpp

@@ -45,21 +45,21 @@ namespace coinbase::tests {
             std::remove("coinbase.log");
         }
 
-        void onMarketDataConnected(WebSocketClient *) {
+        void onMarketDataConnected(WebSocketClient *) override {
             ++md_connected_count_;
             LOG_INFO("MarketData Connected");
         }
 
-        void onMarketDataDisconnected(WebSocketClient *) {
+        void onMarketDataDisconnected(WebSocketClient *) override {
             ++md_disconnected_count_;
             LOG_INFO("MarketData Disconnected");
         }
 
-        void onUserDataConnected(WebSocketClient *) {
+        void onUserDataConnected(WebSocketClient *) override {
             LOG_INFO("UserData Connected");
         }
 
-        void onUserDataDisconnected(WebSocketClient *) {
+        void onUserDataDisconnected(WebSocketClient *) override {
             LOG_INFO("UserData Disconnected");
         }