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

Update JS dependencies for api augment and e2e tests #1965

Merged
merged 5 commits into from
May 9, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
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
24 changes: 0 additions & 24 deletions e2e/.eslintrc.cjs

This file was deleted.

2 changes: 1 addition & 1 deletion e2e/.node-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20.11.1
20.12.2
2 changes: 1 addition & 1 deletion e2e/.tool-versions
Original file line number Diff line number Diff line change
@@ -1 +1 @@
nodejs 20.11.1
nodejs 20.12.2
35 changes: 35 additions & 0 deletions e2e/eslint.config.mjs
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Switching to the new flat config eslint style

Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
// @ts-check

import eslint from '@eslint/js';
import tseslint from 'typescript-eslint';
import mochaPlugin from 'eslint-plugin-mocha';

export default tseslint.config(
eslint.configs.recommended,
...tseslint.configs.strict,
...tseslint.configs.stylistic,
mochaPlugin.configs.flat.recommended,
{
rules: {
'@typescript-eslint/no-empty-interface': 'off',
'@typescript-eslint/no-unused-vars': 'off',
'@typescript-eslint/no-explicit-any': 'off',
'@typescript-eslint/no-non-null-assertion': 'off',
'@typescript-eslint/no-inferrable-types': 'off',
'@typescript-eslint/no-extraneous-class': 'off',
semi: ['error', 'always'],
'mocha/no-setup-in-describe': 'off',
'no-use-before-define': 'off',
'no-unused-vars': 'off',
'no-var': 'error',
'id-length': [
'error',
{
exceptionPatterns: ['[i-k]', 'e', 'c', 'x', 'y', 'r', 's', 'v', 'f', '_'],
properties: 'never',
},
],
'allow-namespace': 'off',
},
}
);
6 changes: 3 additions & 3 deletions e2e/load-tests/signatureRegistry.test.ts
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

All automatic formatting and linting

Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ describe('MSA Initializer Load Tests', function () {
let blockNumber = await getBlockNumber();
console.log('Starting Block Number:', blockNumber);

let signatureCallTests: Array<KeyringPair> = [];
let signatureCallTests: KeyringPair[] = [];

// Make 50k signatures via 50k/2 loops
const loopCount = 25_000;
Expand Down Expand Up @@ -89,7 +89,7 @@ async function generateMsas(count: number = 1): Promise<GeneratedMsa[]> {
const fundingSource = getFundingSource('load-signature-registry');

// Create and fund the control keys
const controlKeyPromises: Array<Promise<KeyringPair>> = [];
const controlKeyPromises: Promise<KeyringPair>[] = [];
let devAccountNonce = await getNonce(fundingSource);
const ed = await getExistentialDeposit();
for (let i = 0; i < count; i++) {
Expand All @@ -100,7 +100,7 @@ async function generateMsas(count: number = 1): Promise<GeneratedMsa[]> {
const controlKeys = await Promise.all(controlKeyPromises);

// Create the msas
const msaPromises: Array<Promise<GeneratedMsa>> = [];
const msaPromises: Promise<GeneratedMsa>[] = [];
for (let i = 0; i < count; i++) {
msaPromises.push(
createMsa(controlKeys[i]).then((id) => ({
Expand Down
Loading
Loading