fixes #31: values after empty url() func

Author: shellscape

lib/parser.js

@@ -293,7 +293,9 @@ module.exports = class Parser {
     // url functions get special treatment, and anything between the function
     // parens get treated as one word, if the contents aren't not a string.
     if (this.current.type === 'func' && this.current.unbalanced &&
-        this.current.value === 'url' && this.currToken[0] !== 'string') {
+        this.current.value === 'url' && this.currToken[0] !== 'string' &&
+        this.currToken[0] !== ')') {
+
       let nextToken = this.nextToken,
         value = this.currToken[1],
         start = {

test/function.js

@@ -23,6 +23,18 @@ describe('Parser → Function', () => {
         { type: 'paren', value: ')' }
       ]
     },
+    {
+      it: 'should parse empty url function with values following',
+      test: 'url() foo bar baz',
+      expected: [
+        { type: 'func', value: 'url' },
+        { type: 'paren', value: '(' },
+        { type: 'paren', value: ')' },
+        { type: 'word', value: 'foo' },
+        { type: 'word', value: 'bar' },
+        { type: 'word', value: 'baz' }
+      ]
+    },
     {
       it: 'should parse url function',
       test: 'url( /gfx/img/bg.jpg )',
@@ -131,8 +143,6 @@ describe('Parser → Function', () => {
       let ast = new Parser(fixture.test).parse(),
         index = 0;
 
-      // console.log(ast.first.first.nodes);
-
       ast.first.walk((node) => {
         let expected = fixture.expected[index];
         index ++;

test/tokenize.js

@@ -21,7 +21,6 @@ describe('Tokenize', () => {
   fixtures.forEach((fixture) => {
     it('should tokenize ' + fixture.value.replace(/\n/g, '\\n').replace(/\t/g, '\\t'), () => {
       let tokens = tokenize(fixture.value);
-      console.log(tokens);
       expect(tokens.length).to.equal(fixture.expectedLength);
     });
   });