Skip to content

Commit 97d75ff

Browse files
committed
Final fixes and test fixes
1 parent c8d3d73 commit 97d75ff

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

lib/rules/assertion-before-screenshot.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,9 @@ const assertionCommands = [
1111
'and',
1212
'contains',
1313

14+
// retries until it gets something
15+
'get',
16+
1417
// not an assertion, but unlikely to require waiting for render
1518
'scrollIntoView',
1619
'scrollTo',

tests/lib/rules/assertion-before-screenshot.js

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,14 @@ const parserOptions = { ecmaVersion: 6 }
1111
ruleTester.run('assertion-before-screenshot', rule, {
1212
valid: [
1313
{ code: 'cy.get(".some-element"); cy.screenshot();', parserOptions },
14-
{ code: 'cy.get(".some-element").screenshot();', parserOptions },
15-
{ code: 'cy.get(".some-element").should("be.visible"); cy.screenshot();', parserOptions },
16-
{ code: 'cy.get(".some-element").screenshot().click()', parserOptions, errors },
17-
{ code: 'cy.get(".some-element"); if(true) cy.screenshot();', parserOptions },
18-
{ code: 'if(true) { cy.get(".some-element"); cy.screenshot(); }', parserOptions },
19-
{ code: 'cy.get(".some-element"); if(true) { cy.screenshot(); }', parserOptions },
20-
{ code: 'const a = () => { cy.get(".some-element"); cy.screenshot(); }', parserOptions, errors },
14+
{ code: 'cy.get(".some-element").should("exist").screenshot();', parserOptions },
15+
{ code: 'cy.get(".some-element").should("exist").screenshot().click()', parserOptions, errors },
16+
{ code: 'cy.get(".some-element").should("exist"); if(true) cy.screenshot();', parserOptions },
17+
{ code: 'if(true) { cy.get(".some-element").should("exist"); cy.screenshot(); }', parserOptions },
18+
{ code: 'cy.get(".some-element").should("exist"); if(true) { cy.screenshot(); }', parserOptions },
19+
{ code: 'const a = () => { cy.get(".some-element").should("exist"); cy.screenshot(); }', parserOptions, errors },
20+
{ code: 'cy.get(".some-element").should("exist").and("be.visible"); cy.screenshot();', parserOptions },
21+
{ code: 'cy.get(".some-element").contains("Text"); cy.screenshot();', parserOptions },
2122
],
2223

2324
invalid: [

0 commit comments

Comments
 (0)