fix: return Promise when opening the db

Author: Naramsim

src/getter.js

@@ -5,22 +5,31 @@ var db
 function openDB(config) {
     if (config.cache && typeof window !== 'undefined') {
         const request = window.indexedDB.open("pokeapi-js-wrapper", 1);
-
-        request.onerror = (event) => {
-            log('IndexedDB not available')
-        }
-        request.onupgradeneeded = (event) => {
-            db = event.target.result;
-            log('db opened and cache created')
-            db.createObjectStore("cache", { autoIncrement: false });
-        }
-        request.onsuccess = (event) => {
-            log('db opened')
-            db = event.target.result;
-        }
-        request.onversionchange = (event) => {
-            db.close()
-        }
+        return new Promise((resolve, reject) => {
+            request.onerror = (event) => {
+                log('IndexedDB not available')
+                reject()
+            }
+            request.onupgradeneeded = (event) => {
+                db = event.target.result;
+                log('db opened and cache created')
+                db.createObjectStore("cache", { autoIncrement: false });
+                resolve(db)
+            }
+            request.onsuccess = (event) => {
+                log('db opened')
+                db = event.target.result;
+                resolve(db)
+            }
+            request.onversionchange = (event) => {
+                db.close()
+                reject()
+            }
+            request.onblocked = (event) => {
+                db.close()
+                reject()
+            }
+        });
     }
 }
 
@@ -37,7 +46,6 @@ async function loadResource(config, url) {
         url = url.replace(/^\//, '');
         url = `${config.protocol}://${config.hostName}/${url}`
     }
-
     if (canUseCache(config, db)) {
         const transaction = db.transaction("cache", "readonly");
         const objectStore = transaction.objectStore("cache");

test/cdn.html

@@ -6,9 +6,10 @@
     // var PJSW_DEBUG=1
   </script>
   <script type="module">
-    import {Pokedex} from "https://cdn.jsdelivr.net/gh/pokeapi/pokeapi-js-wrapper@beta/src/index.js"
-    const P = await Pokedex.init();
-    const version = await P.getVersionByName(1)
+    //import {Pokedex} from "https://cdn.jsdelivr.net/gh/pokeapi/pokeapi-js-wrapper@beta/src/index.js"
+    import {Pokedex} from "../src/index.js"
+    const pokedex = await Pokedex.init();
+    const version = await pokedex.getVersionByName(1)
     document.getElementById("version").textContent = version.name;
   </script>
 </html>

test/test.html.js

@@ -40,7 +40,7 @@ describe("pokedex", function () {
 
   // --- Resource Methods ---
 
-  describe(".resource(Mixed: array)", function () {
+  describe(".resource(Mixed: array) not cached", function () {
     it("should have property name", async function () {
       const res = await customP.resource(['/api/v2/pokemon/36', 'api/v2/berry/8', 'https://pokeapi.co/api/v2/ability/9/']);
       expect(res[0]).to.have.property('name');