[#2195] Add cypress test for show tag when grouping by author and by none

frontend/cypress/config/author-config.csv

@@ -0,0 +1,2 @@
+Repository's Location,Branch,Author's Git Host ID,Author's Emails,Author's Display Name,Author's Git Author Name,Ignore Glob List
+https://github.com/reposense/publish-RepoSense.git,master,yong24s,,,Yong Hao TENG;yong24s,

frontend/cypress/tests/chartView/chartView_mergeGroup.cy.js

@@ -17,9 +17,9 @@ describe('merge group', () => {
       .uncheck()
       .should('not.be.checked');
 
-    // after un-checking merge group, all 14 summary charts will show
+    // after un-checking merge group, all 9 summary charts will show
     cy.get('#summary-charts').find('.summary-chart')
-      .should('have.length', 14);
+      .should('have.length', 9);
   });
 
   it('check and uncheck merge group when group by authors', () => {
@@ -31,19 +31,19 @@ describe('merge group', () => {
       .check()
       .should('be.checked');
 
-    // after checking merge group, 14 merged author groups will show
+    // after checking merge group, 8 merged author groups will show
     cy.get('#summary-charts').find('.summary-chart')
-      .should('have.length', 14);
+      .should('have.length', 8);
 
     cy.get('#summary label.merge-group > input:visible')
       .first()
       .should('be.visible')
       .uncheck()
       .should('not.be.checked');
 
-    // after un-checking merge group, all 14 summary charts will show
+    // after un-checking merge group, all 9 summary charts will show
     cy.get('#summary-charts').find('.summary-chart')
-      .should('have.length', 14);
+      .should('have.length', 9);
   });
 
   it('merge group option should be disabled when group by none', () => {
@@ -62,15 +62,15 @@ describe('merge group', () => {
       .should('be.checked');
 
     // get the chart bars and assert they have the correct initial widths
-    const expectedWidths = [100, 100, 100, 15, 100, 100, 90, 30, 15];
+    const expectedWidths = [100, 100, 20, 100, 100, 3, 20, 5];
     cy.get('.stacked-bar__contrib--bar')
       .should('have.length', expectedWidths.length)
       .then(($bars) => {
         // calculate the percentage of the width relative to the parent container
         const parentWidth = $bars.eq(0).parent().width();
         expectedWidths.forEach((expectedWidth, index) => {
           const width = (parseFloat(window.getComputedStyle($bars[index]).width) / parentWidth) * 100;
-          expect(width).to.be.closeTo(expectedWidth, 1);
+          expect(width).to.be.closeTo(expectedWidth, 2);
         });
       });
   });

frontend/cypress/tests/chartView/chartView_scrollToActiveRepo.cy.js

@@ -49,7 +49,7 @@ describe('scroll to active repo', () => {
       });
 
     cy.url()
-      .should('contain', 'tabAuthor=Yong%20Hao%20TENG')
+      .should('contain', 'tabAuthor=yong24s')
       .should('contain', 'tabRepo=reposense%2Fpublish-RepoSense%5Bmaster%5D');
 
     cy.reload();

frontend/cypress/tests/chartView/chartView_showTags.cy.js

@@ -64,4 +64,91 @@ describe('show tags', () => {
         .should('equal', 'https://github.com/reposense/RepoSense/releases/tag/v1.0');
     });
   });
+
+  it('group by authors works with show tags', () => {
+    cy.get('div.mui-select.grouping > select:visible')
+      .select('groupByAuthors');
+
+    cy.get('div.mui-select.sort-within-group > select:visible')
+      .select('title dsc');
+
+    cy.get('#summary label.show-tags > input:visible')
+      .should('be.visible')
+      .check()
+      .should('be.checked');
+
+    cy.get('.summary-chart')
+      .first()
+      .find('.summary-chart__title--tags')
+      .find('a')
+      .should('have.length', 0);
+
+    cy.get('.summary-chart')
+      .eq(1)
+      .find('.summary-chart__title--tags')
+      .find('a')
+      .should('have.length.gt', 0);
+
+    cy.get('.icon-button.fa-list-ul')
+      .should('exist')
+      .eq(1)
+      .click();
+
+    const correctTags = [];
+
+    cy.get('.zoom__title--tags')
+      .find('.tag')
+      .each(($tag) => correctTags.push($tag.text().trim()))
+      .then(() => {
+        cy.get('.summary-chart')
+          .eq(1)
+          .find('.summary-chart__title--tags')
+          .find('.tag')
+          .each(($tag) => {
+            expect(correctTags).to.include($tag.text().trim());
+          });
+
+        cy.get('.summary-chart')
+          .eq(1)
+          .find('.summary-chart__title--tags')
+          .find('.tag')
+          .should('have.length', correctTags.length);
+      });
+  });
+
+  it('group by none works with show tags', () => {
+    cy.get('div.mui-select.grouping > select:visible')
+      .select('groupByNone');
+
+    cy.get('#summary label.show-tags > input:visible')
+      .should('be.visible')
+      .check()
+      .should('be.checked');
+
+    cy.get('.icon-button.fa-list-ul')
+      .should('exist')
+      .first()
+      .click();
+
+    const correctTags = [];
+
+    cy.get('.zoom__title--tags')
+      .find('.tag')
+      .each(($tag) => correctTags.push($tag.text().trim()))
+      .then(() => {
+        cy.get('.summary-chart')
+          .first()
+          .find('.summary-chart__title--tags')
+          .find('.tag')
+          .each(($tag) => {
+            expect(correctTags).to.include($tag.text().trim());
+          });
+
+        cy.get('.summary-chart')
+          .first()
+          .find('.summary-chart__title--tags')
+          .find('.tag')
+          .should('have.length', correctTags.length);
+      });
+  });
 });

frontend/src/components/c-summary-charts.vue

@@ -980,7 +980,7 @@ export default defineComponent({
     },
 
     getTags(repo: Array<User>, user?: User): Array<string> {
-      if (user) repo = repo.filter((r) => r.name === user.name);
+      if (user) repo = repo.filter((r) => r.name === user.name && r.repoId === user.repoId);
       return [...new Set(repo.flatMap((r) => r.commits).flatMap((c) => c.commitResults).flatMap((r) => r.tags))]
         .filter(Boolean) as Array<string>;
     },