From 56278d2e7e5e20e04a08452da006d86c51061825 Mon Sep 17 00:00:00 2001 From: Vitalii Bedletskyi Date: Wed, 2 Apr 2025 17:25:34 +0300 Subject: [PATCH] Fix running unit tests with node22 --- package.json | 2 +- test/forward_engineering/mappers/arguments.spec.js | 8 ++++---- test/forward_engineering/mappers/directives.spec.js | 2 +- test/forward_engineering/mappers/fields.spec.js | 10 +++++----- .../forward_engineering/mappers/objectLikeType.spec.js | 6 +++--- test/forward_engineering/mappers/rootTypes.spec.js | 4 ++-- test/forward_engineering/mappers/unions.spec.js | 6 +++--- test/reverse_engineering/mappers/arguments.spec.js | 6 +++--- test/reverse_engineering/mappers/field.spec.js | 6 +++--- test/reverse_engineering/mappers/rootTypes.spec.js | 4 ++-- .../mappers/typeDefinitions/customScalars.spec.js | 2 +- .../mappers/typeDefinitions/enum.spec.js | 2 +- .../mappers/typeDefinitions/inputType.spec.js | 4 ++-- .../mappers/typeDefinitions/interface.spec.js | 6 +++--- .../mappers/typeDefinitions/objectType.spec.js | 6 +++--- .../mappers/typeDefinitions/union.spec.js | 2 +- 16 files changed, 38 insertions(+), 38 deletions(-) diff --git a/package.json b/package.json index 32b3ea7..5e5e05c 100644 --- a/package.json +++ b/package.json @@ -109,7 +109,7 @@ }, "scripts": { "lint": "eslint . --max-warnings=0", - "test:unit": "node --experimental-test-module-mocks --test-reporter dot --test ./test", + "test:unit": "node --experimental-test-module-mocks --test-reporter dot --test ./test/**/*.spec.js", "package": "node esbuild.package.js", "postinstall": "npx simple-git-hooks", "check:types": "tsc --noEmit" diff --git a/test/forward_engineering/mappers/arguments.spec.js b/test/forward_engineering/mappers/arguments.spec.js index 24b59f1..ba1f494 100644 --- a/test/forward_engineering/mappers/arguments.spec.js +++ b/test/forward_engineering/mappers/arguments.spec.js @@ -6,22 +6,22 @@ const getArgumentDefaultValueMock = mock.fn(() => ''); const joinInlineStatementsMock = mock.fn(() => ''); const formatFEStatementMock = mock.fn(() => ''); -mock.module('../../../forward_engineering/mappers/directiveUsageStatements', { +mock.module('../../../forward_engineering/mappers/directiveUsageStatements.js', { namedExports: { getDirectivesUsageStatement: getDirectivesUsageStatementMock, }, }); -mock.module('../../../forward_engineering/mappers/argumentDefaultValue', { +mock.module('../../../forward_engineering/mappers/argumentDefaultValue.js', { namedExports: { getArgumentDefaultValue: getArgumentDefaultValueMock, }, }); -mock.module('../../../forward_engineering/helpers/feStatementJoinHelper', { +mock.module('../../../forward_engineering/helpers/feStatementJoinHelper.js', { namedExports: { joinInlineStatements: joinInlineStatementsMock, }, }); -mock.module('../../../forward_engineering/helpers/feStatementFormatHelper', { +mock.module('../../../forward_engineering/helpers/feStatementFormatHelper.js', { namedExports: { formatFEStatement: formatFEStatementMock, }, diff --git a/test/forward_engineering/mappers/directives.spec.js b/test/forward_engineering/mappers/directives.spec.js index 3e4324d..5495cf5 100644 --- a/test/forward_engineering/mappers/directives.spec.js +++ b/test/forward_engineering/mappers/directives.spec.js @@ -3,7 +3,7 @@ const { strictEqual, deepStrictEqual } = require('node:assert'); const getArgumentsMock = mock.fn(() => ({ argumentsStatement: '', argumentsWarningComment: '' })); -mock.module('../../../forward_engineering/mappers/arguments', { +mock.module('../../../forward_engineering/mappers/arguments.js', { namedExports: { getArguments: getArgumentsMock, }, diff --git a/test/forward_engineering/mappers/fields.spec.js b/test/forward_engineering/mappers/fields.spec.js index cfdb341..48dce2d 100644 --- a/test/forward_engineering/mappers/fields.spec.js +++ b/test/forward_engineering/mappers/fields.spec.js @@ -7,27 +7,27 @@ const getArgumentsMock = mock.fn(() => ({ argumentsStatement: '', argumentsWarni const getDirectivesUsageStatementMock = mock.fn(() => ''); const getFieldDefaultValueStatementMock = mock.fn(() => ''); -mock.module('../../../forward_engineering/helpers/feStatementJoinHelper', { +mock.module('../../../forward_engineering/helpers/feStatementJoinHelper.js', { namedExports: { joinInlineStatements: joinInlineStatementsMock, }, }); -mock.module('../../../forward_engineering/helpers/referenceHelper', { +mock.module('../../../forward_engineering/helpers/referenceHelper.js', { namedExports: { getDefinitionNameFromReferencePath: getDefinitionNameFromReferencePathMock, }, }); -mock.module('../../../forward_engineering/mappers/arguments', { +mock.module('../../../forward_engineering/mappers/arguments.js', { namedExports: { getArguments: getArgumentsMock, }, }); -mock.module('../../../forward_engineering/mappers/directiveUsageStatements', { +mock.module('../../../forward_engineering/mappers/directiveUsageStatements.js', { namedExports: { getDirectivesUsageStatement: getDirectivesUsageStatementMock, }, }); -mock.module('../../../forward_engineering/mappers/fieldDefaultValue', { +mock.module('../../../forward_engineering/mappers/fieldDefaultValue.js', { namedExports: { getFieldDefaultValueStatement: getFieldDefaultValueStatementMock, }, diff --git a/test/forward_engineering/mappers/objectLikeType.spec.js b/test/forward_engineering/mappers/objectLikeType.spec.js index 4d19a84..fef1a11 100644 --- a/test/forward_engineering/mappers/objectLikeType.spec.js +++ b/test/forward_engineering/mappers/objectLikeType.spec.js @@ -5,17 +5,17 @@ const joinInlineStatementsMock = mock.fn(); const getDirectivesUsageStatementMock = mock.fn(() => ''); const getImplementsInterfacesStatementMock = mock.fn(() => ''); -mock.module('../../../forward_engineering/helpers/feStatementJoinHelper', { +mock.module('../../../forward_engineering/helpers/feStatementJoinHelper.js', { namedExports: { joinInlineStatements: joinInlineStatementsMock, }, }); -mock.module('../../../forward_engineering/mappers/directiveUsageStatements', { +mock.module('../../../forward_engineering/mappers/directiveUsageStatements.js', { namedExports: { getDirectivesUsageStatement: getDirectivesUsageStatementMock, }, }); -mock.module('../../../forward_engineering/mappers/implementsInterfaces', { +mock.module('../../../forward_engineering/mappers/implementsInterfaces.js', { namedExports: { getImplementsInterfacesStatement: getImplementsInterfacesStatementMock, }, diff --git a/test/forward_engineering/mappers/rootTypes.spec.js b/test/forward_engineering/mappers/rootTypes.spec.js index 85950a3..887a9ae 100644 --- a/test/forward_engineering/mappers/rootTypes.spec.js +++ b/test/forward_engineering/mappers/rootTypes.spec.js @@ -4,12 +4,12 @@ const { strictEqual, deepStrictEqual } = require('assert'); const formatFEStatementMock = mock.fn(); const getRootTypeFieldsMock = mock.fn(); -mock.module('../../../forward_engineering/helpers/feStatementFormatHelper', { +mock.module('../../../forward_engineering/helpers/feStatementFormatHelper.js', { namedExports: { formatFEStatement: formatFEStatementMock, }, }); -mock.module('../../../forward_engineering/mappers/fields', { +mock.module('../../../forward_engineering/mappers/fields.js', { namedExports: { getRootTypeFields: getRootTypeFieldsMock, }, diff --git a/test/forward_engineering/mappers/unions.spec.js b/test/forward_engineering/mappers/unions.spec.js index 1648a38..3ee038c 100644 --- a/test/forward_engineering/mappers/unions.spec.js +++ b/test/forward_engineering/mappers/unions.spec.js @@ -5,19 +5,19 @@ const getDefinitionNameFromReferencePathMock = mock.fn(() => ''); const joinInlineStatementsMock = mock.fn(() => ''); const getDirectivesUsageStatementMock = mock.fn(() => ''); -mock.module('../../../forward_engineering/helpers/referenceHelper', { +mock.module('../../../forward_engineering/helpers/referenceHelper.js', { namedExports: { getDefinitionNameFromReferencePath: getDefinitionNameFromReferencePathMock, }, }); -mock.module('../../../forward_engineering/helpers/feStatementJoinHelper', { +mock.module('../../../forward_engineering/helpers/feStatementJoinHelper.js', { namedExports: { joinInlineStatements: joinInlineStatementsMock, }, }); -mock.module('../../../forward_engineering/mappers/directiveUsageStatements', { +mock.module('../../../forward_engineering/mappers/directiveUsageStatements.js', { namedExports: { getDirectivesUsageStatement: getDirectivesUsageStatementMock, }, diff --git a/test/reverse_engineering/mappers/arguments.spec.js b/test/reverse_engineering/mappers/arguments.spec.js index 5c6bf33..e97c892 100644 --- a/test/reverse_engineering/mappers/arguments.spec.js +++ b/test/reverse_engineering/mappers/arguments.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -24,7 +24,7 @@ const astNodeKindMock = { OBJECT_FIELD: 'ObjectField', }; -mock.module('../../../reverse_engineering/constants/graphqlAST', { +mock.module('../../../reverse_engineering/constants/graphqlAST.js', { namedExports: { astNodeKind: astNodeKindMock, }, @@ -43,7 +43,7 @@ const parseDefaultValueMock = mock.fn(value => { return 'default-value'; }); -mock.module('../../../reverse_engineering/mappers/defaultValue', { +mock.module('../../../reverse_engineering/mappers/defaultValue.js', { namedExports: { parseDefaultValue: parseDefaultValueMock, }, diff --git a/test/reverse_engineering/mappers/field.spec.js b/test/reverse_engineering/mappers/field.spec.js index 08524fa..5155bee 100644 --- a/test/reverse_engineering/mappers/field.spec.js +++ b/test/reverse_engineering/mappers/field.spec.js @@ -4,7 +4,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -25,13 +25,13 @@ const astNodeKindMock = { OBJECT_FIELD: 'ObjectField', }; -mock.module('../../../reverse_engineering/constants/graphqlAST', { +mock.module('../../../reverse_engineering/constants/graphqlAST.js', { namedExports: { astNodeKind: astNodeKindMock, }, }); -mock.module('../../../reverse_engineering/constants/types', { +mock.module('../../../reverse_engineering/constants/types.js', { namedExports: { BUILT_IN_SCALAR_LIST: ['String', 'Int', 'Float', 'Boolean', 'ID'], }, diff --git a/test/reverse_engineering/mappers/rootTypes.spec.js b/test/reverse_engineering/mappers/rootTypes.spec.js index d4de03f..c5e1855 100644 --- a/test/reverse_engineering/mappers/rootTypes.spec.js +++ b/test/reverse_engineering/mappers/rootTypes.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -11,7 +11,7 @@ mock.module('../../../reverse_engineering/mappers/directiveUsage', { // Mock getFieldsSchema const getFieldsSchemaMock = mock.fn(() => ({ properties: {}, required: [] })); -mock.module('../../../reverse_engineering/mappers/field', { +mock.module('../../../reverse_engineering/mappers/field.js', { namedExports: { getFieldsSchema: getFieldsSchemaMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/customScalars.spec.js b/test/reverse_engineering/mappers/typeDefinitions/customScalars.spec.js index c2f2dc3..632be1b 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/customScalars.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/customScalars.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/enum.spec.js b/test/reverse_engineering/mappers/typeDefinitions/enum.spec.js index bdf05e1..bda4cd0 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/enum.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/enum.spec.js @@ -8,7 +8,7 @@ const assert = require('assert'); const { astNodeKind } = require('../../../../reverse_engineering/constants/graphqlAST'); const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/inputType.spec.js b/test/reverse_engineering/mappers/typeDefinitions/inputType.spec.js index 5dfef0f..7fc611d 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/inputType.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/inputType.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -11,7 +11,7 @@ mock.module('../../../../reverse_engineering/mappers/directiveUsage', { // Mock getFieldsSchema instead of mapField const getFieldsSchemaMock = mock.fn(() => ({ properties: {}, required: [] })); -mock.module('../../../../reverse_engineering/mappers/field', { +mock.module('../../../../reverse_engineering/mappers/field.js', { namedExports: { getFieldsSchema: getFieldsSchemaMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/interface.spec.js b/test/reverse_engineering/mappers/typeDefinitions/interface.spec.js index a0db4c0..84fd356 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/interface.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/interface.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -11,14 +11,14 @@ mock.module('../../../../reverse_engineering/mappers/directiveUsage', { // Mock getFieldsSchema instead of mapField const getFieldsSchemaMock = mock.fn(() => ({ properties: {}, required: [] })); -mock.module('../../../../reverse_engineering/mappers/field', { +mock.module('../../../../reverse_engineering/mappers/field.js', { namedExports: { getFieldsSchema: getFieldsSchemaMock, }, }); const mapImplementsInterfacesMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/implementsInterfaces', { +mock.module('../../../../reverse_engineering/mappers/implementsInterfaces.js', { namedExports: { mapImplementsInterfaces: mapImplementsInterfacesMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/objectType.spec.js b/test/reverse_engineering/mappers/typeDefinitions/objectType.spec.js index 9d5aa12..f293147 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/objectType.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/objectType.spec.js @@ -3,7 +3,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, }, @@ -11,14 +11,14 @@ mock.module('../../../../reverse_engineering/mappers/directiveUsage', { // Mock getFieldsSchema instead of mapField const getFieldsSchemaMock = mock.fn(() => ({ properties: {}, required: [] })); -mock.module('../../../../reverse_engineering/mappers/field', { +mock.module('../../../../reverse_engineering/mappers/field.js', { namedExports: { getFieldsSchema: getFieldsSchemaMock, }, }); const mapImplementsInterfacesMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/implementsInterfaces', { +mock.module('../../../../reverse_engineering/mappers/implementsInterfaces.js', { namedExports: { mapImplementsInterfaces: mapImplementsInterfacesMock, }, diff --git a/test/reverse_engineering/mappers/typeDefinitions/union.spec.js b/test/reverse_engineering/mappers/typeDefinitions/union.spec.js index 1863d0c..7ee5bf6 100644 --- a/test/reverse_engineering/mappers/typeDefinitions/union.spec.js +++ b/test/reverse_engineering/mappers/typeDefinitions/union.spec.js @@ -4,7 +4,7 @@ const assert = require('assert'); // Mock dependencies const mapDirectivesUsageMock = mock.fn(() => []); -mock.module('../../../../reverse_engineering/mappers/directiveUsage', { +mock.module('../../../../reverse_engineering/mappers/directiveUsage.js', { namedExports: { mapDirectivesUsage: mapDirectivesUsageMock, },