Merge pull request #351 from terra-money/fix/tests

fix: ibc url, tests and security issue

package.json

@@ -29,7 +29,7 @@
   },
   "scripts": {
     "build": "tsc --module commonjs && webpack --mode production",
-    "test": "jest",
+    "test": "jest --runInBand",
     "format": "prettier --check ./src/**/*.ts",
     "format:fix": "prettier --write ./src/**/*.ts",
     "lint": "eslint src --ext .js,.ts",

src/client/lcd/api/IbcAPI.spec.ts

@@ -5,6 +5,7 @@ import { IbcAPI } from './IbcAPI';
 const terra = new LCDClient({
   chainID: 'pisco-1',
   URL: 'https://pisco-lcd.terra.dev',
+  isClassic: false,
 });
 const ibc = new IbcAPI(terra);
 

src/client/lcd/api/IbcAPI.ts

@@ -142,8 +142,12 @@ export class IbcAPI extends BaseAPI {
    * Gets the current transfer application parameters.
    */
   public async parameters(params: APIParams = {}): Promise<IbcClientParams> {
+    const url = this.lcd.config.isClassic
+      ? `/ibc/client/v1/params`
+      : `/ibc/core/client/v1/params`;
+
     return this.c
-      .get<{ params: IbcClientParams.Data }>(`/ibc/client/v1/params`, params)
+      .get<{ params: IbcClientParams.Data }>(url, params)
       .then(({ params: d }) => ({
         allowed_clients: d.allowed_clients,
       }));

src/client/lcd/api/UpgradeAPI.spec.ts

@@ -25,7 +25,7 @@ describe('UpgradeAPI', () => {
 
   describe('node_versions', () => {
     it('module count', async () => {
-      expect(await upgrade.moduleVersions()).toHaveLength(21);
+      expect(await upgrade.moduleVersions()).toHaveLength(26);
     });
   });
 });