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
Noticed that 2 migrate tests are not in the group 'migrate', one is in 'migration' and the other is 'Migration'. Lets get them into the correct group.
The files are:
core/modules/migrate/tests/src/Unit/Exception/RequirementsExceptionTest.php: * @group migration
core/modules/migrate/tests/src/Unit/MigrationTest.php: * @group Migration
This is simple editing, tagging as novice.
Proposed resolution
Change the test group in the two files to 'migrate.
Remaining tasks
Write a patch
review
commit
Smile!
User interface changes
These tests will appear in the 'migrate' list at admin/config/development/testing
Comment | File | Size | Author |
---|---|---|---|
#8 | correct-test-group-2935822-2.patch | 1007 bytes | priya.chat |
Comments
Comment #2
sonu.raj.chauhan CreditAttribution: sonu.raj.chauhan as a volunteer commentedComment #3
sonu.raj.chauhan CreditAttribution: sonu.raj.chauhan as a volunteer and at Publicis Sapient for Publicis Sapient commentedAll the mentioned changes have been done.
Please review.
Comment #4
quietone CreditAttribution: quietone as a volunteer and commentedApplied the patch and confirmed that the tests appear in the 'migrate' group in the UI and from the CLI.
@sonu.raj.chauhan, thank you. Great work!
Comment #5
snehi CreditAttribution: snehi as a volunteer and at Publicis Sapient for Publicis Sapient commented+1
Comment #6
Gábor HojtsyGreat find, thanks!
Comment #8
priya.chat CreditAttribution: priya.chat as a volunteer and at Publicis Sapient for Publicis Sapient commentedHi, I have made changes as you mentioned and adding a patch for the solution. Please review this.
Comment #9
priya.chat CreditAttribution: priya.chat as a volunteer and at Publicis Sapient for Publicis Sapient commentedComment #11
quietone CreditAttribution: quietone as a volunteer and commented@priya.chat, thank you for joining the Migrate January 2018 Sprint! The patch looks great but this is was resolved and committed in #7. You are welcome to join us in IRC, #drupal-migrate, we can help you find an issue to work on. Cheers