User now Mock as a suffix to match filenames

Author: tschaffter

client/app/main/main.component.spec.ts

@@ -1,9 +1,4 @@
-import {
-    async,
-    ComponentFixture,
-    inject,
-    TestBed,
-} from '@angular/core/testing';
+import { async, ComponentFixture, inject, TestBed } from '@angular/core/testing';
 import { HttpClientTestingModule, HttpTestingController } from '@angular/common/http/testing';
 import { expect } from 'chai';
 import { FormsModule } from '@angular/forms';
@@ -13,22 +8,19 @@ import { PageTitleService } from 'components/page-title/page-title.service';
 import { PageTitleServiceStub } from 'components/page-title/page-title.mock';
 import { MainComponent } from './main.component';
 
-describe('Component: MainComponent', function() {
+describe('Component: MainComponent', function () {
     let comp: MainComponent;
     let fixture: ComponentFixture<MainComponent>;
     let httpTestingController: HttpTestingController;
     // const mockThings = ['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express'];
 
     beforeEach(async(() => {
         TestBed.configureTestingModule({
-            imports: [
-                FormsModule,
-                HttpClientTestingModule,
-            ],
-            declarations: [ MainComponent ], // declare the test component
+            imports: [FormsModule, HttpClientTestingModule],
+            declarations: [MainComponent], // declare the test component
             providers: [
                 { provide: SocketService, useClass: SocketServiceStub },
-                { provide: PageTitleService, useClass: PageTitleServiceStub }
+                { provide: PageTitleService, useClass: PageTitleServiceStub },
             ],
         }).compileComponents();
 

client/components/activity/activity-sidenav/activity-sidenav.component.spec.ts

@@ -16,9 +16,9 @@ import { FiltersModule } from '../../../components/filters/filters.module';
 import { SecondarySidenavService } from '../../../components/sidenav/secondary-sidenav/secondary-sidenav.service';
 import { ProvenanceService } from '../../../components/provenance/provenance.service';
 import { SocketService } from '../../../components/socket/socket.service';
-import { MockProvenanceService } from './../../provenance/provenance.service.mock';
-import { MockSecondarySidenavService } from './../../sidenav/secondary-sidenav/secondary-sidenav.service.mock';
-import { MockSocketService } from './../../socket/socket.service.mock';
+import { ProvenanceServiceMock } from './../../provenance/provenance.service.mock';
+import { SecondarySidenavServiceMock } from './../../sidenav/secondary-sidenav/secondary-sidenav.service.mock';
+import { SocketServiceMock } from './../../socket/socket.service.mock';
 
 describe('ActivitySidenavComponent', () => {
     let component: ActivitySidenavComponent;
@@ -49,9 +49,9 @@ describe('ActivitySidenavComponent', () => {
             declarations: [ActivitySidenavComponent],
             imports: [MaterialModule, CommonModule, RouterTestingModule, FiltersModule],
             providers: [
-                { provide: ProvenanceService, useClass: MockProvenanceService },
-                { provide: SecondarySidenavService, useClass: MockSecondarySidenavService },
-                { provide: SocketService, useClass: MockSocketService },
+                { provide: ProvenanceService, useClass: ProvenanceServiceMock },
+                { provide: SecondarySidenavService, useClass: SecondarySidenavServiceMock },
+                { provide: SocketService, useClass: SocketServiceMock },
                 {
                     provide: HAMMER_LOADER, // this provider prevents warnings in the console regarding missing hammer.js module
                     useValue: () => new Promise(() => {}),

client/components/provenance/provenance.service.mock.ts

@@ -1,7 +1,7 @@
 import sinon from 'sinon';
 import { of } from 'rxjs';
 
-export class MockProvenanceService {
+export class ProvenanceServiceMock {
     getProvenanceGraphByAgent = sinon.fake.returns(of({ results: [] }));
     getProvenanceGraphByReference = sinon.fake.returns(of({ results: [] }));
 }

client/components/sidenav/secondary-sidenav/secondary-sidenav.service.mock.ts

@@ -1,6 +1,6 @@
 import sinon from 'sinon';
 
-export class MockSecondarySidenavService {
+export class SecondarySidenavServiceMock {
     close = sinon.spy();
     destroyContentComponent = sinon.spy();
 }

client/components/socket/socket.mock.ts

@@ -1,4 +1,4 @@
 export class SocketServiceStub {
-    syncUpdates() { }
-    unsyncUpdates() { }
+    syncUpdates() {}
+    unsyncUpdates() {}
 }

client/components/socket/socket.service.mock.ts

@@ -1,5 +1,5 @@
 import sinon from 'sinon';
 
-export class MockSocketService {
+export class SocketServiceMock {
     unsyncUpdates = sinon.spy();
 }

package.json

@@ -1,6 +1,6 @@
 {
     "name": "phccp",
-    "version": "1.0.0-alpha.7",
+    "version": "1.0.0-alpha.8",
     "main": "server/index.js",
     "dependencies": {
         "@babel/polyfill": "^7.4.4",