Skip to content

Commit 2c2e723

Browse files
committed
fix(permission_spec): add more guards
see #397 the spec currently fails in headless mode but passes in headed mode see #513
1 parent 219e91e commit 2c2e723

File tree

1 file changed

+21
-5
lines changed

1 file changed

+21
-5
lines changed

cypress/integration/06_permission_spec.js

Lines changed: 21 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,24 @@ context('Permission Manager', () => {
88
.click()
99
})
1010

11+
// (DP)) we need frequent assertions to work around the fact that the tree collapes on new requests
12+
// see #513 #397
1113
it('should have user entries', () => {
1214
cy.get('[node-id$=security]')
13-
.click()
14-
cy.get('.ReactVirtualized__Grid')
15-
.contains('Users')
16-
.click()
17-
cy.get('[node-id$=user\\/guest]')
1815
.should('be.visible')
16+
.click()
17+
.then(() => {
18+
cy.get('[node-id$=security\\/user]')
19+
.should('be.visible')
20+
.click()
21+
.then(() => {
22+
cy.get(`[node-id=${CSS.escape('admin@' + Cypress.env('API_HOST'))}]`)
23+
.should('be.visible')
24+
.click()
25+
cy.get('[node-id$=user\\/guest]')
26+
.should('be.visible')
27+
})
28+
})
1929
})
2030

2131
it('should have group entries', () => {
@@ -30,6 +40,7 @@ context('Permission Manager', () => {
3040
describe('the user item', () => {
3141
it('should let us create a new user', () => {
3242
cy.get('[node-id$=security]')
43+
.should('be.visible')
3344
.rightclick()
3445
cy.get('[data-command="fusion.add-user"]')
3546
.contains('Add')
@@ -53,6 +64,7 @@ context('Permission Manager', () => {
5364

5465
it('should display user properties card', () => {
5566
cy.get('[node-id$=user\\/guest]')
67+
.should('be.visible')
5668
.rightclick()
5769
cy.get('.p-Menu > ul > .p-Menu-item')
5870
.should('be.visible')
@@ -79,6 +91,7 @@ context('Permission Manager', () => {
7991

8092
it('should let us delete a user', () => {
8193
cy.get('[node-id$=user\\/cy-usr]')
94+
.should('be.visible')
8295
.rightclick()
8396
cy.get('[data-command="fusion.delete-user"]')
8497
.click()
@@ -94,6 +107,7 @@ context('Permission Manager', () => {
94107
describe('the groups item', () => {
95108
it('should let us create a new group', () => {
96109
cy.get('[node-id$=security]')
110+
.should('be.visible')
97111
.rightclick()
98112
cy.get('[data-command="fusion.add-group"]')
99113
.contains('Add')
@@ -107,6 +121,7 @@ context('Permission Manager', () => {
107121

108122
it('should display group properties card', () => {
109123
cy.get('[node-id$=group\\/guest]')
124+
.should('be.visible')
110125
.rightclick()
111126
cy.get('.p-Menu > ul > .p-Menu-item')
112127
.should('be.visible')
@@ -129,6 +144,7 @@ context('Permission Manager', () => {
129144

130145
it('should let us delete a group', () => {
131146
cy.get('[node-id$=group\\/cy-group]')
147+
.should('be.visible')
132148
.rightclick()
133149
cy.get('[data-command="fusion.delete-group"]')
134150
.click()

0 commit comments

Comments
 (0)