From 85c503729e40e4e5b46d9a1fbaf8038c6a87329b Mon Sep 17 00:00:00 2001
From: jtm <jtm@robot.is>
Date: Fri, 6 Apr 2012 14:53:38 +0000
Subject: [PATCH] tests + documentation

---
 lesscpy/lessc/parser.py                       | 41 ++++++++++---------
 lesscpy/plib/deferred.py                      |  5 +++
 lesscpy/test/css/mixin-args-undeclared.css    |  6 +++
 .../test/css/mixin-args-undeclared.min.css    |  2 +
 lesscpy/test/less/imports/import.less         |  3 ++
 lesscpy/test/less/mixin-args-undeclared.less  | 10 +++++
 6 files changed, 48 insertions(+), 19 deletions(-)
 create mode 100644 lesscpy/test/css/mixin-args-undeclared.css
 create mode 100644 lesscpy/test/css/mixin-args-undeclared.min.css
 create mode 100644 lesscpy/test/less/mixin-args-undeclared.less

diff --git a/lesscpy/lessc/parser.py b/lesscpy/lessc/parser.py
index d01772b..d81b505 100644
--- a/lesscpy/lessc/parser.py
+++ b/lesscpy/lessc/parser.py
@@ -73,8 +73,9 @@ class LessParser(object):
         
     def parse(self, filename='', debuglevel=0):
         """ Parse file.
-            @param string: Filename
-            @param int: Debuglevel
+        kwargs:
+            filename (str): File to parse
+            debuglevel (int): Parser debuglevel
         """
         if self.verbose: print('Compiling target: %s' % filename)
         self.scope.push()
@@ -82,16 +83,20 @@ class LessParser(object):
         self.result = self.parser.parse(filename, lexer=self.lex, debug=debuglevel)
         self.post_parse(self.result)
         
-    def post_parse(self, ll):
+    def post_parse(self, lst):
+        """Post process stage. less.js seems to allow calls to mixins not
+        yet declared. Deferreds are used in place of real mixins. After
+        parsing we go over the parse tree and reparse deferred's.
+        args: 
+            lst (list): Parse list
         """
-        """
-        if type(ll) is list:
-            for u in ll: self.post_parse(u)
-        elif type(ll) is Block:
-            ll.parsed = list(utility.flatten([t.parse(self.scope) 
-                                              if type(t) is Deferred else t
-                                              for t in ll.parsed]))
-            self.post_parse(ll.parsed)
+        if type(lst) is list:
+            for u in lst: self.post_parse(u)
+        elif type(lst) is Block:
+            lst.parsed = list(utility.flatten([t.parse(self.scope) 
+                                               if type(t) is Deferred else t
+                                               for t in lst.parsed]))
+            self.post_parse(lst.parsed)
             
     def scopemap(self):
         """ Output scopemap.
@@ -321,10 +326,6 @@ class LessParser(object):
                 res = Deferred(p[1], p[3])
         if res is False:
             res = Deferred(p[1], p[3])
-#            args = ''.join([''.join(a) for a in p[3]]) if p[3] else ''
-#            self.handle_error('Call unknown mixin `%s(%s)`' % 
-#                              (p[1].raw(True), args), 
-#                              p.lineno(2))
         p[0] = res
             
     def p_mixin_args_arguments(self, p):
@@ -777,7 +778,8 @@ class LessParser(object):
 
     def p_error(self, t):
         """ Internal error handler
-            @param Lex token: Error token 
+        args:
+            t (Lex token): Error token 
         """
         if t and self.verbose: 
             print("\x1b[31mE: %s line: %d, Syntax Error, token: `%s`, `%s`\x1b[0m" 
@@ -793,9 +795,10 @@ class LessParser(object):
         
     def handle_error(self, e, line, t='E'):
         """ Custom error handler
-            @param Exception: Exception
-            @param Parser token: Parser token
-            @param string: Error level 
+        args:
+            e (Mixed): Exception or str
+            line (int): line number
+            t(str): Error type
         """
 #        print(e.trace())
         if self.verbose:
diff --git a/lesscpy/plib/deferred.py b/lesscpy/plib/deferred.py
index 6b13950..99bf4c9 100644
--- a/lesscpy/plib/deferred.py
+++ b/lesscpy/plib/deferred.py
@@ -45,4 +45,9 @@ class Deferred(Node):
             if res: return res
         return False
     
+    def fmt(self, fills):
+        """
+        """
+        return ''
+    
     
diff --git a/lesscpy/test/css/mixin-args-undeclared.css b/lesscpy/test/css/mixin-args-undeclared.css
new file mode 100644
index 0000000..b272e95
--- /dev/null
+++ b/lesscpy/test/css/mixin-args-undeclared.css
@@ -0,0 +1,6 @@
+.a {
+	padding: 3px;
+}
+.mixin {
+	color: red;
+}
diff --git a/lesscpy/test/css/mixin-args-undeclared.min.css b/lesscpy/test/css/mixin-args-undeclared.min.css
new file mode 100644
index 0000000..f2c2398
--- /dev/null
+++ b/lesscpy/test/css/mixin-args-undeclared.min.css
@@ -0,0 +1,2 @@
+.a{padding:3px;}
+.mixin{color:red;}
diff --git a/lesscpy/test/less/imports/import.less b/lesscpy/test/less/imports/import.less
index 4338bba..8623df9 100644
--- a/lesscpy/test/less/imports/import.less
+++ b/lesscpy/test/less/imports/import.less
@@ -2,4 +2,7 @@
 
 .mixin {
 	color: red;
+}
+.mmundecl(@index) {
+	padding: @index * 1px;
 }
\ No newline at end of file
diff --git a/lesscpy/test/less/mixin-args-undeclared.less b/lesscpy/test/less/mixin-args-undeclared.less
new file mode 100644
index 0000000..eed64c3
--- /dev/null
+++ b/lesscpy/test/less/mixin-args-undeclared.less
@@ -0,0 +1,10 @@
+/*
+	Calls to undeclared mixins
+*/
+.a {
+	.mmundecl(3px);
+}
+/*
+	Declare
+*/
+@import 'imports/import.less';
\ No newline at end of file