Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
5ebb3d6
Added support for static private fields accessors and methods.
dragomirtitian Feb 16, 2021
7006c09
Added error message for private identifiers used with static decorato…
dragomirtitian Feb 16, 2021
b876613
Fixed emit static bug that used private names outside of classes for …
dragomirtitian Feb 18, 2021
31a8c00
Added tests for private static fields methods and accessors.
dragomirtitian Feb 18, 2021
5a27a32
Merge remote-tracking branch 'remotes/bloomberg/es-private-methods-an…
dragomirtitian Feb 19, 2021
d2ab6fc
Merge branch 'es-private-methods-and-accessors' into es-private-stati…
dragomirtitian Feb 19, 2021
23f6c67
Accept new baseline.
dragomirtitian Feb 19, 2021
374e6f3
Improved duplicate identifier checks for static private class elements.
dragomirtitian Feb 19, 2021
62b4f2f
Added error when using initializers with private static fields when u…
dragomirtitian Feb 23, 2021
b2a692d
Fixed code review issues.
dragomirtitian Feb 24, 2021
363041e
Removed semantically wrong emit on `useDefineForClassFields:true` wit…
dragomirtitian Feb 24, 2021
64757e0
Changed emit for uninitialized private static fields.
dragomirtitian Feb 25, 2021
68a7f80
Added runtime error in helper if a static private field is accessed b…
dragomirtitian Feb 25, 2021
7e7b0b1
Fixed code review comments for private identifier static class elements.
dragomirtitian Feb 25, 2021
2834737
typo (#42899)
pushkine Mar 2, 2021
82de139
Fixed typo error. (#42816)
Mar 2, 2021
0b337f1
Update project.ts (#42900)
pushkine Mar 2, 2021
3d1c6e8
fix(42765): require should not work in ts files (#42792)
Zzzen Mar 2, 2021
c497b48
Add undefined to Symbol.valueDeclaration (#43033)
sandersn Mar 2, 2021
f1c911b
fix(43006): skip trivia in a function name (#43021)
a-tarasyuk Mar 3, 2021
5dbb110
Baseline completions and signature help (#43049)
sandersn Mar 3, 2021
0cf834c
Move fixed chunk size polling as a watch option and move it out of se…
sheetalkamat Mar 3, 2021
6cd87d0
Delete old, different-case name of quickInfoJSDocTags.baseline (#43052)
sandersn Mar 3, 2021
652199a
Added tests
armanio123 Mar 3, 2021
a8742e3
Update package-lock.json
typescript-bot Mar 3, 2021
09a5aae
Merge branch 'es-private-methods-and-accessors' into es-private-stati…
dragomirtitian Mar 3, 2021
497e47f
Fixed code review issues for private class elements.
dragomirtitian Mar 3, 2021
7751ecb
fix receiver on calls of imported and exported functions (#35877)
ajafff Mar 3, 2021
8a81a67
Merge pull request #43055 from armanio123/FixDefineForClassFields
armanio123 Mar 3, 2021
15e69ac
capture thisArg of optionalChaining in parens (#35494)
ajafff Mar 4, 2021
3e05aa6
Update package-lock.json
typescript-bot Mar 4, 2021
1f6a8d7
Do a shallow clone for Docker tests. (#42966)
DanielRosenwasser Mar 4, 2021
e234f0c
Update the DOM: March 03, 2021 (#43070)
sandersn Mar 4, 2021
38fdce9
Expose 'reservedInNestedScopes' option when creating temp and loop va…
rbuckton Mar 4, 2021
ca8d9e4
fix(39836): allow type declaration/unknown type in catch arguments in…
a-tarasyuk Mar 5, 2021
9862b03
Relax requirements on index signatures to 'any' when a type also cont…
DanielRosenwasser Mar 5, 2021
f0a72e2
Update package-lock.json
typescript-bot Mar 5, 2021
408c804
Properly instantiate inferred constraints in conditional types (#42747)
ahejlsberg Mar 5, 2021
25375a2
Unnecessary elaboration about not being assignable to type parameters…
jonhue Mar 5, 2021
612a96e
Update parseInt parameter name and jsdoc (#42756)
sangxxh Mar 5, 2021
7394efc
Update package-lock.json
typescript-bot Mar 6, 2021
ba56fca
Remove undefined from optional properties when inferring to index sig…
ahejlsberg Mar 6, 2021
f918bd9
Exclude enum/number compatibility rule from comparable relation (#42472)
ahejlsberg Mar 6, 2021
f7cd702
Update package-lock.json
typescript-bot Mar 7, 2021
4d50624
Update package-lock.json
typescript-bot Mar 8, 2021
294cc48
Fixes class shadowing when checking access to a private static class …
dragomirtitian Mar 8, 2021
027bdb3
Merge remote-tracking branch 'remotes/origin/master' into es-private-…
dragomirtitian Mar 8, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added tests for private static fields methods and accessors.
  • Loading branch information
dragomirtitian committed Feb 18, 2021
commit 31a8c00fe6e574dff301a9430299f02d5068202c
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(6,13): error TS2339: Property '#prop' does not exist on type '{}'.
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(8,14): error TS2339: Property '#prop' does not exist on type '{}'.
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(7,13): error TS2339: Property '#prop' does not exist on type '{}'.
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(9,14): error TS2339: Property '#prop' does not exist on type '{}'.
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(12,17): error TS2339: Property '#propStatic' does not exist on type '{ prototype: C; }'.
tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts(14,15): error TS2339: Property '#propStatic' does not exist on type '{ prototype: C; }'.


==== tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts (2 errors) ====
==== tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpread.ts (4 errors) ====
class C {
#prop = 1;
static #propStatic = 1;

method(other: C) {
const obj = { ...other };
Expand All @@ -15,5 +18,14 @@ tests/cases/conformance/classes/members/privateNames/privateNameAndObjectRestSpr
rest.#prop;
~~~~~
!!! error TS2339: Property '#prop' does not exist on type '{}'.

const statics = { ... C};
statics.#propStatic
~~~~~~~~~~~
!!! error TS2339: Property '#propStatic' does not exist on type '{ prototype: C; }'.
const { ...sRest } = C;
sRest.#propStatic;
~~~~~~~~~~~
!!! error TS2339: Property '#propStatic' does not exist on type '{ prototype: C; }'.
}
}
19 changes: 18 additions & 1 deletion tests/baselines/reference/privateNameAndObjectRestSpread.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,18 @@
//// [privateNameAndObjectRestSpread.ts]
class C {
#prop = 1;
static #propStatic = 1;

method(other: C) {
const obj = { ...other };
obj.#prop;
const { ...rest } = other;
rest.#prop;

const statics = { ... C};
statics.#propStatic
const { ...sRest } = C;
sRest.#propStatic;
}
}

Expand All @@ -18,6 +24,12 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
}
return privateMap.get(receiver);
};
var __classStaticPrivateFieldGet = (this && this.__classStaticPrivateFieldGet) || function (receiver, classConstructor, propertyDescriptor) {
if (receiver !== classConstructor) {
throw new TypeError("Private static access of wrong provenance");
}
return propertyDescriptor.value;
};
var __rest = (this && this.__rest) || function (s, e) {
var t = {};
for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p) && e.indexOf(p) < 0)
Expand All @@ -29,7 +41,7 @@ var __rest = (this && this.__rest) || function (s, e) {
}
return t;
};
var _C_prop;
var _C_prop, _C_propStatic;
class C {
constructor() {
_C_prop.set(this, 1);
Expand All @@ -39,6 +51,11 @@ class C {
__classPrivateFieldGet(obj, _C_prop);
const rest = __rest(other, []);
__classPrivateFieldGet(rest, _C_prop);
const statics = Object.assign({}, C);
__classStaticPrivateFieldGet(statics, C, _C_propStatic);
const sRest = __rest(C, []);
__classStaticPrivateFieldGet(sRest, C, _C_propStatic);
}
}
_C_prop = new WeakMap();
_C_propStatic = { value: 1 };
33 changes: 25 additions & 8 deletions tests/baselines/reference/privateNameAndObjectRestSpread.symbols
Original file line number Diff line number Diff line change
Expand Up @@ -5,23 +5,40 @@ class C {
#prop = 1;
>#prop : Symbol(C.#prop, Decl(privateNameAndObjectRestSpread.ts, 0, 9))

static #propStatic = 1;
>#propStatic : Symbol(C.#propStatic, Decl(privateNameAndObjectRestSpread.ts, 1, 14))

method(other: C) {
>method : Symbol(C.method, Decl(privateNameAndObjectRestSpread.ts, 1, 14))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 3, 11))
>method : Symbol(C.method, Decl(privateNameAndObjectRestSpread.ts, 2, 27))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 4, 11))
>C : Symbol(C, Decl(privateNameAndObjectRestSpread.ts, 0, 0))

const obj = { ...other };
>obj : Symbol(obj, Decl(privateNameAndObjectRestSpread.ts, 4, 13))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 3, 11))
>obj : Symbol(obj, Decl(privateNameAndObjectRestSpread.ts, 5, 13))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 4, 11))

obj.#prop;
>obj : Symbol(obj, Decl(privateNameAndObjectRestSpread.ts, 4, 13))
>obj : Symbol(obj, Decl(privateNameAndObjectRestSpread.ts, 5, 13))

const { ...rest } = other;
>rest : Symbol(rest, Decl(privateNameAndObjectRestSpread.ts, 6, 15))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 3, 11))
>rest : Symbol(rest, Decl(privateNameAndObjectRestSpread.ts, 7, 15))
>other : Symbol(other, Decl(privateNameAndObjectRestSpread.ts, 4, 11))

rest.#prop;
>rest : Symbol(rest, Decl(privateNameAndObjectRestSpread.ts, 6, 15))
>rest : Symbol(rest, Decl(privateNameAndObjectRestSpread.ts, 7, 15))

const statics = { ... C};
>statics : Symbol(statics, Decl(privateNameAndObjectRestSpread.ts, 10, 13))
>C : Symbol(C, Decl(privateNameAndObjectRestSpread.ts, 0, 0))

statics.#propStatic
>statics : Symbol(statics, Decl(privateNameAndObjectRestSpread.ts, 10, 13))

const { ...sRest } = C;
>sRest : Symbol(sRest, Decl(privateNameAndObjectRestSpread.ts, 12, 15))
>C : Symbol(C, Decl(privateNameAndObjectRestSpread.ts, 0, 0))

sRest.#propStatic;
>sRest : Symbol(sRest, Decl(privateNameAndObjectRestSpread.ts, 12, 15))
}
}
21 changes: 21 additions & 0 deletions tests/baselines/reference/privateNameAndObjectRestSpread.types
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ class C {

#prop = 1;
>#prop : number
>1 : 1

static #propStatic = 1;
>#propStatic : number
>1 : 1

method(other: C) {
Expand All @@ -26,5 +30,22 @@ class C {
rest.#prop;
>rest.#prop : any
>rest : {}

const statics = { ... C};
>statics : { prototype: C; }
>{ ... C} : { prototype: C; }
>C : typeof C

statics.#propStatic
>statics.#propStatic : any
>statics : { prototype: C; }

const { ...sRest } = C;
>sRest : { prototype: C; }
>C : typeof C

sRest.#propStatic;
>sRest.#propStatic : any
>sRest : { prototype: C; }
}
}
3 changes: 1 addition & 2 deletions tests/baselines/reference/privateNameFieldsESNext.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,11 @@ class C {
this.#a = "hello";
console.log(this.#b);
}
static #m;
static #m = "test";
static #x;
static test() {
console.log(this.#m);
console.log(this.#x = "test");
}
#something;
}
C.#m = "test";
20 changes: 20 additions & 0 deletions tests/baselines/reference/privateNameStaticAccessors.errors.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessors.ts(9,12): error TS2540: Cannot assign to '#roProp' because it is a read-only property.


==== tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessors.ts (1 errors) ====
class A1 {
static get #prop() { return ""; }
static set #prop(param: string) { }

static get #roProp() { return ""; }

constructor(name: string) {
A1.#prop = "";
A1.#roProp = ""; // Error
~~~~~~~
!!! error TS2540: Cannot assign to '#roProp' because it is a read-only property.
console.log(A1.#prop);
console.log(A1.#roProp);
}
}

44 changes: 44 additions & 0 deletions tests/baselines/reference/privateNameStaticAccessors.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
//// [privateNameStaticAccessors.ts]
class A1 {
static get #prop() { return ""; }
Comment thread
dragomirtitian marked this conversation as resolved.
static set #prop(param: string) { }

static get #roProp() { return ""; }

constructor(name: string) {
A1.#prop = "";
A1.#roProp = ""; // Error
console.log(A1.#prop);
console.log(A1.#roProp);
}
}


//// [privateNameStaticAccessors.js]
"use strict";
var __classStaticPrivateAccessorSet = (this && this.__classStaticPrivateAccessorSet) || function (receiver, classConstructor, fn, value) {
if (receiver !== classConstructor) {
throw new TypeError("Private static access of wrong provenance");
}
fn.call(receiver, value);
return value;
};
var __classStaticPrivateReadonly = (this && this.__classStaticPrivateReadonly) || function () {
throw new TypeError("Private static element is not writable");
};
var __classStaticPrivateAccessorGet = (this && this.__classStaticPrivateAccessorGet) || function (receiver, classConstructor, fn) {
if (receiver !== classConstructor) {
throw new TypeError("Private static access of wrong provenance");
}
return fn.call(receiver);
};
var _A1_prop_get, _A1_prop_set, _A1_roProp_get;
class A1 {
constructor(name) {
__classStaticPrivateAccessorSet(A1, A1, _A1_prop_set, "");
__classStaticPrivateReadonly(A1, ""); // Error
console.log(__classStaticPrivateAccessorGet(A1, A1, _A1_prop_get));
console.log(__classStaticPrivateAccessorGet(A1, A1, _A1_roProp_get));
}
}
_A1_prop_get = function _A1_prop_get() { return ""; }, _A1_prop_set = function _A1_prop_set(param) { }, _A1_roProp_get = function _A1_roProp_get() { return ""; };
41 changes: 41 additions & 0 deletions tests/baselines/reference/privateNameStaticAccessors.symbols
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
=== tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessors.ts ===
class A1 {
>A1 : Symbol(A1, Decl(privateNameStaticAccessors.ts, 0, 0))

static get #prop() { return ""; }
>#prop : Symbol(A1.#prop, Decl(privateNameStaticAccessors.ts, 0, 10), Decl(privateNameStaticAccessors.ts, 1, 37))

static set #prop(param: string) { }
>#prop : Symbol(A1.#prop, Decl(privateNameStaticAccessors.ts, 0, 10), Decl(privateNameStaticAccessors.ts, 1, 37))
>param : Symbol(param, Decl(privateNameStaticAccessors.ts, 2, 21))

static get #roProp() { return ""; }
>#roProp : Symbol(A1.#roProp, Decl(privateNameStaticAccessors.ts, 2, 39))

constructor(name: string) {
>name : Symbol(name, Decl(privateNameStaticAccessors.ts, 6, 16))

A1.#prop = "";
>A1.#prop : Symbol(A1.#prop, Decl(privateNameStaticAccessors.ts, 0, 10), Decl(privateNameStaticAccessors.ts, 1, 37))
>A1 : Symbol(A1, Decl(privateNameStaticAccessors.ts, 0, 0))

A1.#roProp = ""; // Error
>A1.#roProp : Symbol(A1.#roProp, Decl(privateNameStaticAccessors.ts, 2, 39))
>A1 : Symbol(A1, Decl(privateNameStaticAccessors.ts, 0, 0))

console.log(A1.#prop);
>console.log : Symbol(Console.log, Decl(lib.dom.d.ts, --, --))
>console : Symbol(console, Decl(lib.dom.d.ts, --, --))
>log : Symbol(Console.log, Decl(lib.dom.d.ts, --, --))
>A1.#prop : Symbol(A1.#prop, Decl(privateNameStaticAccessors.ts, 0, 10), Decl(privateNameStaticAccessors.ts, 1, 37))
>A1 : Symbol(A1, Decl(privateNameStaticAccessors.ts, 0, 0))

console.log(A1.#roProp);
>console.log : Symbol(Console.log, Decl(lib.dom.d.ts, --, --))
>console : Symbol(console, Decl(lib.dom.d.ts, --, --))
>log : Symbol(Console.log, Decl(lib.dom.d.ts, --, --))
>A1.#roProp : Symbol(A1.#roProp, Decl(privateNameStaticAccessors.ts, 2, 39))
>A1 : Symbol(A1, Decl(privateNameStaticAccessors.ts, 0, 0))
}
}

49 changes: 49 additions & 0 deletions tests/baselines/reference/privateNameStaticAccessors.types
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
=== tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessors.ts ===
class A1 {
>A1 : A1

static get #prop() { return ""; }
>#prop : string
>"" : ""

static set #prop(param: string) { }
>#prop : string
>param : string

static get #roProp() { return ""; }
>#roProp : string
>"" : ""

constructor(name: string) {
>name : string

A1.#prop = "";
>A1.#prop = "" : ""
>A1.#prop : string
>A1 : typeof A1
>"" : ""

A1.#roProp = ""; // Error
>A1.#roProp = "" : ""
>A1.#roProp : any
>A1 : typeof A1
>"" : ""

console.log(A1.#prop);
>console.log(A1.#prop) : void
>console.log : (...data: any[]) => void
>console : Console
>log : (...data: any[]) => void
>A1.#prop : string
>A1 : typeof A1

console.log(A1.#roProp);
>console.log(A1.#roProp) : void
>console.log : (...data: any[]) => void
>console : Console
>log : (...data: any[]) => void
>A1.#roProp : string
>A1 : typeof A1
}
}

Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessorsAccess.ts(16,4): error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.
tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessorsAccess.ts(19,8): error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.
tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessorsAccess.ts(24,12): error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.


==== tests/cases/conformance/classes/members/privateNames/privateNameStaticAccessorsAccess.ts (3 errors) ====
export {}
class A2 {
static get #prop() { return ""; }
static set #prop(param: string) { }

constructor() {
console.log(A2.#prop);
let a: typeof A2 = A2;
a.#prop;
function foo (){
a.#prop;
}
}
}

A2.#prop; // Error
~~~~~
!!! error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.

function foo (){
A2.#prop; // Error
~~~~~
!!! error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.
}

class B2 {
m() {
A2.#prop;
~~~~~
!!! error TS18013: Property '#prop' is not accessible outside class 'A2' because it has a private identifier.
}
}

Loading