Active
Project:
Drupal core
Version:
main
Component:
javascript
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 May 2018 at 10:10 UTC
Updated:
25 Jun 2018 at 16:01 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
samuel.mortenson👋I'm new to this but made an attempt to add a login, logout, and isLoggedIn command to our Nightwatch instance. Test coverage for the new commands are included. Only settings to "Needs review" for this portion of the work required to port manual tests over, we still need to actually port the tests.
Edit: We could also have a PHP script that returns an appropriate session cookie based on a user ID, then call that with child_process, if setting a username and password feels weird to people.
Comment #3
dawehnerI think it makes sense to add the commands in its own issue.
Humans are surprisingly good in pattern recognition. I think to get the same benefit from a machine we need a more structured approach
I've updated the issue summary
Comment #4
dawehner@samuel.mortenson
I created its own issue for these basics commands: #2973879: Add login/logout Nightwatch commands (and a Drupal "login" command to allow for that) Do you mind posting your patch there? I'm happy to review it.
Comment #5
samuel.mortenson@dawehner Done!
Comment #6
tedbowComment #7
jansete commentedHi!
I've created an issue to add page objects, maybe it could be a new child issue from this.
https://www.drupal.org/project/drupal/issues/2981602