Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.