Refactor/polymorphic tests

[Keyrxng]

Resolves #16

Split from #6

Required by #14

[github-actions]

Lines	Statements	Branches	Functions
	Unknown% (0/0)	Unknown% (0/0)	Unknown% (0/0)

JUnit

Tests	Skipped	Failures	Errors	Time
0	0 💤	0 ❌	0 🔥	3.094s ⏱️

Coverage Report (0%)

File	% Stmts	% Branch	% Funcs	% Lines	Uncovered Line #s
All files	0	0	0	0

[Keyrxng]

the jest workflow shouldn't run yarn build only yarn test which builds for testing

[Keyrxng]

CI is failing here because it's using workflows from development I'm sure

Testing it myself my CI keeps throwing this below but as you can see above all test cases are passing, I just need to figure out why my CI seems to be busted. Lots of related gh issues for it, tried a fair few solutions to no avail, yet.

[rndquu]

@Keyrxng

1. Pls update readme
2. Check this code:

import { HandlerConstructorConfig } from "./dist";
import { RPCHandler } from "./dist";

const config: HandlerConstructorConfig = {
    networkId: 1,
    autoStorage: false,
  };

async function main() {
    const handler = new RPCHandler(config);
    const provider = await handler.getFastestRpcProvider();
    console.log(provider.connection);
    console.log('success');
}

main();

The code above produces this error:

(node:14881) TimeoutOverflowWarning: 1.7976931348623157e+308 does not fit into a 32-bit signed integer.
Timeout duration was set to 1.
[RPCService] Failed to find fastest RPC
/Users/rndquu/Public/projects/backend/rpc-handler/dist/index.js:17021
      throw new Error("Failed to find fastest RPC");
            ^

Error: Failed to find fastest RPC
    at _RPCHandler.testRpcPerformance (/Users/rndquu/Public/projects/backend/rpc-handler/dist/index.js:17021:13)
    at async _RPCHandler.getFastestRpcProvider (/Users/rndquu/Public/projects/backend/rpc-handler/dist/index.js:17001:24)
    at main (/Users/rndquu/Public/projects/backend/rpc-handler/my.ts:11:22)

Pls fix

[Keyrxng]

You should only need to fill in the required config props and your golden. I fresh installed, tried both build envs with your script and with tests/script-test.ts

const config: HandlerConstructorConfig = {
  networkId: 1,
  rpcTimeout: 1500,
  autoStorage: false,
  cacheRefreshCycles: 10,
  networkName: null,
  networkRpcs: null,
  runtimeRpcs: null,
};

[rndquu]

@Keyrxng Why did you remove yarn.lock completely?

[rndquu]

@Keyrxng Could you merge Keyrxng#3 ?

[Keyrxng]

because of the CI errors I was having seeing as the fresh lock didn't change anything and typically no lock is committed.

Think it's best to add it back?

[rndquu]

@Keyrxng Could you describe the main code changes of this exact PR and provide a time estimate for it?

[rndquu]

@Keyrxng Could you describe the main code changes of this exact PR and provide a time estimate for it?

P.S. Tests seems to be passing + the package works fine

[Keyrxng]

• improve package exports reducing import path down to @packageName
• fix latency naming convention
• add configurable and default rpcTimeout
• refactor tests
• create test specific esbuild env

time 2-4 hrs

@rndquu

The task I'd say is making the package polymorphic, the latency and timeout fix happening over the course of review. Tests needed refactored because of how the way the build output was changed in improving exports.

[rndquu]

• improve package exports reducing import path down to @packageName
• fix latency naming convention
• add configurable and default rpcTimeout
• refactor tests
• create test specific esbuild env

time 2-4 hrs

@rndquu

The task I'd say is making the package polymorphic, the latency and timeout fix happening over the course of review. Tests needed refactored because of how the way the build output was changed in improving exports.

Pls self assign #16