From 50b46bdb197904bd6756886770d1775b329b862e Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Fri, 8 Jan 2016 03:19:25 -0500 Subject: [PATCH] Don't bother doing a full DB reset after sync tests Tests should make no assumptions about the presence of bundled files and should do a full resetDB() if they need them. But most tests don't need them, and they're very slow to install. We can reconsider this if we drastically speed up DB resetting in tests (e.g., by caching a pristine data directory). --- test/tests/syncRunnerTest.js | 5 ----- 1 file changed, 5 deletions(-) diff --git a/test/tests/syncRunnerTest.js b/test/tests/syncRunnerTest.js index 65721982e..f92e159d2 100644 --- a/test/tests/syncRunnerTest.js +++ b/test/tests/syncRunnerTest.js @@ -417,11 +417,6 @@ describe("Zotero.Sync.Runner", function () { yield Zotero.Libraries.init(); }) - after(function* () { - yield resetDB({ - thisArg: this - }); - }) it("should perform a sync across all libraries and update library versions", function* () { yield Zotero.Users.setCurrentUserID(1);