Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
There is a d6 and d7 ValidateMigrationStateTest both of which use ValidateMigrationStateTestTrait::testMigrationState() but that actually checks both the d6 and d7 migration states whereas it should only do one. Plus the test will be a bit easier to read without the version index in all the arrays.
Proposed resolution
Get the legacy version from the source database.
Remaining tasks
Patch, review, commit.
Comment | File | Size | Author |
---|---|---|---|
#3 | 3100190-2.patch | 8.67 KB | quietone |
Comments
Comment #2
quietone CreditAttribution: quietone as a volunteer commentedComment #3
quietone CreditAttribution: quietone as a volunteer commentedAnd here is the patch, all straightforward.
Comment #4
mikelutzThis makes sense and seems straightforward. No point in running the same test twice.
Comment #5
alexpottCommitted and pushed 5367f8a5af to 9.0.x and d0476bbefc to 8.9.x. Thanks!
Will backport to 8.8.x when the freeze is over as this is a test only change.
Comment #9
alexpottWe're not in a freeze.