search

sixty-nine / DrupalTesting

PHPUnit with Drupal 7
4 stars 6 forks source link

Is drupalconnectormodule package required? #9

Open andreiashu opened 10 years ago

andreiashu commented 10 years ago

I removed this package from composer.json in a separate branch as it depends on a bunch of other libs which I don't need. Was there a plan to reuse this package in DrupalTesting one?

sixty-nine commented 10 years ago

Idealy we wanted to completely abstract the calls to the Drupal API by using https://github.com/liip/LiipDrupalConnectorModule. But I think all the functions we need are implemented in our DrupalConnector class. So I guess it's ok to remove the dep.

Please double check everything works without the dep before merging.

andreiashu commented 10 years ago

That's what I thought but wanted to make sure. I'll double check that things are working ok before the merge. Thanks!