fix: #176 adding unit-tests

Author: enriquezrene

src/app/modules/shared/components/technologies/technologies.component.ts

@@ -54,10 +54,7 @@ export class TechnologiesComponent implements OnInit {
   }
 
   addTechnology(name: string) {
-    const index = this.selectedTechnologies.indexOf(name);
-    if (index > -1) {
-      this.removeTechnology(index);
-    } else if (this.selectedTechnologies.length < this.MAX_NUM_TECHNOLOGIES) {
+    if (this.selectedTechnologies.length < this.MAX_NUM_TECHNOLOGIES) {
       this.selectedTechnologies = [...this.selectedTechnologies, name];
       this.technologyAdded.emit(this.selectedTechnologies);
       this.showList = false;

src/app/modules/time-clock/pipes/time-details.pipe.spec.ts

@@ -5,4 +5,9 @@ describe('TimeDetailsPipe', () => {
     const pipe = new TimeDetailsPipe();
     expect(pipe).toBeTruthy();
   });
+
+  it('adds a 0 if has one number', () => {
+    const pipe = new TimeDetailsPipe();
+    expect(pipe.formatAsTwoDigit('5')).toBe('05');
+  });
 });

src/app/modules/time-clock/store/entry.actions.spec.ts

@@ -1,6 +1,18 @@
+import { TimeEntriesSummary } from './../models/time.entry.summary';
 import * as actions from './entry.actions';
 
 describe('Actions for Entries', () => {
+
+  it('LoadEntriesSummaryFail type is EntryActionTypes.LOAD_ENTRIES_SUMMARY_FAIL', () => {
+    const action = new actions.LoadEntriesSummaryFail();
+    expect(action.type).toEqual(actions.EntryActionTypes.LOAD_ENTRIES_SUMMARY_FAIL);
+  });
+
+  it('LoadEntriesSummarySuccess type is EntryActionTypes.LOAD_ENTRIES_SUMMARY_SUCCESS', () => {
+    const action = new actions.LoadEntriesSummarySuccess(null);
+    expect(action.type).toEqual(actions.EntryActionTypes.LOAD_ENTRIES_SUMMARY_SUCCESS);
+  });
+
   it('LoadActiveEntrySuccess type is EntryActionTypes.LOAD_ACTIVE_ENTRY_SUCCESS', () => {
     const loadActiveEntrySuccess = new actions.LoadActiveEntrySuccess([]);
     expect(loadActiveEntrySuccess.type).toEqual(actions.EntryActionTypes.LOAD_ACTIVE_ENTRY_SUCCESS);

src/app/modules/time-clock/store/entry.reducer.spec.ts

@@ -25,7 +25,26 @@ describe('entryReducer', () => {
     technologies: ['angular', 'typescript'],
   };
 
-  it('on LOAD_ENTRIES_SUMMARY, is Loading trye', () => {
+  it('sets timeEntriesSummary from action on LOAD_ENTRIES_SUMMARY_SUCCESS', () => {
+    const payload = null;
+    const action = new actions.LoadEntriesSummarySuccess(payload);
+    const state = entryReducer(initialState, action);
+    expect(state.timeEntriesSummary).toBe(payload);
+  });
+
+  it('sets message on LOAD_ACTIVE_ENTRY_FAIL', () => {
+    const action = new actions.LoadActiveEntryFail('');
+    const state = entryReducer(initialState, action);
+    expect(state.message).toBe('Something went wrong fetching active entry!');
+  });
+
+  it('sets timeEntriesSummary as empty on LOAD_ENTRIES_SUMMARY_FAIL', () => {
+    const action = new actions.LoadEntriesSummaryFail();
+    const state = entryReducer(initialState, action);
+    expect(state.timeEntriesSummary).toEqual(emptyTimeEntriesSummary);
+  });
+
+  it('on LOAD_ENTRIES_SUMMARY, is Loading true', () => {
     const action = new actions.LoadEntriesSummary();
     const state = entryReducer(initialState, action);
     expect(state.isLoading).toBe(true);