An open source Valkey client library that supports Valkey and Redis open source 6.2, 7.0 and 7.2. Valkey GLIDE is designed for reliability, optimized performance, and high-availability, for Valkey and Redis OSS based applications. GLIDE is a multi language client library, written in Rust with programming language bindings, such as Java and Python
The main reason of this PR is to fix parsing issue with modules tests,
This pull request includes several changes to the node and utils directories, focusing on improving test configurations, updating dependencies, and refactoring code for better maintainability. The most important changes are grouped into test configuration updates, dependency updates, and code refactoring.
Test Configuration Updates:
node/jest.config.js: Updated the setupFilesAfterEnv configuration to use setup.ts instead of setup.js.
node/package.json: Modified the test-modules script to run tests in-band for better isolation.
node/tests/setup.ts: Introduced a new setup file to initialize global variables and configure the logger.
node/tests/tsconfig.json: Included setup.ts in the TypeScript configuration to ensure it is compiled.
Dependency Updates:
node/package.json: Added ts-node as a new dependency and updated the uuid package version.
utils/package.json: Updated the @types/node and prettier dependencies to their latest versions.
Code Refactoring:
Removed the parseCommandLineArgs function and replaced its usage with global variables in multiple test files (GlideClient.test.ts, GlideClusterClient.test.ts, PubSub.test.ts, ScanTest.test.ts, ServerModules.test.ts). [1][2][3][4][5][6][7][8][9][10][11][12][13][14][15]
Refactored TestUtils.ts to improve code readability and consistency, including formatting changes. [1][2][3][4][5][6]
The main reason of this PR is to fix parsing issue with modules tests,
This pull request includes several changes to the
node
andutils
directories, focusing on improving test configurations, updating dependencies, and refactoring code for better maintainability. The most important changes are grouped into test configuration updates, dependency updates, and code refactoring.Test Configuration Updates:
node/jest.config.js
: Updated thesetupFilesAfterEnv
configuration to usesetup.ts
instead ofsetup.js
.node/package.json
: Modified thetest-modules
script to run tests in-band for better isolation.node/tests/setup.ts
: Introduced a new setup file to initialize global variables and configure the logger.node/tests/tsconfig.json
: Includedsetup.ts
in the TypeScript configuration to ensure it is compiled.Dependency Updates:
node/package.json
: Addedts-node
as a new dependency and updated theuuid
package version.utils/package.json
: Updated the@types/node
andprettier
dependencies to their latest versions.Code Refactoring:
parseCommandLineArgs
function and replaced its usage with global variables in multiple test files (GlideClient.test.ts
,GlideClusterClient.test.ts
,PubSub.test.ts
,ScanTest.test.ts
,ServerModules.test.ts
). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]TestUtils.ts
to improve code readability and consistency, including formatting changes. [1] [2] [3] [4] [5] [6]release-candidate-testing/node/index.js
formatting. [1] [2] [3]Issue link
This Pull Request is linked to issue #2624
Checklist
Before submitting the PR make sure the following are checked: