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

Missing composer command in instructions #29

Open
cpatrone opened this issue Aug 9, 2016 · 1 comment
Open

Missing composer command in instructions #29

cpatrone opened this issue Aug 9, 2016 · 1 comment

Comments

@cpatrone
Copy link

cpatrone commented Aug 9, 2016

When I was trying to install this extension I executed composer require without --dev and for some reason after running that command my local environment gives me an error (no mention in instructions about running composer after editing the json file).

The way I finished installing succesfully this extension was adding the repository in the json file:
"repositories": [
{
"type": "vcs",
"url": "[email protected]:palantirnet/palantir-behat-extension.git"
}
],
And nothing else (no modification in require-dev section).

Finally I run this command: "composer require palantirnet/palantir-behat-extension:dev-master --dev".

This installed succesfully the files in vendor directory.

Another detail: "- Palantirnet\PalantirBehatExtension\Context\DrupalFileContext.php" is missing from the list to add to behat.yml file.

Thanks,

@becw
Copy link
Member

becw commented Aug 9, 2016

Ahh, brilliant, this debugging info is super useful. Thank you!

It turns out that your problems are due to several errors in the README -- I've updated it in 3cd762f:

  • The .php should not be included in the context class names in behat.yml
  • For Drupal 8 sites, we should install dev-drupal8 rather than dev-master

The thing is, if you DO install the dev-drupal8 branch, all of the methods throw a NotUpdatedException... because they are based on Drupal 7 internals. @labbydev said that you need the "I should see a :label field" method, so I've made PR #30 to update that.

Hope this helps!

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

No branches or pull requests

2 participants