Fixed linting and js-api tests

lib/core/src/lib/pipes/date-time.pipe.spec.ts

@@ -14,6 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 import { DateTimePipe } from './date-time.pipe';
 import { addMinutes, isValid } from 'date-fns';
 

lib/extensions/src/lib/services/extension-loader.service.spec.ts

@@ -15,16 +15,14 @@
  * limitations under the License.
  */
 
-import { TestBed } from '@angular/core/testing';
-import { HttpClientTestingModule } from '@angular/common/http/testing';
+import { TestBed, fakeAsync, flushMicrotasks, tick } from '@angular/core/testing';
+import { HttpClientTestingModule, HttpTestingController } from '@angular/common/http/testing';
 import { ExtensionConfig } from '../config/extension.config';
 import { ExtensionLoaderService } from './extension-loader.service';
-import { HttpClient } from '@angular/common/http';
-import { of } from 'rxjs';
 
 describe('ExtensionLoaderService', () => {
     let extensionLoaderService: ExtensionLoaderService;
-    let httpClient: HttpClient;
+    let httpMock: HttpTestingController;
     let appExtensionsConfig: ExtensionConfig;
     const pluginConfig1: ExtensionConfig = {
         $id: 'test1',
@@ -34,33 +32,16 @@ describe('ExtensionLoaderService', () => {
         $license: 'MIT',
         $runtime: '2.6.1'
     };
-    const pluginConfig2: ExtensionConfig = {
-        $id: 'test2',
-        $name: 'test.extension.2',
-        $version: '1.0.0',
-        $vendor: 'Alfresco',
-        $license: 'MIT',
-        $runtime: '2.6.1'
-    };
-    const pluginConfig3: ExtensionConfig = {
-        $id: 'test3',
-        $name: 'test.extension.3',
-        $version: '1.0.0',
-        $vendor: 'Alfresco',
-        $license: 'MIT',
-        $runtime: '2.6.1'
-    };
 
     beforeEach(() => {
         TestBed.configureTestingModule({
             imports: [HttpClientTestingModule],
             providers: [
-                HttpClient,
                 ExtensionLoaderService
             ]
         });
         extensionLoaderService = TestBed.inject(ExtensionLoaderService);
-        httpClient = TestBed.inject(HttpClient);
+        httpMock = TestBed.inject(HttpTestingController);
 
         appExtensionsConfig = {
             $id: 'test',
@@ -72,35 +53,22 @@ describe('ExtensionLoaderService', () => {
             $references: [],
             $ignoreReferenceList: []
         };
-
-        spyOn(httpClient, 'get').and.callFake((url: string) => {
-            if (url === 'assets/app.extensions.json') {
-                return of(appExtensionsConfig);
-            }
-
-            if (url === 'assets/plugins/test.extension.1.json') {
-                return of(pluginConfig1);
-            }
-
-            if (url === 'assets/plugins/test.extension.2.json') {
-                return of(pluginConfig2);
-            }
-
-            if (url === 'assets/plugins/test.extension.3.json') {
-                return of(pluginConfig3);
-            }
-
-            return of(null);
-        });
     });
 
-    it('should load default registered app extensions when no custom $references defined', (done) => {
+    afterEach(() => {
+        httpMock.verify();
+    });
+
+    it('should load default registered app extensions when no custom $references defined', fakeAsync(() => {
         extensionLoaderService.load('assets/app.extensions.json', 'assets/plugins', ['test.extension.1.json']).then((config: ExtensionConfig) => {
             const pluginsReference = config.$references.map((entry: ExtensionConfig) => entry.$name);
             expect(pluginsReference).toEqual(['test.extension.1']);
-            done();
         });
-    });
+        httpMock.expectOne('assets/app.extensions.json').flush(appExtensionsConfig);
+        tick();
+        httpMock.expectOne('assets/plugins/test.extension.1.json').flush(pluginConfig1);
+        flushMicrotasks();
+    }));
 
     it('should ignore default registered app extension if defined in $ignoreReferenceList', (done) => {
         appExtensionsConfig.$ignoreReferenceList = ['test.extension.1.json'];
@@ -110,15 +78,16 @@ describe('ExtensionLoaderService', () => {
             expect(pluginsReference).toEqual([]);
             done();
         });
+
+        httpMock.expectOne('assets/app.extensions.json').flush(appExtensionsConfig);
     });
 
-    it('should load only extensions defined by $references', (done) => {
+    it('should load only extensions defined by $references', fakeAsync(() => {
         appExtensionsConfig.$references = ['test.extension.1.json'];
 
         extensionLoaderService.load('assets/app.extensions.json', 'assets/plugins', ['test.extension.2.json, test.extension.3.json']).then((config: ExtensionConfig) => {
             const pluginsReference = config.$references.map((entry: ExtensionConfig) => entry.$name);
             expect(pluginsReference).toEqual(['test.extension.1']);
-            done();
         });
     });
 

lib/process-services/src/lib/attachment/create-process-attachment/create-process-attachment.component.spec.ts

@@ -59,7 +59,7 @@ describe('CreateProcessAttachmentComponent', () => {
                 MatButtonModule,
                 MatIconModule
             ],
-            providers: [{ provide: AlfrescoApiService, useClass: AlfrescoApiServiceMock },]
+            providers: [{ provide: AlfrescoApiService, useClass: AlfrescoApiServiceMock }]
         });
         fixture = TestBed.createComponent(CreateProcessAttachmentComponent);
         component = fixture.componentInstance;