Maniphest T133024

Unit test ext.popups.logger.js
Closed, ResolvedPublic2 Estimate Story Points
Actions

Description

No coverage whatsoever. Unit test properly.

Details

Related Gerrit Patches:

mediawiki/extensions/Popups : master

Add QUnit tests for ext.popups.logger

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Declined		Jdlrobson	T111329 [GOAL] Page previews on mobileweb
		Resolved		phuedx	T133947 [EPIC] Reduce bloat of Page Previews RL module(s)
		Resolved		Jdlrobson	T144522 [EPIC] Hovercards technical debt and quality enhancements
		Resolved		Jdlrobson	T67103 [EPIC] Hovercards: Test Coverage
		Resolved		Jhernandez	T133024 Unit test ext.popups.logger.js

Event Timeline

Jhernandez created this task.Apr 19 2016, 9:39 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 19 2016, 9:39 AM

Jhernandez moved this task from Needs Analysis to Code Review on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.Apr 19 2016, 9:40 AM

Change 283762 had a related patch set uploaded (by Jhernandez):
Add QUnit tests for ext.popups.logger

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

gerritbot added a project: Patch-For-Review.Apr 19 2016, 9:44 AM

phuedx moved this task from Code Review to -1 (Needs More Work) on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.Apr 19 2016, 1:52 PM

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptApr 19 2016, 1:52 PM

Jhernandez set the point value for this task to 2.Apr 19 2016, 3:43 PM

bmansurov moved this task from -1 (Needs More Work) to Code Review on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.Apr 19 2016, 9:18 PM

phuedx moved this task from Code Review to -1 (Needs More Work) on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.Apr 20 2016, 10:08 AM

bmansurov moved this task from -1 (Needs More Work) to Code Review on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.Apr 20 2016, 12:17 PM

bmansurov removed bmansurov as the assignee of this task.Apr 20 2016, 6:41 PM

bmansurov moved this task from Code Review to Ready for Signoff on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.

bmansurov added a subscriber: bmansurov.

Change 283762 merged by jenkins-bot:
Add QUnit tests for ext.popups.logger

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

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-04-26_(1.27.0-wmf.22)).Apr 20 2016, 7:00 PM

Jdlrobson removed a project: Patch-For-Review.Apr 20 2016, 7:56 PM

Jhernandez closed this task as Resolved.Apr 21 2016, 3:58 PM

Jhernandez claimed this task.

Jhernandez moved this task from Ready for Signoff to Done on the Reading-Web-Sprint-70-Lady-and-the-Trumps board.

Danny_B moved this task from Unsorted to Test coverage on the Technical-Debt board.Apr 21 2016, 9:02 PM