Merge pull request #575 from gregdhill/chore/rm-skip

chore: remove some skipped tests

package.json

@@ -36,12 +36,11 @@
     "test": "run-s build test:*",
     "test:lint": "eslint src --ext .ts",
     "test:unit": "mocha test/unit/*.test.ts test/unit/**/*.test.ts",
-    "test:integration": "run-s test:integration:staging test:integration:release",
+    "test:integration": "run-s test:integration:staging",
     "test:integration:staging": "run-s test:integration:setup test:integration:parallel test:integration:sequential",
     "test:integration:setup": "mocha test/integration/**/staging/setup/initialize.test.ts --timeout 10000000",
     "test:integration:parallel": "mocha test/integration/**/staging/*.test.ts --timeout 10000000 --parallel",
     "test:integration:sequential": "mocha test/integration/**/staging/sequential/*.test.ts --timeout 10000000",
-    "test:integration:release": "mocha test/integration/**/release/*.test.ts --timeout 10000000",
     "watch:build": "tsc -p tsconfig.json -w",
     "watch:test": "mocha --watch test/**/*.test.ts",
     "local-meta": "curl -H 'Content-Type: application/json' -d '{\"id\":\"1\", \"jsonrpc\":\"2.0\", \"method\": \"state_getMetadata\", \"params\":[]}' http://localhost:9933 > src/json/parachain.json",
@@ -123,4 +122,4 @@
     ],
     "recursive": true
   }
-}
+}

test/integration/parachain/staging/sequential/issue.test.ts

@@ -119,31 +119,6 @@ describe("issue", () => {
         );
     });
 
-    // auto-execution tests may stall indefinitely, due to vault client inaction.
-    // This will cause the testing pipeline to time out.
-    // TODO: Discuss if we want to test this in the lib. Should rather be tested in the clients
-    it.skip("should request and auto-execute issue", async () => {
-        for (const vault_1_id of vault_1_ids) {
-            const amount = newMonetaryAmount(0.00121, wrappedCurrency, true);
-
-            const feesToPay = await userInterBtcAPI.issue.getFeesToPay(amount);
-            const issueResult = await issueSingle(
-                userInterBtcAPI,
-                bitcoinCoreClient,
-                userAccount,
-                amount,
-                vault_1_id,
-                true,
-                false
-            );
-            assert.equal(
-                issueResult.finalWrappedTokenBalance.sub(issueResult.initialWrappedTokenBalance).toString(),
-                amount.sub(feesToPay).toString(),
-                "Final balance was not increased by the exact amount specified"
-            );
-        }
-    }).timeout(1000000);
-
     it("should request and manually execute issue", async () => {
         for (const vault_2_id of vault_2_ids) {
             const currencyTicker = (

test/integration/parachain/staging/system.test.ts

@@ -1,12 +1,9 @@
 import { ApiPromise, Keyring } from "@polkadot/api";
-import * as fs from "fs";
-import * as path from "path";
 import { KeyringPair } from "@polkadot/keyring/types";
 
 import { createSubstrateAPI } from "../../../../src/factory";
 import { assert } from "../../../chai";
 import { SUDO_URI, PARACHAIN_ENDPOINT, ESPLORA_BASE_PATH } from "../../../config";
-import { sudo } from "../../../utils/helpers";
 import { BLOCK_TIME_SECONDS, DefaultInterBtcApi, InterBtcApi } from "../../../../src";
 
 describe("systemAPI", () => {
@@ -36,12 +33,6 @@ describe("systemAPI", () => {
         assert.isDefined(statusCode);
     });
 
-    // TODO: Unskip once differences between rococo-local and standalone are fixed
-    it.skip("should setCode", async () => {
-        const code = fs.readFileSync(path.join(__dirname, "../../../mock/rococo_runtime.compact.wasm")).toString("hex");
-        await sudo(interBtcAPI, () => interBtcAPI.system.setCode(code));
-    });
-
     it("should getFutureBlockNumber", async () => {
         const approximately10BlocksTime = 10 * BLOCK_TIME_SECONDS;
         const [currentBlockNumber, futureBlockNumber] = await Promise.all([