Ubercart 7 Test "All The Things"

Postponed on #2569805: For Drupal migration, identify the source module and #2899495: Add source_module and destination_module to the sources and destionations.

Both #2569805: For Drupal migration, identify the source module and #2899495: Add source_module and destination_module to the sources and destionations are in, so no longer postponded.

But let's postpone this until the other similar tests are working.
#2899672: Commerce 1 Test "All The Things" and #2899675: Ubercart 6 Test "All The Things".

This may work.

Oh right, need to finish the field migrations first.

Updating for all the recent migrations that have been committed.

Adjust the node type.

Adjust the entity counts for the latest changes.

I think this is a dependency problem with and uc7_comment_type needs to be added as a dependency in d7_field_instance. As the code is it will get added other field migrations as well but that will should be ok.

uc7\MigrateUpgradeUbercart7Test::testMigrateUpgrade
Migration of a:2:{s:9:"sourceid1";s:4:"node";s:9:"sourceid2";s:7:"article";} to a:0:{} as part of the d7_rdf_mapping migration. The source row status is 3

So, this is failing due to d7_rdf_mapping.

That migration reports errors if the rdf_mapping entity already exists as we can see here.

(8.7.x)$ drush mim d7_rdf_mapping
 [error]  'rdf_mapping' entity with ID 'node.article' already exists. (/opt/sites/d8/core/lib/Drupal/Core/Entity/EntityStorageBase.php:462) 
 [error]  'rdf_mapping' entity with ID 'node.page' already exists. (/opt/sites/d8/core/lib/Drupal/Core/Entity/EntityStorageBase.php:462) 

Opened a core issue and upload a patch for this, #2998666: Warnings after D7 upgrade caused by rdf migration, which fixed this test locally. This is postponed on that.

However, this test is very useful to have in as early as possible and I am tempted to write a workaround for it. Marking postponed while I ponder on that.

No longer postponed

some more entity updates.

How about removing the horrible var_dump

And there are two profile, yes

Maybe this will get the last ones.

Now make a new test base which will be used for U6 and U7 but for now just test with U7.

Make all the entity counts an int.

Fixing more entity counts

Alter views count

Well, my local must be out of date. Let's try again.

Really? One was still a string?

Now convert MigrateUpgradeUbercart6Test to use the Core base test as well.

The setup method in core add content, that is why all the entity counts have changed.

   // Create content.
    $this->createContent();

Does it matter? It just means we are not testing from a clean install of the standard profile. I am ok with that. It is certainly possible that the destination will have content before running this migration.

Let's see if my local is really up to date noe.

Great! Just a few comment changes.

No need to wait on this.