PEXOVS-562 Submit button default ID (#99)

ID on input-submit will default to submit-button unless overridden.

lib/template-mixins.js

@@ -393,7 +393,7 @@ module.exports = function (options, deprecated) {
 
                         var obj = {
                             value: t('buttons.' + value),
-                            id: id
+                            id: id ? id : 'submit-button'
                         };
                         return compiled['partials/forms/input-submit'].render(obj);
                     };

partials/forms/input-submit.html

@@ -1,3 +1,3 @@
 <div class="form-group">
-    <input type="submit" {{#id}}id="{{id}}"{{/id}} value="{{{value}}}" class="button"{{#attributes}} {{attribute}}="{{value}}"{{/attributes}}>
-</div>
+    <input type="submit" id="{{{id}}}" value="{{{value}}}" class="button"{{#attributes}} {{attribute}}="{{value}}"{{/attributes}}>
+</div>

test/spec.index.js

@@ -802,6 +802,24 @@ describe('Template Mixins', function () {
                 }));
             });
 
+            it('id is defaulted to submit-button', function () {
+                middleware(req, res, next);
+                res.locals['input-submit']().call(res.locals, 'button-id');
+                render.should.have.been.calledWith(sinon.match({
+                    value: 'buttons.button-id',
+                    id: 'submit-button'
+                }));
+            });
+
+            it('id is overridden if provided', function () {
+                middleware(req, res, next);
+                res.locals['input-submit']().call(res.locals, 'button-id overridden-id');
+                render.should.have.been.calledWith(sinon.match({
+                    value: 'buttons.button-id',
+                    id: 'overridden-id'
+                }));
+            });
+
             it('prefixes translation lookup with namespace if provided', function () {
                 middleware = mixins({ translate: translate, sharedTranslationsKey: 'name.space' });
                 middleware(req, res, next);