Support from Acquia helps fund testing for Drupal Acquia logo

Comments

akalata created an issue. See original summary.

akalata’s picture

Made some progress with this. So far, the attached patch includes a testing theme for ECK, and testTemplateOverride() checks that this custom theme is enabled and the overriding template is being used when creating an entity.

I haven't quite gotten the adding of fields to the eck entity during the test to work, it tells me the entity type doesn't exist.

Status: Needs review » Needs work

The last submitted patch, 2: 2697093-eck-template-test-2-trying-to-add-fields.patch, failed testing.

The last submitted patch, 2: 2697093-eck-template-test-2-trying-to-add-fields.patch, failed testing.

akalata’s picture

I've spent a few hours trying to move forward on this and am at my wit's end. Maybe the issue has something to do with the entity type itself being dynamically defined? I get the same error even when specifying the entity type and bundle names, rather than using a randomly-generated string.

First arrow points to "Entity type [random string] has been added."

Second arrow points to "The [random string] entity type does not exist."

It would also explain why this module is missing tests on adding FieldUI fields to ECK entities. :)

legolasbo’s picture

Thanks for your work on this akalata. I've just spent a couple of hours moving this forward and we're nearly there. Entity + bundle + fields get created, theme is enabled, but the fields are not showing up. I'm out of time now so just uploading my progress.