Part of #2066993: Use magic methods to sync container property to \Drupal::getContainer in functional tests

CommentFileSizeAuthor
#1 2067007-1.patch2.98 KBandypost

Comments

andypost’s picture

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

Also removed unused variables in AggregatorCronTest

andypost’s picture

andypost
he/him
Primary language Russian
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 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
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
commented
Issue summary: View changes
Parent issue: » #2066993: Use magic methods to sync container property to \Drupal::getContainer in functional tests