refactor: remove unused imports for better developer UX (#2066)

Author: SerkanSipahi

modules/data/spec/actions/entity-action-factory.spec.ts

@@ -1,11 +1,9 @@
 import {
-  EntityAction,
   EntityActionOptions,
   EntityActionPayload,
   EntityOp,
   EntityActionFactory,
   MergeStrategy,
-  CorrelationIdGenerator,
 } from '../../';
 
 class Hero {

modules/data/spec/actions/entity-action-operators.spec.ts

@@ -1,5 +1,4 @@
 import { Action } from '@ngrx/store';
-import { Actions } from '@ngrx/effects';
 
 import { Subject } from 'rxjs';
 

modules/data/spec/actions/entity-cache-changes-set.spec.ts

@@ -1,8 +1,4 @@
-import {
-  ChangeSet,
-  ChangeSetOperation,
-  changeSetItemFactory as cif,
-} from '../../';
+import { ChangeSetOperation, changeSetItemFactory as cif } from '../../';
 
 describe('changeSetItemFactory', () => {
   const hero = { id: 1, name: 'Hero 1' };

modules/data/spec/dataservices/data-service-error.spec.ts

@@ -1,5 +1,5 @@
 import { HttpErrorResponse } from '@angular/common/http';
-import { DataServiceError, RequestData } from '../../';
+import { DataServiceError } from '../../';
 
 describe('DataServiceError', () => {
   describe('#message', () => {

modules/data/spec/dataservices/default-data.service.spec.ts

@@ -1,6 +1,6 @@
 import { TestBed } from '@angular/core/testing';
 
-import { HttpClient, HttpResponse } from '@angular/common/http';
+import { HttpClient } from '@angular/common/http';
 import {
   HttpClientTestingModule,
   HttpTestingController,
@@ -14,7 +14,6 @@ import {
   DefaultDataService,
   DefaultDataServiceFactory,
   DefaultHttpUrlGenerator,
-  EntityHttpResourceUrls,
   HttpUrlGenerator,
   DefaultDataServiceConfig,
   DataServiceError,

modules/data/spec/dataservices/entity-data.service.spec.ts

@@ -7,11 +7,6 @@ import { Observable } from 'rxjs';
 import { Update } from '@ngrx/entity';
 
 import {
-  createEntityDefinition,
-  EntityDefinition,
-  EntityMetadata,
-  EntityMetadataMap,
-  ENTITY_METADATA_TOKEN,
   DefaultDataService,
   DefaultDataServiceFactory,
   HttpUrlGenerator,

modules/data/spec/effects/entity-cache-effects.spec.ts

@@ -2,17 +2,8 @@
 import { TestBed } from '@angular/core/testing';
 import { Action } from '@ngrx/store';
 import { Actions } from '@ngrx/effects';
-
-import {
-  asapScheduler,
-  Observable,
-  of,
-  merge,
-  ReplaySubject,
-  Subject,
-  throwError,
-} from 'rxjs';
-import { first, mergeMap, observeOn, tap } from 'rxjs/operators';
+import { observeOn } from 'rxjs/operators';
+import { asapScheduler, ReplaySubject, Subject } from 'rxjs';
 
 import {
   EntityCacheEffects,

modules/data/spec/effects/entity-effects.marbles.spec.ts

@@ -2,7 +2,7 @@
 import { TestBed } from '@angular/core/testing';
 
 import { cold, hot, getTestScheduler } from 'jasmine-marbles';
-import { Observable, of, Subject } from 'rxjs';
+import { Observable } from 'rxjs';
 
 import { Actions } from '@ngrx/effects';
 import { Update } from '@ngrx/entity';

modules/data/spec/effects/entity-effects.spec.ts

@@ -4,7 +4,7 @@ import { Action } from '@ngrx/store';
 import { Actions } from '@ngrx/effects';
 import { Update } from '@ngrx/entity';
 
-import { Observable, of, merge, ReplaySubject, throwError, timer } from 'rxjs';
+import { of, merge, ReplaySubject, throwError, timer } from 'rxjs';
 import { delay, first, mergeMap } from 'rxjs/operators';
 
 import {

modules/data/spec/entity-data.module.spec.ts

@@ -11,8 +11,8 @@ import { Actions, Effect, EffectsModule } from '@ngrx/effects';
 // Not using marble testing
 import { TestBed } from '@angular/core/testing';
 
-import { Observable, of, Subject } from 'rxjs';
-import { map, skip, tap } from 'rxjs/operators';
+import { Observable } from 'rxjs';
+import { map, skip } from 'rxjs/operators';
 
 import {
   EntityCache,