Skip to content

Commit a1af53b

Browse files
author
Andrea Barbasso
committed
[DSC-2730] remove xdescribe from cypress tests, fix the ones not linked to bad REST requests
1 parent 3465c1f commit a1af53b

8 files changed

Lines changed: 10 additions & 16 deletions

cypress/e2e/admin-search-page.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { testA11y } from 'cypress/support/utils';
22

3-
// TODO: Enable these tests and fix them before the release
4-
xdescribe('Admin Search Page', () => {
3+
describe('Admin Search Page', () => {
54
beforeEach(() => {
65
// Must login as an Admin to see the page
76
cy.visit('/admin/search');

cypress/e2e/admin-workflow-page.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { testA11y } from 'cypress/support/utils';
22

3-
// TODO: Enable these tests and fix them before the release
4-
xdescribe('Admin Workflow Page', () => {
3+
describe('Admin Workflow Page', () => {
54
beforeEach(() => {
65
// Must login as an Admin to see the page
76
cy.visit('/admin/workflow');

cypress/e2e/bulk-access.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { testA11y } from 'cypress/support/utils';
22
import { Options } from 'cypress-axe';
33

4-
// TODO: Enable these tests and fix them before the release
5-
xdescribe('Bulk Access', () => {
4+
describe('Bulk Access', () => {
65
beforeEach(() => {
76
// Must login as an Admin to see the page
87
cy.visit('/access-control/bulk-access');

cypress/e2e/homepage-statistics.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,7 @@ import '../support/commands';
33
import { REGEX_MATCH_NON_EMPTY_TEXT } from 'cypress/support/e2e';
44
import { testA11y } from 'cypress/support/utils';
55

6-
// TODO: Enable these tests and fix them before the release
7-
xdescribe('Site Statistics Page', () => {
6+
describe('Site Statistics Page', () => {
87
it('should load if you click on "Statistics" from homepage', () => {
98
cy.visit('/');
109
cy.get('#expandable-navbar-section-statistics > [data-test="navbar-section-toggler"]').trigger('keyup', { key: 'Enter' });

cypress/e2e/homepage.cy.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { testA11y } from 'cypress/support/utils';
22
import { Options } from 'cypress-axe';
33

44
// TODO: Enable these tests and fix them before the release
5-
xdescribe('Homepage', () => {
5+
describe('Homepage', () => {
66
beforeEach(() => {
77
// All tests start with visiting homepage
88
cy.visit('/');

cypress/e2e/item-statistics.cy.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { REGEX_MATCH_NON_EMPTY_TEXT } from 'cypress/support/e2e';
22
import { testA11y } from 'cypress/support/utils';
33

4-
// TODO: Enable these tests and fix them before the release
5-
xdescribe('Item Statistics Page', () => {
4+
describe('Item Statistics Page', () => {
65
const ITEMSTATISTICSPAGE = '/statistics/items/'.concat(Cypress.env('DSPACE_TEST_ENTITY_PUBLICATION'));
76

87
const TOTALVISITSPERMONTHITEMSTATISTICSPAGE = ITEMSTATISTICSPAGE.concat('?reportType=TotalVisitsPerMonth');
@@ -27,7 +26,8 @@ xdescribe('Item Statistics Page', () => {
2726
it('should contain a "Total visits per month" section', () => {
2827
cy.visit(ITEMSTATISTICSPAGE);
2928
// Check just for existence because this table is empty in CI environment as it's historical data
30-
cy.get('.'.concat(Cypress.env('DSPACE_TEST_ENTITY_PUBLICATION')).concat('_TotalVisitsPerMonth')).should('exist');
29+
cy.get('ul.nav > li.nav-item:nth-child(2) > a.nav-link:nth-child(1)').click();
30+
cy.get('ds-statistics-chart-wrapper').should('exist');
3131
});
3232

3333
it('should pass accessibility tests', () => {

cypress/e2e/my-dspace.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { testA11y } from 'cypress/support/utils';
22

3-
// TODO: Enable these tests and fix them before the release
4-
xdescribe('My DSpace page', () => {
3+
describe('My DSpace page', () => {
54
it('should display recent submissions and pass accessibility tests', () => {
65
cy.visit('/mydspace');
76

cypress/e2e/submission.cy.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@ import { testA11y } from 'cypress/support/utils';
22
//import { TEST_SUBMIT_USER, TEST_SUBMIT_USER_PASSWORD, TEST_SUBMIT_COLLECTION_NAME, TEST_SUBMIT_COLLECTION_UUID, TEST_ADMIN_USER, TEST_ADMIN_PASSWORD } from 'cypress/support/e2e';
33
import { Options } from 'cypress-axe';
44

5-
// TODO: Enable these tests and fix them before the release
6-
xdescribe('New Submission page', () => {
5+
describe('New Submission page', () => {
76

87
// NOTE: We already test that new Item submissions can be started from MyDSpace in my-dspace.spec.ts
98
it('should create a new submission when using /submit path & pass accessibility', () => {

0 commit comments

Comments
 (0)