From aff8e35a0ed3809e43dc497088b201c767d303ac Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Mon, 25 Jan 2016 18:22:25 -0500 Subject: [PATCH] Fix stuck checkUser prompt in sync prefs tests on Ubuntu For some reason the confirmEx prompt behaves differently (as a modal window?) on Ubuntu if it's the only window open, preventing the setTimeout() callback within support.js::waitForWindow() from being run. (Passing -c to the test runner fixed it.) This removes a win.close() in the middle of the file, which was probably a mistake anyway, so that the pref window stays open until the end of the file. --- test/tests/preferences_syncTest.js | 1 - 1 file changed, 1 deletion(-) diff --git a/test/tests/preferences_syncTest.js b/test/tests/preferences_syncTest.js index 20e0e386aa..3b36c2541e 100644 --- a/test/tests/preferences_syncTest.js +++ b/test/tests/preferences_syncTest.js @@ -63,7 +63,6 @@ describe("Sync Preferences", function () { Zotero.alert.restore(); getAPIKeyFromCredentialsStub.restore(); deleteAPIKey.restore(); - win.close(); }); it("should set API key and display full controls with correct credentials", function* () {