[USER32_APITEST] Add more PrivateExtractIcons tests (reactos#7831)

Author: whindsaks

modules/rostests/apitests/user32/PrivateExtractIcons.c

@@ -8,6 +8,20 @@
 
 #include "precomp.h"
 #include <stdio.h>
+#include <versionhelpers.h>
+
+BOOL IsValidIcon(HICON hIco)
+{
+    ICONINFO info = { 0 };
+
+    if (!hIco || !GetIconInfo(hIco, &info))
+        return FALSE;
+
+    DeleteObject(info.hbmMask);
+    if (info.hbmColor)
+        DeleteObject(info.hbmColor);
+    return TRUE;
+}
 
 BOOL FileExists(PCWSTR FileName)
 {
@@ -110,11 +124,14 @@ static struct
     {L"%temp%\\cpimg2e.exe", IDR_EXE_NORMAL}
 };
 
+void TestPairExtraction(void);
+
 START_TEST(PrivateExtractIcons)
 {
     HICON ahIcon;
     UINT i, aIconId, cIcons, cIcoTotal;
     WCHAR PathBuffer[MAX_PATH];
+    UINT Shell32WinIcoCount = IsWindowsVistaOrGreater() ? 326 : 239; /* 239 on W2K3SP2, 326 on Win10 */
 
     /* Extract icons */
     for (i = 0; i < _countof(IconFiles); ++i)
@@ -125,14 +142,16 @@ START_TEST(PrivateExtractIcons)
             goto Cleanup;
     }
 
+    TestPairExtraction();
+
     for (i = 0; i < _countof(IconTests); ++i)
     {
         /* Get total number of icon groups in file.
          * None of the hard numbers in the function matter since we have
          * two NULLs for the Icon Handle and Count to be set. */
         cIcoTotal = PrivateExtractIconsW(IconTests[i].FilePath, 0, 16, 16, NULL, NULL, 0, 0);
         ok((i == 3 ?
-              cIcoTotal > 232 && cIcoTotal < 240 :    /* shell32 case: ROS has 233, W2K2SP2 has 239 icon groups. */
+              cIcoTotal > 232 && cIcoTotal <= Shell32WinIcoCount :  /* shell32 case: ROS has 233, Windows has >= 239 icon groups. */
               cIcoTotal == IconTests[i].cTotalIcons),
            "PrivateExtractIconsW(%u): "
            "got %u, expected %u\n", i, cIcoTotal, IconTests[i].cTotalIcons);
@@ -167,3 +186,67 @@ START_TEST(PrivateExtractIcons)
         DeleteFileW(PathBuffer);
     }
 }
+
+static const struct tagPAIRSTESTS
+{
+    BYTE InCount;
+    BYTE UseHigh; // Extract a pair (high and low word sizes)
+    BYTE Library;
+    BYTE ReturnNT5;
+    BYTE CountNT5;
+    BYTE ReturnNT6;
+    BYTE CountNT6;
+} g_pairs[] =
+{
+    { 0, 0, 0,     1, 1, 1, 1 },
+    { 0, 0, 1,     0, 0, 0, 0 },
+    { 0, 1, 0,     1, 2, 1, 2 },
+    { 0, 1, 1,     0, 0, 0, 0 },
+    { 1, 0, 0,     1, 1, 1, 1 },
+    { 1, 0, 1,     1, 1, 1, 1 },
+    { 1, 1, 0,     1, 2, 1, 2 },
+    { 1, 1, 1,     2, 2, 0, 0 },
+    { 2, 0, 0,     1, 1, 1, 1 },
+    { 2, 0, 1,     2, 2, 2, 2 },
+    { 2, 1, 0,     1, 2, 1, 2 },
+    { 2, 1, 1,     2, 2, 2, 2 }, // This is the only way to extract a pair from a PE on NT6!
+    { 3, 0, 0,     1, 1, 1, 1 },
+    { 3, 0, 1,     3, 3, 3, 3 },
+    { 3, 1, 0,     1, 2, 1, 2 },
+    { 3, 1, 1,     4, 4, 0, 0 },
+    { 4, 0, 0,     1, 1, 1, 1 },
+    { 4, 0, 1,     4, 4, 4, 4 },
+    { 4, 1, 0,     1, 2, 1, 2 },
+    { 4, 1, 1,     4, 4, 4, 4 }
+};
+
+void TestPairExtraction(void)
+{
+    const HICON hInvalidIcon = (HICON)(INT_PTR)-2;
+    const BOOL IsNT6 = IsWindowsVistaOrGreater();
+    for (UINT i = 0; i < _countof(g_pairs); ++i)
+    {
+        UINT j, Count, ExpectedCount;
+        int RetVal, ExpectedRet;
+        UINT IcoSize = MAKELONG(32, g_pairs[i].UseHigh ? 16 : 0);
+        PCWSTR pszPath = g_pairs[i].Library ? L"%SystemRoot%\\system32\\shell32.dll" : L"%temp%\\sysicon.ico";
+        HICON hIcons[8];
+        for (j = 0; j < _countof(hIcons); ++j)
+            hIcons[j] = hInvalidIcon;
+
+        RetVal = PrivateExtractIconsW(pszPath, 0, IcoSize, IcoSize, hIcons, NULL, g_pairs[i].InCount, 0);
+        for (j = 0, Count = 0; j < _countof(hIcons); ++j)
+        {
+            if (hIcons[j] != hInvalidIcon && IsValidIcon(hIcons[j]))
+            {
+                DestroyIcon(hIcons[j]);
+                ++Count;
+            }
+        }
+
+        ExpectedRet = !IsNT6 ? g_pairs[i].ReturnNT5 : g_pairs[i].ReturnNT6;
+        ExpectedCount = !IsNT6 ? g_pairs[i].CountNT5 : g_pairs[i].CountNT6;
+        ok(RetVal == ExpectedRet, "Test %u: RetVal must be %d but got %d\n", i, ExpectedRet, RetVal);
+        ok(Count == ExpectedCount, "Test %u: Count must be %u but got %u\n", i, ExpectedCount, Count);
+    }
+}