The test module files

core/modules/aggregator/tests$ ls
aggregator_test_atom.xml  aggregator_test_rss091.xml          lib
aggregator_test.info.yml  aggregator_test_title_entities.xml
aggregator_test.module    config

should probably be in their own directory
core/module/aggregator/tests/modules/aggregator_test/
as elsewhere in core.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ramlev’s picture

I have moved the files into it's own subdirectory.

chrisjlee’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, aggregator-test-files-in-own-dir-2004022-1.patch, failed testing.

Niklas Fiekas’s picture

It appears to me like the new files are missing. Files can be moved with git mv old-location new-location (or git add new-location, git rm old-location). git diff --cached will then have the correct diff. If not already set git config diff.renames true would be helpful before the diff, so that renames are detected as such.

ollu’s picture

Status: Needs work » Needs review
FileSize
3.32 KB

Files moved to new location. Needs review.

Status: Needs review » Needs work
Issue tags: -Novice

The last submitted patch, aggregrator_new_location.patch, failed testing.

Niklas Fiekas’s picture

Status: Needs work » Needs review

Looks like d.o git was down. #5: aggregrator_new_location.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, aggregrator_new_location.patch, failed testing.

ollu’s picture

Status: Needs work » Needs review

#5: aggregrator_new_location.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, aggregrator_new_location.patch, failed testing.

ollu’s picture

Status: Needs work » Needs review
Issue tags: +Novice

#5: aggregrator_new_location.patch queued for re-testing.

ollu’s picture

The file was missing empty line at the end so I guess that's the reason for it to fail.

Status: Needs review » Needs work
Issue tags: -Novice

The last submitted patch, aggregrator_new_location.patch, failed testing.

ollu’s picture

Status: Needs work » Needs review

#12: aggregrator_new_location.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Novice

The last submitted patch, aggregrator_new_location.patch, failed testing.

ParisLiakos’s picture

ParisLiakos’s picture

Status: Needs work » Needs review
FileSize
4.9 KB

Status: Needs review » Needs work
Issue tags: -Novice

The last submitted patch, drupal-aggregator_move_test_modules-2004022-17.patch, failed testing.

ParisLiakos’s picture

Status: Needs work » Needs review
Issue tags: +Novice
tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

It's always good to be consistent.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 0cc160d and pushed to 8.x. Thanks!

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