[PATCH v2 4/4] msi/tests: Test deferral of SelfRegModules and SelfUnregModules.

[Hans Leidekker]

From: Zebediah Figura

Signed-off-by: Zebediah Figura 
Signed-off-by: Hans Leidekker 
---
 dlls/msi/tests/action.c    | 14 ++++++++++++++
 dlls/msi/tests/custom.c    | 25 +++++++++++++++++++++++++
 dlls/msi/tests/custom.spec |  2 ++
 3 files changed, 41 insertions(+)

diff --git a/dlls/msi/tests/action.c b/dlls/msi/tests/action.c
index 9dfa623758..ac94c38ed9 100644
--- a/dlls/msi/tests/action.c
+++ b/dlls/msi/tests/action.c
@@ -774,15 +774,28 @@ static const char sr_install_exec_seq_dat[] =
     "InstallInitialize\t\t1500\n"
     "ProcessComponents\t\t1600\n"
     "SelfUnregModules\t\t3800\n"
+    "su_immediate\tREMOVE\t3801\n"
+    "su_deferred\tREMOVE\t3802\n"
     "RemoveFiles\t\t3900\n"
     "InstallFiles\t\t4000\n"
     "SelfRegModules\t\t4100\n"
+    "sr_immediate\tNOT REMOVE\t4101\n"
+    "sr_deferred\tNOT REMOVE\t4102\n"
     "CreateShortcuts\t\t4600\n"
     "RegisterProduct\t\t5100\n"
     "PublishFeatures\t\t5200\n"
     "PublishProduct\t\t5300\n"
     "InstallFinalize\t\t6600\n";

+static const char sr_custom_action_dat[] =
+    "Action\tType\tSource\tTarget\n"
+    "s72\ti2\tS64\tS0\n"
+    "CustomAction\tAction\n"
+    "sr_immediate\t1\tcustom.dll\tsr_absent\n"
+    "sr_deferred\t1025\tcustom.dll\tsr_present\n"
+    "su_immediate\t1\tcustom.dll\tsr_present\n"
+    "su_deferred\t1025\tcustom.dll\tsr_absent\n";
+
 static const char font_media_dat[] =
     "DiskId\tLastSequence\tDiskPrompt\tCabinet\tVolumeLabel\tSource\n"
     "i2\ti4\tL64\tS255\tS32\tS72\n"
@@ -2128,6 +2141,7 @@ static const msi_table sr_tables[] =
     ADD_TABLE(sr_file),
     ADD_TABLE(sr_selfreg),
     ADD_TABLE(sr_install_exec_seq),
+    ADD_TABLE(sr_custom_action),
     ADD_TABLE(media),
     ADD_TABLE(property)
 };
diff --git a/dlls/msi/tests/custom.c b/dlls/msi/tests/custom.c
index b7da2b8fcc..3fb538203f 100644
--- a/dlls/msi/tests/custom.c
+++ b/dlls/msi/tests/custom.c
@@ -1833,3 +1833,28 @@ todo_wine_if(!MsiGetMode(hinst, MSIRUNMODE_SCHEDULED))

return ERROR_SUCCESS;
 }
+
+UINT WINAPI sr_present(MSIHANDLE hinst)
+{
+    HKEY key;
+    LONG res;
+
+    res = RegOpenKeyA(HKEY_CLASSES_ROOT, "selfreg_test", &key);
+todo_wine_if(!MsiGetMode(hinst, MSIRUNMODE_SCHEDULED))
+    ok(hinst, !res, "got %u\n", res);
+    RegCloseKey(key);
+
+    return ERROR_SUCCESS;
+}
+
+UINT WINAPI sr_absent(MSIHANDLE hinst)
+{
+    HKEY key;
+    LONG res;
+
+    res = RegOpenKeyA(HKEY_CLASSES_ROOT, "selfreg_test", &key);
+todo_wine_if(!MsiGetMode(hinst, MSIRUNMODE_SCHEDULED))
+    ok(hinst, res == ERROR_FILE_NOT_FOUND, "got %u\n", res);
+
+    return ERROR_SUCCESS;
+}
diff --git a/dlls/msi/tests/custom.spec b/dlls/msi/tests/custom.spec
index 058a9172f1..2ec5b6cc6d 100644
--- a/dlls/msi/tests/custom.spec
+++ b/dlls/msi/tests/custom.spec
@@ -43,6 +43,8 @@
 @ stdcall sds_absent(long)
 @ stdcall sis_present(long)
 @ stdcall sis_absent(long)
+@ stdcall sr_present(long)
+@ stdcall sr_absent(long)
 @ stdcall sss_started(long)
 @ stdcall sss_stopped(long)
 @ stdcall tl_present(long)

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=38819

Your paranoid android.

=== w7u (32 bit install) ===
The previous 1 run(s) terminated abnormally

=== w7u (32 bit msi) ===
The previous 1 run(s) terminated abnormally