Part of #2066993: Use \Drupal consistently in tests

CommentFileSizeAuthor
#1 2067007-1.patch2.98 KBandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Status: Active » Needs review
FileSize
2067007-1.patch2.98 KB

Also removed unused variables in AggregatorCronTest

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Component: system.module » aggregator.module
Issue tags: +Novice

assign module and taggin
PS: related issue #2002650: [meta, no patch] improve maintainability by removing unused local variables

ParisLiakos’s picture

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

thanks for killing those unused variables in the cron test, i am trying to get rid of it in various irrelevant issues but it never works somehow:P

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 as a volunteer commented
Issue summary: View changes
Parent issue: » #2066993: Use \Drupal consistently in tests