The current test fixture for migrations has limited content available for testing the more complex migrations being developed right now. It is ideal to commit a new version which is both slimmed down in size (by clearing out some unneeded table), and has a more complex setup.

The new fixtures has settings and content to test widgets, formatters, view modes, and nested field collections.

CommentFileSizeAuthor
#2 2944844-2.paragraphs.Update-testing-database-fixture-to-allow-for-testing-more-complex-interactions.patch2.68 MBmikelutz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikelutz created an issue. See original summary.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
CreditAttribution: mikelutz as a volunteer commented
Status: Active » Needs review
FileSize
2944844-2.paragraphs.Update-testing-database-fixture-to-allow-for-testing-more-complex-interactions.patch2.68 MB
mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
CreditAttribution: mikelutz as a volunteer commented
Related issues: +#2897021: [META] Migrate support for importing field collections as paragraphs
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented

Can you explain what things you added/removed from the d7 base fixture? I see some things, but I'm not going to see much in a huge dump like this.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
CreditAttribution: mikelutz as a volunteer commented

Specifically, starting from https://www.drupal.org/project/paragraphs/issues/2911241#comment-12463873 I made the following changes (actually started with a fresh fixture out of necessity):

Changed the content_type machine name from paragraphs_migration_test to just paragraphs_test. This was necessary because the comment migration was creating a machine name of comment_bundle_ which was too long and causing errors during tests.

Replaced the paragraph_bundle_test paragraph type with two bundles, paragraph_bundle_one and paragraph_bundle_two

Added two new field collection types, field_nested_fc_inner and field_nested_fc_outer. Outer contains the field for inner.

The content type now has two paragraph fields, one which allows any paragraph (field_paragraph_any), and a second with only allows bundles of type paragraph_bundle_one (field_paragraph_one_only)

I adjusted the widget settings for the first paragraph field to non-default settings to test the migration of the widget settings, and set the view mode to a non-default view mode to test the formatter settings.

I hid the second paragraph field in the content type display to test that transfering.

I added a field to the content type for the nested field collection

I created a new revision of the language neutral content item to test revisions migrations

I added basic test content to support the new fields above.

Finally, I cleared the cache, and truncated the access log and search index tables to reduce the size.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Needs review » Reviewed & tested by the community

Thanks for that thoughtful description. I think this is ready to land now.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Great work, committed. Ready for next step. :-)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.