Problem/Motivation

The module aaa_update_test actually is in 2 locations: core/modules/update/tests/aaa_update_test and core/modules/update/tests/modules/aaa_update_test

Proposed resolution

Remove it from core/modules/update/tests/aaa_update_test

Remaining tasks

Do it

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 2990508-2.patch450 bytestedbow

Comments

tedbow created an issue. See original summary.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Active » Needs review
StatusFileSize
new 2990508-2.patch450 bytes
dani3lr0se’s picture

dani3lr0se commented
Status: Needs review » Reviewed & tested by the community

The patch applied successfully. Confirmed it's now removed. Thanks for the work on this patch @tedbow. :)

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

The patch applied using simplytest.me. Already marked RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Nice catch and totally confusing. I checked that the duplication is not part of testing and it does not appear to be. As a bug and test only change backported to 8.6.x

Committed and pushed 9077b0e47a to 8.7.x and 3479c574ef to 8.6.x. Thanks!

  • alexpott committed 9077b0e on 8.7.x 
    Issue #2990508 by tedbow: Update test module aaa_update_test is...

  • alexpott committed 3479c57 on 8.6.x 
    Issue #2990508 by tedbow: Update test module aaa_update_test is...

Status: Fixed » Closed (fixed)

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