From da8d36cca8f1b0dd1b1008a5811f01f1faff1fb5 Mon Sep 17 00:00:00 2001 From: Robert Strong Date: Sat, 11 Jun 2016 08:31:20 -0700 Subject: [PATCH] Bug 1168743 - test changes - Only use env vars in the test updater and general app update cleanup. r=mhowell --- .../mozapps/update/tests/TestAUSHelper.cpp | 10 + .../chrome/test_0111_neverButton_basic.xul | 2 +- .../test_0112_neverButton_billboard.xul | 2 +- ...113_showNeverForVersionRemovedWithPref.xul | 2 +- .../test_0121_check_requireBuiltinCert.xul | 4 +- ...eck_allowNonBuiltinCert_validCertAttrs.xul | 4 +- ...k_allowNonBuiltinCert_noCertAttrsCheck.xul | 2 +- ...t_0131_check_invalidCertAttrs_noUpdate.xul | 2 +- ..._0132_check_invalidCertAttrs_hasUpdate.xul | 2 +- ..._0141_notify_invalidCertAttrs_noUpdate.xul | 2 +- ...0142_notify_invalidCertAttrs_hasUpdate.xul | 2 +- .../chrome/test_0171_check_noPerms_manual.xul | 2 +- .../test_0172_notify_noPerms_manual.xul | 2 +- .../update/tests/chrome/test_9999_cleanup.xul | 2 +- toolkit/mozapps/update/tests/chrome/utils.js | 16 +- toolkit/mozapps/update/tests/data/shared.js | 88 +- .../update/tests/data/sharedUpdateXML.js | 1 - .../update/tests/data/xpcshellUtilsAUS.js | 1564 +++++++++++------ toolkit/mozapps/update/tests/moz.build | 12 +- .../unit_aus_update/cleanupSuccessLogMove.js | 13 +- .../unit_aus_update/cleanupSuccessLogsFIFO.js | 19 +- .../downloadInterruptedRecovery.js | 4 +- .../tests/unit_aus_update/head_update.js | 2 + .../tests/unit_aus_update/updateManagerXML.js | 2 +- .../tests/unit_aus_update/urlConstruction.js | 4 +- .../update/tests/unit_aus_update/xpcshell.ini | 2 - .../tests/unit_base_updater/head_update.js | 2 + .../marAppApplyDirLockedStageFailure_win.js | 75 +- ...pApplyUpdateAppBinInUseStageSuccess_win.js | 239 +-- .../marAppApplyUpdateStageSuccess.js | 222 +-- .../marAppApplyUpdateSuccess.js | 125 +- .../marAppInUseStageFailureComplete_win.js | 69 +- .../marAppInUseStageSuccessComplete_unix.js | 160 +- .../marAppInUseSuccessComplete.js | 59 +- .../marCallbackAppStageSuccessComplete_win.js | 49 +- .../marCallbackAppStageSuccessPartial_win.js | 49 +- .../marCallbackAppSuccessComplete_win.js | 34 +- .../marCallbackAppSuccessPartial_win.js | 34 +- .../unit_base_updater/marFailurePartial.js | 40 +- .../marFileInUseStageFailureComplete_win.js | 69 +- .../marFileInUseStageFailurePartial_win.js | 69 +- .../marFileInUseSuccessComplete_win.js | 57 +- .../marFileInUseSuccessPartial_win.js | 57 +- .../marFileLockedFailureComplete_win.js | 60 +- .../marFileLockedFailurePartial_win.js | 60 +- .../marFileLockedStageFailureComplete_win.js | 79 +- .../marFileLockedStageFailurePartial_win.js | 78 +- ...MRFDirFileInUseStageFailureComplete_win.js | 80 +- ...RMRFDirFileInUseStageFailurePartial_win.js | 76 +- .../marRMRFDirFileInUseSuccessComplete_win.js | 67 +- .../marRMRFDirFileInUseSuccessPartial_win.js | 64 +- .../marStageFailurePartial.js | 31 +- .../marStageSuccessComplete.js | 155 +- .../marStageSuccessPartial.js | 113 +- .../unit_base_updater/marSuccessComplete.js | 86 +- .../unit_base_updater/marSuccessPartial.js | 74 +- .../unit_base_updater/marVersionDowngrade.js | 46 +- .../unit_base_updater/marWrongChannel.js | 46 +- .../tests/unit_base_updater/xpcshell.ini | 5 +- .../unit_service_updater/bootstrapSvc.js | 22 +- .../checkUpdaterSigSvc.js | 4 +- .../tests/unit_service_updater/head_update.js | 2 + ...marAppApplyDirLockedStageFailureSvc_win.js | 79 +- ...plyUpdateAppBinInUseStageSuccessSvc_win.js | 243 +-- .../marAppApplyUpdateStageSuccessSvc.js | 226 +-- .../marAppApplyUpdateSuccessSvc.js | 129 +- .../marAppInUseStageFailureCompleteSvc_win.js | 69 +- .../marAppInUseSuccessCompleteSvc.js | 59 +- ...rCallbackAppStageSuccessCompleteSvc_win.js | 51 +- ...arCallbackAppStageSuccessPartialSvc_win.js | 51 +- .../marCallbackAppSuccessCompleteSvc_win.js | 34 +- .../marCallbackAppSuccessPartialSvc_win.js | 34 +- .../marFailurePartialSvc.js | 43 +- ...marFileInUseStageFailureCompleteSvc_win.js | 69 +- .../marFileInUseStageFailurePartialSvc_win.js | 69 +- .../marFileInUseSuccessCompleteSvc_win.js | 57 +- .../marFileInUseSuccessPartialSvc_win.js | 57 +- .../marFileLockedFailureCompleteSvc_win.js | 60 +- .../marFileLockedFailurePartialSvc_win.js | 60 +- ...arFileLockedStageFailureCompleteSvc_win.js | 79 +- ...marFileLockedStageFailurePartialSvc_win.js | 78 +- ...DirFileInUseStageFailureCompleteSvc_win.js | 79 +- ...FDirFileInUseStageFailurePartialSvc_win.js | 76 +- ...rRMRFDirFileInUseSuccessCompleteSvc_win.js | 67 +- ...arRMRFDirFileInUseSuccessPartialSvc_win.js | 64 +- .../marStageFailurePartialSvc.js | 36 +- .../marStageSuccessCompleteSvc.js | 161 +- .../marStageSuccessPartialSvc.js | 119 +- .../marSuccessCompleteSvc.js | 90 +- .../marSuccessPartialSvc.js | 78 +- 90 files changed, 3262 insertions(+), 3379 deletions(-) diff --git a/toolkit/mozapps/update/tests/TestAUSHelper.cpp b/toolkit/mozapps/update/tests/TestAUSHelper.cpp index 8d207a742597..e3c5d40af127 100644 --- a/toolkit/mozapps/update/tests/TestAUSHelper.cpp +++ b/toolkit/mozapps/update/tests/TestAUSHelper.cpp @@ -327,6 +327,16 @@ int NS_main(int argc, NS_tchar **argv) #endif } + if (!NS_tstrcmp(argv[1], NS_T("is-process-running"))) { +#ifdef XP_WIN + LPCWSTR application = argv[2]; + return (ERROR_NOT_FOUND == IsProcessRunning(application)) ? 0 : 1; +#else + // Not implemented on non-Windows platforms + return 1; +#endif + } + if (NS_tchdir(argv[1]) != 0) { return 1; } diff --git a/toolkit/mozapps/update/tests/chrome/test_0111_neverButton_basic.xul b/toolkit/mozapps/update/tests/chrome/test_0111_neverButton_basic.xul index c3b2fdfa8250..3851a0982ca7 100644 --- a/toolkit/mozapps/update/tests/chrome/test_0111_neverButton_basic.xul +++ b/toolkit/mozapps/update/tests/chrome/test_0111_neverButton_basic.xul @@ -19,7 +19,7 @@