Skip to content

Commit 8cfab2c

Browse files
committed
introduce simple user provided caches for parse/validate
BREAKING CHANGE: parse/validate become potentially async when provided with a cache with an async getter, Introduces parseSync/validateSync to throw when an async value is returned. Updates all non-cache tests to use parseSync/validateSync.
1 parent cca3f98 commit 8cfab2c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

50 files changed

+951
-55
lines changed

src/__tests__/starWarsValidation-test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import { expect } from 'chai';
22
import { describe, it } from 'mocha';
33

4-
import { parse } from '../language/parser.js';
4+
import { parseSync as parse } from '../language/parser.js';
55
import { Source } from '../language/source.js';
66

7-
import { validate } from '../validation/validate.js';
7+
import { validateSync as validate } from '../validation/validate.js';
88

99
import { StarWarsSchema } from './starWarsSchema.js';
1010

src/error/__tests__/GraphQLError-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { describe, it } from 'mocha';
44
import { dedent } from '../../__testUtils__/dedent.js';
55

66
import { Kind } from '../../language/kinds.js';
7-
import { parse } from '../../language/parser.js';
7+
import { parseSync as parse } from '../../language/parser.js';
88
import { Source } from '../../language/source.js';
99

1010
import { GraphQLError } from '../GraphQLError.js';

src/execution/__tests__/abort-signal-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { expectJSON } from '../../__testUtils__/expectJSON.js';
55
import { resolveOnNextTick } from '../../__testUtils__/resolveOnNextTick.js';
66

77
import type { DocumentNode } from '../../language/ast.js';
8-
import { parse } from '../../language/parser.js';
8+
import { parseSync as parse } from '../../language/parser.js';
99

1010
import { buildSchema } from '../../utilities/buildASTSchema.js';
1111

src/execution/__tests__/abstract-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { describe, it } from 'mocha';
33

44
import { expectJSON } from '../../__testUtils__/expectJSON.js';
55

6-
import { parse } from '../../language/parser.js';
6+
import { parseSync as parse } from '../../language/parser.js';
77

88
import {
99
assertInterfaceType,

src/execution/__tests__/defer-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { resolveOnNextTick } from '../../__testUtils__/resolveOnNextTick.js';
88
import { promiseWithResolvers } from '../../jsutils/promiseWithResolvers.js';
99

1010
import type { DocumentNode } from '../../language/ast.js';
11-
import { parse } from '../../language/parser.js';
11+
import { parseSync as parse } from '../../language/parser.js';
1212

1313
import {
1414
GraphQLList,

src/execution/__tests__/directives-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { expect } from 'chai';
22
import { describe, it } from 'mocha';
33

4-
import { parse } from '../../language/parser.js';
4+
import { parseSync as parse } from '../../language/parser.js';
55

66
import { GraphQLObjectType } from '../../type/definition.js';
77
import { GraphQLString } from '../../type/scalars.js';

src/execution/__tests__/executor-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import { resolveOnNextTick } from '../../__testUtils__/resolveOnNextTick.js';
77
import { inspect } from '../../jsutils/inspect.js';
88

99
import { Kind } from '../../language/kinds.js';
10-
import { parse } from '../../language/parser.js';
10+
import { parseSync as parse } from '../../language/parser.js';
1111

1212
import {
1313
GraphQLInterfaceType,

src/execution/__tests__/lists-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { expectJSON } from '../../__testUtils__/expectJSON.js';
55

66
import type { PromiseOrValue } from '../../jsutils/PromiseOrValue.js';
77

8-
import { parse } from '../../language/parser.js';
8+
import { parseSync as parse } from '../../language/parser.js';
99

1010
import type { GraphQLFieldResolver } from '../../type/definition.js';
1111
import {

src/execution/__tests__/mutations-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { describe, it } from 'mocha';
44
import { expectJSON } from '../../__testUtils__/expectJSON.js';
55
import { resolveOnNextTick } from '../../__testUtils__/resolveOnNextTick.js';
66

7-
import { parse } from '../../language/parser.js';
7+
import { parseSync as parse } from '../../language/parser.js';
88

99
import { GraphQLObjectType } from '../../type/definition.js';
1010
import { GraphQLInt } from '../../type/scalars.js';

src/execution/__tests__/nonnull-test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { resolveOnNextTick } from '../../__testUtils__/resolveOnNextTick.js';
66

77
import type { PromiseOrValue } from '../../jsutils/PromiseOrValue.js';
88

9-
import { parse } from '../../language/parser.js';
9+
import { parseSync as parse } from '../../language/parser.js';
1010

1111
import { GraphQLNonNull, GraphQLObjectType } from '../../type/definition.js';
1212
import { GraphQLString } from '../../type/scalars.js';

0 commit comments

Comments
 (0)