Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/tinny standalone #722

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Feat/tinny standalone #722

wants to merge 20 commits into from

Conversation

Ansonhkg
Copy link
Collaborator

@Ansonhkg Ansonhkg commented Nov 19, 2024

WHAT

This PR takes the E2E tests I’ve already built for the SDK and turns them into a reusable package. Internal members asked for this so they can use it to test custom networks and tie it into observability projects to keep an eye on service health. This makes the tests more flexible and useful for everyone.

COMMANDS

To build

yarn build:tinny

To Publish (Have to manually update package version)

cd local-tests && npm publish 

Package usage

Overview

See this repo for more details

import {
  tinnyTests,
  runTestsParallel,
  TinnyEnvironment,
  getEoaSessionSigs
} from "@lit-protocol/tinny";

const { testEthAuthSigToEncryptDecryptString } = tinnyTests;

const devEnv = new TinnyEnvironment({
  NETWORK: 'datil-dev'
});

(async () => {
  await devEnv.init();

  const testConfig = {
    tests: {
      testEthAuthSigToEncryptDecryptString,
    },
    devEnv,
  }

  const exitRes = await runTestsParallel(testConfig);
})();

@Ansonhkg Ansonhkg requested a review from hwrdtm November 19, 2024 22:41
},
};

export const tinnyTests = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we import this huge json into test.ts:testConfig.tests to avoid the huge imports block there?

await bundle();
console.log(`[build.mjs] 🚀 Build time: ${Date.now() - start}ms`);
} catch (error) {
console.error(`[build.mjs] ❌ Build failed:`, error);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will hide the error. We can add a process.exit(1) to notify caller

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants