Merge pull request UoEMainLibrary#3 from UoEMainLibrary/dspace-angula…

…r-accessibility Dspace angular accessibility
UoEMainLibrary · Mar 14, 2024 · 6844d13 · 6844d13
2 parents 8ebabd1 + 407ded0
commit 6844d13
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 14 deletions.
diff --git a/src/app/admin/admin-sidebar/admin-sidebar.component.html b/src/app/admin/admin-sidebar/admin-sidebar.component.html
@@ -34,11 +34,11 @@ <h4 class="section-header-text mb-0">{{ 'menu.header.admin' | translate }}</h4>
     </div>
     <div class="navbar-nav">
         <div class="sidebar-section" id="sidebar-collapse-toggle">
-            <!-- <a class="nav-item nav-link sidebar-section d-flex flex-row flex-nowrap"
+            <a class="nav-item nav-link sidebar-section d-flex flex-row flex-nowrap"
                href="javascript:void(0);"
                (click)="toggle($event)"
                (keyup.space)="toggle($event)"
-            > -->
+            > 
               <div class="shortcut-icon">
                 <i *ngIf="(menuCollapsed | async)" class="fas fa-fw fa-angle-double-right"
                    [title]="'menu.section.icon.pin' | translate"></i>
@@ -49,7 +49,7 @@ <h4 class="section-header-text mb-0">{{ 'menu.header.admin' | translate }}</h4>
                 <span *ngIf="menuCollapsed | async" class="section-header-text">{{'menu.section.pin' | translate }}</span>
                 <span *ngIf="!(menuCollapsed | async)" class="section-header-text">{{'menu.section.unpin' | translate }}</span>
               </div>
-            <!-- </a> -->
+            </a> 
         </div>
     </div>
 </nav>
diff --git a/src/app/item-page/full/full-item-page.component.spec.ts b/src/app/item-page/full/full-item-page.component.spec.ts
@@ -141,7 +141,7 @@ describe('FullItemPageComponent', () => {
   });
 
   it('should display the item\'s metadata', () => {
-    const table = fixture.debugElement.query(By.css('table'));
+    const table = fixture.debugElement.query(By.css('div.table-responsive'));
     for (const metadatum of mockItem.allMetadata(Object.keys(mockItem.metadata))) {
       expect(table.nativeElement.innerHTML).toContain(metadatum.value);
     }

diff --git a/src/app/statistics-page/statistics-table/statistics-table.component.spec.ts b/src/app/statistics-page/statistics-table/statistics-table.component.spec.ts
@@ -78,20 +78,20 @@ describe('StatisticsTableComponent', () => {
 
     it ('should display a table with the correct data', () => {
 
-      expect(de.query(By.css('table'))).toBeTruthy();
+      expect(de.query(By.css('div.table'))).toBeTruthy();
 
-      expect(de.query(By.css('th.views-header')).nativeElement.innerText)
+      expect(de.query(By.css('div.views-header')).nativeElement.innerText)
         .toEqual('views');
-      expect(de.query(By.css('th.downloads-header')).nativeElement.innerText)
+      expect(de.query(By.css('div.downloads-header')).nativeElement.innerText)
         .toEqual('downloads');
 
-      expect(de.query(By.css('td.item_1-views-data')).nativeElement.innerText)
+      expect(de.query(By.css('div.item_1-views-data')).nativeElement.innerText)
         .toEqual('7');
-      expect(de.query(By.css('td.item_1-downloads-data')).nativeElement.innerText)
+      expect(de.query(By.css('div.item_1-downloads-data')).nativeElement.innerText)
         .toEqual('4');
-      expect(de.query(By.css('td.item_2-views-data')).nativeElement.innerText)
+      expect(de.query(By.css('div.item_2-views-data')).nativeElement.innerText)
         .toEqual('8');
-      expect(de.query(By.css('td.item_2-downloads-data')).nativeElement.innerText)
+      expect(de.query(By.css('div.item_2-downloads-data')).nativeElement.innerText)
         .toEqual('8');
     });
   });

diff --git a/src/app/thumbnail/thumbnail.component.spec.ts b/src/app/thumbnail/thumbnail.component.spec.ts
@@ -227,7 +227,7 @@ describe('ThumbnailComponent', () => {
 
         fixture.detectChanges();
         const image: HTMLElement = fixture.debugElement.query(By.css('img')).nativeElement;
-        expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt);
+        expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt + ' for PDF');
       });
     });
 
@@ -271,7 +271,7 @@ describe('ThumbnailComponent', () => {
         comp.ngOnChanges({});
         fixture.detectChanges();
         const image: HTMLElement = fixture.debugElement.query(By.css('img')).nativeElement;
-        expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt);
+        expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt + ' for PDF');
       });
     });
 
@@ -311,7 +311,7 @@ describe('ThumbnailComponent', () => {
           comp.ngOnChanges({});
           fixture.detectChanges();
           const image: HTMLElement = de.query(By.css('img')).nativeElement;
-          expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt);
+          expect(image.getAttribute('alt')).toBe('TRANSLATED ' + comp.alt + ' for PDF');
         });
       });