diff --git a/packages/snjs/mocha/application.test.js b/packages/snjs/mocha/application.test.js index 974ee68ee..b3f5af9e5 100644 --- a/packages/snjs/mocha/application.test.js +++ b/packages/snjs/mocha/application.test.js @@ -6,9 +6,6 @@ chai.use(chaiAsPromised) const expect = chai.expect describe('application instances', () => { - let application1 - let application2 - const syncOptions = { checkIntegrity: true, awaitAll: true, @@ -20,12 +17,6 @@ describe('application instances', () => { afterEach(async () => { localStorage.clear() - if (application1 !== undefined) { - await Factory.safeDeinit(application1) - } - if (application2 !== undefined) { - await Factory.safeDeinit(application2) - } }) it('two distinct applications should not share model manager state', async () => { @@ -45,84 +36,86 @@ describe('application instances', () => { }) it('two distinct applications should not share storage manager state', async () => { - application1 = await Factory.createAndInitializeApplication('app1') - application2 = await Factory.createAndInitializeApplication('app2') + const app1 = await Factory.createAndInitializeApplication('app1') + const app2 = await Factory.createAndInitializeApplication('app2') - await Factory.createMappedNote(application1) - await application1.syncService.sync(syncOptions) + await Factory.createMappedNote(app1) + await app1.syncService.sync(syncOptions) - expect((await application1.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) - expect((await application2.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems) + expect((await app1.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) + expect((await app2.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems) - await Factory.createMappedNote(application2) - await application2.syncService.sync(syncOptions) + await Factory.createMappedNote(app2) + await app2.syncService.sync(syncOptions) - expect((await application1.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) - expect((await application2.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) + expect((await app1.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) + expect((await app2.diskStorageService.getAllRawPayloads()).length).length.to.equal(BaseItemCounts.DefaultItems + 1) + await Factory.safeDeinit(app1) + await Factory.safeDeinit(app2) }) it('deinit application while storage persisting should be handled gracefully', async () => { /** This test will always succeed but should be observed for console exceptions */ - application1 = await Factory.createAndInitializeApplication('app') + const app = await Factory.createAndInitializeApplication('app') /** Don't await */ - application1.diskStorageService.persistValuesToDisk() - await application1.prepareForDeinit() - await Factory.safeDeinit(application1) + app.diskStorageService.persistValuesToDisk() + await app.prepareForDeinit() + await Factory.safeDeinit(app) }) it('changing default host should not affect already signed in accounts', async () => { /** This test will always succeed but should be observed for console exceptions */ - application1 = await Factory.createAndInitializeApplication( + const app = await Factory.createAndInitializeApplication( 'app', Environment.Web, Platform.MacWeb, Factory.getDefaultHost(), ) await Factory.registerUserToApplication({ - application: application1, + application: app, email: UuidGenerator.GenerateUuid(), password: 'password', }) - await application1.prepareForDeinit() - await Factory.safeDeinit(application1) + await app.prepareForDeinit() + await Factory.safeDeinit(app) /** Recreate app with different host */ - application2 = await Factory.createAndInitializeApplication( + const recreatedApp = await Factory.createAndInitializeApplication( 'app', Environment.Web, Platform.MacWeb, 'http://nonsense.host', ) - expect(application2.getHost()).to.not.equal('http://nonsense.host') - expect(application2.getHost()).to.equal(Factory.getDefaultHost()) + expect(recreatedApp.getHost()).to.not.equal('http://nonsense.host') + expect(recreatedApp.getHost()).to.equal(Factory.getDefaultHost()) }) it('signing out application should delete snjs_version', async () => { const identifier = 'app' - application1 = await Factory.createAndInitializeApplication(identifier) + const app = await Factory.createAndInitializeApplication(identifier) expect(localStorage.getItem(`${identifier}-snjs_version`)).to.be.ok - await application1.user.signOut() + await app.user.signOut() expect(localStorage.getItem(`${identifier}-snjs_version`)).to.not.be.ok }) it('locking application while critical func in progress should wait up to a limit', async () => { /** This test will always succeed but should be observed for console exceptions */ - application1 = await Factory.createAndInitializeApplication('app') + const app = await Factory.createAndInitializeApplication('app') /** Don't await */ const MaximumWaitTime = 0.5 - application1.diskStorageService.executeCriticalFunction(async () => { + app.diskStorageService.executeCriticalFunction(async () => { /** If we sleep less than the maximum, locking should occur safely. * If we sleep more than the maximum, locking should occur with exception on * app deinit. */ await Factory.sleep(MaximumWaitTime - 0.05) /** Access any deviceInterface function */ - application1.diskStorageService.deviceInterface.getAllDatabaseEntries(application1.identifier) + app.diskStorageService.deviceInterface.getAllDatabaseEntries(app.identifier) }) - await application1.lock() + await app.lock() }) describe.skip('signOut()', () => {