2f0c8b2bea
* wip: add types * wip * Add cases * Add some case * Add more check * accept baseline * add abstract abd declare method * add override in declaration * accept baseline * add property override * Fix decalre modifier * update baseline * Add more cases * make lint happy * make lint happy * Update description * Add codefix * simplify code * accept baseline * Update desc * Accept baseline * Add override completions * filter out implements field in override context * fix tests * Add parameter property check * Accept baseline * acept baseline * Add parameter property to declaration code action * Add quickfix for override parameter property * fix code style * Add override with interface tests * Add more cases about modifier position * rename flag * rename flags * Added tests. * Accepted baselines. * Always issue errors for unnecessary 'override' modifiers. * Accepted baselines. * Override perf (#4) * try cache check result * pre check for override * Do not issue error if implement abstract * Add abstract-spec check * Avoid override dead lock * Add more case * Add codefix for new error * Fix error message * Add jsdoc override tag (#6) * Override jsdoc tag (#7) * accept baseline * Disallow codefix in js * update baseline * Omit override in d.ts * Add more case in js * Accept baseline * fix override js test * fix crlf * Revert merge conflict changes * Accept baseline * Avoid space * Fix CR issues * Accept baseline * Fix typo and add more check * Fix error name Co-authored-by: Daniel Rosenwasser <Daniel.Rosenwasser@microsoft.com>
39 lines
1.7 KiB
Plaintext
39 lines
1.7 KiB
Plaintext
tests/cases/conformance/override/override4.ts(11,5): error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
tests/cases/conformance/override/override4.ts(13,5): error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
tests/cases/conformance/override/override4.ts(17,5): error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
tests/cases/conformance/override/override4.ts(23,5): error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
|
|
|
|
==== tests/cases/conformance/override/override4.ts (4 errors) ====
|
|
class B {
|
|
p1: number = 1;
|
|
p2: number = 1;
|
|
p3: () => void;
|
|
p4: () => void;
|
|
foo (v: string) {}
|
|
fooo (v: string) {}
|
|
}
|
|
|
|
class D extends B {
|
|
p1: number = 2;
|
|
~~
|
|
!!! error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
override p2: number = 3;
|
|
p3: () => void;
|
|
~~
|
|
!!! error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
override p4: () => void;
|
|
override foo (v: string) {}
|
|
|
|
fooo (v: string) {}
|
|
~~~~
|
|
!!! error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
|
|
}
|
|
|
|
class DD extends B {
|
|
override foo: () => void
|
|
fooo: () => void;
|
|
~~~~
|
|
!!! error TS4114: This member must have an 'override' modifier because it overrides a member in the base class 'B'.
|
|
} |