- extended README to contain a npm version badge of this library

- extended README to contain installation instructions on how to install this library via npm
- bumped dependency version to current versions
- added two tests to read 100% coverage

Author: madoar

README.md

@@ -1,11 +1,12 @@
-# Overview ng2-wizard
+# Overview ng2-archwizard
 
 [![Build Status](https://travis-ci.org/madoar/ng2-wizard.png)](https://travis-ci.org/madoar/ng2-wizard)
 [![Dependency Status](https://david-dm.org/madoar/ng2-wizard.svg)](https://david-dm.org/madoar/ng2-wizard)
 [![Dev-Dependency Status](https://david-dm.org/madoar/ng2-wizard/dev-status.svg)](https://david-dm.org/madoar/ng2-wizard?type=dev)
 [![Dependency Licence Status](https://dependencyci.com/github/madoar/ng2-wizard/badge)](https://dependencyci.com/github/madoar/ng2-wizard)
 [![Code Climate](https://codeclimate.com/github/madoar/ng2-wizard/badges/gpa.svg)](https://codeclimate.com/github/madoar/ng2-wizard)
 [![Test Coverage](https://codeclimate.com/github/madoar/ng2-wizard/badges/coverage.svg)](https://codeclimate.com/github/madoar/ng2-wizard/coverage)
+[![NPM Version](https://img.shields.io/npm/v/ng2-archwizard.svg)](https://www.npmjs.com/package/ng2-archwizard)
 
 This project contains a functional wizard component for [Angular 2](https://angular.io/).
 
@@ -17,6 +18,25 @@ Run `npm run build` to build the project. The build artifacts will be stored in
 
 Run `npm test` to execute the unit tests via [Karma](https://karma-runner.github.io).
 
+## Installation
+
+Ng2-archwizard is available as an NPM package. To install ng2-archwizard in your project directory run:
+```
+$ npm install --save ng2-archwizard
+```
+
+Afterwards you can import ng2-archwizard in your angular 2 project by adding the `WizardModule` to your Module declaration as followed:
+```typescript
+import {WizardModule} from "ng2-archwizard/dist";
+
+@NgModule({
+  imports: [
+    WizardModule
+  ],
+})
+export class Module { }
+```
+
 ## How to use the wizard
 
 To use the this wizard component in an angular 2 project simply add a wizard component to the html template of your component, like this:

package.json

@@ -31,68 +31,68 @@
     "build": "rimraf dist && ngc -p src/tsconfig.aot.json && gulp"
   },
   "dependencies": {
-    "@angular/common": "^2.3.1",
-    "@angular/compiler": "^2.3.1",
-    "@angular/core": "^2.3.1",
-    "@angular/http": "^2.3.1",
-    "@angular/platform-browser": "^2.3.1",
-    "@angular/platform-browser-dynamic": "^2.3.1",
-    "@angular/router": "^3.3.1",
+    "@angular/common": "^2.4.7",
+    "@angular/compiler": "^2.4.7",
+    "@angular/core": "^2.4.7",
+    "@angular/http": "^2.4.7",
+    "@angular/platform-browser": "^2.4.7",
+    "@angular/platform-browser-dynamic": "^2.4.7",
+    "@angular/router": "^3.4.7",
     "core-js": "^2.4.1",
-    "rxjs": "^5.0.1",
+    "rxjs": "^5.1.1",
     "ts-helpers": "^1.1.2",
-    "zone.js": "^0.7.2"
+    "zone.js": "^0.7.7"
   },
   "devDependencies": {
-    "@angular/compiler-cli": "^2.3.1",
-    "@angularclass/hmr": "^1.0.1",
+    "@angular/compiler-cli": "^2.4.7",
+    "@angularclass/hmr": "^1.2.2",
     "@angularclass/hmr-loader": "^3.0.2",
-    "@ngtools/webpack": "^1.2.8",
-    "@types/jasmine": "^2.2.29",
-    "@types/node": "^6.0.38",
-    "@types/selenium-webdriver": "2.53.33",
+    "@ngtools/webpack": "^1.2.9",
+    "@types/jasmine": "^2.5.42",
+    "@types/node": "^7.0.5",
+    "@types/selenium-webdriver": "2.53.39",
     "angular-router-loader": "^0.5.0",
-    "angular2-template-loader": "^0.6.1",
-    "autoprefixer": "^6.3.2",
+    "angular2-template-loader": "^0.6.2",
+    "autoprefixer": "^6.7.3",
     "awesome-typescript-loader": "^3.0.3",
-    "codelyzer": "2.0.0-beta.4",
+    "codelyzer": "2.0.0",
     "copy-webpack-plugin": "^4.0.0",
-    "css-loader": "^0.25.0",
+    "css-loader": "^0.26.1",
     "extract-text-webpack-plugin": "^2.0.0-beta.4",
-    "file-loader": "^0.9.0",
+    "file-loader": "^0.10.0",
     "gulp": "^3.9.1",
     "gulp-less": "^3.3.0",
     "html-loader": "^0.4.0",
-    "html-webpack-plugin": "^2.8.1",
-    "istanbul-instrumenter-loader": "^0.2.0",
-    "jasmine-core": "^2.3.4",
+    "html-webpack-plugin": "^2.28.0",
+    "istanbul-instrumenter-loader": "^2.0.0",
+    "jasmine-core": "^2.5.2",
     "jasmine-spec-reporter": "^2.7.0",
     "json-loader": "^0.5.3",
-    "karma": "1.3.0",
+    "karma": "1.4.1",
     "karma-chrome-launcher": "^2.0.0",
     "karma-coverage": "^1.0.0",
     "karma-jasmine": "^1.0.2",
     "karma-mocha-reporter": "^2.0.3",
     "karma-phantomjs-launcher": "^1.0.0",
     "karma-remap-coverage": "^0.1.4",
     "karma-sourcemap-loader": "^0.3.7",
-    "karma-webpack": "1.8.0",
+    "karma-webpack": "2.0.2",
     "less": "^2.7.1",
     "less-loader": "^2.2.3",
-    "node-sass": "^3.4.2",
+    "node-sass": "^4.5.0",
     "phantomjs-prebuilt": "^2.1.14",
-    "postcss-loader": "^1.1.0",
-    "protractor": "^4.0.10",
+    "postcss-loader": "^1.3.0",
+    "protractor": "^5.1.1",
     "raw-loader": "0.5.1",
     "reflect-metadata": "^0.1.9",
-    "remap-istanbul": "^0.6.4",
+    "remap-istanbul": "^0.9.1",
     "rimraf": "^2.5.1",
-    "sass-loader": "^4.0.0",
+    "sass-loader": "^6.0.0",
     "style-loader": "^0.13.0",
     "to-string-loader": "^1.1.5",
     "tslint": "^4.4.2",
-    "tslint-loader": "^3.3.0",
-    "typescript": "2.1.5",
+    "tslint-loader": "^3.4.2",
+    "typescript": "2.1.6",
     "url-loader": "^0.5.6",
     "webpack": "^2.2.0",
     "webpack-dev-server": "2.3.0",

src/components/components/wizard-step.component.spec.ts

@@ -104,4 +104,11 @@ describe('WizardStepComponent', () => {
 
     expect(wizardTest.eventLog).toEqual(['enter Forwards 1', 'exit Forwards 1', 'enter Forwards 3', 'exit Backwards 3', 'enter Backwards 2']);
   });
+
+  it('should stay at first step correctly', () => {
+    wizardTest.wizard.goToStep(0);
+    wizardTestFixture.detectChanges();
+
+    expect(wizardTest.eventLog).toEqual(['enter Forwards 1', 'exit Stay 1', 'enter Stay 1']);
+  });
 });

src/components/directives/go-to-step.directive.spec.ts

@@ -16,6 +16,7 @@ import {OptionalStepDirective} from "./optional-step.directive";
     <wizard>
       <wizard-step title='Steptitle 1'>
         Step 1
+        <button type="button" goToStep="0" (finalize)="finalizeStep(1)">Stay at this step</button>
         <button type="button" goToStep="1" (finalize)="finalizeStep(1)">Go to second step</button>
         <button type="button" goToStep="2" (finalize)="finalizeStep(1)">Go to third step</button>
       </wizard-step>
@@ -60,13 +61,14 @@ describe('GoToStepDirective', () => {
   it('should create an instance', () => {
     expect(wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:first-child'))).toBeTruthy();
     expect(wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(2)'))).toBeTruthy();
+    expect(wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(3)'))).toBeTruthy();
     expect(wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 2"] > button[goToStep]'))).toBeTruthy();
     expect(wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 3"] > button[goToStep]'))).toBeTruthy();
-    expect(wizardTestFixture.debugElement.queryAll(By.css('wizard-step > button[goToStep]')).length).toBe(4);
+    expect(wizardTestFixture.debugElement.queryAll(By.css('wizard-step > button[goToStep]')).length).toBe(5);
   });
 
   it('should move to step correctly', () => {
-    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:first-child')).nativeElement;
+    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(2)')).nativeElement;
     const secondStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 2"] > button[goToStep]')).nativeElement;
 
     expect(wizardTest.wizard.currentStepIndex).toBe(0);
@@ -88,7 +90,7 @@ describe('GoToStepDirective', () => {
   });
 
   it('should jump over an optional step correctly', () => {
-    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(2)')).nativeElement;
+    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(3)')).nativeElement;
     const thirdStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 3"] > button[goToStep]')).nativeElement;
 
     expect(wizardTest.wizard.currentStepIndex).toBe(0);
@@ -109,9 +111,22 @@ describe('GoToStepDirective', () => {
     expect(wizardTestFixture.debugElement.query(By.css('wizard-step.current')).nativeElement.title).toBe("Steptitle 1");
   });
 
+  it('should stay at current step correctly', () => {
+    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(1)')).nativeElement;
+
+    expect(wizardTest.wizard.currentStepIndex).toBe(0);
+    expect(wizardTestFixture.debugElement.query(By.css('wizard-step.current')).nativeElement.title).toBe("Steptitle 1");
+
+    // click button
+    firstStepGoToButton.click();
+    wizardTestFixture.detectChanges();
+
+    expect(wizardTest.wizard.currentStepIndex).toBe(0);
+    expect(wizardTestFixture.debugElement.query(By.css('wizard-step.current')).nativeElement.title).toBe("Steptitle 1");
+  });
 
   it('should finalize step correctly', () => {
-    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(2)')).nativeElement;
+    const firstStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 1"] > button[goToStep]:nth-child(3)')).nativeElement;
     const thirdStepGoToButton = wizardTestFixture.debugElement.query(By.css('wizard-step[title="Steptitle 3"] > button[goToStep]')).nativeElement;
 
     expect(wizardTest.wizard.currentStepIndex).toBe(0);