Fix ORCID redirect URI double slash

src/app/core/auth/auth.service.spec.ts

@@ -343,6 +343,8 @@ describe('AuthService test', () => {
     });
 
     it('should return a token object', () => {
+      mockStore.overrideSelector(getAuthenticationToken, token);
+      mockStore.refreshState();
       const result = authService.getToken();
       expect(result).toEqual(token);
     });

src/app/core/orcid/orcid-auth.service.spec.ts

@@ -298,7 +298,7 @@ describe('OrcidAuthService', () => {
 
     it('should build the url properly', () => {
       const result = service.getOrcidAuthorizeUrl(mockItemUnlinkedToOrcid);
-      const redirectUri: string = new URLCombiner(nativeWindowService.nativeWindow.origin, encodeURIComponent(routerStub.url.split('?')[0])).toString();
+      const redirectUri: string = new URLCombiner(nativeWindowService.nativeWindow.origin, routerStub.url.split('?')[0]).toString();
       const url = 'orcid.authorize-url?client_id=orcid.application-client-id&redirect_uri=' + redirectUri + '&response_type=code&scope=/authenticate /read-limited';
 
       const expected = cold('(a|)', {

src/app/core/orcid/orcid-auth.service.ts

@@ -127,7 +127,7 @@ export class OrcidAuthService {
       this.configurationService.findByPropertyName('orcid.scope').pipe(getFirstSucceededRemoteDataPayload())],
     ).pipe(
       map(([authorizeUrl, clientId, scopes]) => {
-        const redirectUri = new URLCombiner(this._window.nativeWindow.origin, encodeURIComponent(this.router.url.split('?')[0]));
+        const redirectUri = new URLCombiner(this._window.nativeWindow.origin, this.router.url.split('?')[0]);
         return authorizeUrl.values[0] + '?client_id=' + clientId.values[0]   + '&redirect_uri=' + redirectUri + '&response_type=code&scope='
           + scopes.values.join(' ');
       }));