fix(sugar): @a.b(1) not compile

Author: litlighilit

src/pylib/pysugar/stmt/class.nim

@@ -173,6 +173,8 @@ then you will find it compile but `O.f()` gives `0` instead of `3`
   template clsType: NimNode =
     nnkBracketExpr.newTree(ident"typedesc", curClass())
 
+  if decor.name.len != 0:
+    return false
   case $decor.name
   of "staticmethod":
     purgeBase()
@@ -283,9 +285,14 @@ so if wantting the attr inherited from SupCls, just write it as-is (e.g. `self.a
   for def in items(body):
     var pragmas = defPragmas  # will be set as empty if `isConstruct` or not base
     case def.kind
-    of nnkCall: # attr define, e.g. a: int / a: int = 1
-      let tup = parseDeclWithType(def)
-      addAttr tup.name, tup.typ, tup.val
+    of nnkCall:
+      # - attr define, e.g. a: int / a: int = 1
+      # - dotted called decorator, e.g. @unittest.skipIf(COND, MSG)
+      if def.len == 2 and def[0].kind == nnkIdent:
+        let tup = parseDeclWithType(def)
+        addAttr tup.name, tup.typ, tup.val
+      else:
+        parser.pushDecorator extractDottedCalledDecorator def
     of nnkAsgn:  #  a = 1
       addAttr def[0], emptyn, def[1]
     of nnkCommand:  # TODO: support async

src/pylib/pysugar/stmt/decorator.nim

@@ -63,3 +63,16 @@ proc consumeDecorator*(mparser; procDef: NimNode): NimNode =
   let blk = newBlockStmt blkExpr
   result = newLetStmt(procName, blk)
 
+proc extractDottedCalledDecorator*(decorator: NimNode): NimNode =
+  ## Extract dotted call from `decorator`.
+  ## e.g. `@a.b.c(1, 2)` -> `a.b.c(1, 2)`
+  ## 
+  ## .. note:: Currently `decorator` is to be modified in place, while result is still returned
+  expectKind decorator, nnkCall
+  expectMinLen decorator, 2
+  result = decorator
+  var cur = decorator
+  while cur[0].kind != nnkPrefix:
+    cur = cur[0]
+  assert cur[0][0].eqIdent "@"
+  cur[0] = cur[0][1]

src/pylib/pysugar/stmt/tonim.nim

@@ -120,6 +120,9 @@ proc parsePyStmt*(mparser; statement: NimNode): NimNode =
       # TODO: impl by define such class in global but mangling its name
       # It has to be global as  class's def is implemented via `method`,
       # which is only allowed at global scope 
+    of "@":
+      # dotted called decorator, e.g. @unittest.skipIf(COND, MSG)
+      mparser.pushDecorator extractDottedCalledDecorator statement
     else:
       var cmd = newNimNode nnkCommand
       for i in statement: