scriptoriumadmin/zotero
1
0
Fork 0
mirror of https://github.com/zotero/zotero.git synced 2026-10-10 03:28:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Wait for test success file to be written before quitting

Browse source 
The write is asynchronous, and on Windows the app quit before it
finished, making passing test runs report failure.
This commit is contained in:
Dan Stillman 2026-08-19 14:44:36 -04:00
parent 1aecadce0f
commit 6e7e4678f1
1 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
test/content/runtests.js
View file

@ -10,15 +10,20 @@ chai.config.truncateThreshold = 0
function quit(failed) {
// Quit with exit status
var promise = Promise.resolve();
if(!failed) {
IOUtils.write(PathUtils.join(FileUtils.getDir("ProfD", []).path, "success"), new Uint8Array(0));
// Wait for the write, which the runner uses to detect success, to finish before
// quitting -- on Windows, quitting could otherwise beat it
promise = IOUtils.write(PathUtils.join(FileUtils.getDir("ProfD", []).path, "success"), new Uint8Array(0));
}
if(!TestOptions.noquit) {
setTimeout(function () {
Components.classes['@mozilla.org/toolkit/app-startup;1']
.getService(Components.interfaces.nsIAppStartup)
.quit(Components.interfaces.nsIAppStartup.eForceQuit);
}, 250);
promise.then(function () {
setTimeout(function () {
Components.classes['@mozilla.org/toolkit/app-startup;1']
.getService(Components.interfaces.nsIAppStartup)
.quit(Components.interfaces.nsIAppStartup.eForceQuit);
}, 250);
});
}
}