-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make tests spec compliant by avoiding duplicate declarations in input files #9522
Conversation
@@ -65,7 +65,7 @@ describe("arrow function conversion", () => { | |||
_supercall(); | |||
() => _supercall(); | |||
`, | |||
{ methodName: "constructor" }, | |||
{ methodName: "constructor", extend: true }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think extend: true
was missing here and not left out on purpose, even though all other tests in this file use it correctly.
super call requires the class to extend another class, so this would throw when #9493 gets merged
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/10171/ |
@@ -1 +0,0 @@ | |||
function null() { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1 +0,0 @@ | |||
function true() { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1 +0,0 @@ | |||
function false() { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1 +0,0 @@ | |||
function if() { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You renamed a lot of function declarations: it doesn't really matter, but they can be duplicated, right? |
in some cases yes, i just renamed all duplication in tests that were failing |
This does NOT change any real code, only test fixtures are change.
This changes make sure there are no duplicate declarations in any of the
input.m?js
files. I extracted this from #9493 as it can be already added independently and makes the review of #9493 easier.For reviewing here the important part is that I did not change the meaning of any test by renaming stuff. This is the reason
import type, { imp } from "";
was extracted into a separate file as it collides withimport type from "";
, but renamingtype
to something else would probably change the meaning.There is some additional fixes see inline comments.