Better error message for top-level private names

babel · Sep 18, 2019 · 8e836dd · 8e836dd
1 parent 1973ba3
commit 8e836dd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/packages/babel-parser/src/tokenizer/index.js b/packages/babel-parser/src/tokenizer/index.js
@@ -381,14 +381,8 @@ export default class Tokenizer extends LocationParser {
     }
 
     if (
-      (this.hasPlugin("classPrivateProperties") ||
-        this.hasPlugin("classPrivateMethods")) &&
-      this.state.classLevel > 0
-    ) {
-      ++this.state.pos;
-      this.finishToken(tt.hash);
-      return;
-    } else if (
+      this.hasPlugin("classPrivateProperties") ||
+      this.hasPlugin("classPrivateMethods") ||
       this.getPluginOption("pipelineOperator", "proposal") === "smart"
     ) {
       this.finishOp(tt.hash, 1);

diff --git a/...ser/test/fixtures/experimental/class-private-properties/undeclared-top-level/options.json b/...ser/test/fixtures/experimental/class-private-properties/undeclared-top-level/options.json
@@ -1,4 +1,4 @@
 {
   "plugins": ["classPrivateProperties"],
-  "throws": "Unexpected character '#' (1:15)"
+  "throws": "Private name #priv is not defined (1:15)"
 }