[Ubercart] Migrate store

This is not complete, the address, owner and billing_countries need to be figured out.

If you want to work on this patch, and you want to use a test fixture, use the one at #2871811: New complete fixture for Ubercart D6 schema.

Grrr, this is the wrong patch.

Here is the right patch. Still needs some variables, namely the address, migrated. Not sure why the address isn't being saved.

Adding the address migration. To run the tests get the latest patch at #2871811: New complete fixture for Ubercart D6 schema.

Still to investigate: owner, billing countries and adding the store migration as a dependency on other d6 ubercart migrations.

Merged d6 and d7 store migration.

To run the tests get the latest patch at #2871811: New complete fixture for Ubercart D6 schema and #2871812: New complete fixture for Ubercart D7 schema.

Still to investigate: owner, billing countries and adding the store migration as a dependency on other ubercart migrations.

+++ b/migration_templates/d6_ubercart_store.yml
@@ -0,0 +1,42 @@
+  default_currency: uc_currency_code

The currency code is migrated, but the currency may not be installed. Maybe a new migration to add all the currencies that exist on the source site.

Adding the dependency to currency here.

These migrations need a source provider defined. And there are new line errors in the patch.

Add the source provider.

Next, is to add this as a dependency of other migrations.

I seem to be all thumbs today, let's see if this passes tests.

How to run tests? Or is this another manifestation of my 'all thumbs' day?

Yep, all thumbs,

That was not my fingers, HEAD moved at the same minute I uploaded the patch.

Here is the reroll.

Forgot to test d7. Just needed to add the currency migration to the test.

Fix comments, add the uid migration to the d7 store migration and update the test.

1,3,4. About dependencies, migrate will figure it out. We don't need to add d6_ubercart_store to every migrations. Just thinking about having to add every needed migration to a new migration makes my head hurt.

2. The migrate helpers can be rather useful. There appear in the D6 migrations and work well. For example if one needs users, the simply use $this->migrateUsers(FALSE); saving having to remember or search for all the migrations, entities and schemas that need to be installed for the user migrations. In this case, it makes sense to me to have this since every migration will need a store and even though it is only two migrations it seem natural to think in terms of give me a store, without worrying about the details.
And thats for the question, I spotted that the currency migration was done twice.

Just noting that this is blocking #2874667: Add a trait assertEntity functions for tests.