Skip to content

Commit

Permalink
refactor baseline tests for import/export name collision
Browse files Browse the repository at this point in the history
  • Loading branch information
collin5 committed May 3, 2019
1 parent 6915c19 commit 1d07dde
Show file tree
Hide file tree
Showing 10 changed files with 26 additions and 10 deletions.
13 changes: 13 additions & 0 deletions tests/baselines/reference/mergeWithImportedNamespace.errors.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
tests/cases/compiler/f2.ts(1,9): error TS2440: Import declaration conflicts with local declaration of 'N'.


==== tests/cases/compiler/f1.ts (0 errors) ====
export namespace N { export var x = 1; }

==== tests/cases/compiler/f2.ts (1 errors) ====
import {N} from "./f1";
~
!!! error TS2440: Import declaration conflicts with local declaration of 'N'.
export namespace N {
export interface I {x: any}
}
1 change: 0 additions & 1 deletion tests/baselines/reference/mergeWithImportedNamespace.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ export namespace N { export var x = 1; }

//// [f2.ts]
import {N} from "./f1";
// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export namespace N {
export interface I {x: any}
}
Expand Down
5 changes: 2 additions & 3 deletions tests/baselines/reference/mergeWithImportedNamespace.symbols
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,10 @@ export namespace N { export var x = 1; }
import {N} from "./f1";
>N : Symbol(N, Decl(f2.ts, 0, 8), Decl(f2.ts, 0, 23))

// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export namespace N {
>N : Symbol(N, Decl(f2.ts, 0, 23))

export interface I {x: any}
>I : Symbol(I, Decl(f2.ts, 2, 20))
>x : Symbol(I.x, Decl(f2.ts, 3, 24))
>I : Symbol(I, Decl(f2.ts, 1, 20))
>x : Symbol(I.x, Decl(f2.ts, 2, 24))
}
1 change: 0 additions & 1 deletion tests/baselines/reference/mergeWithImportedNamespace.types
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ export namespace N { export var x = 1; }
import {N} from "./f1";
>N : typeof N

// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export namespace N {
export interface I {x: any}
>x : any
Expand Down
11 changes: 11 additions & 0 deletions tests/baselines/reference/mergeWithImportedType.errors.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
tests/cases/compiler/f2.ts(1,9): error TS2440: Import declaration conflicts with local declaration of 'E'.


==== tests/cases/compiler/f1.ts (0 errors) ====
export enum E {X}

==== tests/cases/compiler/f2.ts (1 errors) ====
import {E} from "./f1";
~
!!! error TS2440: Import declaration conflicts with local declaration of 'E'.
export type E = E;
1 change: 0 additions & 1 deletion tests/baselines/reference/mergeWithImportedType.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ export enum E {X}

//// [f2.ts]
import {E} from "./f1";
// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export type E = E;

//// [f1.js]
Expand Down
1 change: 0 additions & 1 deletion tests/baselines/reference/mergeWithImportedType.symbols
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ export enum E {X}
import {E} from "./f1";
>E : Symbol(E, Decl(f2.ts, 0, 8), Decl(f2.ts, 0, 23))

// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export type E = E;
>E : Symbol(E, Decl(f2.ts, 0, 23))
>E : Symbol(E, Decl(f2.ts, 0, 8), Decl(f2.ts, 0, 23))
Expand Down
1 change: 0 additions & 1 deletion tests/baselines/reference/mergeWithImportedType.types
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ export enum E {X}
import {E} from "./f1";
>E : typeof E

// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export type E = E;
>E : E

1 change: 0 additions & 1 deletion tests/cases/compiler/mergeWithImportedNamespace.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ export namespace N { export var x = 1; }

// @filename: f2.ts
import {N} from "./f1";
// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export namespace N {
export interface I {x: any}
}
1 change: 0 additions & 1 deletion tests/cases/compiler/mergeWithImportedType.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,4 @@ export enum E {X}

// @filename: f2.ts
import {E} from "./f1";
// partial revert of https://github.com/Microsoft/TypeScript/pull/7583 to prevent breaking changes
export type E = E;

0 comments on commit 1d07dde

Please sign in to comment.