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.

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

Dusk tests #4

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Dusk tests #4

wants to merge 3 commits into from

Conversation

bennothommo
Copy link
Member

@bennothommo bennothommo commented May 2, 2021

This PR will set up multiple Dusk tests for the Test plugin, giving us a comprehensive suite of acceptance tests with the goal of testing some front-end functionality of the Backend that unit tests can't properly test. As an added benefit, this is also giving our Dusk plugin a good test case as well.

Another goal is to set up a gallery of screenshots that cover most, if not all, of the elements one would expect to find in the Backend, which will be useful for the main repo when PRs are submitted that change the look or functionality of widgets, elements or the entire backend.

@bennothommo
Copy link
Member Author

Example screenshots being taken by the Dusk tests:

1-People-List-1-Initial
1-People-List-2-Hover-List-Item
1-People-List-3-Hover-Button
1-People-List-4-Enter-Search-Query
1-People-List-5-Clear-Search-Query
1-People-List-6-Show-List-Setup
1-People-List-7-Change-List-Setup
1-People-List-8-Apply-List-Setup

@LukeTowers
Copy link
Member

Is there anyway that we could have some sort of visual diff / fixtures to compare against to make it easier to identify changes in the screenshots?

@bennothommo
Copy link
Member Author

@LukeTowers BrowserStack has the "Percy" section which I believe does exactly that. I'd just have to figure out how to get images generated in Dusk sent through to there.

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

Successfully merging this pull request may close these issues.

2 participants