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.

Sign up for GitHub

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

[LIT-2333] Add e2e test for initializing multiple clients #331

Merged
merged 3 commits into from
Feb 2, 2024
Merged

[LIT-2333] Add e2e test for initializing multiple clients #331

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
14f35f0
Remove globals for loggers and client
cairomassimo Jan 29, 2024
ef931a2
Test e2e parallel connections to multiple networks
cairomassimo Jan 29, 2024
06f166c
Revert "Remove globals for loggers and client"
cairomassimo Jan 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions e2e-nodejs/group-connection/test-connection-multi-network.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
import path from 'path';
import { success, fail, testThis, log } from '../../tools/scripts/utils.mjs';
import { LitNodeClient } from '@lit-protocol/lit-node-client';

export async function main() {
const networks = ['cayenne', 'habanero', 'manzano'];

for (const network of networks) {
// ==================== Test Logic ====================
const client = new LitNodeClient({
litNetwork: network,
debug: globalThis.LitCI.debug,
});
log(`connecting to ${network.toUpperCase()}...`);
await client.connect();

// ==================== Post-Validation ====================
if (!client.ready) {
return fail('client not ready');
}
if (client.config.litNetwork !== network) {
Copy link

Choose a reason for hiding this comment

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

Could we add a check on globalThis to assert on the new global namespacing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't implement the global namespacing, I think it's a risky approach (see also #331 (comment)). There could be a use case for global namespaces I'm not aware of yet: in that case please let me know.

return fail(`client not connected to ${network}`);
}
}

// ==================== Success ====================
return success(`Connected to ${networks.join(', ')}`);
}

await testThis({ name: path.basename(import.meta.url), fn: main });
Loading