fix: unit test now expects correct value for pins

Author: Jaskowicz1

include/dpp/cluster.h

@@ -2375,7 +2375,7 @@ class DPP_EXPORT cluster {
 	 * @see https://discord.com/developers/docs/resources/channel#get-pinned-messages
 	 * @param channel_id Channel ID to get pins for
 	 * @param callback Function to call when the API call completes.
-	 * On success the callback will contain a dpp::message_map object in confirmation_callback_t::value. On failure, the value is undefined and confirmation_callback_t::is_error() method will return true. You can obtain full error details with confirmation_callback_t::get_error().
+	 * On success the callback will contain a dpp::message_pin_map object in confirmation_callback_t::value. On failure, the value is undefined and confirmation_callback_t::is_error() method will return true. You can obtain full error details with confirmation_callback_t::get_error().
 	 */
 	void channel_pins_get(snowflake channel_id, command_completion_event_t callback);
 
@@ -2386,7 +2386,7 @@ class DPP_EXPORT cluster {
 	 * @param before Get messages pinned before this timestamp.
 	 * @param limit Max number of pins to return (1-50). Defaults to 50 if not set.
 	 * @param callback Function to call when the API call completes.
-	 * On success the callback will contain a dpp::message_map object in confirmation_callback_t::value. On failure, the value is undefined and confirmation_callback_t::is_error() method will return true. You can obtain full error details with confirmation_callback_t::get_error().
+	 * On success the callback will contain a dpp::message_pin_map object in confirmation_callback_t::value. On failure, the value is undefined and confirmation_callback_t::is_error() method will return true. You can obtain full error details with confirmation_callback_t::get_error().
 	 */
 	void channel_pins_get(snowflake channel_id, std::optional<time_t> before, std::optional<uint64_t> limit, command_completion_event_t callback);
 

src/dpp/cluster/message.cpp

@@ -211,8 +211,7 @@ void cluster::channel_pins_get(snowflake channel_id, command_completion_event_t
 }
 
 void cluster::channel_pins_get(snowflake channel_id, std::optional<time_t> before, std::optional<uint64_t> limit, command_completion_event_t callback) {
-	if (!limit.has_value())
-	{
+	if (!limit.has_value()) {
 		limit = GET_CHANNEL_PINS_MAX;
 	}
 
@@ -238,6 +237,7 @@ void cluster::channel_pins_get(snowflake channel_id, std::optional<time_t> befor
 				list[pinned_msg.pinned_message.id] = pinned_msg;
 			}
 		}
+
 		if (callback) {
 			callback(confirmation_callback_t(this, list, http));
 		}

src/unittest/test.cpp

@@ -2424,7 +2424,7 @@ Markdown lol \\|\\|spoiler\\|\\| \\~\\~strikethrough\\~\\~ \\`small \\*code\\* b
 			singleparam_api_test_list(channels_get, TEST_GUILD_ID, dpp::channel_map, GETCHANS);
 			singleparam_api_test_list(guild_get_invites, TEST_GUILD_ID, dpp::invite_map, GETINVS);
 			multiparam_api_test_list(guild_get_bans, TEST_GUILD_ID, dpp::ban_map, GETBANS);
-			singleparam_api_test_list(channel_pins_get, TEST_TEXT_CHANNEL_ID, dpp::message_map, GETPINS);
+			singleparam_api_test_list(channel_pins_get, TEST_TEXT_CHANNEL_ID, dpp::message_pin_map, GETPINS);
 			singleparam_api_test_list(guild_events_get, TEST_GUILD_ID, dpp::scheduled_event_map, GETEVENTS);
 			twoparam_api_test(guild_event_get, TEST_GUILD_ID, TEST_EVENT_ID, dpp::scheduled_event, GETEVENT);
 			twoparam_api_test_list(guild_event_users_get, TEST_GUILD_ID, TEST_EVENT_ID, dpp::event_member_map, GETEVENTUSERS);