Don't dispatch click event for disabled button

src/vaadin-button.html

@@ -176,6 +176,15 @@
         get focusElement() {
           return this.$.button;
         }
+
+        /**
+         * @protected
+         */
+        click() {
+          if (!this.disabled) {
+            super.click();
+          }
+        }
       }
 
       customElements.define(ButtonElement.is, ButtonElement);

test/.eslintrc

@@ -7,6 +7,7 @@
     "it": false,
     "expect": false,
     "gemini": false,
-    "MockInteractions": false
+    "MockInteractions": false,
+    "sinon": false
   }
 }

test/vaadin-button_test.html

@@ -54,6 +54,21 @@
         MockInteractions.downAndUp(vaadinButton);
       });
 
+      it('should fire click event for button', () => {
+        const spy = sinon.spy();
+        vaadinButton.addEventListener('click', spy);
+        vaadinButton.click();
+        expect(spy.called).to.be.true;
+      });
+
+      it('should not fire click event for disabled button', () => {
+        const spy = sinon.spy();
+        vaadinButton.disabled = true;
+        vaadinButton.addEventListener('click', spy);
+        vaadinButton.click();
+        expect(spy.called).to.be.false;
+      });
+
       it('host should have the `button` role', () => {
         expect(vaadinButton.getAttribute('role')).to.be.eql('button');
       });