Heya, I'm very new to the Drupal community so forgive me if I'm committing a Drupal faux-pas.

It seems that the community 'migrate' module is a dependency of the date module. I can't see this listed on the main content page nor on the documentation page. I also couldn't see any comments or google hits to help address my lack of understanding.

Will the behaviour of this Date module be problematic if I don't have the migrate module? The simpletest for Date fails without it.

Kind regards,

Michael J.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Stevel’s picture

Title: Migrate module dependency » DateMigrateExampleUnitTest fails if migrate is missing
Version: 7.x-2.7 » 7.x-2.x-dev
Component: Documentation » Miscellaneous
Assigned: Unassigned » Stevel
Priority: Minor » Normal
Issue tags: -migrate dependency date

The migrate module is not a dependency for the Date API module, but it is required for the DateMigrateExampleUnitTest. If the migrate module is missing and tests are run, this test fails:

Call to undefined function migrate_get_module_apis()

I propose to move this test to the "Date Migration Example" module, which does depend on the Migrate module. I'll roll a patch for this.

Stevel’s picture

Status: Active » Needs review
FileSize
8.37 KB

The attached patch moves date_migrate.test to the date_migrate_example module and updates the associated info files.

Stevel’s picture

FileSize
422 bytes

Sorry, the previous patch didn't work, because it still tries to run the tests. New patch adds the dependencies to migrate and features to the getInfo() module, as they are later required in the setUp() method.

DamienMcKenna’s picture

Status: Needs review » Reviewed & tested by the community
Parent issue: » #2510292: Plan for Date 7.x-2.10 release

Thanks again, @Stevel!

  • vijaycs85 committed a769db7 on 7.x-2.x authored by Stevel
    Issue #2412049 by Stevel: DateMigrateExampleUnitTest fails if migrate is...
vijaycs85’s picture

Status: Reviewed & tested by the community » Fixed

Thanks @Stevel and @DamienMcKenna

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.