Maniphest T133024

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

Description

No coverage whatsoever. Unit test properly.

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Add QUnit tests for ext.popups.logger	mediawiki/extensions/Popups	master	+136 -0

Customize query in gerrit

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 subscribed.

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