#78 PR comments

Author: enriquezrene

src/app/app-routing.module.ts

@@ -20,7 +20,7 @@ const routes: Routes = [
       { path: 'time-clock', component: TimeClockComponent },
       { path: 'time-entries', component: TimeEntriesComponent },
       { path: 'activities-management', component: ActivitiesManagementComponent },
-      { path: 'customer-managment', component: CustomerComponent },
+      { path: 'customers-management', component: CustomerComponent },
       { path: '', pathMatch: 'full', redirectTo: 'time-clock' },
     ],
   },

src/app/modules/customer-managment/components/customer-info/components/create-customer/create-customer.spec.ts

@@ -18,7 +18,7 @@ describe('InputCustomerComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.html

@@ -12,10 +12,12 @@
     >
       <td class="col-sm-9">{{ customer.name }}</td>
       <td class="col-sm-3 text-center">
-        <button (click)="activateCustomerForm()" type="button" class="btn btn-sm btn-secondary">
+        <button type="button" class="btn btn-sm btn-secondary">
           <i class="fa fa-pencil fa-xs"></i>
         </button>
-        <button type="button" class="btn btn-sm btn-danger ml-2"><i class="fas fa-trash-alt fa-xs"></i></button>
+        <button type="button" class="btn btn-sm btn-danger ml-2">
+          <i class="fas fa-trash-alt fa-xs"></i>
+        </button>
       </td>
     </tr>
   </tbody>

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.spec.ts

@@ -20,14 +20,8 @@ describe('CustomerTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 
-  it('changeShowCustomerForm should listen form changes', () => {
-    component.showCustomerForm = true;
-    spyOn(component.changeShowCustomerForm, 'emit');
-    component.activateCustomerForm();
-    expect(component.changeShowCustomerForm.emit).toHaveBeenCalled();
-  });
 });

src/app/modules/customer-managment/components/customer-info/components/customer-list/customer-list.component.ts

@@ -1,4 +1,4 @@
-import { Component, Input, Output, EventEmitter } from '@angular/core';
+import { Component } from '@angular/core';
 import { ITEMS_PER_PAGE } from 'src/environments/environment';
 
 @Component({
@@ -29,13 +29,6 @@ export class CustomerListComponent {
     },
   ];
 
-  @Input() showCustomerForm;
-  @Output() changeShowCustomerForm = new EventEmitter<boolean>();
-
   constructor() {}
 
-  activateCustomerForm() {
-    this.showCustomerForm = true;
-    this.changeShowCustomerForm.emit(this.showCustomerForm);
-  }
 }

src/app/modules/customer-managment/components/projects-type/components/create-project-type/create-project-type.component.html

@@ -2,13 +2,9 @@
   <form style="width: 600px;">
     <div class="form-group">
       <input type="text" class="form-control form-control-sm" id="" aria-describedby="" placeholder="Name" />
-      <textarea
-        class="form-control form-control-sm mt-2"
-        id="descriptionTextArea"
-        rows="3"
-        placeholder="Description"
-      ></textarea>
-      <button type="submit" class="btn btn-sm btn-primary mb-2 mt-2">Save</button>
+      <textarea class="form-control form-control-sm mt-2" id="descriptionTextArea" rows="3"
+        placeholder="Description"></textarea>
+      <button type="submit" class="btn btn-sm btn-primary">Save</button>
       <button type="submit" class="btn btn-sm btn-secondary mb-2 ml-2 mt-2">Cancel</button>
     </div>
   </form>

src/app/modules/customer-managment/components/projects-type/components/create-project-type/create-project-type.component.spec.ts

@@ -18,7 +18,7 @@ describe('InputProjectTypeComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects-type/components/project-type-list/project-type-list.component.spec.ts

@@ -20,7 +20,7 @@ describe('ProjectTypeTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects/components/create-project/create-project.component.spec.ts

@@ -18,7 +18,7 @@ describe('InputProjectComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });

src/app/modules/customer-managment/components/projects/components/project-list/project-list.component.spec.ts

@@ -20,7 +20,7 @@ describe('ProjectTableListComponent', () => {
     fixture.detectChanges();
   });
 
-  it('component should create', () => {
+  it('component should be created', () => {
     expect(component).toBeTruthy();
   });
 });