<macro> tag now requires a name as its first attribute, which may or may not be an argument attribute

taglibs/core/macro-tag.js

@@ -1,13 +1,10 @@
 module.exports = function codeGenerator(elNode, codegen) {
 
     var attributes = elNode.attributes;
-    if (!attributes.length) {
-        return;
-    }
-
     var defAttr = attributes[0];
-    if (defAttr.argument == null) {
+
-        return;
+    if(!defAttr || defAttr.value !== undefined) {
+        return codegen.addError('The <macro> tag must contain a name as its first attribute, example: <macro greeting()>');
     }
 
     var body = elNode.body;

test/autotests/render/error-macro-no-name/marko.json

@@ -0,0 +1,5 @@
+{
+    "<test-open-tag-only>": {
+        "open-tag-only": true
+    }
+}

test/autotests/render/error-macro-no-name/template.marko

@@ -0,0 +1,7 @@
+<macro>
+    This is bad.
+</macro>
+
+<macro test=123>
+    This is also bad.
+</macro>

test/autotests/render/error-macro-no-name/test.js

@@ -0,0 +1,11 @@
+var expect = require('chai').expect;
+
+exports.templateData = {};
+
+exports.checkError = function(e) {
+    expect(Array.isArray(e.errors)).to.equal(true);
+    expect(e.errors.length).to.equal(2);
+
+    var message = e.toString();
+    expect(message).to.contain('The <macro> tag must contain a name as its first attribute, example: <macro greeting()>');
+};

test/autotests/render/macro-no-args/expected.html

@@ -1 +1 @@
-Marko is awesomeMarko is awesome
+Marko is awesomeMarko is awesomeMarko is really awesomeMarko is really awesome

test/autotests/render/macro-no-args/template.marko

@@ -2,5 +2,11 @@
     Marko is awesome
 </macro>
 
+<macro greeting2>
+    Marko is really awesome
+</macro>
+
 <greeting/>
-<greeting/>
+<greeting()/>
+<greeting2/>
+<greeting2()/>