fix(steps): always fire @Enter only once

Author: daKmoR

packages/steps/src/LionStep.js

@@ -119,8 +119,10 @@ export class LionStep extends LionLitElement {
     this.status = 'entered';
     if (updateController === true) {
       this.controller.current = this.getControllerIndex();
+      // controller will trigger enter() again which will dispatch the enter event
+    } else {
+      this.dispatchEvent(new CustomEvent('enter', { bubbles: true, composed: true }));
     }
-    this.dispatchEvent(new CustomEvent('enter', { bubbles: true, composed: true }));
   }
 
   leave() {

packages/steps/test/lion-steps.test.js

@@ -159,7 +159,7 @@ describe('lion-steps', () => {
   });
 
   describe('events', () => {
-    it('will trigger lion-step @leave event before changing .current', async () => {
+    it('will fire lion-step @leave event before changing .current', async () => {
       let currentInLeaveEvent;
       const onLeave = ev => {
         currentInLeaveEvent = ev.target.controller.current;
@@ -175,7 +175,7 @@ describe('lion-steps', () => {
       expect(currentInLeaveEvent).to.equal(0);
     });
 
-    it('will trigger lion-step @enter event after changing .current', async () => {
+    it('will fire lion-step @enter event after changing .current', async () => {
       let currentInEnterEvent;
       const onEnter = ev => {
         currentInEnterEvent = ev.target.controller.current;
@@ -190,6 +190,23 @@ describe('lion-steps', () => {
       el.next();
       expect(currentInEnterEvent).to.equal(1);
     });
+
+    it('will fire initial @enter event only once if [initial-step] is not on first step', async () => {
+      const firstEnterSpy = sinon.spy();
+      const secondEnterSpy = sinon.spy();
+      await fixture(html`
+        <lion-steps>
+          <lion-step @enter=${firstEnterSpy}>
+            <div>test1</div>
+          </lion-step>
+          <lion-step initial-step @enter=${secondEnterSpy}>
+            <div>test2</div>
+          </lion-step>
+        </lion-steps>
+      `);
+      expect(firstEnterSpy).to.not.have.been.called;
+      expect(secondEnterSpy).to.have.been.calledOnce;
+    });
   });
 
   describe('workflow with data and conditions', () => {