Problem/Motivation

Just noticed that 4 of the moved functional tests are using the migrate_drupal test fixture instead of the aggregator one.

Steps to reproduce

Proposed resolution

s/migrate_drupal/tracker

Remaining tasks

Patch
Review
Commit

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

CommentFileSizeAuthor
#2 3267508-2.patch3.2 KBquietone

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Issue summary: View changes
StatusFileSize
new 3267508-2.patch3.2 KB

Simple change to 4 files.

quietone’s picture

quietone commented
Status: Active » Needs review
spokje’s picture

spokje
Primary language Dutch
commented
Status: Needs review » Reviewed & tested by the community

Does what it says on the tin:

  • 4 time loadFixture() from the aggregator module instead of loading it from the migrate_drupal module.
  • Other 2 migration Functional Tests are already using the fixture from aggregator module
  • Green TestBot

RTBC

  • catch committed 19c7a3c on 10.0.x 
    Issue #3267508 by quietone, Spokje: Use aggregator fixture in migrate...
catch’s picture

catch
he/him
Primary language English
commented
Version: 10.0.x-dev » 9.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x and cherry-picked to 9.4.x, thanks!

Version: 9.4.x-dev » 10.0.x-dev
Status: Fixed » Reviewed & tested by the community
  • catch committed 4280bd6 on 9.4.x 
    Issue #3267508 by quietone, Spokje: Use aggregator fixture in migrate...
drumm’s picture

drumm
he/him
Location NY, US
commented

(Just updating to resolve cross-post.)

Status: Fixed » Closed (fixed)

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