Skip to content

Commit 3459b85

Browse files
Mattia VianelliMattia Vianelli
Mattia Vianelli
authored and
Mattia Vianelli
committed
DSC-2764 Lint fix
1 parent 9425e05 commit 3459b85

5 files changed

Lines changed: 18 additions & 82 deletions

File tree

‎src/app/lucky-search/search/lucky-search.component.spec.ts‎

Lines changed: 10 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ import {
1414
import { TranslateModule } from '@ngx-translate/core';
1515
import { of } from 'rxjs';
1616

17+
import { APP_CONFIG } from '../../../config/app-config.interface';
1718
import { getBitstreamDownloadRoute } from '../../app-routing-paths';
1819
import {
1920
SortDirection,
@@ -52,34 +53,6 @@ import { TruncatablePartComponent } from '../../shared/truncatable/truncatable-p
5253
import { FileSizePipe } from '../../shared/utils/file-size-pipe';
5354
import { LuckySearchService } from '../lucky-search.service';
5455
import { LuckySearchComponent } from './lucky-search.component';
55-
import {ComponentFixture, TestBed} from '@angular/core/testing';
56-
57-
import {LuckySearchComponent} from './lucky-search.component';
58-
import {LuckySearchService} from '../lucky-search.service';
59-
import {SearchConfigurationService} from '../../core/shared/search/search-configuration.service';
60-
import {Router, UrlTree} from '@angular/router';
61-
import {createSuccessfulRemoteDataObject, createSuccessfulRemoteDataObject$} from '../../shared/remote-data.utils';
62-
import {createPaginatedList} from '../../shared/testing/utils.test';
63-
import {Item} from '../../core/shared/item.model';
64-
import {of as observableOf} from 'rxjs';
65-
import {PaginatedSearchOptions} from '../../shared/search/models/paginated-search-options.model';
66-
import {PaginationComponentOptions} from '../../shared/pagination/pagination-component-options.model';
67-
import {SortDirection, SortOptions} from '../../core/cache/models/sort-options.model';
68-
import {TranslateModule} from '@ngx-translate/core';
69-
import {By} from '@angular/platform-browser';
70-
import {SearchResult} from '../../shared/search/models/search-result.model';
71-
import {DSpaceObject} from '../../core/shared/dspace-object.model';
72-
import {BitstreamDataService, MetadataFilter} from '../../core/data/bitstream-data.service';
73-
import {Bitstream} from '../../core/shared/bitstream.model';
74-
import {RouterMock} from '../../shared/mocks/router.mock';
75-
import {MetadataMap, MetadataValue} from '../../core/shared/metadata.models';
76-
import {FileSizePipe} from '../../shared/utils/file-size-pipe';
77-
import {HardRedirectService} from '../../core/services/hard-redirect.service';
78-
import {getBitstreamDownloadRoute} from '../../app-routing-paths';
79-
import {PLATFORM_ID} from '@angular/core';
80-
import {NotificationsService} from '../../shared/notifications/notifications.service';
81-
import {NotificationsServiceStub} from '../../shared/testing/notifications-service.stub';
82-
import {APP_CONFIG} from '../../../config/app-config.interface';
8356

8457
describe('LuckySearchComponent', () => {
8558
let fixture: ComponentFixture<LuckySearchComponent>;
@@ -159,7 +132,7 @@ describe('LuckySearchComponent', () => {
159132
{ provide: HardRedirectService, useValue: hardRedirectService },
160133
{ provide: PLATFORM_ID, useValue: 'browser' },
161134
{ provide: NotificationsService, useValue: new NotificationsServiceStub() },
162-
{provide: APP_CONFIG, useValue: {}},
135+
{ provide: APP_CONFIG, useValue: {} },
163136
],
164137
}).overrideComponent(LuckySearchComponent, {
165138
remove: {
@@ -395,11 +368,11 @@ describe('LuckySearchComponent', () => {
395368
// @ts-ignore: Accessing private method for testing
396369
component.appConfig = {
397370
luckySearchRedirects: {
398-
default: 301
399-
}
371+
default: 301,
372+
},
400373
} as any;
401374
// @ts-ignore: Accessing private method for testing
402-
component.currentFilter = {identifier: 'unknown'};
375+
component.currentFilter = { identifier: 'unknown' };
403376

404377
// @ts-ignore: Accessing private method for testing
405378
const result = component.getRedirectCode();
@@ -410,10 +383,10 @@ describe('LuckySearchComponent', () => {
410383
it('should return 302 when default is not set and identifier is not found', () => {
411384
// @ts-ignore: Accessing private method for testing
412385
component.appConfig = {
413-
luckySearchRedirects: {}
386+
luckySearchRedirects: {},
414387
} as any;
415388
// @ts-ignore: Accessing private method for testing
416-
component.currentFilter = {identifier: 'unknown'};
389+
component.currentFilter = { identifier: 'unknown' };
417390

418391
// @ts-ignore: Accessing private method for testing
419392
const result = component.getRedirectCode();
@@ -425,11 +398,11 @@ describe('LuckySearchComponent', () => {
425398
component.appConfig = {
426399
luckySearchRedirects: {
427400
default: 302,
428-
'legacy-id': 301
429-
}
401+
'legacy-id': 301,
402+
},
430403
};
431404
// @ts-ignore: Accessing private method for testing
432-
component.currentFilter = {identifier: 'legacy-id'};
405+
component.currentFilter = { identifier: 'legacy-id' };
433406

434407
// @ts-ignore: Accessing private method for testing
435408
const result = component.getRedirectCode();

‎src/app/lucky-search/search/lucky-search.component.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,10 @@ import {
3030
withLatestFrom,
3131
} from 'rxjs/operators';
3232

33+
import {
34+
APP_CONFIG,
35+
AppConfig,
36+
} from '../../../config/app-config.interface';
3337
import { getBitstreamDownloadRoute } from '../../app-routing-paths';
3438
import {
3539
BitstreamDataService,
@@ -64,7 +68,6 @@ import { TruncatableComponent } from '../../shared/truncatable/truncatable.compo
6468
import { TruncatablePartComponent } from '../../shared/truncatable/truncatable-part/truncatable-part.component';
6569
import { FileSizePipe } from '../../shared/utils/file-size-pipe';
6670
import { LuckySearchService } from '../lucky-search.service';
67-
import {APP_CONFIG, AppConfig} from '../../../config/app-config.interface';
6871

6972
@Component({
7073
selector: 'ds-lucky-search',

‎src/config/app-config.interface.ts‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ import {
3838
LayoutConfig,
3939
} from './layout-config.interfaces';
4040
import { LoaderConfig } from './loader-config.interfaces';
41+
import { LuckySearchRedirectConfig } from './lucky-search-redirect-config';
4142
import { MarkdownConfig } from './markdown-config.interface';
4243
import { MatomoConfig } from './matomo-config.interface';
4344
import { MediaViewerConfig } from './media-viewer-config.interface';
@@ -56,7 +57,6 @@ import { SubmissionConfig } from './submission-config.interface';
5657
import { SuggestionConfig } from './suggestion-config.interfaces';
5758
import { ThemeConfig } from './theme.config';
5859
import { UIServerConfig } from './ui-server-config.interface';
59-
import {LuckySearchRedirectConfig} from './lucky-search-redirect-config';
6060

6161
interface AppConfig extends Config {
6262
ui: UIServerConfig;
@@ -110,9 +110,6 @@ interface AppConfig extends Config {
110110
metadataLinkViewPopoverData: MetadataLinkViewPopoverDataConfig;
111111
identifierSubtypes: IdentifierSubtypesConfig[];
112112
datadogRum?: DatadogRumConfig;
113-
location: LocationConfig;
114-
epflUnpaywallMetadata: EpflUnpaywallMetadata;
115-
loader: LoaderConfig;
116113
permanentRedirectPaths?: string[];
117114
luckySearchRedirects?: LuckySearchRedirectConfig;
118115
}

‎src/config/default-app-config.ts‎

Lines changed: 2 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ import {
3838
SuggestionConfig,
3939
} from './layout-config.interfaces';
4040
import { LoaderConfig } from './loader-config.interfaces';
41+
import { LuckySearchRedirectConfig } from './lucky-search-redirect-config';
4142
import { MarkdownConfig } from './markdown-config.interface';
4243
import { MatomoConfig } from './matomo-config.interface';
4344
import { MediaViewerConfig } from './media-viewer-config.interface';
@@ -55,36 +56,6 @@ import { ServerConfig } from './server-config.interface';
5556
import { SubmissionConfig } from './submission-config.interface';
5657
import { ThemeConfig } from './theme.config';
5758
import { UIServerConfig } from './ui-server-config.interface';
58-
import { BundleConfig } from './bundle-config.interface';
59-
import { ActuatorsConfig } from './actuators.config';
60-
import { InfoConfig } from './info-config.interface';
61-
import { CommunityListConfig } from './community-list-config.interface';
62-
import { HomeConfig } from './homepage-config.interface';
63-
import { MarkdownConfig } from './markdown-config.interface';
64-
import { FilterVocabularyConfig } from './filter-vocabulary-config';
65-
import { DiscoverySortConfig } from './discovery-sort.config';
66-
import { AddToAnyPluginConfig } from './add-to-any-plugin-config';
67-
import { CmsMetadata } from './cms-metadata';
68-
import { CrisLayoutConfig, LayoutConfig, SuggestionConfig } from './layout-config.interfaces';
69-
import { MetadataSecurityConfig } from './metadata-security-config';
70-
import { FollowAuthorityMetadata } from './search-follow-metadata.interface';
71-
import { MetricVisualizationConfig } from './metric-visualization-config.interfaces';
72-
import {
73-
AdvancedAttachmentElementType,
74-
AdvancedAttachmentRenderingConfig
75-
} from './advanced-attachment-rendering.config';
76-
import { AttachmentRenderingConfig } from './attachment-rendering.config';
77-
import { SearchResultConfig } from './search-result-config.interface';
78-
import { VirtualCollectionConfig } from './virtual-collection-config.interface';
79-
import { EpflUnpaywallMetadata } from './epfl-unpaywall-metadata';
80-
import { MiradorConfig } from './mirador-config.interfaces';
81-
import { LoaderConfig } from './loader-config.interfaces';
82-
import { MetaTagsConfig } from './meta-tags.config';
83-
import { MetadataLinkViewPopoverDataConfig } from './metadata-link-view-popoverdata-config.interface';
84-
import { IdentifierSubtypesConfig, IdentifierSubtypesIconPositionEnum } from './identifier-subtypes-config.interface';
85-
import { DatadogRumConfig } from './datadog-rum-config.interfaces';
86-
import { LocationConfig } from './location-config.interface';
87-
import {LuckySearchRedirectConfig} from './lucky-search-redirect-config';
8859

8960
export class DefaultAppConfig implements AppConfig {
9061
production = false;
@@ -1085,16 +1056,8 @@ export class DefaultAppConfig implements AppConfig {
10851056
defaultPrivacyLevel: 'mask-user-input',
10861057
};
10871058

1088-
location: LocationConfig = {
1089-
nominatimApi: {
1090-
searchEndpoint: 'https://nominatim.openstreetmap.org/search',
1091-
reverseSearchEndpoint: 'https://nominatim.openstreetmap.org/reverse',
1092-
statusEndpoint: 'https://nominatim.openstreetmap.org/status',
1093-
}
1094-
};
1095-
10961059
luckySearchRedirects: LuckySearchRedirectConfig = {
10971060
'legacy-id': 301,
1098-
default: 302
1061+
default: 302,
10991062
};
11001063
}

‎src/config/lucky-search-redirect-config.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {Config} from './config.interface';
1+
import { Config } from './config.interface';
22

33
export interface LuckySearchRedirectConfig extends Config {
44
[indexName: string]: number;

0 commit comments

Comments
 (0)