Added await-thenable rule to prevent unnecessary await, and fixed utils.getRandomBytes

Author: CMCDragonkai

.eslintrc

@@ -108,6 +108,7 @@
     "@typescript-eslint/no-misused-promises": ["error", {
       "checksVoidReturn": false
     }],
+    "@typescript-eslint/await-thenable": ["error"],
     "@typescript-eslint/naming-convention": [
       "error",
       {

tests/DBTransaction.test.ts

@@ -652,7 +652,7 @@ describe(DBTransaction.name, () => {
       await tran.put('f', '6');
       await tran.put('j', '10');
       await tran.put('k', '11');
-      const iterator = await tran.iterator();
+      const iterator = tran.iterator();
       iterator.seek('a');
       expect(await iterator.next()).toStrictEqual([
         Buffer.from('a'),

tests/utils.ts

@@ -4,7 +4,16 @@ const ivSize = 16;
 const authTagSize = 16;
 
 async function getRandomBytes(size: number): Promise<Buffer> {
-  return Buffer.from(await random.getBytes(size), 'binary');
+  const p = new Promise<string>((resolve, reject) => {
+    random.getBytes(size, (e, bytes) => {
+      if (e != null) {
+        reject(e);
+      } else {
+        resolve(bytes);
+      }
+    });
+  });
+  return Buffer.from(await p, 'binary');
 }
 
 function getRandomBytesSync(size: number): Buffer {