remove unused #100

.github/actions/build/action.yml

@@ -10,9 +10,9 @@ runs:
     - name: Npm audit
       shell: bash
       run: npm audit --audit-level=moderate
-    # - name: Test
-    #   shell: bash
-    #   run: npm run test:ci
+    - name: Test
+      shell: bash
+      run: npm run test:ci
     - name: Build
       shell: bash
       run: |

package.json

@@ -28,9 +28,9 @@
   "sideEffects": false,
   "description": "Typescript library to interact with the Nil blockchain. Can be used in the browser or in Node.js.",
   "scripts": {
-    "test": "vitest -c ./test/vitest.config.ts",
+    "test:unit": "vitest -c ./test/vitest.config.ts",
     "test:ci": "CI=true vitest -c ./test/vitest.config.ts",
-    "integration-test": "CI=true vitest -c ./test/vitest.integration.config.ts --isolate false",
+    "test:integration": "CI=true vitest -c ./test/vitest.integration.config.ts --isolate false",
     "test:coverage": "npm run test:ci -- --coverage",
     "build": "rimraf dist && rollup -c ./rollup/rollup.config.js --bundleConfigAsCjs",
     "prepare": "npx simple-git-hooks",

src/clients/PublicClient.test.ts

@@ -170,7 +170,10 @@ test("getMessageByHash", async ({ expect }) => {
 
 test("getMessageReceiptByHash", async ({ expect }) => {
   const fn = vi.fn();
-  fn.mockReturnValue({});
+  fn.mockReturnValue({
+    gasPrice: "0x100",
+    gasUsed: "0x100",
+  });
   const client = new PublicClient({
     transport: new MockTransport(fn),
     shardId: 1,

src/clients/PublicClient.ts

@@ -300,6 +300,7 @@ class PublicClient extends BaseClient {
     shardId = this.shardId,
   ): Promise<ProcessedReceipt | null> {
     assertIsValidShardId(shardId);
+
     const mapReceipt = (receipt: IReceipt): ProcessedReceipt => {
       return {
         ...receipt,

src/contracts/Faucet/Faucet.test.ts

@@ -1,5 +1,5 @@
 import { PublicClient } from "../../clients/index.js";
-import { bytesToHex } from "../../index.js";
+import { bytesToHex } from "../../encoding/fromBytes.js";
 import {
   LocalECDSAKeySigner,
   generateRandomPrivateKey,

src/contracts/WalletV1/WalletV1.test.ts

@@ -2,17 +2,18 @@ import { PublicClient } from "../../clients/index.js";
 import { generateRandomPrivateKey } from "../../index.js";
 import { LocalECDSAKeySigner } from "../../signers/LocalECDSAKeySigner.js";
 import { MockTransport } from "../../transport/MockTransport.js";
-import { HttpTransport } from "../../transport/index.js";
 import { WalletV1 } from "./WalletV1.js";
 
 const signer = new LocalECDSAKeySigner({
   privateKey: generateRandomPrivateKey(),
 });
 const pubkey = await signer.getPublicKey();
+
+const fn = vi.fn();
+fn.mockReturnValue({});
+
 const client = new PublicClient({
-  transport: new HttpTransport({
-    endpoint: "http://127.0.0.1:8529",
-  }),
+  transport: new MockTransport(fn),
   shardId: 1,
 });
 
@@ -121,7 +122,7 @@ test("Deploy through wallet", async ({ expect }) => {
     salt: 100n,
     shardId: 1,
     value: 100n,
-    gas: 100_000n * 10n,
+    feeCredit: 100_000n,
   });
   expect(fn.mock.calls).toHaveLength(1);
   expect(fn.mock.calls[0][0].method).toBe("eth_sendRawTransaction");

src/contracts/WalletV1/WalletV1.ts

@@ -4,7 +4,8 @@ import { bytesToHex, encodeFunctionData } from "viem";
 import type { PublicClient } from "../../clients/PublicClient.js";
 import { prepareDeployPart } from "../../encoding/deployPart.js";
 import { externalMessageEncode } from "../../encoding/externalMessage.js";
-import { hexToBytes, toHex } from "../../index.js";
+import { hexToBytes } from "../../encoding/fromHex.js";
+import { toHex } from "../../encoding/toHex.js";
 import type { ISigner } from "../../signers/index.js";
 import type { IDeployData } from "../../types/IDeployData.js";
 import { getShardIdFromAddress, refineAddress } from "../../utils/address.js";

src/encoding/fromBytes.ts

@@ -1,4 +1,5 @@
-import { type Hex, toHex } from "../index.js";
+import type { Hex } from "../index.js";
+import { toHex } from "./toHex.js";
 
 const decoder = new TextDecoder("utf8");
 

src/integrations/bounce.test.ts → test/integration/bounce.test.ts

@@ -1,5 +1,4 @@
 import { bytesToHex, encodeFunctionData } from "viem";
-import { testEnv } from "../../test/testEnv.js";
 import {
   Faucet,
   HttpTransport,
@@ -9,7 +8,8 @@ import {
   convertEthToWei,
   generateRandomPrivateKey,
   waitTillCompleted,
-} from "../index.js";
+} from "../../src/index.js";
+import { testEnv } from "../testEnv.js";
 const client = new PublicClient({
   transport: new HttpTransport({
     endpoint: testEnv.endpoint,

src/integrations/calling.test.ts → test/integration/calling.test.ts

@@ -1,4 +1,3 @@
-import { testEnv } from "../../test/testEnv.js";
 import {
   Faucet,
   HttpTransport,
@@ -9,7 +8,8 @@ import {
   convertEthToWei,
   generateRandomPrivateKey,
   waitTillCompleted,
-} from "../index.js";
+} from "../../src/index.js";
+import { testEnv } from "../testEnv.js";
 const client = new PublicClient({
   transport: new HttpTransport({
     endpoint: testEnv.endpoint,

src/integrations/deploy.test.ts → test/integration/deploy.test.ts

@@ -1,5 +1,4 @@
 import type { Abi } from "abitype";
-import { testEnv } from "../../test/testEnv.js";
 import {
   Faucet,
   HttpTransport,
@@ -11,7 +10,8 @@ import {
   externalDeploymentMessage,
   generateRandomPrivateKey,
   waitTillCompleted,
-} from "../index.js";
+} from "../../src/index.js";
+import { testEnv } from "../testEnv.js";
 const client = new PublicClient({
   transport: new HttpTransport({
     endpoint: testEnv.endpoint,

src/integrations/receipt.test.ts → test/integration/receipt.test.ts

@@ -1,13 +1,14 @@
-import { testEnv } from "../../test/testEnv.js";
-import { Faucet, WalletV1 } from "../contracts/index.js";
 import {
+  Faucet,
   HttpTransport,
+  LocalECDSAKeySigner,
   PublicClient,
+  WalletV1,
   convertEthToWei,
+  generateRandomPrivateKey,
   waitTillCompleted,
-} from "../index.js";
-import { LocalECDSAKeySigner } from "../signers/LocalECDSAKeySigner.js";
-import { generateRandomPrivateKey } from "../signers/privateKey.js";
+} from "../../src/index.js";
+import { testEnv } from "../../test/testEnv.js";
 
 const client = new PublicClient({
   transport: new HttpTransport({

src/integrations/tokens.test.ts → test/integration/tokens.test.ts

@@ -1,5 +1,4 @@
 import { encodeFunctionData } from "viem";
-import { testEnv } from "../../test/testEnv.js";
 import {
   Faucet,
   HttpTransport,
@@ -14,7 +13,8 @@ import {
   hexToBigInt,
   toHex,
   waitTillCompleted,
-} from "../index.js";
+} from "../../src/index.js";
+import { testEnv } from "../testEnv.js";
 const client = new PublicClient({
   transport: new HttpTransport({
     endpoint: testEnv.endpoint,

test/testEnv.ts

@@ -1,9 +1,6 @@
 const defaultRpcEndpoint = "http://127.0.0.1:8529";
-const defaultPrivateKey =
-  "41285f03e8692676bf80a98e4052a008026427a7302ca97cb06edcd60689850b";
 
 const testEnv = {
-  localPrivKey: process.env.LOCAL_PRIV_KEY ?? defaultPrivateKey,
   endpoint: process.env.RPC_ENDPOINT ?? defaultRpcEndpoint,
 } as const;
 

test/vitest.integration.config.ts

@@ -4,7 +4,7 @@ import { defineConfig } from "vitest/config";
 export default defineConfig({
   test: {
     environment: "node",
-    include: ["src/integrations/*.test.ts"],
+    include: ["test/integration/*.test.ts"],
     hookTimeout: 20_000,
     testTimeout: 40_000,
     globals: true,