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

811 lines
17 KiB
Plaintext
Raw 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
// === /tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts ===
// interface A {
// readonly /*FIND ALL REFS*/[|x|]: number | string;
// }
// interface B extends A {
// readonly [|x|]: number;
// }
// const a: A = { [|x|]: 0 };
// const b: B = { [|x|]: 0 };
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) A.x: string | number",
"textSpan": {
"start": 27,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "A",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "string",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "|",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 18,
"length": 28
}
},
"references": [
{
"textSpan": {
"start": 27,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 18,
"length": 28
},
"isWriteAccess": false,
"isDefinition": true
},
{
"textSpan": {
"start": 114,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 114,
"length": 4
},
"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": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) B.x: number",
"textSpan": {
"start": 86,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "B",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 77,
"length": 19
}
},
"references": [
{
"textSpan": {
"start": 86,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 77,
"length": 19
},
"isWriteAccess": false,
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": 137,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 137,
"length": 4
},
"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
}
]
}
]
// === /tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts ===
// interface A {
// readonly [|x|]: number | string;
// }
// interface B extends A {
// readonly /*FIND ALL REFS*/[|x|]: number;
// }
// const a: A = { [|x|]: 0 };
// const b: B = { [|x|]: 0 };
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) A.x: string | number",
"textSpan": {
"start": 27,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "A",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "string",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "|",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 18,
"length": 28
}
},
"references": [
{
"textSpan": {
"start": 27,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 18,
"length": 28
},
"isWriteAccess": false,
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": 114,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 114,
"length": 4
},
"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": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) B.x: number",
"textSpan": {
"start": 86,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "B",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 77,
"length": 19
}
},
"references": [
{
"textSpan": {
"start": 86,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 77,
"length": 19
},
"isWriteAccess": false,
"isDefinition": true
},
{
"textSpan": {
"start": 137,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 137,
"length": 4
},
"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
}
]
}
]
// === /tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts ===
// interface A {
// readonly [|x|]: number | string;
// }
// interface B extends A {
// readonly [|x|]: number;
// }
// const a: A = { /*FIND ALL REFS*/[|x|]: 0 };
// const b: B = { [|x|]: 0 };
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) A.x: string | number",
"textSpan": {
"start": 27,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "A",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "string",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "|",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 18,
"length": 28
}
},
"references": [
{
"textSpan": {
"start": 27,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 18,
"length": 28
},
"isWriteAccess": false,
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": 114,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 114,
"length": 4
},
"isWriteAccess": true,
"isDefinition": true
}
]
},
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) B.x: number",
"textSpan": {
"start": 86,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "B",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 77,
"length": 19
}
},
"references": [
{
"textSpan": {
"start": 86,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 77,
"length": 19
},
"isWriteAccess": false,
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": 137,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 137,
"length": 4
},
"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
}
]
}
]
// === /tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts ===
// interface A {
// readonly [|x|]: number | string;
// }
// interface B extends A {
// readonly [|x|]: number;
// }
// const a: A = { [|x|]: 0 };
// const b: B = { /*FIND ALL REFS*/[|x|]: 0 };
[
{
"definition": {
"containerKind": "",
"containerName": "",
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) A.x: string | number",
"textSpan": {
"start": 27,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "A",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "string",
"kind": "keyword"
},
{
"text": " ",
"kind": "space"
},
{
"text": "|",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 18,
"length": 28
}
},
"references": [
{
"textSpan": {
"start": 27,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 18,
"length": 28
},
"isWriteAccess": false,
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": 114,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 114,
"length": 4
},
"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": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"kind": "property",
"name": "(property) B.x: number",
"textSpan": {
"start": 86,
"length": 1
},
"displayParts": [
{
"text": "(",
"kind": "punctuation"
},
{
"text": "property",
"kind": "text"
},
{
"text": ")",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "B",
"kind": "interfaceName"
},
{
"text": ".",
"kind": "punctuation"
},
{
"text": "x",
"kind": "propertyName"
},
{
"text": ":",
"kind": "punctuation"
},
{
"text": " ",
"kind": "space"
},
{
"text": "number",
"kind": "keyword"
}
],
"contextSpan": {
"start": 77,
"length": 19
}
},
"references": [
{
"textSpan": {
"start": 86,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 77,
"length": 19
},
"isWriteAccess": false,
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": 137,
"length": 1
},
"fileName": "/tests/cases/fourslash/findAllRefsRedeclaredPropertyInDerivedInterface.ts",
"contextSpan": {
"start": 137,
"length": 4
},
"isWriteAccess": true,
"isDefinition": true
}
]
}
]
Reference in a new issue Copy permalink