Linted the code

Author: tomaszslabon

src/BitcoinHelpers.js

@@ -290,7 +290,10 @@ const BitcoinHelpers = {
       throw new Error("Electrum client not configured.")
     }
 
-    const electrumClient = new ElectrumClient(BitcoinHelpers.electrumConfig, BitcoinHelpers.electrsConfig)
+    const electrumClient = new ElectrumClient(
+      BitcoinHelpers.electrumConfig,
+      BitcoinHelpers.electrsConfig
+    )
 
     await electrumClient.connect()
 

src/lib/ElectrumClient.js

@@ -66,7 +66,8 @@ export default class Client {
    * @param {Config} config Electrum client connection configuration.
    * @param {string} apiUrl Url to the electrs server
    */
-  constructor(config, apiUrl) { // TODO: config will be removed once all ported
+  constructor(config, apiUrl) {
+    // TODO: config will be removed once all ported
     this.electrumClient = new ElectrumClient(
       config.server,
       config.port,

test/BitcoinSPVTest.js

@@ -1,6 +1,6 @@
 import { BitcoinSPV } from "../src/lib/BitcoinSPV.js"
 import ElectrumClient from "../src/lib/ElectrumClient.js"
-import { electrumConfig } from "./config/network.js"
+import { electrsConfig, electrumConfig } from "./config/network.js"
 import { readFileSync } from "fs"
 import { assert } from "chai"
 
@@ -12,7 +12,10 @@ describe("BitcoinSPV", async () => {
   before(async () => {
     const txData = readFileSync("./test/data/tx.json", "utf8")
     tx = JSON.parse(txData)
-    electrumClient = new ElectrumClient(electrumConfig["testnet"])
+    electrumClient = new ElectrumClient(
+      electrumConfig["testnet"],
+      electrsConfig["testnet"]
+    )
     bitcoinSPV = new BitcoinSPV(electrumClient)
     await electrumClient.connect()
   })

test/ElectrumClientTest.js

@@ -193,31 +193,31 @@ describe("ElectrumClient", async () => {
       console.log("result", result)
     })
 
-      // This test cannot be executed as part of CI. It takes too long to wait
-      // for confirmations. Use this for manual testing.
-      // TODO: Make it possible with electrum server mock.
-      it.skip("wait for three more confirmations", async () => {
-        const txHash = tx.hash
+    // This test cannot be executed as part of CI. It takes too long to wait
+    // for confirmations. Use this for manual testing.
+    // TODO: Make it possible with electrum server mock.
+    it.skip("wait for three more confirmations", async () => {
+      const txHash = tx.hash
 
-        const currentTx = await client.getTransaction(txHash)
+      const currentTx = await client.getTransaction(txHash)
 
-        const requiredConfirmations = currentTx.confirmations + 3
+      const requiredConfirmations = currentTx.confirmations + 3
 
-        const callback = async function() {
-          // Get current state of the transaction.
-          const tx = await client.getTransaction(txHash)
+      const callback = async function() {
+        // Get current state of the transaction.
+        const tx = await client.getTransaction(txHash)
 
-          // Check if transaction has already enough number of confirmations.
+        // Check if transaction has already enough number of confirmations.
 
-          console.log("current confirmations:", tx.confirmations)
-          if (tx.confirmations >= requiredConfirmations) {
-            return tx.txid
-          }
+        console.log("current confirmations:", tx.confirmations)
+        if (tx.confirmations >= requiredConfirmations) {
+          return tx.txid
         }
+      }
 
-        const result = await client.onNewBlock(callback)
+      const result = await client.onNewBlock(callback)
 
-        console.log("result", result)
-      })
+      console.log("result", result)
+    })
   })
 })