fix: close root datastore after initialized check

Author: achingbrain

src/index.js

@@ -74,13 +74,21 @@ class IpfsRepo {
    * @returns {Promise<Boolean>}
    */
   async isInitialized () {
+    if (!this.closed) {
+      // repo is open, must be initialized
+      return true
+    }
+
     try {
+      // have to open the root datastore in the browser before
+      // we can check whether it's been initialized
       await this._openRoot()
       await this._checkInitialized()
-      // necessary? await this.root.close()
+      await this.root.close()
 
       return true
     } catch (err) {
+      // FIXME: do not use exceptions for flow control
       return false
     }
   }

test/is-initialized.js

@@ -25,4 +25,17 @@ describe('isInitialized', () => {
     await repo.init({})
     expect(await repo.isInitialized()).to.be.true()
   })
+
+  it('should be true after initialization and opening', async () => {
+    await repo.init({})
+    await repo.open()
+    expect(await repo.isInitialized()).to.be.true()
+  })
+
+  it('should be true after initialization, opening and closing', async () => {
+    await repo.init({})
+    await repo.open()
+    await repo.close()
+    expect(await repo.isInitialized()).to.be.true()
+  })
 })