-
Notifications
You must be signed in to change notification settings - Fork 20
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
Add BehatCommands for Drush 9 #13
Conversation
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.
This looks good. Just these nits.
Want to open a PR against the DrupalExtension that targets this branch of the behat drush endpoint to see if tests start passing again? There's this related issue jhedstrom/drupalextension#458
BehatCommands.php
Outdated
@@ -0,0 +1,202 @@ | |||
<?php | |||
namespace Drush\Commands\example_site_wide_command; |
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.
This should probably be updated to BehatDrushEndpoint
or similar?
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.
It will be behat_drush_endpoint
BehatCommands.php
Outdated
* | ||
* @see Drupal\Driver\Cores\AbstractCore\expandEntityFields | ||
*/ | ||
function expandEntityFields($entity_type, \stdClass $entity) { |
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.
This and the other methods that don't already should probably specify visibility (protected
, public
, etc) for clarity.
3574ae5
to
8293eb5
Compare
Requires drush-ops/drush#3687, which I will merge shortly.