assert when selectors are missing

closes reworkcss#50

Author: jonathanong

History.md

@@ -6,6 +6,7 @@
    - `filename` is now the optional filename
  * changed default `options.position` value to `true`
  * remove comments from properties and values
+ * asserts when selectors are missing
 
 1.7.0 / 2013-12-21
 ==================

index.js

@@ -109,7 +109,7 @@ module.exports = function(css, options){
     var rules = [];
     whitespace();
     comments(rules);
-    while (css.charAt(0) != '}' && (node = atrule() || rule())) {
+    while (css.length && css.charAt(0) != '}' && (node = atrule() || rule())) {
       rules.push(node);
       comments(rules);
     }
@@ -483,7 +483,7 @@ module.exports = function(css, options){
     var pos = position();
     var sel = selector();
 
-    if (!sel) return;
+    if (!sel) return error('selector missing');
     comments();
 
     return pos({

test/cases/invalid.css

@@ -3,11 +3,12 @@
  * Module dependencies.
  */
 
-var parse = require('..')
-  , fs = require('fs')
-  , path = require('path')
-  , read = fs.readFileSync
-  , readdir = fs.readdirSync;
+var parse = require('..');
+var fs = require('fs');
+var path = require('path');
+var read = fs.readFileSync;
+var readdir = fs.readdirSync;
+var assert = require('assert');
 
 describe('parse(str)', function(){
   readdir('test/cases').forEach(function(file){
@@ -34,4 +35,14 @@ describe('parse(str)', function(){
     position.filename.should.equal('booty.css');
     position.source.should.equal(css);
   });
+
+  it('should throw when a selector is missing', function(){
+    assert.throws(function(){
+      parse('{size: large}');
+    });
+
+    assert.throws(function(){
+      parse('b { color: red; }\n{ color: green; }\na {color: blue; }');
+    });
+  })
 })