destroy fixtures to prevent random unit test failures

src/app/dashboard/dashboard.component.spec.ts

@@ -33,6 +33,10 @@ describe("DashboardComponent", () => {
     fixture.detectChanges();
   });
 
+  afterEach(() => {
+    fixture.destroy();
+  });
+
   it("should be created", () => {
     console.log ("should be created");
     expect(component).toBeTruthy();

src/app/dataset-detail/dataset-detail.component.spec.ts

@@ -13,7 +13,6 @@ import {
   MockPublishedDataApi
 } from "../MockStubs";
 import { NgxJsonLdModule } from "@ngx-lite/json-ld";
-import { PublishedDataApi } from "../shared/sdk/services/custom";
 import { RouterTestingModule } from "@angular/router/testing";
 import { async, ComponentFixture, TestBed } from "@angular/core/testing";
 
@@ -31,7 +30,6 @@ describe("DatasetDetailComponent", () => {
       declarations: [DatasetDetailComponent, FileSizePipe],
       providers: [
         { provide: HttpClient, useClass: MockHttp },
-        { provide: PublishedDataApi, useClass: MockPublishedDataApi },
         { provide: DatasetService, useClass: MockDatasetService },
         { provide: NgxJsonLdModule, useClass: MockNgx },
         { provide: APP_CONFIG }
@@ -46,6 +44,10 @@ describe("DatasetDetailComponent", () => {
     fixture.detectChanges();
   });
 
+  afterEach(() => {
+    fixture.destroy();
+  });
+
   it("should be created", () => {
     expect(component).toBeTruthy();
   });

src/app/datasets/datasets.component.spec.ts

@@ -5,9 +5,7 @@ import { HttpClient } from "@angular/common/http";
 import { InMemoryDataService } from "../in-memory-data.service";
 import { InMemoryWebApiModule } from "angular-in-memory-web-api";
 import { MatCardModule } from "@angular/material";
-import { MockDatasetService, MockHttp, MockPublishedDataApi } from "../MockStubs";
-import { NgxDatatableModule } from "@swimlane/ngx-datatable";
-import { PublishedDataApi } from "../shared/sdk/services/custom";
+import { MockDatasetService, MockHttp } from "../MockStubs";
 import { RouterTestingModule } from "@angular/router/testing";
 import { async, ComponentFixture, TestBed } from "@angular/core/testing";
 
@@ -19,14 +17,12 @@ describe("DatasetsComponent", () => {
     TestBed.configureTestingModule({
       imports: [
         RouterTestingModule,
-        NgxDatatableModule,
         MatCardModule,
         InMemoryWebApiModule.forRoot(InMemoryDataService)
       ],
       declarations: [DatasetsComponent],
       providers: [
         { provide: HttpClient, useClass: MockHttp },
-        { provide: PublishedDataApi, useClass: MockPublishedDataApi },
         { provide: DatasetService, useClass: MockDatasetService },
         { provide: APP_CONFIG }
       ]
@@ -40,6 +36,10 @@ describe("DatasetsComponent", () => {
     fixture.detectChanges();
   });
 
+  afterEach(() => {
+    fixture.destroy();
+  });
+
   it("should be created", () => {
     expect(component).toBeTruthy();
   });

src/app/datasets/datasets.component.ts

@@ -3,14 +3,10 @@ import { Component, Inject, OnInit } from "@angular/core";
 import { DatasetService } from "../dataset.service";
 import { PublishedData } from "../shared/sdk/models";
 import { APP_CONFIG, AppConfig } from "../app-config.module";
-import { HttpClient, HttpHeaders } from "@angular/common/http";
+import { HttpClient } from "@angular/common/http";
 import { map } from "rxjs/operators";
 import { Router } from "@angular/router";
 
-const httpOptions = {
-  headers: new HttpHeaders({ "Content-Type": "application/json" })
-};
-
 interface MyType {
   doi: string;
   value: string;