Merge pull request #3 from PistachoSoft/feature/add-travis-support

Feature/add travis support

Author: RecuencoJones

.travis.yml

@@ -0,0 +1,14 @@
+language: node_js
+node_js:
+  - "5"
+  - "6"
+
+before_install:
+  - "npm i -g [email protected]"
+
+install:
+  - npm install
+  - npm run setup
+
+script:
+  - npm test

README.md

@@ -0,0 +1,37 @@
+[![Build Status](https://travis-ci.org/PistachoSoft/NMM-frontend.png?branch=develop)](https://travis-ci.org/PistachoSoft/NMM-frontend)
+
+# NMM-Frontend
+
+React browser application for NMM.
+
+## Building the application
+
+Before anything, run
+
+```
+npm install
+npm run setup
+npm start
+```
+
+Once this is done, you can open [localhost:8080](http://localhost:8080) file to browse
+the application.
+
+The test account is `test:test`.
+
+## Scripts
+
+- `npm run setup`: run this command to install typings.
+- `npm start`: run the default dev task.
+- `npm run build`: build TypeScript sources and UMD distributable.
+- `npm test`: run all tests.
+- `npm run test:unit`: run unit tests only.
+- `npm run test:coverage`: run coverage tests.
+
+## Coverage
+
+The coverage reports can be found under `test/results/coverage`.
+
+## Sonar
+
+You will need to add the plugin [SonarTsPlugin](https://github.com/Pablissimo/SonarTsPlugin).

app/App.tsx

@@ -17,15 +17,15 @@ import SongsView from './views/songs/SongsView';
 import AuthService from './services/AuthService';
 
 export default class App extends React.Component<{}, {}> {
-  constructor() {
+  public constructor() {
     super();
   }
 
-  getFirstPage() {
+  public getFirstPage() {
     return AuthService.isAuth() ? '/home' : '/login';
   }
 
-  requireAuth(next: RouterState, replace: Function) {
+  public requireAuth(next: RouterState, replace: Function) {
     if (!AuthService.isAuth()) {
       replace({
         pathname: '/',
@@ -34,7 +34,7 @@ export default class App extends React.Component<{}, {}> {
     }
   }
 
-  render() {
+  public render() {
     return (
       <Router history={browserHistory}>
         <Route path="/">

app/components/navbar/Navbar.tsx

@@ -3,11 +3,11 @@ import {Link} from 'react-router';
 import AuthService from '../../services/AuthService';
 
 export default class Navbar extends React.Component<{}, {}> {
-  logout() {
+  public logout() {
     AuthService.removeAuth();
   }
 
-  render() {
+  public render() {
     return (
       <div className="navbar">
         <div className="navbar-logo">

app/components/player/Player.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react';
 
 export default class Player extends React.Component<{}, {}> {
-  render() {
+  public render() {
     return (
       <div className="player">
         Player

app/components/song/Song.tsx

@@ -1,13 +1,13 @@
 import * as React from 'react';
-import {SongModel} from '../../models/SongModel';
+import {ISongModel} from '../../models/ISongModel';
 
 export interface ISongProps {
-  model: SongModel;
+  model: ISongModel;
   className: string;
 }
 
 export default class Song extends React.Component<ISongProps, {}> {
-  render() {
+  public render() {
     return (
       <div className={`${this.props.className} song`}>
         {this.props.model.id}. {this.props.model.name}

app/mocks/Repository.ts

@@ -4,55 +4,55 @@ import {albums} from './Albums';
 import {songs} from './Songs';
 
 export default class Repository {
-  static findAllArtists() {
+  public static findAllArtists() {
     return Promise.resolve({
       artists
     });
   }
 
-  static findAllAlbums() {
+  public static findAllAlbums() {
     return Promise.resolve({
       albums
     });
   }
 
-  static findAllSongs() {
+  public static findAllSongs() {
     return Promise.resolve({
       songs
     });
   }
 
-  static findSongsByAlbum(albumId: number) {
+  public static findSongsByAlbum(albumId: number) {
     return Promise.resolve({
       songs: songs.filter((song) => {
         return song.albumId === albumId;
       })
     });
   }
 
-  static findAlbumsByArtist(artistId: number) {
+  public static findAlbumsByArtist(artistId: number) {
     return Promise.resolve({
       albums: albums.filter((album) => {
         return album.artistId === artistId;
       })
     });
   }
 
-  static findSongsByArtist(artistId: number) {
+  public static findSongsByArtist(artistId: number) {
     return Promise.resolve({
       songs: songs.filter((song) => {
         return song.artistId === artistId;
       })
     });
   }
 
-  static findArtistById(id: number) {
+  public static findArtistById(id: number) {
     return Promise.resolve(artists.filter((artist) => {
       return artist.id === id;
     })[0]);
   }
 
-  static findAlbumById(id: number) {
+  public static findAlbumById(id: number) {
     return Promise.resolve(albums.filter((album) => {
       return album.id === id;
     })[0]);

app/models/AlbumModel.ts renamed to app/models/IAlbumModel.ts

@@ -1,4 +1,4 @@
-export interface AlbumModel {
+export interface IAlbumModel {
   id: number;
   name: string;
   image: string;

app/models/ArtistModel.ts renamed to app/models/IArtistModel.ts

@@ -1,4 +1,4 @@
-export interface ArtistModel {
+export interface IArtistModel {
   id: number;
   name: string;
   image: string;

app/models/SongModel.ts renamed to app/models/ISongModel.ts

@@ -1,4 +1,4 @@
-export interface SongModel {
+export interface ISongModel {
   id: number;
   name: string;
   albumId?: number;

app/models/responses/AlbumsResponse.ts

@@ -0,0 +1,5 @@
+import {IAlbumModel} from '../IAlbumModel';
+
+export interface IAlbumsResponse {
+  albums: Array<IAlbumModel>;
+}

app/models/responses/ArtistsResponse.ts

@@ -0,0 +1,5 @@
+import {IArtistModel} from '../IArtistModel';
+
+export interface IArtistsResponse {
+  artists: Array<IArtistModel>;
+}

app/models/responses/ErrorResponse.ts

@@ -0,0 +1,3 @@
+export interface IErrorResponse {
+  message: string;
+}

app/models/responses/IAlbumsResponse.ts

@@ -0,0 +1,5 @@
+import {ISongModel} from '../ISongModel';
+
+export interface ISongsResponse {
+  songs: Array<ISongModel>;
+}

app/models/responses/IArtistsResponse.ts

@@ -0,0 +1,3 @@
+export interface ITokenResponse {
+  token: string;
+}

app/models/responses/IErrorResponse.ts

@@ -1,17 +1,17 @@
 let auth = localStorage.getItem('auth') || false;
 
 export default class AuthService {
-  static isAuth() {
+  public static isAuth() {
     return !!auth;
   }
 
-  static setAuth(token: string) {
+  public static setAuth(token: string) {
     auth = token;
 
     localStorage.setItem('auth', token);
   }
 
-  static removeAuth() {
+  public static removeAuth() {
     auth = null;
 
     localStorage.removeItem('auth');

app/models/responses/ISongsResponse.ts

@@ -1,21 +1,19 @@
 import {Promise} from 'es6-promise';
-import {ErrorResponse} from '../models/responses/ErrorResponse';
-import {TokenResponse} from '../models/responses/TokenResponse';
+import {IErrorResponse} from '../models/responses/IErrorResponse';
+import {ITokenResponse} from '../models/responses/ITokenResponse';
 
 export default class LoginService {
-  static login(user, pass): Promise<TokenResponse> {
+  public static login(user, pass): Promise<ITokenResponse> {
     return new Promise((resolve, reject) => {
-      console.log('Login:', user, pass);
-
       if (user.trim() === 'test' && pass.trim() === 'test' ||
           user.trim() === 'kimjonun') {
         resolve({
           token: 'ok'
-        } as TokenResponse);
+        } as ITokenResponse);
       } else {
         reject({
           message: 'Oops, try again!'
-        } as ErrorResponse);
+        } as IErrorResponse);
       }
     });
   }

app/models/responses/ITokenResponse.ts

@@ -1,57 +1,41 @@
 import {Promise} from 'es6-promise';
 import Repository from '../mocks/Repository';
-import {SongsResponse} from '../models/responses/SongsResponse';
-import {AlbumsResponse} from '../models/responses/AlbumsResponse';
-import {ArtistsResponse} from '../models/responses/ArtistsResponse';
-import {ArtistModel} from '../models/ArtistModel';
-import {AlbumModel} from '../models/AlbumModel';
+import {ISongsResponse} from '../models/responses/ISongsResponse';
+import {IAlbumsResponse} from '../models/responses/IAlbumsResponse';
+import {IArtistsResponse} from '../models/responses/IArtistsResponse';
+import {IArtistModel} from '../models/IArtistModel';
+import {IAlbumModel} from '../models/IAlbumModel';
 
 export default class MusicService {
-  static getAllSongs(): Promise<SongsResponse> {
-    console.info('GET', '/songs');
-
+  public static getAllSongs(): Promise<ISongsResponse> {
     return Repository.findAllSongs();
   }
 
-  static getSongsByAlbum(albumId: number) {
-    console.info('GET', `/albums/${albumId}/songs`);
-
+  public static getSongsByAlbum(albumId: number) {
     return Repository.findSongsByAlbum(albumId);
   }
 
-  static getAllAlbums(): Promise<AlbumsResponse> {
-    console.info('GET', '/albums');
-
+  public static getAllAlbums(): Promise<IAlbumsResponse> {
     return Repository.findAllAlbums();
   }
 
-  static getAlbumsByArtist(artistId: number) {
-    console.info('GET', `/artists/${artistId}/albums`);
-
+  public static getAlbumsByArtist(artistId: number) {
     return Repository.findAlbumsByArtist(artistId);
   }
 
-  static getSongsByArtist(artistId: number) {
-    console.info('GET', `/artists/${artistId}/songs`);
-
+  public static getSongsByArtist(artistId: number) {
     return Repository.findSongsByArtist(artistId);
   }
 
-  static getAllArtists(): Promise<ArtistsResponse> {
-    console.info('GET', '/artists');
-
+  public static getAllArtists(): Promise<IArtistsResponse> {
     return Repository.findAllArtists();
   }
 
-  static getArtistById(id: number): Promise<ArtistModel> {
-    console.info('GET', `/artists/${id}`);
-
+  public static getArtistById(id: number): Promise<IArtistModel> {
     return Repository.findArtistById(id);
   }
 
-  static getAlbumById(id: number): Promise<AlbumModel> {
-    console.info('GET', `/albums/${id}`);
-
+  public static getAlbumById(id: number): Promise<IAlbumModel> {
     return Repository.findAlbumById(id);
   }
 }