Skip to content

Commit

Permalink
[ACS-6576] Local storage prefix is not always created/updated properl…
Browse files Browse the repository at this point in the history
…y in ADW (#3602)

* [ACS-6576] set local storage prefix

* [ACS-6576] remove redundant async
  • Loading branch information
nikita-web-ua committed Jan 16, 2024
1 parent dc2c7c9 commit c77528c
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 3 deletions.
19 changes: 16 additions & 3 deletions projects/aca-shared/src/lib/services/app.service.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ import {
PageTitleService,
AlfrescoApiServiceMock,
TranslationMock,
TranslationService
TranslationService,
UserPreferencesService
} from '@alfresco/adf-core';
import { BehaviorSubject, Observable, of, Subject } from 'rxjs';
import { HttpClientModule } from '@angular/common/http';
Expand Down Expand Up @@ -66,6 +67,7 @@ describe('AppService', () => {
let sharedLinksApiService: SharedLinksApiService;
let contentApi: ContentApiService;
let groupService: GroupService;
let preferencesService: UserPreferencesService;

beforeEach(() => {
TestBed.configureTestingModule({
Expand Down Expand Up @@ -110,10 +112,12 @@ describe('AppService', () => {
useValue: {
onLogin: new Subject<any>(),
onLogout: new Subject<any>(),
isLoggedIn: () => false
isLoggedIn: () => false,
getUsername: () => null
}
},
{ provide: TranslationService, useClass: TranslationMock }
{ provide: TranslationService, useClass: TranslationMock },
{ provide: UserPreferencesService, useValue: { setStoragePrefix: () => null } }
]
});

Expand All @@ -126,6 +130,7 @@ describe('AppService', () => {
contentApi = TestBed.inject(ContentApiService);
groupService = TestBed.inject(GroupService);
service = TestBed.inject(AppService);
preferencesService = TestBed.inject(UserPreferencesService);
});

it('should be ready if [withCredentials] mode is used', (done) => {
Expand All @@ -152,6 +157,14 @@ describe('AppService', () => {
await expect(isReady).toEqual(true);
});

it('should set local storage prefix after login', () => {
spyOn(preferencesService, 'setStoragePrefix');
spyOn(auth, 'getUsername').and.returnValue('test-username');
auth.onLogin.next();

expect(preferencesService.setStoragePrefix).toHaveBeenCalledWith('test-username');
});

it('should reset search to defaults upon logout', async () => {
const resetToDefaults = spyOn(searchQueryBuilderService, 'resetToDefaults');
auth.onLogout.next(true);
Expand Down
1 change: 1 addition & 0 deletions projects/aca-shared/src/lib/services/app.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ export class AppService implements OnDestroy {

this.authenticationService.onLogin.subscribe(() => {
this.ready.next(true);
this.preferencesService.setStoragePrefix(this.authenticationService.getUsername());
});

this.authenticationService.onLogout.subscribe(() => {
Expand Down

0 comments on commit c77528c

Please sign in to comment.