Page MenuHomePhabricator

TestResults: Replace testResults fetching state with equivalent getter
Closed, ResolvedPublicBUG REPORT

Description

Description

During https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikiLambda/+/1005142 we've replaced the fetching boolean state with the promises of pending requests.
However, the getter getFetchingTestResults is necessary for the components FunctionReport and FunctionReportItem.

We should restore this getter and replace its previous functionality (returning a boolean state flag) with a different logic that asserts the fetching state from the promises.


Devices and Design (URLs or screenshots, if applicable):

  • Desktop: ...
  • Small screens/mobile: ...

Completion checklist

Event Timeline

Change 1010874 had a related patch set uploaded (by Genoveva Galarza; author: Genoveva Galarza):

[mediawiki/extensions/WikiLambda@master] Add fetching state as a local flag instead of global getter

https://gerrit.wikimedia.org/r/1010874

Change 1010874 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] Add fetching state as a local flag instead of global getter

https://gerrit.wikimedia.org/r/1010874

Jdforrester-WMF moved this task from To Triage to To deploy on the Abstract Wikipedia team board.