maxmustermann/TypeScript
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity
TypeScript/tests/baselines/reference/findAllRefsForDefaultExport_reExport.baseline.jsonc

1055 lines
20 KiB
Plaintext
Raw Permalink Normal View History

Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
// === /export.ts ===
// const /*FIND ALL REFS*/[|foo|] = 1;
// export default [|foo|];
// === /re-export-dep.ts ===
// import [|fooDefault|] from "./re-export";
// === /re-export.ts ===
// export { [|default|] } from "./export";
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/export.ts",
"kind": "const",
"name": "const foo: 1",
"textSpan": {
"start": 6,
"length": 3
},
"displayParts": [
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
}
],
"contextSpan": {
"start": 0,
"length": 14
}
},
"references": [
{
"textSpan": {
"start": 6,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 0,
"length": 14
},
"isWriteAccess": true,
"isDefinition": true
},
{
"textSpan": {
"start": 30,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 15,
"length": 19
},
"isWriteAccess": false,
"isDefinition": false
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export.ts",
"kind": "alias",
"name": "(alias) const foo: 1\nexport default",
"textSpan": {
"start": 9,
"length": 7
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "export",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "default",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 35
}
},
"references": [
{
"textSpan": {
"start": 9,
"length": 7
},
"fileName": "/re-export.ts",
"contextSpan": {
"start": 0,
"length": 35
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export-dep.ts",
"kind": "alias",
"name": "(alias) const fooDefault: 1\nimport fooDefault",
"textSpan": {
"start": 7,
"length": 10
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "import",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 37
}
},
"references": [
{
"textSpan": {
"start": 7,
"length": 10
},
"fileName": "/re-export-dep.ts",
"contextSpan": {
"start": 0,
"length": 37
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
}
]
// === /export.ts ===
// const [|foo|] = 1;
// export default /*FIND ALL REFS*/[|foo|];
// === /re-export-dep.ts ===
// import [|fooDefault|] from "./re-export";
// === /re-export.ts ===
// export { [|default|] } from "./export";
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/export.ts",
"kind": "const",
"name": "const foo: 1",
"textSpan": {
"start": 6,
"length": 3
},
"displayParts": [
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
}
],
"contextSpan": {
"start": 0,
"length": 14
}
},
"references": [
{
"textSpan": {
"start": 6,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 0,
"length": 14
},
"isWriteAccess": true,
"isDefinition": true
},
{
"textSpan": {
"start": 30,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 15,
"length": 19
},
"isWriteAccess": false,
"isDefinition": false
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export.ts",
"kind": "alias",
"name": "(alias) const foo: 1\nexport default",
"textSpan": {
"start": 9,
"length": 7
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "export",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "default",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 35
}
},
"references": [
{
"textSpan": {
"start": 9,
"length": 7
},
"fileName": "/re-export.ts",
"contextSpan": {
"start": 0,
"length": 35
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export-dep.ts",
"kind": "alias",
"name": "(alias) const fooDefault: 1\nimport fooDefault",
"textSpan": {
"start": 7,
"length": 10
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "import",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 37
}
},
"references": [
{
"textSpan": {
"start": 7,
"length": 10
},
"fileName": "/re-export-dep.ts",
"contextSpan": {
"start": 0,
"length": 37
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
}
]
// === /export.ts ===
// const [|foo|] = 1;
// export default [|foo|];
// === /re-export-dep.ts ===
// import [|fooDefault|] from "./re-export";
// === /re-export.ts ===
// export { /*FIND ALL REFS*/[|default|] } from "./export";
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export.ts",
"kind": "alias",
"name": "(alias) const foo: 1\nexport default",
"textSpan": {
"start": 9,
"length": 7
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "export",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "default",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 35
}
},
"references": [
{
"textSpan": {
"start": 9,
"length": 7
},
"fileName": "/re-export.ts",
"contextSpan": {
"start": 0,
"length": 35
},
"isWriteAccess": true,
"isDefinition": true
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export-dep.ts",
"kind": "alias",
"name": "(alias) const fooDefault: 1\nimport fooDefault",
"textSpan": {
"start": 7,
"length": 10
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "import",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 37
}
},
"references": [
{
"textSpan": {
"start": 7,
"length": 10
},
"fileName": "/re-export-dep.ts",
"contextSpan": {
"start": 0,
"length": 37
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/export.ts",
"kind": "const",
"name": "const foo: 1",
"textSpan": {
"start": 6,
"length": 3
},
"displayParts": [
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
}
],
"contextSpan": {
"start": 0,
"length": 14
}
},
"references": [
{
"textSpan": {
"start": 6,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 0,
"length": 14
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
},
{
"textSpan": {
"start": 30,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 15,
"length": 19
},
"isWriteAccess": false,
"isDefinition": false
}
]
}
]
// === /export.ts ===
// const [|foo|] = 1;
// export default [|foo|];
// === /re-export-dep.ts ===
// import /*FIND ALL REFS*/[|fooDefault|] from "./re-export";
// === /re-export.ts ===
// export { [|default|] } from "./export";
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export-dep.ts",
"kind": "alias",
"name": "(alias) const fooDefault: 1\nimport fooDefault",
"textSpan": {
"start": 7,
"length": 10
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "import",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "fooDefault",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 37
}
},
"references": [
{
"textSpan": {
"start": 7,
"length": 10
},
"fileName": "/re-export-dep.ts",
"contextSpan": {
"start": 0,
"length": 37
},
"isWriteAccess": true,
"isDefinition": true
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/re-export.ts",
"kind": "alias",
"name": "(alias) const foo: 1\nexport default",
"textSpan": {
"start": 9,
"length": 7
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "alias",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
},
{
"text": "\n",
"kind": "lineBreak"
},
{
"text": "export",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "default",
"kind": "aliasName"
}
],
"contextSpan": {
"start": 0,
"length": 35
}
},
"references": [
{
"textSpan": {
"start": 9,
"length": 7
},
"fileName": "/re-export.ts",
"contextSpan": {
"start": 0,
"length": 35
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/export.ts",
"kind": "const",
"name": "const foo: 1",
"textSpan": {
"start": 6,
"length": 3
},
"displayParts": [
{
"text": "const",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "foo",
"kind": "localName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "1",
"kind": "stringLiteral"
}
],
"contextSpan": {
"start": 0,
"length": 14
}
},
"references": [
{
"textSpan": {
"start": 6,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 0,
"length": 14
},
"isWriteAccess": true,
Make isDefinition aware of declaring symbol (#45920) * Make isDefinition aware of target symbol Initial code, haven't fixed any tests yet. * Update baselines This commit includes a regression for commonjs aliases: ```js // @filename: a.js function f() { } module.exports.f = f // @filename: b.js const { f } = require('./a') f/**/ ``` Now says that `f` in b.js has 1 reference -- the alias `module.exports.f = f`. This is not correct (or not exactly correct), but correctly fixing will involve re-creating the ad-hoc commonjs alias resolution code from the checker. I don't think it's worth it for an edge case like this. * update more unit tests * Fix symbol lookup for constructors * More baselines + two fixes 1. Fix `default` support. 2. Add a secondary declaration location for commonjs assignment declarations. * Update rest of baselines * Switch a few more tests over to baselines
2021-09-22 22:43:52 +02:00
"isDefinition": false
Remove referenceGroups/noReferences from fourslash (#45988) * Remove referenceGroups/noReferences from fourslash I left most singleReferenceGroups because there were so many, and my current PR doesn't affect many of them. * update fourslash/server too * Explicitly sort keys Needed for node 10 compatibility * Revert "Explicitly sort keys" This reverts commit 1d1c58a4e3a13759d228297bceb3b7aabb891f97. It shouldn't be needed now that we're not testing with node 10
2021-09-22 01:25:52 +02:00
},
{
"textSpan": {
"start": 30,
"length": 3
},
"fileName": "/export.ts",
"contextSpan": {
"start": 15,
"length": 19
},
"isWriteAccess": false,
"isDefinition": false
}
]
}
]
Reference in a new issue Copy permalink