Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sling paginator regression tests #135

Closed
wants to merge 10 commits into from
Closed

Sling paginator regression tests #135

wants to merge 10 commits into from

Conversation

renanrbs
Copy link
Contributor

@renanrbs renanrbs commented Nov 5, 2018

Short description of what this resolves:

Adds regression test for sling paginator

Changes proposed in this pull request:

Uses nightwatch library and take screenshots

How to run

the first time run npm run e2e:setup
run the component with npm start sling-web-component-paginator
to execute the tests run npm run nightwatch

@coveralls
Copy link

coveralls commented Nov 5, 2018 •
edited
Loading

Pull Request Test Coverage Report for Build 920

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.939%
Totals Coverage Status
Change from base Build 903: 0.0%
Covered Lines: 632
Relevant Lines: 989
💛 - Coveralls

@breakzplatform breakzplatform changed the title Sling calendar regression tests Sling paginator regression tests Nov 6, 2018
breakzplatform
breakzplatform suggested changes Nov 8, 2018
View reviewed changes
Copy link
Contributor

@breakzplatform breakzplatform left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error while running .clickElement() protocol action: stale element reference: element is not attached to the page document

@leofavre leofavre deleted the feature/CMPDC-1040 branch July 3, 2019 22:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@breakzplatform breakzplatform breakzplatform requested changes

@italomarcel italomarcel Awaiting requested review from italomarcel

@leofavre leofavre Awaiting requested review from leofavre

@caiomdias caiomdias Awaiting requested review from caiomdias

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@renanrbs @coveralls @breakzplatform @italomarcel