Merge pull request #4426 from connec/class-dynamic-key-scope

[CS2] Fix scope of assignments in dynamic class keys

Author: lydell

lib/coffee-script/nodes.js

@@ -2009,8 +2009,10 @@ exports.Code = class Code extends Base
 
     # We need to compile the body before method names to ensure super references are handled
     if @isMethod
+      [methodScope, o.scope] = [o.scope, o.scope.parent]
       name = @name.compileToFragments o
       name.shift() if name[0].code is '.'
+      o.scope = methodScope
 
     answer = @joinFragmentArrays (@makeCode m for m in modifiers), ' '
     answer.push @makeCode ' ' if modifiers.length and name

src/nodes.coffee

@@ -845,6 +845,22 @@ test "#1392 calling `super` in methods defined on namespaced classes", ->
     @a::m = -> super
   eq 5, (new C.a).m()
 
+
+test "dynamic method names", ->
+  class A
+    "#{name = 'm'}": -> 1
+  eq 1, new A().m()
+
+  class B extends A
+    "#{name = 'm'}": -> super
+  eq 1, new B().m()
+
+  getName = -> 'm'
+  class C
+    "#{name = getName()}": -> 1
+  eq 1, new C().m()
+
+
 test "dynamic method names and super", ->
   class Base
     @m: -> 6