Merged in task/dspace-cris-2023_02_x/DSC-2443 (pull request DSpace#3571)

[DSC-2443] fix header e2e tests

Approved-by: Francesco Molinaro

Author: Andrea Barbasso

src/app/shared/lang-switch/lang-switch.component.html

@@ -1,21 +1,24 @@
-<div ngbDropdown class="navbar-nav" *ngIf="moreThanOneLanguage" display="dynamic" placement="bottom-right">
-  <a href="javascript:void(0);" role="menuitem"
-     [attr.aria-label]="'nav.language' |translate"
-     aria-controls="language-menu-list"
-     aria-haspopup="menu"
-     [title]="'nav.language' | translate"
-     (click)="$event.preventDefault()" data-toggle="dropdown" ngbDropdownToggle
-     data-test="lang-switch"
-     tabindex="0">
-    <i class="fas fa-globe-asia fa-lg fa-fw"></i>
-  </a>
-  <ul ngbDropdownMenu class="dropdown-menu" [attr.aria-label]="'nav.language' |translate" id="language-menu-list" role="menu">
-    <li class="dropdown-item" tabindex="0" #langSelect *ngFor="let lang of translate.getLangs()"
-        role="menuitem"
-            (keyup.enter)="useLang(lang)"
-            (click)="useLang(lang)"
-            [class.active]="lang === translate.currentLang">
-                {{ langLabel(lang) }}
-    </li>
-  </ul>
-</div>
+<ng-container *dsRenderOnlyForBrowser="true">
+  <div ngbDropdown class="navbar-nav" *ngIf="moreThanOneLanguage" display="dynamic" placement="bottom-right">
+    <a href="javascript:void(0);" role="menuitem"
+       [attr.aria-label]="'nav.language' |translate"
+       aria-controls="language-menu-list"
+       aria-haspopup="menu"
+       [title]="'nav.language' | translate"
+       (click)="$event.preventDefault()" data-toggle="dropdown" ngbDropdownToggle
+       data-test="lang-switch"
+       tabindex="0">
+      <i class="fas fa-globe-asia fa-lg fa-fw"></i>
+    </a>
+    <ul ngbDropdownMenu class="dropdown-menu" [attr.aria-label]="'nav.language' | translate"
+        id="language-menu-list" role="menu">
+      <li class="dropdown-item" tabindex="0" #langSelect *ngFor="let lang of translate.getLangs()"
+          role="menuitem"
+          (keyup.enter)="useLang(lang)"
+          (click)="useLang(lang)"
+          [class.active]="lang === translate.currentLang">
+        {{ langLabel(lang) }}
+      </li>
+    </ul>
+  </div>
+</ng-container>

src/app/shared/lang-switch/lang-switch.component.spec.ts

@@ -9,6 +9,7 @@ import { HttpClientTestingModule, HttpTestingController } from '@angular/common/
 import { LangSwitchComponent } from './lang-switch.component';
 import { LangConfig } from '../../../config/lang-config.interface';
 import { LocaleService } from '../../core/locale/locale.service';
+import { BrowserOnlyDirective } from '../utils/browser-only.directive';
 
 // This test is completely independent from any message catalogs or keys in the codebase
 // The translation module is instantiated with these bogus messages that we aren't using anyway.
@@ -67,7 +68,9 @@ describe('LangSwitchComponent', () => {
       };
 
       TestBed.configureTestingModule({
-        imports: [HttpClientTestingModule, TranslateModule.forRoot(
+        imports: [HttpClientTestingModule,
+          BrowserOnlyDirective,
+          TranslateModule.forRoot(
           {
             loader: { provide: TranslateLoader, useClass: CustomLoader }
           }

src/app/shared/shared.module.ts

@@ -364,6 +364,7 @@ import { MetadataLinkViewOrcidComponent } from './metadata-link-view/metadata-li
 import { SwitchComponent } from './switch/switch.component';
 import {StickyPopoverDirective} from './metadata-link-view/sticky-popover.directive';
 import { SortPipe } from './utils/sort.pipe';
+import { BrowserOnlyDirective } from './utils/browser-only.directive';
 
 const MODULES = [
   CommonModule,
@@ -640,10 +641,11 @@ const DIRECTIVES = [
 ];
 
 @NgModule({
-  imports: [
-    ...MODULES,
-    ...ROOT_MODULES,
-  ],
+    imports: [
+        ...MODULES,
+        ...ROOT_MODULES,
+        BrowserOnlyDirective,
+    ],
   declarations: [
     ...PIPES,
     ...COMPONENTS,