[PATCH v4 6/6] bluetoothapis/tests: Add test for BluetoothSdpGetAttributeValue.

[Vibhav Pant]

From: Vibhav Pant

---
 dlls/bluetoothapis/tests/sdp.c | 34 +++++++++++++++++++++-------------
 1 file changed, 21 insertions(+), 13 deletions(-)

diff --git a/dlls/bluetoothapis/tests/sdp.c b/dlls/bluetoothapis/tests/sdp.c
index daefedd0799..a772a181687 100644
--- a/dlls/bluetoothapis/tests/sdp.c
+++ b/dlls/bluetoothapis/tests/sdp.c
@@ -514,13 +514,20 @@ struct attr_callback_data
     SIZE_T i;
 };

+static BYTE sdp_record_bytes[] = {
+    0x35, 0x48, 0x09, 0x00, 0x00, 0x0a, 0x00, 0x01, 0x00, 0x00, 0x09, 0x00, 0x01, 0x35, 0x03,
+    0x19, 0x12, 0x00, 0x09, 0x00, 0x05, 0x35, 0x03, 0x19, 0x10, 0x02, 0x09, 0x00, 0x09, 0x35,
+    0x08, 0x35, 0x06, 0x19, 0x12, 0x00, 0x09, 0x01, 0x03, 0x09, 0x02, 0x00, 0x09, 0x01, 0x03,
+    0x09, 0x02, 0x01, 0x09, 0x1d, 0x6b, 0x09, 0x02, 0x02, 0x09, 0x02, 0x46, 0x09, 0x02, 0x03,
+    0x09, 0x05, 0x4d, 0x09, 0x02, 0x04, 0x28, 0x01, 0x09, 0x02, 0x05, 0x09, 0x00, 0x02 };
+
 static BOOL WINAPI enum_attr_callback( ULONG attr_id, BYTE *stream, ULONG stream_size, void *param )
 {
     struct attr_callback_data *params = param;
     winetest_push_context( TEST_CASE_NAME( params->i + 1) );
     if (params->i < params->attrs_n)
-    {
-        SDP_ELEMENT_DATA data = {0};
+        {
+        SDP_ELEMENT_DATA data = {0}, data2 = {0};
         DWORD result;

ok( attr_id == params->attrs_id[params->i], "Expected attribute id %lu, got %lu.\n",
@@ -530,6 +537,14 @@ static BOOL WINAPI enum_attr_callback( ULONG attr_id, BYTE *stream, ULONG stream
         ok( !memcmp( &params->attrs[params->i], &data, sizeof( data ) ), "Expected %s, got %s.\n",
             debugstr_SDP_ELEMENT_DATA( &params->attrs[params->i] ),
             debugstr_SDP_ELEMENT_DATA( &data ) );
+
+        result = BluetoothSdpGetAttributeValue( sdp_record_bytes, ARRAY_SIZE( sdp_record_bytes ),
+                                               params->attrs_id[params->i], &data2 );
+        ok( result == ERROR_SUCCESS, "BluetoothSdpGetAttributeValue failed: %ld.\n", result );
+        ok( !memcmp( &params->attrs[params->i], &data2, sizeof( data2 ) ), "Expected %s, got %s.\n",
+            debugstr_SDP_ELEMENT_DATA( &params->attrs[params->i] ),
+            debugstr_SDP_ELEMENT_DATA( &data2 ) );
+
         params->i++;
     }
     winetest_pop_context();
@@ -538,18 +553,11 @@ static BOOL WINAPI enum_attr_callback( ULONG attr_id, BYTE *stream, ULONG stream

static void test_BluetoothSdpEnumAttributes( void )
 {
-    static BYTE record_bytes[] = {
-        0x35, 0x48, 0x09, 0x00, 0x00, 0x0a, 0x00, 0x01, 0x00, 0x00, 0x09, 0x00, 0x01, 0x35, 0x03,
-        0x19, 0x12, 0x00, 0x09, 0x00, 0x05, 0x35, 0x03, 0x19, 0x10, 0x02, 0x09, 0x00, 0x09, 0x35,
-        0x08, 0x35, 0x06, 0x19, 0x12, 0x00, 0x09, 0x01, 0x03, 0x09, 0x02, 0x00, 0x09, 0x01, 0x03,
-        0x09, 0x02, 0x01, 0x09, 0x1d, 0x6b, 0x09, 0x02, 0x02, 0x09, 0x02, 0x46, 0x09, 0x02, 0x03,
-        0x09, 0x05, 0x4d, 0x09, 0x02, 0x04, 0x28, 0x01, 0x09, 0x02, 0x05, 0x09, 0x00, 0x02 };
-
     static SDP_ELEMENT_DATA attributes[] = {
         {SDP_TYPE_UINT, SDP_ST_UINT32, {.uint32 = 0x10000}},
-        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&record_bytes[13], 5}}},
-        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&record_bytes[21], 5}}},
-        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&record_bytes[29], 10}}},
+        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&sdp_record_bytes[13], 5}}},
+        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&sdp_record_bytes[21], 5}}},
+        {SDP_TYPE_SEQUENCE, SDP_ST_NONE, {.sequence = {&sdp_record_bytes[29], 10}}},
         {SDP_TYPE_UINT, SDP_ST_UINT16, {.uint16 = 0x0103}},
         {SDP_TYPE_UINT, SDP_ST_UINT16, {.uint16 = 0x1d6b}},
         {SDP_TYPE_UINT, SDP_ST_UINT16, {.uint16 = 0x0246}},
@@ -563,7 +571,7 @@ static void test_BluetoothSdpEnumAttributes( void )
     BOOL ret;

SetLastError( 0xdeadbeef );
-    ret = BluetoothSdpEnumAttributes( record_bytes, ARRAY_SIZE( record_bytes ), enum_attr_callback,
+    ret = BluetoothSdpEnumAttributes( sdp_record_bytes, ARRAY_SIZE( sdp_record_bytes ), enum_attr_callback,
                                       &data );
     ok( ret, "BluetoothSdpEnumAttributes failed with %ld.\n", GetLastError() );
     ok( data.i == data.attrs_n, "%lu != %lu\n", data.i, data.attrs_n );