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
Improve check for dangerous links... #13731
Conversation
It might be better to run But I have to admit, the difference is very minor on my system. And the performance gain will cannibalize itself, as soon as you use more wildcards. |
@@ -82,11 +82,11 @@ describe("TagManagerTeaser", function () { | |||
}); | |||
|
|||
it('should be possible to disable page and redirect to home', function (done) { | |||
capturePage(done, 'admin_page_disable', function (page) { | |||
expect.page("").contains("#dashboard", function (page) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that change on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is no screenshot name etc... and shouldn't it capture?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. it now only checks if the loaded page has the element #dashboard
. Imho there is no need to compare the actual content as a screenshot
closing in favor of #13741 |
and some other UI test related stuff