Skip to content

Commit

Permalink
review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
hyunnaye committed Jan 8, 2025
1 parent 8e7abc1 commit eca88d4
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cypress/e2e/group2/myworkflows.ts
Original file line number Diff line number Diff line change
Expand Up @@ -429,7 +429,7 @@ describe('Dockstore my workflows part 2', () => {
gotoVersionsAndClickActions();
// Request DOI
cy.fixture('versionWithDoi.json').then((json) => {
cy.intercept('GET', '/api/workflows/11/workflowVersions?limit=10&offset=0&sortCol=lastModified&sortOrder=desc&include=metrics', {
cy.intercept('GET', '/api/workflows/11/workflowVersions?limit=10&offset=0&sortOrder=desc&include=metrics', {
body: json,
statusCode: 200,
}).as('getVersionWithDoi');
Expand All @@ -448,7 +448,7 @@ describe('Dockstore my workflows part 2', () => {

// Export to ORCID
cy.fixture('versionAfterOrcidExport.json').then((json) => {
cy.intercept('GET', '/api/workflows/11/workflowVersions?limit=10&offset=0&sortCol=lastModified&sortOrder=desc&include=metrics', {
cy.intercept('GET', '/api/workflows/11/workflowVersions?limit=10&offset=0&sortOrder=desc&include=metrics', {
body: json,
statusCode: 200,
}).as('getVersionAfterOrcidExport');
Expand Down
7 changes: 7 additions & 0 deletions cypress/e2e/group3/metrics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,13 @@ describe('Dockstore Metrics', () => {
}).as('getMetrics');
});
cy.visit('/workflows/github.com/A/l:master');
cy.fixture('versionWithMetrics.json').then((json) => {
cy.intercept('GET', '/api/workflows/published/11/workflowVersions?limit=10&offset=0&sortOrder=desc&include=metrics', {
body: json,
statusCode: 200,
}).as('getVersionWithMetrics');
});
cy.wait('@getVersionWithMetrics');
goToTab('Versions');
cy.get('[data-cy=execution-metrics-icon]').should('be.visible');
cy.get('[data-cy=validation-metrics-icon]').should('be.visible');
Expand Down

0 comments on commit eca88d4

Please sign in to comment.