Merge pull request #190 from sveltejs/gh-188

throw if default export is not an object literal

src/validate/js/index.js

@@ -16,24 +16,19 @@ export default function validateJs ( validator, js ) {
 		}
 
 		if ( node.type === 'ExportDefaultDeclaration' ) {
-			if ( validator.defaultExport ) {
-				validator.error( `Duplicate default export`, node.start );
+			if ( node.declaration.type !== 'ObjectExpression' ) {
+				return validator.error( `Default export must be an object literal`, node.declaration.start );
 			}
 
-			validator.defaultExport = node;
-		}
-	});
-
-	// ensure all exported props are valid
-	if ( validator.defaultExport ) {
-		checkForComputedKeys( validator, validator.defaultExport.declaration.properties );
-		checkForDupes( validator, validator.defaultExport.declaration.properties );
+			checkForComputedKeys( validator, node.declaration.properties );
+			checkForDupes( validator, node.declaration.properties );
 
-		validator.defaultExport.declaration.properties.forEach( prop => {
+			node.declaration.properties.forEach( prop => {
 				validator.templateProperties[ prop.key.name ] = prop;
 			});
 
-		validator.defaultExport.declaration.properties.forEach( prop => {
+			// ensure all exported props are valid
+			node.declaration.properties.forEach( prop => {
 				const propValidator = propValidators[ prop.key.name ];
 
 				if ( propValidator ) {
@@ -54,5 +49,8 @@ export default function validateJs ( validator, js ) {
 				const ns = validator.templateProperties.namespace.value.value;
 				validator.namespace = namespaces[ ns ] || ns;
 			}
+
+			validator.defaultExport = node;
 		}
+	});
 }

test/validate.js

@@ -46,9 +46,9 @@ describe( 'validate', () => {
 				if ( err.name !== 'ParseError' ) throw err;
 
 				try {
-					const expected = require( `./validator/${dir}/error.json` );
+					const expected = require( `./validator/${dir}/errors.json` )[0];
 
-					assert.equal( err.shortMessage, expected.message );
+					assert.equal( err.message, expected.message );
 					assert.deepEqual( err.loc, expected.loc );
 					assert.equal( err.pos, expected.pos );
 				} catch ( err2 ) {

test/validator/export-default-duplicated/errors.json

@@ -0,0 +1,8 @@
+[{
+	"message": "Duplicate export 'default'",
+	"pos": 37,
+	"loc": {
+		"line": 3,
+		"column": 8
+	}
+}]

test/validator/export-default-duplicated/input.html

@@ -0,0 +1,4 @@
+<script>
+	export default {};
+	export default {};
+</script>

test/validator/export-default-must-be-object/errors.json

@@ -0,0 +1,8 @@
+[{
+	"message": "Default export must be an object literal",
+	"pos": 25,
+	"loc": {
+		"line": 2,
+		"column": 16
+	}
+}]

test/validator/export-default-must-be-object/input.html

@@ -0,0 +1,3 @@
+<script>
+	export default potato;
+</script>