Skip to content

Conversation

@subhojit777
Copy link
Contributor

@subhojit777 subhojit777 commented Sep 27, 2016

Issue #313
This is just the start, let me know if I am going in the right direction.

This is only for Drupal 7

@jhedstrom jhedstrom added this to the 4.0 release milestone Sep 27, 2016
@jhedstrom
Copy link
Owner

@subhojit777 this looks like a good start.

One thought though, is that since this is using the admin UI to place the blocks, it ties it (as you mention) directly to Drupal 7. I would prefer adding the place block method to the https://github.com/jhedstrom/DrupalDriver, and then this new step definition need only call a method, rather than placing the blocks through the UI. This would have the added benefit of supporting more than one version of Drupal.

Once that is done, we can add a new test in features to verify the behavior.

@subhojit777
Copy link
Contributor Author

subhojit777 commented Sep 28, 2016

But using DrupalDriver we won't be able to access the UI. I haven't tried this before, but we might be able to place the block by running an update query in the block table (atleast in Drupal 7) - may be that can work. But the challenge is to obtain the block delta. For example, facetapi it generates a hash-ed block delta, and then again we have to access facetapi's API to generate the hash-ed delta. This will add dependency to the project.

@subhojit777
Copy link
Contributor Author

Using Mink inside the DrupalDriver will also defeat its purpose.

@jhedstrom
Copy link
Owner

But using DrupalDriver we won't be able to access the UI

Using Mink inside the DrupalDriver will also defeat its purpose.

Right--from the Drupal Driver, the API itself would be used to place the block. A step definition for placing a block probably doesn't need to use the UI, unless the express purpose was to test the UI itself, in which case that would be outside the scope of this project.

@subhojit777 subhojit777 force-pushed the issue#313 branch 2 times, most recently from ab64a2f to a0c25c4 Compare September 29, 2016 18:19
@subhojit777 subhojit777 force-pushed the issue#313 branch 3 times, most recently from 97c2ebe to 4718c9a Compare October 27, 2016 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants