[#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>;
  },