Use node 18 (lts)

Author: lserra-iov

.nvmrc

@@ -0,0 +1 @@
+v18.17.0

package-lock.json

@@ -53,12 +53,12 @@
     "!*/__tests__"
   ],
   "dependencies": {
-    "@loopback/boot": "^3.1.2",
-    "@loopback/core": "^2.13.1",
-    "@loopback/repository": "^3.3.0",
-    "@loopback/rest": "^9.1.1",
-    "@loopback/rest-explorer": "^5.0.9",
-    "@loopback/service-proxy": "^3.0.5",
+    "@loopback/boot": "^6.1.3",
+    "@loopback/core": "^5.1.3",
+    "@loopback/repository": "^6.1.3",
+    "@loopback/rest": "^13.1.3",
+    "@loopback/rest-explorer": "^6.1.3",
+    "@loopback/service-proxy": "^6.1.3",
     "@rsksmart/bridge-state-data-parser": "git+https://github.com/rsksmart/bridge-state-data-parser.git#allow-block-argument",
     "@rsksmart/rsk-precompiled-abis": "git+https://github.com/rsksmart/precompiled-abis.git#5.0.0-FINGERROOT",
     "@types/mongoose": "^5.11.97",
@@ -81,20 +81,20 @@
     "web3-utils": "^1.3.4"
   },
   "devDependencies": {
-    "@loopback/build": "^6.2.8",
-    "@loopback/eslint-config": "^10.0.4",
-    "@loopback/testlab": "^3.2.10",
+    "@loopback/build": "^10.1.3",
+    "@loopback/eslint-config": "^14.0.4",
+    "@loopback/testlab": "^6.1.3",
     "@types/big.js": "^6.1.2",
     "@types/bs58": "^4.0.1",
-    "@types/node": "^10.17.48",
-    "@typescript-eslint/eslint-plugin": "^5.30.5",
-    "@typescript-eslint/parser": "^5.30.5",
-    "eslint": "^8.19.0",
+    "@types/node": "^16.18.50",
+    "@typescript-eslint/eslint-plugin": "^6.4.1",
+    "@typescript-eslint/parser": "^6.4.1",
+    "eslint": "^8.48.0",
     "eslint-config-airbnb-base": "^15.0.0",
     "eslint-plugin-import": "^2.26.0",
     "sinon": "^11.1.2",
     "sonarqube-scanner": "^3.0.1",
-    "source-map-support": "^0.5.19",
-    "typescript": "~4.1.2"
+    "source-map-support": "^0.5.21",
+    "typescript": "~5.2.2"
   }
 }

package.json

@@ -382,7 +382,7 @@ describe('Service: RskChainSyncService', () => {
 
     // RSK is contacted with these blocks (latest and 3 to 5)
     sinon.assert.callCount(mockedRskNodeService.getBlock, 4);
-    sinon.assert.calledWithExactly(mockedRskNodeService.getBlock, 'latest', false);
+    sinon.assert.calledWithExactly(mockedRskNodeService.getBlock, 'latest');
     sinon.assert.calledWithExactly(mockedRskNodeService.getBlock, 3);
     sinon.assert.calledWithExactly(mockedRskNodeService.getBlock, 4);
     sinon.assert.calledWithExactly(mockedRskNodeService.getBlock, 5);

src/__tests__/unit/services/rsk-chain-sync.service.unit.ts

@@ -9,7 +9,7 @@ const getInitialBlock = () => new RskBlock(2863627, '0xba5e', '0x');
 describe('Service: RskNodeService', () => {
     it('Searches the block using initial block conf', async () => {
         const thisService = new RskNodeService();
-        const block = await thisService.getBlock(getInitialBlock().height, false);
+        const block = await thisService.getBlock(getInitialBlock().height);
         sinon.assert.match(block.number, getInitialBlock().height);
     });
     it('Verify ${process.env.RSK_NODE_HOST} configuration', async () => {
@@ -42,7 +42,7 @@ describe('Service: RskNodeService', () => {
     });
     it('Searches the block', async () => {
         const thisService = new RskNodeService();
-        const block = await thisService.getBlock(getInitialBlock().height, false);
+        const block = await thisService.getBlock(getInitialBlock().height);
         expect(block).to.not.be.null;
     });
     it('Searches the block number', async () => {

src/__tests__/unit/services/rsk-node.services.unit.ts

@@ -324,7 +324,7 @@ describe('tx Fee controller', () => {
     }))).to.be.true();
   });
   it('Should ensure the change output has a higher value than dust environment variable ', async () => {
-    const dustValue = process.env.BURN_DUST_VALUE ?? 2000;
+    const dustValue = Number(process.env.BURN_DUST_VALUE) || 2000;
     const minFastFee = new SatoshiBig(process.env.FEE_PER_KB_FAST_MIN ?? 100, 'satoshi');
     const minAverageFee = new SatoshiBig(process.env.FEE_PER_KB_AVERAGE_MIN ?? 100, 'satoshi');
     const minSlowFee = new SatoshiBig(process.env.FEE_PER_KB_SLOW_MIN ?? 100, 'satoshi');

src/__tests__/unit/tx-fee.controller.unit.ts

@@ -114,7 +114,7 @@ export class RskChainSyncService {
 
   public async sync(): Promise<void> {
     let dbBestBlock = await this.getSyncStatus();
-    const rskBestBlock = RskBlock.fromWeb3Block(await this.rskNodeService.getBlock('latest', false));
+    const rskBestBlock = RskBlock.fromWeb3Block(await this.rskNodeService.getBlock('latest'));
     // In case the db is synced with a forked chain and that forked chain is longer than the main chain,
     // remove all extra forked blocks from the db + 1 as an offset so the following logic handles it appropriately.
     if(rskBestBlock.height < dbBestBlock.rskBlockHeight) {

src/services/rsk-chain-sync.service.ts

@@ -9,8 +9,8 @@ export class RskNodeService {
   constructor() {
     this.web3 = new Web3(`${process.env.RSK_NODE_HOST}`);
   }
-  getBlock(block: string | number, withTransactions = true): Promise<BlockTransactionObject> {
-    return this.web3.eth.getBlock(block, withTransactions);
+  getBlock(block: string | number): Promise<BlockTransactionObject> {
+    return this.web3.eth.getBlock(block, true);
   }
   getTransactionReceipt(txHash: string): Promise<any> {
     return this.web3.eth.getTransactionReceipt(txHash);