Page MenuHomePhabricator

Unit test ext.popups.settings.js
Closed, ResolvedPublic3 Estimated Story Points


No coverage whatsoever. Unit test properly.

This module has mixed functionality from a view (unit testable) and other functionality that should be tested with acceptance tests (browser tests) like the add browser footer link thingy.

This task is about unit testing the parts that make sense. There's a followup task for adding browser tests for the addfooterlink thing.

Event Timeline

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

Change 286024 had a related patch set uploaded (by Bmansurov):
Add QUnit test for ext.popups.settings

Change 286024 merged by jenkins-bot:
Add QUnit test for ext.popups.settings

phuedx claimed this task.
phuedx added a subscriber: phuedx.

Since this is a technical task…