[DURACOM-327] Fix lint

Author: atarix83

modules/core/src/lib/core/admin/admin-notify-message/admin-notify-messages.service.spec.ts

@@ -6,15 +6,29 @@ import {
 } from 'rxjs';
 import { take } from 'rxjs/operators';
 
+import { mockAdminNotifyMessages } from '../../../../../../../src/app/admin/admin-notify-dashboard/admin-notify-search-result/admin-notify-search-result.component.spec';
 import {
-  mockAdminNotifyMessages
-} from '../../../../../../../src/app/admin/admin-notify-dashboard/admin-notify-search-result/admin-notify-search-result.component.spec';
-import { AdminNotifyMessage, AdminNotifyMessagesService } from './';
-import { ItemDataService, RemoteData, RequestEntry, RequestEntryState, RequestService } from '../../data';
-import { ObjectCacheService, RemoteDataBuildService, RestResponse } from '../../cache';
+  createSuccessfulRemoteDataObject$,
+  NotificationsService,
+} from '../../';
+import {
+  ObjectCacheService,
+  RemoteDataBuildService,
+  RestResponse,
+} from '../../cache';
+import {
+  ItemDataService,
+  RemoteData,
+  RequestEntry,
+  RequestEntryState,
+  RequestService,
+} from '../../data';
 import { HALEndpointService } from '../../shared';
 import { LdnServicesService } from '../';
-import { createSuccessfulRemoteDataObject$, NotificationsService } from '../../';
+import {
+  AdminNotifyMessage,
+  AdminNotifyMessagesService,
+} from './';
 
 describe('AdminNotifyMessagesService test', () => {
   let service: AdminNotifyMessagesService;

modules/core/src/lib/core/admin/ldn-services-data/ldn-services-data.service.spec.ts

@@ -5,35 +5,31 @@ import {
 import { of as observableOf } from 'rxjs';
 import { TestScheduler } from 'rxjs/testing';
 
-import { testFindAllDataImplementation } from '../../data/base/find-all-data.spec';
+import { mockLdnService } from '../../../../../../../src/app/admin/admin-ldn-services/ldn-service-serviceMock/ldnServicesRD$-mock';
+import { RemoteDataBuildService } from '../../cache/builders/remote-data-build.service';
+import { RequestParam } from '../../cache/models/request-param.model';
+import { ObjectCacheService } from '../../cache/object-cache.service';
+import { RestResponse } from '../../cache/response.models';
+import { CreateData } from '../../data/base/create-data';
+import { testCreateDataImplementation } from '../../data/base/create-data.spec';
+import { DeleteData } from '../../data/base/delete-data';
 import { testDeleteDataImplementation } from '../../data/base/delete-data.spec';
-import { testSearchDataImplementation } from '../../data/base/search-data.spec';
+import { FindAllData } from '../../data/base/find-all-data';
+import { testFindAllDataImplementation } from '../../data/base/find-all-data.spec';
+import { PatchData } from '../../data/base/patch-data';
 import { testPatchDataImplementation } from '../../data/base/patch-data.spec';
-import { testCreateDataImplementation } from '../../data/base/create-data.spec';
-import { createPaginatedList } from '../../utilities/testing/utils.test';
-import {
-  mockLdnService
-} from '../../../../../../../src/app/admin/admin-ldn-services/ldn-service-serviceMock/ldnServicesRD$-mock';
-import { LdnServicesService } from './ldn-services-data.service';
+import { SearchData } from '../../data/base/search-data';
+import { testSearchDataImplementation } from '../../data/base/search-data.spec';
+import { FindListOptions } from '../../data/find-list-options.model';
+import { RemoteData } from '../../data/remote-data';
 import { RequestService } from '../../data/request.service';
-import { RemoteDataBuildService } from '../../cache/builders/remote-data-build.service';
-import { ObjectCacheService } from '../../cache/object-cache.service';
-import { HALEndpointService } from '../../shared/hal-endpoint.service';
-import { NotificationsService } from '../../notifications/notifications.service';
 import { RequestEntry } from '../../data/request-entry.model';
-import { RemoteData } from '../../data/remote-data';
 import { RequestEntryState } from '../../data/request-entry-state.model';
+import { NotificationsService } from '../../notifications/notifications.service';
+import { HALEndpointService } from '../../shared/hal-endpoint.service';
 import { createSuccessfulRemoteDataObject$ } from '../../utilities/remote-data.utils';
-import { RequestParam } from '../../cache/models/request-param.model';
-import { FindListOptions } from '../../data/find-list-options.model';
-import { FindAllData } from '../../data/base/find-all-data';
-import { DeleteData } from '../../data/base/delete-data';
-import { SearchData } from '../../data/base/search-data';
-import { PatchData } from '../../data/base/patch-data';
-import { CreateData } from '../../data/base/create-data';
-import { RestResponse } from '../../cache/response.models';
-
-
+import { createPaginatedList } from '../../utilities/testing/utils.test';
+import { LdnServicesService } from './ldn-services-data.service';
 
 describe('LdnServicesService test', () => {
   let scheduler: TestScheduler;

package.json

@@ -24,7 +24,7 @@
     "test:lint": "npm run build:lint && npm run test:lint:nobuild",
     "test:lint:nobuild": "jasmine --config=lint/jasmine.json",
     "lint": "npm run build:lint && npm run lint:nobuild",
-    "lint:nobuild": "nx lint",
+    "lint:nobuild": "nx lint --quiet",
     "lint-fix": "npm run build:lint && nx lint --fix=true",
     "docs:lint": "ts-node --project ./lint/tsconfig.json ./lint/generate-docs.ts",
     "e2e": "cross-env NODE_ENV=production nx e2e",

src/app/access-control/access-control-routes.ts

@@ -1,14 +1,16 @@
 import { AbstractControl } from '@angular/forms';
 import { Route } from '@angular/router';
+import {
+  EPersonResolver,
+  groupAdministratorGuard,
+  i18nBreadcrumbResolver,
+  siteAdministratorGuard,
+} from '@dspace/core';
 import {
   DYNAMIC_ERROR_MESSAGES_MATCHER,
   DynamicErrorMessagesMatcher,
 } from '@ng-dynamic-forms/core';
 
-import { i18nBreadcrumbResolver } from '@dspace/core';
-import { groupAdministratorGuard } from '@dspace/core';
-import { siteAdministratorGuard } from '@dspace/core';
-import { EPersonResolver } from '@dspace/core';
 import {
   EPERSON_PATH,
   GROUP_PATH,

src/app/access-control/access-control-routing-paths.ts

@@ -1,6 +1,7 @@
-import { getAccessControlModuleRoute } from '../app-routing-paths';
 import { URLCombiner } from '@dspace/core';
 
+import { getAccessControlModuleRoute } from '../app-routing-paths';
+
 export const EPERSON_PATH = 'epeople';
 
 export function getEPersonsRoute(): string {

src/app/access-control/bulk-access/browse/bulk-access-browse.component.spec.ts

@@ -4,21 +4,23 @@ import {
   TestBed,
   waitForAsync,
 } from '@angular/core/testing';
+import {
+  buildPaginatedList,
+  createSuccessfulRemoteDataObject,
+  PageInfo,
+  SelectableListService,
+} from '@dspace/core';
 import {
   NgbAccordionModule,
   NgbNavModule,
 } from '@ng-bootstrap/ng-bootstrap';
 import { TranslateModule } from '@ngx-translate/core';
 import { of } from 'rxjs';
 
-import { buildPaginatedList } from '@dspace/core';
-import { PageInfo } from '@dspace/core';
-import { createSuccessfulRemoteDataObject } from '@dspace/core';
+import { SelectableObject } from '../../../../../modules/core/src/lib/core/states/selectable-list/selectable-list.service.spec';
 import { getMockThemeService } from '../../../shared/mocks/theme-service.mock';
 import { ListableObjectComponentLoaderComponent } from '../../../shared/object-collection/shared/listable-object/listable-object-component-loader.component';
 import { SelectableListItemControlComponent } from '../../../shared/object-collection/shared/selectable-list-item-control/selectable-list-item-control.component';
-import { SelectableListService } from '@dspace/core';
-import { SelectableObject } from '../../../../../modules/core/src/lib/core/states/selectable-list/selectable-list.service.spec';
 import { PaginationComponent } from '../../../shared/pagination/pagination.component';
 import { ThemedSearchComponent } from '../../../shared/search/themed-search.component';
 import { ThemeService } from '../../../shared/theme-support/theme.service';

src/app/access-control/bulk-access/browse/bulk-access-browse.component.ts

@@ -5,6 +5,18 @@ import {
   OnDestroy,
   OnInit,
 } from '@angular/core';
+import {
+  buildPaginatedList,
+  createSuccessfulRemoteDataObject,
+  ListableObject,
+  PageInfo,
+  PaginatedList,
+  PaginationComponentOptions,
+  RemoteData,
+  SearchConfigurationService,
+  SelectableListService,
+  SelectableListState,
+} from '@dspace/core';
 import { hasValue } from '@dspace/shared/utils';
 import {
   NgbAccordionModule,
@@ -21,21 +33,9 @@ import {
   map,
 } from 'rxjs/operators';
 
-import {
-  buildPaginatedList,
-  PaginatedList,
-} from '@dspace/core';
-import { RemoteData } from '@dspace/core';
-import { ListableObject } from '@dspace/core';
-import { PageInfo } from '@dspace/core';
-import { PaginationComponentOptions } from '@dspace/core';
-import { SearchConfigurationService } from '@dspace/core';
-import { createSuccessfulRemoteDataObject } from '@dspace/core';
 import { SEARCH_CONFIG_SERVICE } from '../../../my-dspace-page/my-dspace-configuration.service';
 import { ListableObjectComponentLoaderComponent } from '../../../shared/object-collection/shared/listable-object/listable-object-component-loader.component';
 import { SelectableListItemControlComponent } from '../../../shared/object-collection/shared/selectable-list-item-control/selectable-list-item-control.component';
-import { SelectableListState } from '@dspace/core';
-import { SelectableListService } from '@dspace/core';
 import { PaginationComponent } from '../../../shared/pagination/pagination.component';
 import { ThemedSearchComponent } from '../../../shared/search/themed-search.component';
 import { BrowserOnlyPipe } from '../../../shared/utils/browser-only.pipe';

src/app/access-control/bulk-access/bulk-access.component.spec.ts

@@ -4,17 +4,19 @@ import {
   TestBed,
 } from '@angular/core/testing';
 import { RouterTestingModule } from '@angular/router/testing';
+import {
+  createSuccessfulRemoteDataObject$,
+  NotificationsService,
+  NotificationsServiceStub,
+  Process,
+  SelectableListService,
+  SelectableListState,
+} from '@dspace/core';
 import { TranslateModule } from '@ngx-translate/core';
 import { of } from 'rxjs';
 
-import { NotificationsService } from '@dspace/core';
-import { Process } from '@dspace/core';
-import { createSuccessfulRemoteDataObject$ } from '@dspace/core';
-import { NotificationsServiceStub } from '@dspace/core';
 import { BulkAccessControlService } from '../../shared/access-control-form-container/bulk-access-control.service';
 import { getMockThemeService } from '../../shared/mocks/theme-service.mock';
-import { SelectableListState } from '@dspace/core';
-import { SelectableListService } from '@dspace/core';
 import { ThemeService } from '../../shared/theme-support/theme.service';
 import { BulkAccessComponent } from './bulk-access.component';
 import { BulkAccessSettingsComponent } from './settings/bulk-access-settings.component';

src/app/access-control/bulk-access/bulk-access.component.ts

@@ -3,6 +3,10 @@ import {
   OnInit,
   ViewChild,
 } from '@angular/core';
+import {
+  SelectableListService,
+  SelectableListState,
+} from '@dspace/core';
 import { TranslateModule } from '@ngx-translate/core';
 import {
   BehaviorSubject,
@@ -15,8 +19,6 @@ import {
 
 import { BulkAccessControlService } from '../../shared/access-control-form-container/bulk-access-control.service';
 import { BtnDisabledDirective } from '../../shared/btn-disabled.directive';
-import { SelectableListState } from '@dspace/core';
-import { SelectableListService } from '@dspace/core';
 import { BulkAccessBrowseComponent } from './browse/bulk-access-browse.component';
 import { BulkAccessSettingsComponent } from './settings/bulk-access-settings.component';
 

src/app/access-control/epeople-registry/epeople-registry.component.spec.ts

@@ -20,6 +20,25 @@ import {
 } from '@angular/platform-browser';
 import { Router } from '@angular/router';
 import { RouterTestingModule } from '@angular/router/testing';
+import {
+  AuthorizationDataService,
+  buildPaginatedList,
+  createSuccessfulRemoteDataObject$,
+  EPerson,
+  EPersonDataService,
+  EPersonMock,
+  EPersonMock2,
+  FindListOptions,
+  NotificationsService,
+  NotificationsServiceStub,
+  PageInfo,
+  PaginatedList,
+  PaginationService,
+  PaginationServiceStub,
+  RemoteData,
+  RequestService,
+  RouterMock,
+} from '@dspace/core';
 import {
   NgbModal,
   NgbModule,
@@ -30,27 +49,6 @@ import {
   of as observableOf,
 } from 'rxjs';
 
-import { AuthorizationDataService } from '@dspace/core';
-import { FindListOptions } from '@dspace/core';
-import {
-  buildPaginatedList,
-  PaginatedList,
-} from '@dspace/core';
-import { RemoteData } from '@dspace/core';
-import { RequestService } from '@dspace/core';
-import { EPersonDataService } from '@dspace/core';
-import { EPerson } from '@dspace/core';
-import { RouterMock } from '@dspace/core';
-import { NotificationsService } from '@dspace/core';
-import { PaginationService } from '@dspace/core';
-import { PageInfo } from '@dspace/core';
-import { createSuccessfulRemoteDataObject$ } from '@dspace/core';
-import {
-  EPersonMock,
-  EPersonMock2,
-} from '@dspace/core';
-import { NotificationsServiceStub } from '@dspace/core';
-import { PaginationServiceStub } from '@dspace/core';
 import { BtnDisabledDirective } from '../../shared/btn-disabled.directive';
 import { FormBuilderService } from '../../shared/form/builder/form-builder.service';
 import { ThemedLoadingComponent } from '../../shared/loading/themed-loading.component';