Skip to content

Expand saveScreenshot() functionality#523

Merged
DavertMik merged 5 commits intocodeceptjs:masterfrom
HughZurname:master
May 17, 2017
Merged

Expand saveScreenshot() functionality#523
DavertMik merged 5 commits intocodeceptjs:masterfrom
HughZurname:master

Conversation

@HughZurname
Copy link
Contributor

Resolves #522.

Changes (all helpers):
  • saveScreenshot() - Takes a second argument of true to capture full page.
  • _failed - Updated behaviour to take advantage of new functionality, failed tests now capture the full page.
  • Documentation updated to reflect new functionality.
  • Unit tests added for new functionality.

@DavertMik
Copy link
Contributor

Wow, thanks for updating all helpers and tests!
Awesome.

@DavertMik DavertMik merged commit b37bf09 into codeceptjs:master May 17, 2017
@HughZurname
Copy link
Contributor Author

No worries, happy to help 😁

@arifkhanaj
Copy link

Somehow, this still doesn't take full page screenshot for me :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] Expand saveScreenshot() to capture full page

3 participants

Comments