Fix loader: throw validation errors

* add test verifying that we properly throw validation errors
* restructure test suite

Author: nikku

index.js

@@ -11,13 +11,7 @@ module.exports = function(source, map) {
     let { code, map } = compile(source, {
       filename: filename,
       format: query.format || 'es',
-      name: query.name,
-      onerror: (err) => {
-        this.emitError(err);
-      },
-      onwarn: (warn) => {
-        this.emitWarn(warn);
-      }
+      name: query.name
     });
 
     this.callback(null, code, map);

test/fixtures/export-error.html

@@ -0,0 +1,7 @@
+<p>Count: {{count}}</p>
+
+<script>
+  export {
+    foo: 'BAR'
+  };
+</script>

test/fixtures/bad.html renamed to test/fixtures/parse-error.html

@@ -0,0 +1,9 @@
+<p>Count: {{count}}</p>
+
+<script>
+  export default {
+    computed: {
+      foo: 'BAR'
+    }
+  };
+</script>

test/fixtures/validation-error.html

@@ -28,8 +28,6 @@ describe('loader', function() {
       loader.call({
         cacheable: cacheableSpy,
         callback: callbackSpy,
-        emitWarn: function(warning) {},
-        emitError: function(e) {},
         filename: fileName,
         query,
       }, fileContents, null);
@@ -40,7 +38,7 @@ describe('loader', function() {
   }
 
 
-  it('should compile good',
+  it('should compile',
     testLoader('test/fixtures/good.html', function(err, code, map) {
       expect(err).not.to.exist;
 
@@ -50,65 +48,119 @@ describe('loader', function() {
   );
 
 
-  it('should compile bad',
-    testLoader('test/fixtures/bad.html', function(err, code, map, context) {
+  describe('error handling', function() {
 
-      expect(err).to.exist;
+    it('should handle parse error',
+      testLoader('test/fixtures/parse-error.html', function(err, code, map, context) {
 
-      expect(err.message).to.eql(
-        'Expected }}} (1:18)\n' +
-        '1: <p>Count: {{{count}}</p>\n' +
-        '                     ^\n' +
-        '2: <button on:click=\'set({ count: count + 1 })\'>+1</button>'
-      );
+        expect(err).to.exist;
 
-      expect(code).not.to.exist;
-      expect(map).not.to.exist;
-    })
-  );
+        expect(err.message).to.eql(
+          'Expected }}} (1:18)\n' +
+          '1: <p>Count: {{{count}}</p>\n' +
+          '                     ^\n' +
+          '2: <button on:click=\'set({ count: count + 1 })\'>+1</button>'
+        );
 
+        expect(code).not.to.exist;
+        expect(map).not.to.exist;
+      })
+    );
 
-  it('should compile with import / ES2015 features',
-    testLoader('test/fixtures/es2015.html', function(err, code, map) {
-      expect(err).not.to.exist;
 
-      expect(code).to.exist;
-      expect(map).to.exist;
+    it('should handle wrong export',
+      testLoader('test/fixtures/export-error.html', function(err, code, map, context) {
 
-      // es2015 statements remain
-      expect(code).to.contain('import { hello } from \'./utils\';');
-      expect(code).to.contain('data() {');
-    })
-  );
+        expect(err).to.exist;
 
+        expect(err.message).to.eql(
+          'Unexpected token (5:7)\n' +
+          '3: <script>\n' +
+          '4:   export {\n' +
+          '5:     foo: \'BAR\'\n' +
+          '          ^\n' +
+          '6:   };\n' +
+          '7: </script>'
+        );
 
-  it('should compile Component with with nesting',
-    testLoader('test/fixtures/parent.html', function(err, code, map) {
-      expect(err).not.to.exist;
+        expect(code).not.to.exist;
+        expect(map).not.to.exist;
+      })
+    );
 
-      // es2015 statements remain
-      expect(code).to.contain('import Nested from \'./nested\';');
 
-      expect(code).to.exist;
-      expect(map).to.exist;
-    })
-  );
+    it('should validation error',
+      testLoader('test/fixtures/validation-error.html', function(err, code, map, context) {
 
+        expect(err).to.exist;
 
-  it('should compile Component to CJS if requested',
-    testLoader('test/fixtures/good.html', function(err, code, map) {
-      expect(err).not.to.exist;
-      expect(code).to.contain('module.exports = SvelteComponent;');
-    }, { format: 'cjs' })
-  );
+        expect(err.message).to.eql(
+          'Computed properties can be function expressions or arrow function expressions (6:11)\n' +
+          '4:   export default {\n' +
+          '5:     computed: {\n' +
+          '6:       foo: \'BAR\'\n' +
+          '              ^\n' +
+          '7:     }\n' +
+          '8:   };'
+        );
 
+        expect(code).not.to.exist;
+        expect(map).not.to.exist;
+      })
+    );
 
-  it('should compile Component to UMD if requested',
-    testLoader('test/fixtures/good.html', function(err, code, map) {
-      expect(err).not.to.exist;
-      expect(code).to.contain('(global.FooComponent = factory());');
-    }, { format: 'umd', name: 'FooComponent' })
-  );
+  });
+
+
+  describe('ES2015 features', function() {
+
+    it('should keep imports / methods',
+      testLoader('test/fixtures/es2015.html', function(err, code, map) {
+        expect(err).not.to.exist;
+
+        expect(code).to.exist;
+        expect(map).to.exist;
+
+        // es2015 statements remain
+        expect(code).to.contain('import { hello } from \'./utils\';');
+        expect(code).to.contain('data() {');
+      })
+    );
+
+
+    it('should keep nested Component import',
+      testLoader('test/fixtures/parent.html', function(err, code, map) {
+        expect(err).not.to.exist;
+
+        // es2015 statements remain
+        expect(code).to.contain('import Nested from \'./nested\';');
+
+        expect(code).to.exist;
+        expect(map).to.exist;
+      })
+    );
+
+  });
+
+
+  describe('configuration via query', function() {
+
+    it('should configure CommonJS output',
+      testLoader('test/fixtures/good.html', function(err, code, map) {
+        expect(err).not.to.exist;
+        expect(code).to.contain('module.exports = SvelteComponent;');
+      }, { format: 'cjs' })
+    );
+
+
+    it('should configure named UMD output',
+      testLoader('test/fixtures/good.html', function(err, code, map) {
+        expect(err).not.to.exist;
+        expect(code).to.contain('(global.FooComponent = factory());');
+      }, { format: 'umd', name: 'FooComponent' })
+    );
+
+  });
 
 });