[PATCH 0/1] MR7458: kernelbase/tests: Skip test if EnumSystemFirmwareTables is not available.

[Bernhard Übelacker]

Tests show test kernelbase:memory is failing for many Windows 7 machines. https://test.winehq.org/data/patterns-tb-win.html#kernelbase:memory

This patch adds skipping the test if function `EnumSystemFirmwareTables` is not found.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7458

[PATCH 1/1] kernelbase/tests: Skip test if EnumSystemFirmwareTables is not available.

[Bernhard Übelacker]

From: Bernhard Übelacker <bernhardu(a)mailbox.org>

---
 dlls/kernelbase/tests/memory.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dlls/kernelbase/tests/memory.c b/dlls/kernelbase/tests/memory.c
index b7841d57fee..b38cea2c258 100644
--- a/dlls/kernelbase/tests/memory.c
+++ b/dlls/kernelbase/tests/memory.c
@@ -34,6 +34,12 @@ static void test_enum_system_firmware_tables(void)
     UINT res;
     DWORD err;
 
+    if (!pEnumSystemFirmwareTables)
+    {
+        skip("Function EnumSystemFirmwareTables not available, skipping.\n");
+        return;
+    }
+
     /* Applications may use e.g. 'ACPI', which we currently don't support.
      * We should at the very least return valid error codes for it.
      * Test with a definitely-invalid provider so it also fails on real Windows. */
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7458

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/kernelbase/tests/memory.c:
> UINT res;
>      DWORD err;
>
> +    if (!pEnumSystemFirmwareTables)
> +    {
> +        skip("Function EnumSystemFirmwareTables not available, skipping.\n");

```suggestion:-0+0
        win_skip("Function EnumSystemFirmwareTables not available, skipping.\n");
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7458#note_96349

[Jinoh Kang (＠iamahuman)]

Looks good otherwise.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7458#note_96350