feat(DH): Fix unit test and some e2e tests

geonetwork · Jan 9, 2024 · fff7713 · fff7713
1 parent 3c0ca21
commit fff7713
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 15 deletions.
diff --git a/apps/datahub-e2e/src/e2e/datasetDetailPage.cy.ts b/apps/datahub-e2e/src/e2e/datasetDetailPage.cy.ts
@@ -120,12 +120,7 @@ describe('dataset pages', () => {
           .find('[id="about"]')
           .find('gn-ui-metadata-contact')
           .should('have.length', 1)
-        cy.get('datahub-record-metadata')
-          .find('[id="about"]')
-          .find('gn-ui-metadata-contact')
-          .children('div')
-          .children('a')
-          .eq(0)
+        cy.get('[data-cy="contact-email"]')
           .invoke('text')
           .should('include', '@')
         cy.get('datahub-record-metadata')
@@ -174,6 +169,7 @@ describe('dataset pages', () => {
           .children('div')
           .children('div')
           .find('a')
+          .first()
           .as('proviLink')
 
         cy.get('@proviLink')
@@ -185,14 +181,7 @@ describe('dataset pages', () => {
           })
       })
       it('should go to dataset search page when clicking on org name and filter by org', () => {
-        cy.get('datahub-record-metadata')
-          .find('[id="about"]')
-          .find('gn-ui-metadata-contact')
-          .children('div')
-          .children('div')
-          .children('div')
-          .first()
-          .click()
+        cy.get('[data-cy="organization-name"]').click()
         cy.url().should('include', '/search?publisher=')
       })
       it('should go to dataset search page when clicking on keyword and filter by keyword', () => {

diff --git a/libs/ui/elements/src/lib/metadata-contact/metadata-contact.component.html b/libs/ui/elements/src/lib/metadata-contact/metadata-contact.component.html
@@ -15,6 +15,7 @@
       <div
         class="text-primary font-title text-21 mr-2 cursor-pointer hover:underline"
         (click)="onOrganizationClick()"
+        data-cy="organization-name"
       >
         {{ shownOrganization.name }}
       </div>
@@ -55,6 +56,7 @@
         [href]="'mailto:' + contacts[0].email"
         class="text-sm hover:underline ml-2"
         target="_blank"
+        data-cy="contact-email"
         >{{ contacts[0].email }}</a
       >
     </div>

diff --git a/libs/ui/elements/src/lib/metadata-contact/metadata-contact.component.spec.ts b/libs/ui/elements/src/lib/metadata-contact/metadata-contact.component.spec.ts
@@ -89,7 +89,7 @@ describe('MetadataContactComponent', () => {
   describe('content', () => {
     let email
     beforeEach(() => {
-      email = fixture.debugElement.query(By.css('a'))
+      email = fixture.debugElement.queryAll(By.css('a'))[1]
     })
     it('displays the contact name', () => {
       const el = fixture.debugElement.query(