launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #09353
[Merge] lp:~rharding/launchpad/bug_yui35_three into lp:launchpad
Richard Harding has proposed merging lp:~rharding/launchpad/bug_yui35_three into lp:launchpad.
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
Related bugs:
Bug #793579 in Launchpad itself: "LPClient JS test class is duplicated; should be consolidated in lib/lp/testing and documented/polished"
https://bugs.launchpad.net/launchpad/+bug/793579
For more details, see:
https://code.launchpad.net/~rharding/launchpad/bug_yui35_three/+merge/112590
= Summary =
This branch helps update several of the test suites in the bug JS to work in YUI3.5 and to use our common test runner.
== Implementation Notes ==
This fixes an outstand XXX bug by moving LPClient into the newish testing.helpers module. Three tests are updated that were using it.
Updating the tests is mostly just reorganizing them to add an actual module. There are some smaller bugs fixed such as the quotes around selectors using attributes.
== Tests ==
./bin/test -x -cvv --layer=YUITestLayer
== Lint ==
Updated for lint since we indented a lot of code.
== LoC Qualification ==
It's a negative LoC change.
--
https://code.launchpad.net/~rharding/launchpad/bug_yui35_three/+merge/112590
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rharding/launchpad/bug_yui35_three into lp:launchpad.
=== modified file 'lib/lp/app/javascript/testing/helpers.js'
--- lib/lp/app/javascript/testing/helpers.js 2012-06-27 17:29:08 +0000
+++ lib/lp/app/javascript/testing/helpers.js 2012-06-28 15:32:36 +0000
@@ -19,6 +19,40 @@
};
+ ns.LPClient = function () {
+ if (!(this instanceof ns.LPClient)) {
+ throw new Error("Constructor called as a function");
+ }
+ this.received = [];
+ // We create new functions every time because we allow them to be
+ // configured.
+ this.named_post = function(url, func, config) {
+ this._call('named_post', config, arguments);
+ };
+ this.patch = function(bug_filter, data, config) {
+ this._call('patch', config, arguments);
+ };
+ }
+ ns.LPClient.prototype._call = function(name, config, args) {
+ this.received.push(
+ [name, Array.prototype.slice.call(args)]);
+ if (!Y.Lang.isValue(args.callee.args)) {
+ throw new Error("Set call_args on "+name);
+ }
+ var do_action = function () {
+ if (Y.Lang.isValue(args.callee.fail) && args.callee.fail) {
+ config.on.failure.apply(undefined, args.callee.args);
+ } else {
+ config.on.success.apply(undefined, args.callee.args);
+ }
+ };
+ if (Y.Lang.isValue(args.callee.halt) && args.callee.halt) {
+ args.callee.resume = do_action;
+ } else {
+ do_action();
+ }
+ };
+
}, '0.1', {
'requires': [ 'history']
});
=== modified file 'lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.html'
--- lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.html 2012-03-14 04:41:36 +0000
+++ lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.html 2012-06-28 15:32:36 +0000
@@ -21,6 +21,8 @@
<script type="text/javascript"
src="../../../../../build/js/lp/app/testing/testrunner.js"></script>
+ <script type="text/javascript"
+ src="../../../../../build/js/lp/app/testing/helpers.js"></script>
<link rel="stylesheet" href="../../../app/javascript/testing/test.css" />
@@ -58,7 +60,7 @@
<body class="yui3-skin-sam">
<ul id="suites">
<!-- <li>lp.large_indicator.test</li> -->
- <li>lp.bug_subscription_portlet.test</li>
+ <li>lp.bugs.subscription_portlet.test</li>
</ul>
</body>
</html>
=== modified file 'lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.js'
--- lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.js 2011-09-27 16:14:09 +0000
+++ lib/lp/bugs/javascript/tests/test_bug_subscription_portlet.js 2012-06-28 15:32:36 +0000
@@ -1,682 +1,630 @@
-YUI({
- base: '../../../../canonical/launchpad/icing/yui/',
- filter: 'raw', combine: false, fetchCSS: false
- }).use('test', 'console', 'lp.bugs.bugtask_index.portlets.subscription',
- 'node-event-simulate', function(Y) {
-
-var suite = new Y.Test.Suite(
- "lp.bugs.bugtask_index.portlets.subscription Tests");
-var module = Y.lp.bugs.bugtask_index.portlets.subscription;
-
-/**
- * XXX gary 2011-05-26 bug 793579
- * LPClient is copied three times (see also test_structural_subscription.js
- * and test_subscription.js). It should be pushed to a shared test module.
- */
-function LPClient(){
- if (!(this instanceof LPClient)) {
- throw new Error("Constructor called as a function");
- }
- this.received = [];
- // We create new functions every time because we allow them to be
- // configured.
- this.named_post = function(url, func, config) {
- this._call('named_post', config, arguments);
- };
- this.patch = function(bug_filter, data, config) {
- this._call('patch', config, arguments);
- };
-}
-LPClient.prototype._call = function(name, config, args) {
- this.received.push(
- [name, Array.prototype.slice.call(args)]);
- if (!Y.Lang.isValue(args.callee.args)) {
- throw new Error("Set call_args on "+name);
- }
- var do_action = function () {
- if (Y.Lang.isValue(args.callee.fail) && args.callee.fail) {
- config.on.failure.apply(undefined, args.callee.args);
- } else {
- config.on.success.apply(undefined, args.callee.args);
- }
- };
- if (Y.Lang.isValue(args.callee.halt) && args.callee.halt) {
- args.callee.resume = do_action;
- } else {
- do_action();
- }
-};
-
-function make_status_node() {
- var status = Y.Node.create('<div/>')
- .set('id', 'current_user_subscription')
- .append(Y.Node.create('<span/>'));
- Y.one('body').appendChild(status);
- return status;
-}
-
-function add_link_to_status_node() {
- var status = Y.one('#current_user_subscription');
- status.append(
- Y.Node.create('<a/>')
- .addClass('menu-link-subscription')
- .addClass('sprite')
- .addClass('modify')
- .addClass('edit')
- .set('href', 'http://example.com')
- .set('text', 'Example text')
- );
-}
-
-function make_mute_node() {
- var parent = Y.Node.create('<div/>')
- .set('id', 'mute-link-container')
- .append(Y.Node.create('<a/>')
- .addClass('menu-link-mute_subscription')
- .addClass(module.UNMUTED_CLASS)
- .set('text', 'This is a mute link')
- .set('href', 'http://www.example.com/+mute')
- );
- Y.one('body').appendChild(parent);
- return parent;
-}
-
-function setup_LP(bug_link) {
- window.LP = {
- cache: {
- notifications_text: {
- not_only_other_subscription: 'You are',
- only_other_subscription:
- 'You have subscriptions that may cause you to receive ' +
- 'notifications, but you are',
- direct_all: 'subscribed to all notifications for this bug.',
- direct_metadata:
- 'subscribed to all notifications except comments for ' +
- 'this bug.',
- direct_lifecycle:
- 'subscribed to notifications when this bug is closed ' +
- 'or reopened.',
- not_direct:
- "not directly subscribed to this bug's notifications.",
- muted:
- 'Your personal email notifications from this bug ' +
- 'are muted.'
+/* Copyright (c) 2012 Canonical Ltd. All rights reserved. */
+
+YUI.add('lp.bugs.subscription_portlet.test', function (Y) {
+ var module = Y.lp.bugs.bugtask_index.portlets.subscription;
+
+ // Notification levels.
+ var DISCUSSION = 'Discussion';
+ var DETAILS = 'Details';
+ var LIFECYCLE = 'Lifecycle';
+
+ function make_status_node() {
+ var status = Y.Node.create('<div/>')
+ .set('id', 'current_user_subscription')
+ .append(Y.Node.create('<span/>'));
+ Y.one('body').appendChild(status);
+ return status;
+ }
+
+ function add_link_to_status_node() {
+ var status = Y.one('#current_user_subscription');
+ status.append(
+ Y.Node.create('<a/>')
+ .addClass('menu-link-subscription')
+ .addClass('sprite')
+ .addClass('modify')
+ .addClass('edit')
+ .set('href', 'http://example.com')
+ .set('text', 'Example text')
+ );
+ }
+
+ function make_mute_node() {
+ var parent = Y.Node.create('<div/>')
+ .set('id', 'mute-link-container')
+ .append(Y.Node.create('<a/>')
+ .addClass('menu-link-mute_subscription')
+ .addClass(module.UNMUTED_CLASS)
+ .set('text', 'This is a mute link')
+ .set('href', 'http://www.example.com/+mute')
+ );
+ Y.one('body').appendChild(parent);
+ return parent;
+ }
+
+ function setup_LP(bug_link) {
+ window.LP = {
+ cache: {
+ notifications_text: {
+ not_only_other_subscription: 'You are',
+ only_other_subscription:
+ 'You have subscriptions that may cause you to receive ' +
+ 'notifications, but you are',
+ direct_all: 'subscribed to all notifications for this bug.',
+ direct_metadata:
+ 'subscribed to all notifications except comments for ' +
+ 'this bug.',
+ direct_lifecycle:
+ 'subscribed to notifications when this bug is closed ' +
+ 'or reopened.',
+ not_direct:
+ "not directly subscribed to this bug's notifications.",
+ muted:
+ 'Your personal email notifications from this bug ' +
+ 'are muted.'
+ },
+ context: {web_link: 'http://example.com', bug_link: bug_link},
+ other_subscription_notifications: false
},
- context: {web_link: 'http://example.com', bug_link: bug_link},
- other_subscription_notifications: false
- },
- links: {me: '~tweedledee'}
- };
-}
-
-// Notification levels.
-var DISCUSSION = 'Discussion';
-var DETAILS = 'Details';
-var LIFECYCLE = 'Lifecycle';
-
-function make_subscription(level) {
- if (Y.Lang.isUndefined(level)) {
- level = DISCUSSION;
- }
- window.LP.cache.subscription = {'bug_notification_level': level};
-}
-
-/**
- * Test update_subscription_status.
- */
-suite.add(new Y.Test.Case({
- name: 'Test update_subscription_status',
-
- setUp: function() {
- this.status_node = make_status_node();
- this.mute_node = make_mute_node();
- add_link_to_status_node();
- setup_LP();
- },
-
- tearDown: function() {
- this.status_node.remove();
- this.mute_node.remove();
- delete window.LP;
- },
-
- test_can_create_link: function() {
- this.status_node.one('a').remove();
- make_subscription();
- Y.Assert.isTrue(Y.Lang.isNull(this.status_node.one('a')));
- module.update_subscription_status();
- var link = this.status_node.one('a');
- Y.Assert.isTrue(Y.Lang.isValue(link));
- Y.Assert.isTrue(link.hasClass('menu-link-subscription'));
- Y.Assert.isTrue(link.hasClass('sprite'));
- Y.Assert.isTrue(link.hasClass('modify'));
- Y.Assert.isTrue(link.hasClass('edit'));
- Y.Assert.isTrue(link.hasClass('js-action'));
- Y.Assert.areEqual(
- // window.LP.context.web_link + '/+subscribe',
- 'http://example.com/+subscribe',
- link.get('href'));
- },
-
- test_no_subscription: function() {
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- "not directly subscribed to this bug's notifications.",
- this.status_node.one('a').get('text'));
- },
-
- test_other_subscription: function() {
- window.LP.cache.other_subscription_notifications = true;
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You have subscriptions that may cause you to receive ' +
- 'notifications, but you are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- "not directly subscribed to this bug's notifications.",
- this.status_node.one('a').get('text'));
- },
-
- test_full_subscription: function() {
- make_subscription(DISCUSSION);
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- "subscribed to all notifications for this bug.",
- this.status_node.one('a').get('text'));
- },
-
- test_metadata_subscription: function() {
- make_subscription(DETAILS);
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- 'subscribed to all notifications except comments for this bug.',
- this.status_node.one('a').get('text'));
- },
-
- test_lifecycle_subscription: function() {
- make_subscription(LIFECYCLE);
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- 'subscribed to notifications when this bug is closed or ' +
- 'reopened.',
- this.status_node.one('a').get('text'));
- },
-
- test_direct_subscription_has_precedence: function() {
- window.LP.cache.other_subscription_notifications = true;
- make_subscription(LIFECYCLE);
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- 'subscribed to notifications when this bug is closed or ' +
- 'reopened.',
- this.status_node.one('a').get('text'));
- },
-
- test_muted_subscription: function() {
- make_subscription(LIFECYCLE);
- this.mute_node.one('a').replaceClass(
- module.UNMUTED_CLASS, module.MUTED_CLASS);
- Y.Assert.isTrue(Y.Lang.isValue(this.status_node.one('a')));
- module.update_subscription_status();
- Y.Assert.areEqual(
- 'Your personal email notifications from this bug are muted.',
- this.status_node.one('span').get('text'));
- Y.Assert.isFalse(Y.Lang.isValue(this.status_node.one('a')));
- }
-
-}));
-
-/**
- * Test setup_mute_link_handlers.
- */
-suite.add(new Y.Test.Case({
- name: 'Test setup_mute_link_handlers',
-
- setUp: function() {
- this.status_node = make_status_node();
- this.mute_node = make_mute_node();
- this.link = this.mute_node.one('a');
- add_link_to_status_node();
- this.bug_link = 'http://example.net/firefox/bug/1';
- setup_LP(this.bug_link);
- make_subscription(DISCUSSION);
- module.update_subscription_status();
- module.setup_mute_link_handlers();
- module._lp_client = new LPClient();
- module._lp_client.named_post.args = [];
- },
-
- tearDown: function() {
- this.status_node.remove();
- this.mute_node.remove();
- delete window.LP;
- delete module._lp_client;
- var error_overlay = Y.one('.yui3-lazr-formoverlay');
- if (Y.Lang.isValue(error_overlay)) {
- error_overlay.remove();
- }
- },
-
- test_mute_success: function() {
- this.link.simulate('click');
- Y.Assert.areEqual(1, module._lp_client.received.length);
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('mute', args[1]);
- Y.ObjectAssert.areEqual({}, args[2].parameters);
- Y.Assert.isTrue(this.link.hasClass(module.MUTED_CLASS));
- Y.Assert.isFalse(this.link.hasClass('spinner'));
- Y.Assert.isFalse(this.link.hasClass(module.UNMUTED_CLASS));
- Y.Assert.areEqual(
- 'Your personal email notifications from this bug are muted.',
- this.status_node.one('span').get('text'));
- },
-
- test_unmute_success: function() {
- this.link.replaceClass(module.UNMUTED_CLASS, module.MUTED_CLASS);
- this.link.simulate('click');
- Y.Assert.areEqual(1, module._lp_client.received.length);
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('unmute', args[1]);
- Y.ObjectAssert.areEqual({}, args[2].parameters);
- Y.Assert.isTrue(this.link.hasClass(module.UNMUTED_CLASS));
- Y.Assert.isFalse(this.link.hasClass('spinner'));
- Y.Assert.isFalse(this.link.hasClass(module.MUTED_CLASS));
- Y.Assert.areEqual(
- 'You are',
- this.status_node.one('span').get('text'));
- Y.Assert.areEqual(
- "subscribed to all notifications for this bug.",
- this.status_node.one('a').get('text'));
- },
-
- test_mute_spinner_and_failure: function() {
- module._lp_client.named_post.fail = true;
- module._lp_client.named_post.args = [
- true,
- {status: 400, responseText: 'Rutebegas!'}];
- module._lp_client.named_post.halt = true;
- this.link.simulate('click');
- // Right now, this is as if we are waiting for the server to
- // reply. The link is spinning.
- Y.Assert.isTrue(this.link.hasClass('spinner'));
- Y.Assert.isFalse(this.link.hasClass(module.UNMUTED_CLASS));
- // Now the server replies with an error.
- module._lp_client.named_post.resume();
- // We have no spinner.
- Y.Assert.isTrue(this.link.hasClass(module.UNMUTED_CLASS));
- Y.Assert.isFalse(this.link.hasClass('spinner'));
- // The page has rendered the error overlay.
- var error_box = Y.one('.yui3-lazr-formoverlay-errors');
- Y.Assert.isTrue(Y.Lang.isValue(error_box));
- }
-
-}));
-
-/**
- * Test setup_subscription_link_handlers.
- */
-suite.add(new Y.Test.Case({
- name: 'Test setup_subscription_link_handlers',
-
- setUp: function() {
- this.status_node = make_status_node();
- this.mute_node = make_mute_node();
- this.mute_link = this.mute_node.one('a');
- this.bug_link = 'http://example.net/firefox/bug/1';
- setup_LP(this.bug_link);
- module._lp_client = new LPClient();
- },
-
- tearDown: function() {
- this.status_node.remove();
- this.mute_node.remove();
- delete window.LP;
- delete module._lp_client;
- var error_overlay = Y.one('.yui3-lazr-formoverlay');
- if (Y.Lang.isValue(error_overlay)) {
- error_overlay.remove();
- }
- var pretty_overlay = Y.one('.pretty-overlay-window');
- if (Y.Lang.isValue(pretty_overlay)) {
- pretty_overlay.remove();
- }
- },
-
- init: function(sub_level, has_other_subs, response) {
- if (Y.Lang.isValue(sub_level)) {
- make_subscription(sub_level);
- } else if (!has_other_subs) {
- this.mute_node.addClass('hidden');
- }
- window.LP.cache.other_subscription_notifications = has_other_subs;
- var args = [];
- if (Y.Lang.isValue(response)) {
- args = [
- {getAttrs: function () {
- return {bug_notification_level: response};
- }}];
- }
- module._lp_client.named_post.args = args;
- module.update_subscription_status();
- module.setup_mute_link_handlers();
- },
-
- link: function() {
- return this.status_node.one('a');
- },
-
- test_overlay_add_subscription: function() {
- this.init(null, false);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- // We have the "Add" title, not the "Change" title.
- Y.Assert.areEqual('Add a mail subscription for this bug',
- overlay.one('h2').get('text'));
- // There is no status.
- Y.Assert.isFalse(Y.Lang.isValue(overlay.one('.subscription-status')));
- // The action links are visible.
- var action_links = overlay.one('.subscription-actions');
- Y.Assert.isFalse(action_links.one(
- '.Discussion a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
- // The "Remove" link is not present.
- Y.Assert.isFalse(Y.Lang.isValue(overlay.one('a.remove')));
- // The link is spinning.
- Y.Assert.isTrue(this.link().hasClass('spinner'));
- Y.Assert.isFalse(this.link().hasClass('edit'));
- // When we click on the "x," the overlay goes away, and the spinner
- // is gone.
- overlay.one('a.close-button').simulate('click');
- Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- Y.Assert.isFalse(this.link().hasClass('spinner'));
- Y.Assert.isTrue(this.link().hasClass('edit'));
- },
-
- test_overlay_with_discussion_subscription: function() {
- // Starting with a DISCUSSION subscription, we get a "change" overlay.
- this.init(DISCUSSION, false);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- // We have the "Change" title, not the "Add" title.
- Y.Assert.areEqual('Change your mail subscription for this bug',
- overlay.one('h2').get('text'));
- // We show the Discussion status.
- var status = overlay.one('.subscription-status');
- Y.Assert.isFalse(overlay.one('span.Discussion').hasClass('hidden'));
- Y.Assert.isTrue(overlay.one('span.Details').hasClass('hidden'));
- Y.Assert.isTrue(overlay.one('span.Lifecycle').hasClass('hidden'));
- // The action links are visible except for Discussion.
- var action_links = overlay.one('.subscription-actions');
- Y.Assert.isTrue(action_links.one('.Discussion a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
- // We have a remove link.
- Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
- Y.Assert.areEqual(
- 'Remove your direct subscription',
- overlay.one('div.subscription-actions+div').get('text'));
- },
-
- test_overlay_with_details_subscription: function() {
- // Test overlay with existing DETAILS subscription.
- this.init(DETAILS, true);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- // We show the Lifecycle status.
- var status = overlay.one('.subscription-status');
- Y.Assert.isTrue(overlay.one('span.Discussion').hasClass('hidden'));
- Y.Assert.isFalse(overlay.one('span.Details').hasClass('hidden'));
- Y.Assert.isTrue(overlay.one('span.Lifecycle').hasClass('hidden'));
- // The action links are visible except for Lifecycle.
- var action_links = overlay.one('.subscription-actions');
- Y.Assert.isFalse(action_links.one(
- '.Discussion a').hasClass('hidden'));
- Y.Assert.isTrue(action_links.one('.Details a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
- // We have a remove link.
- Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
- Y.Assert.areEqual(
- 'Remove your direct subscription',
- overlay.one('div.subscription-actions+div').get('text'));
- },
-
- test_overlay_with_lifecycle_subscription: function() {
- // Test overlay with existing LIFECYCLE subscription.
- this.init(LIFECYCLE, true);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- // We show the Lifecycle status.
- var status = overlay.one('.subscription-status');
- Y.Assert.isTrue(overlay.one('span.Discussion').hasClass('hidden'));
- Y.Assert.isTrue(overlay.one('span.Details').hasClass('hidden'));
- Y.Assert.isFalse(overlay.one('span.Lifecycle').hasClass('hidden'));
- // The action links are visible except for Lifecycle.
- var action_links = overlay.one('.subscription-actions');
- Y.Assert.isFalse(action_links.one(
- '.Discussion a').hasClass('hidden'));
- Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
- Y.Assert.isTrue(action_links.one('.Lifecycle a').hasClass('hidden'));
- // We have a remove link.
- Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
- Y.Assert.areEqual(
- 'Remove your direct subscription',
- overlay.one('div.subscription-actions+div').get('text'));
- },
-
- test_subscribe_discussion: function() {
- this.init(null, false, DISCUSSION);
- // The mute node is hidden initially.
- Y.Assert.isTrue(this.mute_node.hasClass('hidden'));
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('.subscription-actions .Discussion a').simulate('click');
- // The overlay has been destroyed, and the spinner is gone.
- Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- Y.Assert.isFalse(this.link().hasClass('spinner'));
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('subscribe', args[1]);
- Y.ObjectAssert.areEqual({person: '~tweedledee', level: DISCUSSION},
- args[2].parameters);
- // There is now a subscription object in the cache.
- Y.Assert.isTrue(Y.Lang.isValue(window.LP.cache.subscription));
- // The mute node is not hidden.
- Y.Assert.isFalse(this.mute_node.hasClass('hidden'));
- // The link has updated its text.
- Y.Assert.areEqual(
- "You are subscribed to all notifications for this bug.",
- this.status_node.get('text'));
- },
-
- test_subscribe_details: function() {
- this.init(null, false, DETAILS);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('.subscription-actions .Details a').simulate('click');
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('subscribe', args[1]);
- Y.ObjectAssert.areEqual({person: '~tweedledee', level: DETAILS},
- args[2].parameters);
- // The link has updated its text.
- Y.Assert.areEqual(
- "You are subscribed to all notifications except comments for "+
- "this bug.",
- this.status_node.get('text'));
- },
-
- test_subscribe_lifecycle: function() {
- this.init(null, false, LIFECYCLE);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('.subscription-actions .Lifecycle a').simulate('click');
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('subscribe', args[1]);
- Y.ObjectAssert.areEqual({person: '~tweedledee', level: LIFECYCLE},
- args[2].parameters);
- // The link has updated its text.
- Y.Assert.areEqual(
- "You are subscribed to notifications when this bug is closed "+
- "or reopened.",
- this.status_node.get('text'));
- },
-
- test_unsubscribe: function() {
- this.init(LIFECYCLE, false);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('a.remove').simulate('click');
- // There is no warning.
- var warning = overlay.one('.private-bug-warning');
- Y.Assert.isNull(warning);
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('unsubscribe', args[1]);
- Y.ObjectAssert.areEqual({}, args[2].parameters);
- // The overlay has been destroyed, and the spinner is gone.
- Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- Y.Assert.isFalse(this.link().hasClass('spinner'));
- // The mute node is hidden.
- Y.Assert.isTrue(this.mute_node.hasClass('hidden'));
- // The link has updated its text.
- Y.Assert.areEqual(
- "You are not directly subscribed to this bug's notifications.",
- this.status_node.get('text'));
- },
-
- test_unsubscribe_from_private_bug: function() {
- this.init(LIFECYCLE, false);
- window.LP.cache.bug_is_private = true;
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('a.remove').simulate('click');
- // The overlay now has a warning.
- var warning = overlay.one('.private-bug-warning');
- Y.Assert.isTrue(Y.Lang.isValue(warning));
- // Find and click the OK button.
- var ok_btn = warning.one('.ok-btn');
- ok_btn.simulate('click');
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('unsubscribe', args[1]);
- Y.ObjectAssert.areEqual({}, args[2].parameters);
- // The overlay has been destroyed, and the spinner is gone.
- Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- Y.Assert.isFalse(this.link().hasClass('spinner'));
- },
-
- test_unsubscribe_from_private_bug_cancel: function() {
- this.init(LIFECYCLE, false);
- window.LP.cache.bug_is_private = true;
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('a.remove').simulate('click');
- // The overlay now has a warning.
- var warning = overlay.one('.private-bug-warning');
- Y.Assert.isTrue(Y.Lang.isValue(warning));
- // Find and click the Cancel button.
- var cancel_btn = warning.one('.cancel-btn');
- cancel_btn.simulate('click');
- // The warning is gone.
- warning = overlay.one('.private-bug-warning');
- Y.Assert.isNull(warning);
- },
-
- test_unsubscribe_with_other: function() {
- this.init(LIFECYCLE, true);
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- overlay.one('a.remove').simulate('click');
- // We got an appropriate call to the webservice.
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(this.bug_link, args[0]);
- Y.Assert.areEqual('unsubscribe', args[1]);
- Y.ObjectAssert.areEqual({}, args[2].parameters);
- // The overlay has been destroyed, and the spinner is gone.
- Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- Y.Assert.isFalse(this.link().hasClass('spinner'));
- // The mute node is not hidden.
- Y.Assert.isFalse(this.mute_node.hasClass('hidden'));
- // The link has updated its text.
- Y.Assert.areEqual(
- "You have subscriptions that may cause you to receive "+
- "notifications, but you are not directly subscribed to this "+
- "bug's notifications.",
- this.status_node.get('text'));
- },
-
- test_io_spinner_and_error: function() {
- this.init(LIFECYCLE, true);
- module._lp_client.named_post.fail = true;
- module._lp_client.named_post.args = [
- true,
- {status: 400, responseText: 'Rutebegas!'}];
- module._lp_client.named_post.halt = true;
- this.link().simulate('click');
- var overlay = Y.one('.pretty-overlay-window');
- var unsub = overlay.one('a.remove');
- unsub.simulate('click');
- // Right now, this is as if we are waiting for the server to
- // reply. The link is spinning.
- Y.Assert.isTrue(unsub.hasClass('spinner'));
- Y.Assert.isFalse(unsub.hasClass('remove'));
- // Now the server replies with an error.
- module._lp_client.named_post.resume();
- // We have no spinner.
- Y.Assert.isTrue(unsub.hasClass('remove'));
- Y.Assert.isFalse(unsub.hasClass('spinner'));
- // The page has rendered the error overlay.
- var error_box = Y.one('.yui3-lazr-formoverlay-errors');
- Y.Assert.isTrue(Y.Lang.isValue(error_box));
- // The overlay is still hanging around too, as expected, but for
- // better or worse (popup-on-popup is not the best UI).
- Y.Assert.isTrue(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
- }
-
-}));
-
-
-Y.Test.Runner.on('complete', function(data) {
- window.status = '::::' + JSON.stringify(data);
- });
-Y.Test.Runner.add(suite);
-
-var console = new Y.Console({newestOnTop: false});
-console.render('#log');
-
-Y.on('domready', function() {
- Y.Test.Runner.run();
-});
+ links: {me: '~tweedledee'}
+ };
+ }
+
+ function make_subscription(level) {
+ if (Y.Lang.isUndefined(level)) {
+ level = DISCUSSION;
+ }
+ window.LP.cache.subscription = {'bug_notification_level': level};
+ }
+
+ var tests = Y.namespace('lp.bugs.subscription_portlet.test');
+ tests.suite = new Y.Test.Suite('bugs.subscription_portlet Tests');
+
+ /**
+ * Test update_subscription_status.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test update_subscription_status',
+
+ setUp: function() {
+ this.status_node = make_status_node();
+ this.mute_node = make_mute_node();
+ add_link_to_status_node();
+ setup_LP();
+ },
+
+ tearDown: function() {
+ this.status_node.remove();
+ this.mute_node.remove();
+ delete window.LP;
+ },
+
+ test_can_create_link: function() {
+ this.status_node.one('a').remove();
+ make_subscription();
+ Y.Assert.isTrue(Y.Lang.isNull(this.status_node.one('a')));
+ module.update_subscription_status();
+ var link = this.status_node.one('a');
+ Y.Assert.isTrue(Y.Lang.isValue(link));
+ Y.Assert.isTrue(link.hasClass('menu-link-subscription'));
+ Y.Assert.isTrue(link.hasClass('sprite'));
+ Y.Assert.isTrue(link.hasClass('modify'));
+ Y.Assert.isTrue(link.hasClass('edit'));
+ Y.Assert.isTrue(link.hasClass('js-action'));
+ Y.Assert.areEqual(
+ // window.LP.context.web_link + '/+subscribe',
+ 'http://example.com/+subscribe',
+ link.get('href'));
+ },
+
+ test_no_subscription: function() {
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ "not directly subscribed to this bug's notifications.",
+ this.status_node.one('a').get('text'));
+ },
+
+ test_other_subscription: function() {
+ window.LP.cache.other_subscription_notifications = true;
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You have subscriptions that may cause you to receive ' +
+ 'notifications, but you are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ "not directly subscribed to this bug's notifications.",
+ this.status_node.one('a').get('text'));
+ },
+
+ test_full_subscription: function() {
+ make_subscription(DISCUSSION);
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ "subscribed to all notifications for this bug.",
+ this.status_node.one('a').get('text'));
+ },
+
+ test_metadata_subscription: function() {
+ make_subscription(DETAILS);
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ 'subscribed to all notifications except comments for this bug.',
+ this.status_node.one('a').get('text'));
+ },
+
+ test_lifecycle_subscription: function() {
+ make_subscription(LIFECYCLE);
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ 'subscribed to notifications when this bug is closed or ' +
+ 'reopened.',
+ this.status_node.one('a').get('text'));
+ },
+
+ test_direct_subscription_has_precedence: function() {
+ window.LP.cache.other_subscription_notifications = true;
+ make_subscription(LIFECYCLE);
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ 'subscribed to notifications when this bug is closed or ' +
+ 'reopened.',
+ this.status_node.one('a').get('text'));
+ },
+
+ test_muted_subscription: function() {
+ make_subscription(LIFECYCLE);
+ this.mute_node.one('a').replaceClass(
+ module.UNMUTED_CLASS, module.MUTED_CLASS);
+ Y.Assert.isTrue(Y.Lang.isValue(this.status_node.one('a')));
+ module.update_subscription_status();
+ Y.Assert.areEqual(
+ 'Your personal email notifications from this bug are muted.',
+ this.status_node.one('span').get('text'));
+ Y.Assert.isFalse(Y.Lang.isValue(this.status_node.one('a')));
+ }
+
+ }));
+
+ /**
+ * Test setup_mute_link_handlers.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test setup_mute_link_handlers',
+
+ setUp: function() {
+ this.status_node = make_status_node();
+ this.mute_node = make_mute_node();
+ this.link = this.mute_node.one('a');
+ add_link_to_status_node();
+ this.bug_link = 'http://example.net/firefox/bug/1';
+ setup_LP(this.bug_link);
+ make_subscription(DISCUSSION);
+ module.update_subscription_status();
+ module.setup_mute_link_handlers();
+ module._lp_client = new Y.lp.testing.helpers.LPClient();
+ module._lp_client.named_post.args = [];
+ },
+
+ tearDown: function() {
+ this.status_node.remove();
+ this.mute_node.remove();
+ delete window.LP;
+ delete module._lp_client;
+ var error_overlay = Y.one('.yui3-lazr-formoverlay');
+ if (Y.Lang.isValue(error_overlay)) {
+ error_overlay.remove();
+ }
+ },
+
+ test_mute_success: function() {
+ this.link.simulate('click');
+ Y.Assert.areEqual(1, module._lp_client.received.length);
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('mute', args[1]);
+ Y.ObjectAssert.areEqual({}, args[2].parameters);
+ Y.Assert.isTrue(this.link.hasClass(module.MUTED_CLASS));
+ Y.Assert.isFalse(this.link.hasClass('spinner'));
+ Y.Assert.isFalse(this.link.hasClass(module.UNMUTED_CLASS));
+ Y.Assert.areEqual(
+ 'Your personal email notifications from this bug are muted.',
+ this.status_node.one('span').get('text'));
+ },
+
+ test_unmute_success: function() {
+ this.link.replaceClass(module.UNMUTED_CLASS, module.MUTED_CLASS);
+ this.link.simulate('click');
+ Y.Assert.areEqual(1, module._lp_client.received.length);
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('unmute', args[1]);
+ Y.ObjectAssert.areEqual({}, args[2].parameters);
+ Y.Assert.isTrue(this.link.hasClass(module.UNMUTED_CLASS));
+ Y.Assert.isFalse(this.link.hasClass('spinner'));
+ Y.Assert.isFalse(this.link.hasClass(module.MUTED_CLASS));
+ Y.Assert.areEqual(
+ 'You are',
+ this.status_node.one('span').get('text'));
+ Y.Assert.areEqual(
+ "subscribed to all notifications for this bug.",
+ this.status_node.one('a').get('text'));
+ },
+
+ test_mute_spinner_and_failure: function() {
+ module._lp_client.named_post.fail = true;
+ module._lp_client.named_post.args = [
+ true,
+ {status: 400, responseText: 'Rutebegas!'}];
+ module._lp_client.named_post.halt = true;
+ this.link.simulate('click');
+ // Right now, this is as if we are waiting for the server to
+ // reply. The link is spinning.
+ Y.Assert.isTrue(this.link.hasClass('spinner'));
+ Y.Assert.isFalse(this.link.hasClass(module.UNMUTED_CLASS));
+ // Now the server replies with an error.
+ module._lp_client.named_post.resume();
+ // We have no spinner.
+ Y.Assert.isTrue(this.link.hasClass(module.UNMUTED_CLASS));
+ Y.Assert.isFalse(this.link.hasClass('spinner'));
+ // The page has rendered the error overlay.
+ var error_box = Y.one('.yui3-lazr-formoverlay-errors');
+ Y.Assert.isTrue(Y.Lang.isValue(error_box));
+ }
+ }));
+
+ /**
+ * Test setup_subscription_link_handlers.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test setup_subscription_link_handlers',
+
+ setUp: function() {
+ this.status_node = make_status_node();
+ this.mute_node = make_mute_node();
+ this.mute_link = this.mute_node.one('a');
+ this.bug_link = 'http://example.net/firefox/bug/1';
+ setup_LP(this.bug_link);
+ module._lp_client = new Y.lp.testing.helpers.LPClient();
+ },
+
+ tearDown: function() {
+ this.status_node.remove();
+ this.mute_node.remove();
+ delete window.LP;
+ delete module._lp_client;
+ var error_overlay = Y.one('.yui3-lazr-formoverlay');
+ if (Y.Lang.isValue(error_overlay)) {
+ error_overlay.remove();
+ }
+ var pretty_overlay = Y.one('.pretty-overlay-window');
+ if (Y.Lang.isValue(pretty_overlay)) {
+ pretty_overlay.remove();
+ }
+ },
+
+ init: function(sub_level, has_other_subs, response) {
+ if (Y.Lang.isValue(sub_level)) {
+ make_subscription(sub_level);
+ } else if (!has_other_subs) {
+ this.mute_node.addClass('hidden');
+ }
+ window.LP.cache.other_subscription_notifications = has_other_subs;
+ var args = [];
+ if (Y.Lang.isValue(response)) {
+ args = [
+ {getAttrs: function () {
+ return {bug_notification_level: response};
+ }}];
+ }
+ module._lp_client.named_post.args = args;
+ module.update_subscription_status();
+ module.setup_mute_link_handlers();
+ },
+
+ link: function() {
+ return this.status_node.one('a');
+ },
+
+ test_overlay_add_subscription: function() {
+ this.init(null, false);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ // We have the "Add" title, not the "Change" title.
+ Y.Assert.areEqual('Add a mail subscription for this bug',
+ overlay.one('h2').get('text'));
+ // There is no status.
+ Y.Assert.isFalse(Y.Lang.isValue(overlay.one('.subscription-status')));
+ // The action links are visible.
+ var action_links = overlay.one('.subscription-actions');
+ Y.Assert.isFalse(action_links.one(
+ '.Discussion a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
+ // The "Remove" link is not present.
+ Y.Assert.isFalse(Y.Lang.isValue(overlay.one('a.remove')));
+ // The link is spinning.
+ Y.Assert.isTrue(this.link().hasClass('spinner'));
+ Y.Assert.isFalse(this.link().hasClass('edit'));
+ // When we click on the "x," the overlay goes away, and the spinner
+ // is gone.
+ overlay.one('a.close-button').simulate('click');
+ Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ Y.Assert.isFalse(this.link().hasClass('spinner'));
+ Y.Assert.isTrue(this.link().hasClass('edit'));
+ },
+
+ test_overlay_with_discussion_subscription: function() {
+ // Starting with a DISCUSSION subscription, we get a "change" overlay.
+ this.init(DISCUSSION, false);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ // We have the "Change" title, not the "Add" title.
+ Y.Assert.areEqual('Change your mail subscription for this bug',
+ overlay.one('h2').get('text'));
+ // We show the Discussion status.
+ var status = overlay.one('.subscription-status');
+ Y.Assert.isFalse(overlay.one('span.Discussion').hasClass('hidden'));
+ Y.Assert.isTrue(overlay.one('span.Details').hasClass('hidden'));
+ Y.Assert.isTrue(overlay.one('span.Lifecycle').hasClass('hidden'));
+ // The action links are visible except for Discussion.
+ var action_links = overlay.one('.subscription-actions');
+ Y.Assert.isTrue(action_links.one('.Discussion a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
+ // We have a remove link.
+ Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
+ Y.Assert.areEqual(
+ 'Remove your direct subscription',
+ overlay.one('div.subscription-actions+div').get('text'));
+ },
+
+ test_overlay_with_details_subscription: function() {
+ // Test overlay with existing DETAILS subscription.
+ this.init(DETAILS, true);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ // We show the Lifecycle status.
+ var status = overlay.one('.subscription-status');
+ Y.Assert.isTrue(overlay.one('span.Discussion').hasClass('hidden'));
+ Y.Assert.isFalse(overlay.one('span.Details').hasClass('hidden'));
+ Y.Assert.isTrue(overlay.one('span.Lifecycle').hasClass('hidden'));
+ // The action links are visible except for Lifecycle.
+ var action_links = overlay.one('.subscription-actions');
+ Y.Assert.isFalse(action_links.one(
+ '.Discussion a').hasClass('hidden'));
+ Y.Assert.isTrue(action_links.one('.Details a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Lifecycle a').hasClass('hidden'));
+ // We have a remove link.
+ Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
+ Y.Assert.areEqual(
+ 'Remove your direct subscription',
+ overlay.one('div.subscription-actions+div').get('text'));
+ },
+
+ test_overlay_with_lifecycle_subscription: function() {
+ // Test overlay with existing LIFECYCLE subscription.
+ this.init(LIFECYCLE, true);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ // We show the Lifecycle status.
+ var status = overlay.one('.subscription-status');
+ Y.Assert.isTrue(overlay.one('span.Discussion').hasClass('hidden'));
+ Y.Assert.isTrue(overlay.one('span.Details').hasClass('hidden'));
+ Y.Assert.isFalse(overlay.one('span.Lifecycle').hasClass('hidden'));
+ // The action links are visible except for Lifecycle.
+ var action_links = overlay.one('.subscription-actions');
+ Y.Assert.isFalse(action_links.one(
+ '.Discussion a').hasClass('hidden'));
+ Y.Assert.isFalse(action_links.one('.Details a').hasClass('hidden'));
+ Y.Assert.isTrue(action_links.one('.Lifecycle a').hasClass('hidden'));
+ // We have a remove link.
+ Y.Assert.isTrue(Y.Lang.isValue(overlay.one('a.remove')));
+ Y.Assert.areEqual(
+ 'Remove your direct subscription',
+ overlay.one('div.subscription-actions+div').get('text'));
+ },
+
+ test_subscribe_discussion: function() {
+ this.init(null, false, DISCUSSION);
+ // The mute node is hidden initially.
+ Y.Assert.isTrue(this.mute_node.hasClass('hidden'));
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('.subscription-actions .Discussion a').simulate('click');
+ // The overlay has been destroyed, and the spinner is gone.
+ Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ Y.Assert.isFalse(this.link().hasClass('spinner'));
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('subscribe', args[1]);
+ Y.ObjectAssert.areEqual({person: '~tweedledee', level: DISCUSSION},
+ args[2].parameters);
+ // There is now a subscription object in the cache.
+ Y.Assert.isTrue(Y.Lang.isValue(window.LP.cache.subscription));
+ // The mute node is not hidden.
+ Y.Assert.isFalse(this.mute_node.hasClass('hidden'));
+ // The link has updated its text.
+ Y.Assert.areEqual(
+ "You are subscribed to all notifications for this bug.",
+ this.status_node.get('text'));
+ },
+
+ test_subscribe_details: function() {
+ this.init(null, false, DETAILS);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('.subscription-actions .Details a').simulate('click');
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('subscribe', args[1]);
+ Y.ObjectAssert.areEqual({person: '~tweedledee', level: DETAILS},
+ args[2].parameters);
+ // The link has updated its text.
+ Y.Assert.areEqual(
+ "You are subscribed to all notifications except comments for "+
+ "this bug.",
+ this.status_node.get('text'));
+ },
+
+ test_subscribe_lifecycle: function() {
+ this.init(null, false, LIFECYCLE);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('.subscription-actions .Lifecycle a').simulate('click');
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('subscribe', args[1]);
+ Y.ObjectAssert.areEqual({person: '~tweedledee', level: LIFECYCLE},
+ args[2].parameters);
+ // The link has updated its text.
+ Y.Assert.areEqual(
+ "You are subscribed to notifications when this bug is closed "+
+ "or reopened.",
+ this.status_node.get('text'));
+ },
+
+ test_unsubscribe: function() {
+ this.init(LIFECYCLE, false);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('a.remove').simulate('click');
+ // There is no warning.
+ var warning = overlay.one('.private-bug-warning');
+ Y.Assert.isNull(warning);
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('unsubscribe', args[1]);
+ Y.ObjectAssert.areEqual({}, args[2].parameters);
+ // The overlay has been destroyed, and the spinner is gone.
+ Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ Y.Assert.isFalse(this.link().hasClass('spinner'));
+ // The mute node is hidden.
+ Y.Assert.isTrue(this.mute_node.hasClass('hidden'));
+ // The link has updated its text.
+ Y.Assert.areEqual(
+ "You are not directly subscribed to this bug's notifications.",
+ this.status_node.get('text'));
+ },
+
+ test_unsubscribe_from_private_bug: function() {
+ this.init(LIFECYCLE, false);
+ window.LP.cache.bug_is_private = true;
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('a.remove').simulate('click');
+ // The overlay now has a warning.
+ var warning = overlay.one('.private-bug-warning');
+ Y.Assert.isTrue(Y.Lang.isValue(warning));
+ // Find and click the OK button.
+ var ok_btn = warning.one('.ok-btn');
+ ok_btn.simulate('click');
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('unsubscribe', args[1]);
+ Y.ObjectAssert.areEqual({}, args[2].parameters);
+ // The overlay has been destroyed, and the spinner is gone.
+ Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ Y.Assert.isFalse(this.link().hasClass('spinner'));
+ },
+
+ test_unsubscribe_from_private_bug_cancel: function() {
+ this.init(LIFECYCLE, false);
+ window.LP.cache.bug_is_private = true;
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('a.remove').simulate('click');
+ // The overlay now has a warning.
+ var warning = overlay.one('.private-bug-warning');
+ Y.Assert.isTrue(Y.Lang.isValue(warning));
+ // Find and click the Cancel button.
+ var cancel_btn = warning.one('.cancel-btn');
+ cancel_btn.simulate('click');
+ // The warning is gone.
+ warning = overlay.one('.private-bug-warning');
+ Y.Assert.isNull(warning);
+ },
+
+ test_unsubscribe_with_other: function() {
+ this.init(LIFECYCLE, true);
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ overlay.one('a.remove').simulate('click');
+ // We got an appropriate call to the webservice.
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(this.bug_link, args[0]);
+ Y.Assert.areEqual('unsubscribe', args[1]);
+ Y.ObjectAssert.areEqual({}, args[2].parameters);
+ // The overlay has been destroyed, and the spinner is gone.
+ Y.Assert.isFalse(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ Y.Assert.isFalse(this.link().hasClass('spinner'));
+ // The mute node is not hidden.
+ Y.Assert.isFalse(this.mute_node.hasClass('hidden'));
+ // The link has updated its text.
+ Y.Assert.areEqual(
+ "You have subscriptions that may cause you to receive "+
+ "notifications, but you are not directly subscribed to this "+
+ "bug's notifications.",
+ this.status_node.get('text'));
+ },
+
+ test_io_spinner_and_error: function() {
+ this.init(LIFECYCLE, true);
+ module._lp_client.named_post.fail = true;
+ module._lp_client.named_post.args = [
+ true,
+ {status: 400, responseText: 'Rutebegas!'}];
+ module._lp_client.named_post.halt = true;
+ this.link().simulate('click');
+ var overlay = Y.one('.pretty-overlay-window');
+ var unsub = overlay.one('a.remove');
+ unsub.simulate('click');
+ // Right now, this is as if we are waiting for the server to
+ // reply. The link is spinning.
+ Y.Assert.isTrue(unsub.hasClass('spinner'));
+ Y.Assert.isFalse(unsub.hasClass('remove'));
+ // Now the server replies with an error.
+ module._lp_client.named_post.resume();
+ // We have no spinner.
+ Y.Assert.isTrue(unsub.hasClass('remove'));
+ Y.Assert.isFalse(unsub.hasClass('spinner'));
+ // The page has rendered the error overlay.
+ var error_box = Y.one('.yui3-lazr-formoverlay-errors');
+ Y.Assert.isTrue(Y.Lang.isValue(error_box));
+ // The overlay is still hanging around too, as expected, but for
+ // better or worse (popup-on-popup is not the best UI).
+ Y.Assert.isTrue(Y.Lang.isValue(Y.one('.pretty-overlay-window')));
+ }
+ }));
+
+}, '0.1', {
+ 'requires': ['test', 'lp.testing.helpers', 'console',
+ 'lp.bugs.bugtask_index.portlets.subscription', 'node-event-simulate']
});
=== modified file 'lib/lp/bugs/javascript/tests/test_subscription.html'
--- lib/lp/bugs/javascript/tests/test_subscription.html 2012-03-14 04:41:36 +0000
+++ lib/lp/bugs/javascript/tests/test_subscription.html 2012-06-28 15:32:36 +0000
@@ -21,6 +21,8 @@
<script type="text/javascript"
src="../../../../../build/js/lp/app/testing/testrunner.js"></script>
+ <script type="text/javascript"
+ src="../../../../../build/js/lp/app/testing/helpers.js"></script>
<link rel="stylesheet" href="../../../app/javascript/testing/test.css" />
@@ -61,8 +63,7 @@
</head>
<body class="yui3-skin-sam">
<ul id="suites">
- <!-- <li>lp.large_indicator.test</li> -->
- <li>lp.subscription.test</li>
+ <li>lp.bugs.subscription.test</li>
</ul>
<!-- Example markup required by test suite -->
<div id="test-root"></div>
=== modified file 'lib/lp/bugs/javascript/tests/test_subscription.js'
--- lib/lp/bugs/javascript/tests/test_subscription.js 2011-09-21 06:38:51 +0000
+++ lib/lp/bugs/javascript/tests/test_subscription.js 2012-06-28 15:32:36 +0000
@@ -1,2836 +1,2795 @@
-YUI({
- base: '../../../../canonical/launchpad/icing/yui/',
- filter: 'raw', combine: false, fetchCSS: false
- }).use('test', 'console', 'lp.bugs.subscription', 'node-event-simulate',
- 'lazr.effects',
- function(Y) {
-
-var suite = new Y.Test.Suite("lp.bugs.subscription Tests");
-var module = Y.lp.bugs.subscription;
-
-/**
- * XXX gary 2011-05-26 bug 793579
- * LPClient is copied three times (see also test_structural_subscription.js
- * and test_bug_subscription_portlet.js). It should be pushed to a
- * shared test module.
- */
-function LPClient(){
- if (!(this instanceof LPClient)) {
- throw new Error("Constructor called as a function");
- }
- this.received = [];
- // We create new functions every time because we allow them to be
- // configured.
- this.named_post = function(url, func, config) {
- this._call('named_post', config, arguments);
- };
- this.patch = function(bug_filter, data, config) {
- this._call('patch', config, arguments);
- };
-}
-LPClient.prototype._call = function(name, config, args) {
- this.received.push(
- [name, Array.prototype.slice.call(args)]);
- if (!Y.Lang.isValue(args.callee.args)) {
- throw new Error("Set call_args on "+name);
- }
- var do_action = function () {
- if (Y.Lang.isValue(args.callee.fail) && args.callee.fail) {
- config.on.failure.apply(undefined, args.callee.args);
- } else {
- config.on.success.apply(undefined, args.callee.args);
- }
- };
- if (Y.Lang.isValue(args.callee.halt) && args.callee.halt) {
- args.callee.resume = do_action;
- } else {
- do_action();
- }
-};
-
-/**
- * Test selection of the string by the number.
- * We expect to receive a plural string for all numbers
- * not equal to 1, and a singular string otherwise.
- */
-suite.add(new Y.Test.Case({
- name: 'Choose object by number',
-
- test_singular: function() {
- Y.Assert.areEqual(
- 'SINGULAR',
- module._choose_by_number(1, 'SINGULAR', 'PLURAL'));
- },
-
- test_plural: function() {
- Y.Assert.areEqual(
- 'PLURAL',
- module._choose_by_number(5, 'SINGULAR', 'PLURAL'));
- },
-
- test_zero: function() {
- Y.Assert.areEqual(
- 'PLURAL',
- module._choose_by_number(0, 'SINGULAR', 'PLURAL'));
- }
-}));
-
-/**
- * Replacing references to cache objects with actual objects.
- */
-suite.add(new Y.Test.Case({
- name: 'Replacing references with real objects',
-
- test_nothing: function() {
- // When there are no references, nothing gets replaced.
- var object = {
- something: 'nothing'
- };
- var cache = {};
- module._replace_textual_references(object, cache);
- Y.Assert.areEqual('nothing', object.something);
- },
-
- test_simple: function() {
- // With a simple reference, it gets substituted.
- var object = {
- something: 'subscription-cache-reference-1'
- };
- var cache = {
- 'subscription-cache-reference-1': 'OK'
- };
- module._replace_textual_references(object, cache);
- Y.Assert.areEqual('OK', object.something);
- },
-
- test_multiple: function() {
- // With multiple references, they all get substituted.0
- var object = {
- something: 'subscription-cache-reference-1',
- other: 'subscription-cache-reference-2'
- };
- var cache = {
- 'subscription-cache-reference-1': 'OK 1',
- 'subscription-cache-reference-2': 'OK 2'
- };
- module._replace_textual_references(object, cache);
- Y.Assert.areEqual('OK 1', object.something);
- Y.Assert.areEqual('OK 2', object.other);
- },
-
- test_recursive: function() {
- // Even references in nested objects get replaced.
- var object = {
- nested: {
+/* Copyright (c) 2012 Canonical Ltd. All rights reserved. */
+YUI.add('lp.bugs.subscription.test', function (Y) {
+ var module = Y.lp.bugs.subscription;
+
+ var reduction_ids = [module._action_ids.mute,
+ module._action_ids.subscribe_only_metadata,
+ module._action_ids.subscribe_only_closed,
+ module._action_ids.unsubscribe];
+
+ var increasing_ids = [module._action_ids.unmute,
+ module._action_ids.subscribe_all,
+ module._action_ids.subscribe_metadata,
+ module._action_ids.subscribe_closed];
+
+ /**
+ * Helper to construct a single 'category' of subscriptions,
+ * grouped by type (personally, as team member and as team admin).
+ */
+ function _constructCategory(personal, as_member, as_admin) {
+ if (personal === undefined) {
+ personal = [];
+ }
+ if (as_member === undefined) {
+ as_member = [];
+ }
+ if (as_admin === undefined) {
+ as_admin = [];
+ }
+ return {
+ count: personal.length + as_admin.length + as_member.length,
+ personal: personal,
+ as_team_member: as_member,
+ as_team_admin: as_admin
+ };
+ }
+
+ var tests = Y.namespace('lp.bugs.subscription.test');
+ tests.suite = new Y.Test.Suite('bugs.subscription Tests');
+
+ tests.suite.add(new Y.Test.Case({
+ name: 'bugs.subscription_tests',
+
+ setUp: function () {},
+ tearDown: function () {},
+
+ test_library_exists: function () {
+ Y.Assert.isObject(Y.lp.bugs.subscription,
+ "Could not locate the lp.bugs.subscription module");
+ }
+
+ }));
+
+ /**
+ * Test selection of the string by the number.
+ * We expect to receive a plural string for all numbers
+ * not equal to 1, and a singular string otherwise.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Choose object by number',
+
+ test_singular: function() {
+ Y.Assert.areEqual(
+ 'SINGULAR',
+ module._choose_by_number(1, 'SINGULAR', 'PLURAL'));
+ },
+
+ test_plural: function() {
+ Y.Assert.areEqual(
+ 'PLURAL',
+ module._choose_by_number(5, 'SINGULAR', 'PLURAL'));
+ },
+
+ test_zero: function() {
+ Y.Assert.areEqual(
+ 'PLURAL',
+ module._choose_by_number(0, 'SINGULAR', 'PLURAL'));
+ }
+ }));
+
+ /**
+ * Replacing references to cache objects with actual objects.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Replacing references with real objects',
+
+ test_nothing: function() {
+ // When there are no references, nothing gets replaced.
+ var object = {
+ something: 'nothing'
+ };
+ var cache = {};
+ module._replace_textual_references(object, cache);
+ Y.Assert.areEqual('nothing', object.something);
+ },
+
+ test_simple: function() {
+ // With a simple reference, it gets substituted.
+ var object = {
something: 'subscription-cache-reference-1'
- }
- };
- var cache = {
- 'subscription-cache-reference-1': 'OK'
- };
- module._replace_textual_references(object, cache);
- Y.Assert.areEqual('OK', object.nested.something);
- }
-}));
-
-
-/**
- * Gather subscription records for all assignments.
- */
-suite.add(new Y.Test.Case({
- name: 'Gather assignment subscription information',
-
- test_nothing: function() {
- // When there are no subscriptions as assignee, returns empty list.
- var mock_category = {
- count: 0,
- personal: [],
- as_team_member: [],
- as_team_admin: []
- };
- Y.ArrayAssert.itemsAreEqual(
- [],
- module._gather_subscriptions_as_assignee(mock_category));
- },
-
- test_personal: function() {
- // When a person is directly the bug assignee, we get that
- // subscription details returned.
- var mock_category = {
- count: 1,
- personal: [{}],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.YOU_ASSIGNED, subs[0].reason);
- Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
- },
-
- test_team_member: function() {
- // When a person is the bug assignee through team membership,
- // we get that subscription details returned.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{ principal: 'my team'}],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAM_ASSIGNED, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple: function() {
- // If a person is a member of multiple teams are assigned to work
- // on a single bug (eg. on different bug tasks) they get only one
- // subscription returned.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: 'team1'},
- { principal: 'team2'}],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAMS_ASSIGNED, subs[0].reason);
- // And there is a 'teams' variable containing all the team objects.
- Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
- subs[0].vars.teams);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple_duplicate: function() {
- // As with the previous test, but we need to show that each team is
- // only represented once even if they are responsible for multiple
- // bug tasks.
- // We test with full-fledged objects to make sure they work with the
- // mechanism used to find dupes.
- var team1 = {display_name: 'team 1',
- web_link: 'http://launchpad.net/~team1'},
- team2 = {display_name: 'team 2',
- web_link: 'http://launchpad.net/~team2'},
- mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: team1 },
- { principal: team2 },
- { principal: team2 }],
- as_team_admin: []
- },
- subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAMS_ASSIGNED, subs[0].reason);
- // And there is a 'teams' variable containing all the team objects.
- var teams_found = [];
- var index;
- for (index = 0; index < subs[0].vars.teams.length; index++) {
- teams_found.push(subs[0].vars.teams[index].title);
- }
- Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
- },
-
- test_team_admin: function() {
- // When a person is the bug assignee through team membership,
- // and a team admin at the same time, that subscription is returned.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'my team' }]
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAM_ASSIGNED, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
- },
-
- test_team_admin_multiple: function() {
- // If a person is a member of multiple teams are assigned to work
- // on a single bug (eg. on different bug tasks) they get only one
- // subscription returned.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'team1'},
- { principal: 'team2'}]
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAMS_ASSIGNED, subs[0].reason);
- // And there is a 'teams' variable containing all the team objects.
- Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
- subs[0].vars.teams);
- Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
- },
-
- test_team_admin_multiple_duplicate: function() {
- // As with the previous test, but we need to show that each team is
- // only represented once even if they are responsible for multiple
- // bug tasks.
- // We test with full-fledged objects to make sure they work with the
- // mechanism used to find dupes.
- var team1 = {display_name: 'team 1',
- web_link: 'http://launchpad.net/~team1'},
- team2 = {display_name: 'team 2',
- web_link: 'http://launchpad.net/~team2'},
- mock_category = {
- count: 2,
- personal: [],
- as_team_admin: [{ principal: team1 },
- { principal: team2 },
- { principal: team2 }],
- as_team_member: []
- },
- subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(1, subs.length);
- // And there is a 'teams' variable containing all the team objects.
- var teams_found = [];
- for (index = 0; index < subs[0].vars.teams.length; index++) {
- teams_found.push(subs[0].vars.teams[index].title);
- }
- Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
- },
-
- test_combined: function() {
- // Test that multiple assignments, even if they are in different
- // categories, work properly.
- var mock_category = {
- count: 3,
- personal: [{}],
- as_team_member: [{ principal: 'users' }],
- as_team_admin: [{ principal: 'admins' }]
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(3, subs.length);
- },
-
- test_object_links: function() {
- // Test that team assignments actually provide decent link data.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [
- { principal: { display_name: 'My team',
- web_link: 'http://link' } }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual('My team', subs[0].vars.team.title);
- Y.Assert.areEqual('http://link', subs[0].vars.team.url);
- }
-}));
-
-/**
- * Gather subscription records for bug supervisor.
- */
-suite.add(new Y.Test.Case({
- name: 'Gather bug supervisor subscription information',
-
- test_nothing: function() {
- // When there are no subscriptions as bug supervisor,
- // returns empty list.
- var mock_category = {
- count: 0,
- personal: [],
- as_team_member: [],
- as_team_admin: []
- };
- Y.ArrayAssert.itemsAreEqual(
- [],
- module._gather_subscriptions_as_supervisor(mock_category));
- },
-
- test_personal: function() {
- // Person is the implicit bug supervisor by being the owner
- // of the project with no bug supervisor.
- var mock_category = {
- count: 1,
- personal: [{pillar: 'project'}],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.YOU_OWNER, subs[0].reason);
- Y.Assert.areEqual('project', subs[0].vars.pillar);
- Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
- },
-
- test_personal_multiple: function() {
- // Person is the implicit bug supervisor by being the owner
- // of several projects (eg. multiple bug tasks) with no bug
- // supervisor.
- var mock_category = {
- count: 2,
- personal: [ {pillar: {title: 'project'} },
- {pillar: {title:'distro'} }],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_team_member: function() {
- // Person is a member of the team which is the implicit
- // bug supervisor.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{ principal: 'my team',
- pillar: 'project' }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAM_OWNER, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual('project', subs[0].vars.pillar);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple: function() {
- // Person is a member of several teams which are implicit bug
- // supervisors on multiple bugtasks, we get subscription
- // records separately.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: 'team1',
- pillar: {display_name: 'project'} },
- { principal: 'team2',
- pillar: {display_name: 'distro'} }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_team_admin: function() {
- // Person is an admin of the team which is the implicit
- // bug supervisor.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'my team',
- pillar: 'project' }]
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAM_OWNER, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual('project', subs[0].vars.pillar);
- Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
- },
-
- test_team_admin_multiple: function() {
- // Person is an admin of several teams which are implicit bug
- // supervisors on multiple bugtasks, we get subscription
- // records separately.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'team1',
- pillar: {display_name: 'project'} },
- { principal: 'team2',
- pillar: {display_name: 'distro'} }]
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_repeated_pillars: function() {
- // Different bug tasks might still be on the same pillar,
- // and we should only get one action.
- var mock_pillar = { display_name: 'project',
- web_link: 'http://project/' };
- var mock_category = {
- count: 1,
- personal: [{pillar: mock_pillar},
- {pillar: mock_pillar}],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.YOU_OWNER, subs[0].reason);
- Y.Assert.areEqual(mock_pillar, subs[0].vars.pillar.self);
- Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
- },
-
- test_combined: function() {
- // Test that multiple implicit bug supervisor roles
- // are all returned.
- var mock_category = {
- count: 3,
- personal: [{pillar: 'project1'}],
- as_team_member: [{ principal: 'users', pillar: 'project2' }],
- as_team_admin: [{ principal: 'admins', pillar: 'distro' }]
- };
- var subs = module._gather_subscriptions_as_assignee(mock_category);
- Y.Assert.areEqual(3, subs.length);
- },
-
- test_object_links: function() {
- // Test that team-as-supervisor actually provide decent link data,
- // along with pillars as well.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{
- principal: { display_name: 'My team',
- web_link: 'http://link' },
- pillar: { display_name: 'My project',
- web_link: 'http://project/' }
- }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_as_supervisor(mock_category);
- Y.Assert.areEqual('My team', subs[0].vars.team.title);
- Y.Assert.areEqual('http://link', subs[0].vars.team.url);
-
- Y.Assert.areEqual('My project', subs[0].vars.pillar.title);
- Y.Assert.areEqual('http://project/', subs[0].vars.pillar.url);
- }
-}));
-
-/**
- * Gather subscription records for dupe bug subscriptions.
- */
-suite.add(new Y.Test.Case({
- name: 'Gather subscription information for duplicates',
-
- test_nothing: function() {
- // When there are no duplicate subscriptions, returns empty list.
- var mock_category = {
- count: 0,
- personal: [],
- as_team_member: [],
- as_team_admin: []
- };
- Y.ArrayAssert.itemsAreEqual(
- [],
- module._gather_subscriptions_from_duplicates(mock_category));
- },
-
- test_personal: function() {
- // A person is subscribed to a duplicate bug.
- var mock_category = {
- count: 1,
- personal: [{bug: 'dupe bug'}],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.YOU_SUBSCRIBED_TO_DUPLICATE, subs[0].reason);
- Y.Assert.areEqual('dupe bug', subs[0].vars.duplicate_bug);
- Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
- subs[0].action);
- },
-
- test_personal_multiple: function() {
- // A person is subscribed to multiple duplicate bugs.
- // They are returned together as one subscription record.
- var mock_category = {
- count: 2,
- personal: [{bug: 'dupe1'}, {bug: 'dupe2'}],
- as_team_member: [],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.YOU_SUBSCRIBED_TO_DUPLICATES, subs[0].reason);
- Y.ArrayAssert.itemsAreEqual(
- ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
- Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
- subs[0].action);
- },
-
- test_team_member: function() {
- // A person is a member of the team subscribed to a duplicate bug.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{ principal: 'my team',
- bug: 'dupe' }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.TEAM_SUBSCRIBED_TO_DUPLICATE, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- // And a 'duplicate_bug' variable pointing to the dupe.
- Y.Assert.areEqual('dupe', subs[0].vars.duplicate_bug);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple_bugs: function() {
- // A person is a member of the team subscribed to multiple
- // duplicate bugs.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{
- principal: 'my team',
- bug: 'dupe1'
- }, {
- principal: 'my team',
- bug: 'dupe2'
- }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.TEAM_SUBSCRIBED_TO_DUPLICATES, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- // And a 'duplicate_bugs' variable with the list of dupes.
- Y.ArrayAssert.itemsAreEqual(
- ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple: function() {
- // A person is a member of several teams subscribed to
- // duplicate bugs.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: 'team1',
- bug: 'dupe1' },
- { principal: 'team2',
- bug: 'dupe1' }],
- as_team_admin: []
- };
-
- // Result is two separate subscription records.
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_team_admin: function() {
- // A person is an admin of the team subscribed to a duplicate bug.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'my team',
- bug: 'dupe' }]
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAM_SUBSCRIBED_TO_DUPLICATE,
- subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- // And a 'duplicate_bug' variable pointing to the dupe.
- Y.Assert.areEqual('dupe', subs[0].vars.duplicate_bug);
- Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
- subs[0].action);
- },
-
- test_team_admin_multiple_bugs: function() {
- // A person is an admin of the team subscribed to multiple
- // duplicate bugs.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [],
- as_team_admin: [{
- principal: 'my team',
- bug: 'dupe1'
- }, {
- principal: 'my team',
- bug: 'dupe2'
- }]
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAM_SUBSCRIBED_TO_DUPLICATES,
- subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- // And a 'duplicate_bugs' variable with the list of dupes.
- Y.ArrayAssert.itemsAreEqual(
- ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
- Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
- subs[0].action);
- },
-
- test_team_admin_multiple: function() {
- // A person is an admin of several teams subscribed to
- // duplicate bugs.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'team1',
- bug: 'dupe1' },
- { principal: 'team2',
- bug: 'dupe1' }]
- };
-
- // Result is two separate subscription records.
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_object_links: function() {
- // Test that team dupe subscriptions actually provide decent
- // link data, including duplicate bugs link data.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{
- principal: { display_name: 'My team',
- web_link: 'http://link' },
- bug: { id: 1,
- web_link: 'http://launchpad/bug/1' }
- }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_from_duplicates(
- mock_category);
- Y.Assert.areEqual('My team', subs[0].vars.team.title);
- Y.Assert.areEqual('http://link', subs[0].vars.team.url);
-
- Y.Assert.areEqual('#1', subs[0].vars.duplicate_bug.title);
- Y.Assert.areEqual(
- 'http://launchpad/bug/1', subs[0].vars.duplicate_bug.url);
- }
-}));
-
-/**
- * Gather subscription records for direct team subscriptions.
- */
-suite.add(new Y.Test.Case({
- name: 'Gather team subscription information',
-
- test_nothing: function() {
- // When there are no subscriptions through team, returns empty list.
- var mock_category = {
- count: 0,
- personal: [],
- as_team_member: [],
- as_team_admin: []
- };
- Y.ArrayAssert.itemsAreEqual(
- [],
- module._gather_subscriptions_through_team(mock_category));
- },
-
- test_personal: function() {
- // A personal subscription is not considered a team subscription.
- var mock_category = {
- count: 1,
- personal: [{}],
- as_team_member: [],
- as_team_admin: []
- };
- Y.ArrayAssert.itemsAreEqual(
- [],
- module._gather_subscriptions_through_team(mock_category));
- },
-
- test_team_member: function() {
- // Person is a member of the team subscribed to the bug.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [{ principal: 'my team'}],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAM_SUBSCRIBED, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple: function() {
- // Person is a member of several teams subscribed to the bug.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: 'team1'},
- { principal: 'team2'}],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(module._reasons.TEAMS_SUBSCRIBED, subs[0].reason);
- // And there is a 'teams' variable containing all the team objects.
- Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
- subs[0].vars.teams);
- Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
- },
-
- test_team_member_multiple_duplicate: function() {
- // As with the previous test, but we need to show that each team is
- // only represented once even if they are responsible for multiple
- // bug tasks.
- // We test with full-fledged objects to make sure they work with the
- // mechanism used to find dupes.
- var team1 = {display_name: 'team 1',
- web_link: 'http://launchpad.net/~team1'},
- team2 = {display_name: 'team 2',
- web_link: 'http://launchpad.net/~team2'},
- mock_category = {
- count: 2,
- personal: [],
- as_team_member: [{ principal: team1 },
- { principal: team2 },
- { principal: team2 }],
- as_team_admin: []
- },
- subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- // And there is a 'teams' variable containing all the team objects.
- var teams_found = [];
- for (index = 0; index < subs[0].vars.teams.length; index++) {
- teams_found.push(subs[0].vars.teams[index].title);
- }
- Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
- },
-
- test_team_admin: function() {
- // Person is an admin of the team subscribed to the bug.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'my team' }]
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAM_SUBSCRIBED, subs[0].reason);
- // And there is a 'team' variable containing the team object.
- Y.Assert.areEqual('my team', subs[0].vars.team);
- Y.Assert.areEqual(module._actions.CHANGE_TEAM_SUBSCRIPTIONS,
- subs[0].action);
- },
-
- test_team_admin_multiple: function() {
- // Person is an admin of the several teams subscribed to the bug.
- var mock_category = {
- count: 2,
- personal: [],
- as_team_member: [],
- as_team_admin: [{ principal: 'team1'},
- { principal: 'team2'}]
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- Y.Assert.areEqual(
- module._reasons.ADMIN_TEAMS_SUBSCRIBED, subs[0].reason);
- // And there is a 'teams' variable containing all the team objects.
- Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
- subs[0].vars.teams);
- Y.Assert.areEqual(module._actions.CHANGE_TEAM_SUBSCRIPTIONS,
- subs[0].action);
- },
-
- test_team_admin_multiple_duplicate: function() {
- // As with the previous test, but we need to show that each team is
- // only represented once even if they are responsible for multiple
- // bug tasks.
- // We test with full-fledged objects to make sure they work with the
- // mechanism used to find dupes.
- var team1 = {display_name: 'team 1',
- web_link: 'http://launchpad.net/~team1'},
- team2 = {display_name: 'team 2',
- web_link: 'http://launchpad.net/~team2'},
- mock_category = {
- count: 2,
- personal: [],
- as_team_admin: [{ principal: team1 },
- { principal: team2 },
- { principal: team2 }],
- as_team_member: []
- },
- subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(1, subs.length);
- // And there is a 'teams' variable containing all the team objects.
- var teams_found = [];
- for (index = 0; index < subs[0].vars.teams.length; index++) {
- teams_found.push(subs[0].vars.teams[index].title);
- }
- Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
- },
-
- test_combined: function() {
- // Test that multiple subscriptions, even if they are in different
- // categories, work properly, and that personal subscriptions are
- // still ignored.
- var mock_category = {
- count: 3,
- personal: [{}],
- as_team_member: [{ principal: 'users' }],
- as_team_admin: [{ principal: 'admins' }]
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual(2, subs.length);
- },
-
- test_object_links: function() {
- // Test that team subscriptions actually provide decent link data.
- var mock_category = {
- count: 1,
- personal: [],
- as_team_member: [
- { principal: { display_name: 'My team',
- web_link: 'http://link' } }],
- as_team_admin: []
- };
- var subs = module._gather_subscriptions_through_team(mock_category);
- Y.Assert.areEqual('My team', subs[0].vars.team.title);
- Y.Assert.areEqual('http://link', subs[0].vars.team.url);
- }
-}));
-
-
-/**
- * Helper to construct a single 'category' of subscriptions,
- * grouped by type (personally, as team member and as team admin).
- */
-function _constructCategory(personal, as_member, as_admin) {
- if (personal === undefined) {
- personal = [];
- }
- if (as_member === undefined) {
- as_member = [];
- }
- if (as_admin === undefined) {
- as_admin = [];
- }
- return {
- count: personal.length + as_admin.length + as_member.length,
- personal: personal,
- as_team_member: as_member,
- as_team_admin: as_admin
- };
-}
-
-/**
- * Get the reason for a direct subscription.
- * Tests for method get_direct_subscription_information().
- */
-suite.add(new Y.Test.Case({
- name: 'Get reason and actions for a direct subscription',
-
- _should: {
- error: {
- test_multiple_direct_subscriptions:
- new Error('Programmer error: a person should not have more than '+
- 'one direct personal subscription.'),
- test_direct_subscription_at_unknown_level:
- new Error('Programmer error: unknown bug notification level: '+
- 'The Larch')
- }
- },
-
- setUp: function() {
- window.LP = {cache: {subscription_info: []}};
- },
-
- tearDown: function() {
- delete window.LP;
- },
-
- test_multiple_direct_subscriptions: function() {
- // It should not be possible to have multiple direct,
- // personal subscriptions.
- // This errors out (see _should.error above).
- var info = {
- direct: _constructCategory(['1', '2']),
- count: 2
- };
- module._get_direct_subscription_information(info);
- },
-
- test_no_subscriptions_at_all: function() {
- // There are no subscriptions at all.
- var info = {
- direct: _constructCategory(),
- from_duplicates: _constructCategory()
- };
- info.count = info.direct.count + info.from_duplicates.count;
-
- direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areEqual(
- module._reasons.NOT_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['select-direct-subscription-discussion',
- 'select-direct-subscription-metadata',
- 'select-direct-subscription-lifecycle'],
- direct_info.increases);
- },
-
- test_only_structural_subscriptions: function() {
- // There are only structural subscriptions.
- var info = {
- direct: _constructCategory(),
- from_duplicates: _constructCategory()
- };
- info.count = info.direct.count + info.from_duplicates.count;
- window.LP.cache.subscription_info.push(true);
-
- direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.NOT_PERSONALLY_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['select-direct-subscription-discussion'],
- direct_info.increases);
- },
-
- test_no_direct_subscriptions: function() {
- // There is no direct subscription, but there are
- // other subscriptions.
- var info = {
- direct: _constructCategory(),
- from_duplicates: _constructCategory(['dupe'])
- };
- info.count = info.direct.count + info.from_duplicates.count;
- direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.NOT_PERSONALLY_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['select-direct-subscription-discussion'],
- direct_info.increases);
- },
-
- test_muted_subscription: function() {
- // The direct subscription is muted.
- var info = {
- direct: _constructCategory(['direct']),
- muted: true
- };
- info.count = info.direct.count;
- direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.MUTED_SUBSCRIPTION,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['unmute-direct-subscription'],
- direct_info.increases);
- },
-
- test_direct_subscription_at_discussion_level: function() {
- // The larch^D^D^D^D^D^D simple direct subscription.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Discussion'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
-
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.increases);
- },
-
- test_direct_subscription_at_metadata_level: function() {
- // The simple direct subscription at metadata level.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
-
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['select-direct-subscription-discussion'],
- direct_info.increases);
- },
-
- test_direct_subscription_at_lifecycle_level: function() {
- // The simple direct subscription at lifecycle level.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Lifecycle'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
-
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- ['select-direct-subscription-discussion',
- 'select-direct-subscription-metadata'],
- direct_info.increases);
- },
-
- test_direct_subscription_at_unknown_level: function() {
- // The simple direct subscription at unknown level.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'The Larch'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
- // This should raise an error.
- module._get_direct_subscription_information(info);
- },
-
- test_direct_subscription_as_reporter: function() {
- // The direct subscription created for bug reporter.
- var sub = {
- bug: {},
- principal_is_reporter: true,
- subscription: {bug_notification_level: 'Discussion'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
-
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_REPORTED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.increases);
- },
-
- test_direct_subscription_for_supervisor: function() {
- // The direct subscription created on private bugs for
- // the bug supervisor.
- var sub = {
- bug: {
- 'private': true
- },
- subscription: {bug_notification_level: 'Discussion'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED_BUG_SUPERVISOR,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.increases);
- },
-
- test_direct_subscription_for_security_contact: function() {
- // The simple direct subscription.
- var sub = {
- bug: {
- security_related: true
- },
- subscription: {bug_notification_level: 'Discussion'}
- };
- var info = {
- direct: _constructCategory([sub]),
- count: 1
- };
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED_SECURITY_CONTACT,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.increases);
- },
-
- test_direct_subscription_and_other_subscriptions: function() {
- // Other subscriptions are present along with the simple direct
- // subscription.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Discussion'}
- };
- var info = {
- direct: _constructCategory([sub]),
- from_duplicates: _constructCategory(['dupe']),
- count: 2
- };
-
- var direct_info = module._get_direct_subscription_information(info);
- Y.Assert.areSame(
- module._reasons.YOU_SUBSCRIBED,
- direct_info.reason);
- Y.ArrayAssert.itemsAreEqual(
- ['mute-direct-subscription',
- 'select-only-direct-subscription-metadata',
- 'select-only-direct-subscription-lifecycle',
- 'remove-direct-subscription-with-warning'],
- direct_info.reductions);
- Y.ArrayAssert.itemsAreEqual(
- [],
- direct_info.increases);
- }
-
-}));
-
-/**
- * Test for get_objectlink_html() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test conversion of ObjectLink to HTML.',
-
- _should: {
- error: {
- test_non_link: new Error('Not a proper ObjectLink.')
- }
- },
-
- test_string: function() {
- // When a string is passed in, it is returned unmodified.
- var link = 'test';
- Y.Assert.areEqual(
- link,
- module._get_objectlink_html(link));
- },
-
- test_non_link: function() {
- // When an object that doesn't have both 'title' and 'url'
- // passed in, it fails. (see _should.error above)
- var link = {};
- module._get_objectlink_html(link);
- },
-
- test_simple: function() {
- // When a string is passed in, it is returned unmodified.
- var link = {
- title: 'Title',
- url: 'http://url/'
- };
- Y.Assert.areEqual(
- '<a href="http://url/";>Title</a>',
- module._get_objectlink_html(link));
- },
-
- test_escaping_title: function() {
- // Even with title containing HTML characters, they are properly
- // escaped.
- var link = {
- title: 'Title<script>',
- url: 'http://url/'
- };
- Y.Assert.areEqual(
- '<a href="http://url/";>Title<script></a>',
- module._get_objectlink_html(link));
- },
-
- test_escaping_url: function() {
- // Even with title containing HTML characters, they are properly
- // escaped.
- var url = 'http://url/"; onclick="javascript:alert(\'test\');" a="';
- var link = {
- title: 'Title',
- url: url
- };
- // Firefox returns:
- // '<a href="http://url/%22%20onclick=%22' +
- // 'javascript:alert%28%27test%27%29;%22%20a=%22">Title</a>'
- // WebKit returns:
- // '<a href="http://url/" onclick="'+
- // 'javascript:alert(\'test\');" a="">Title</a>'
- Y.Assert.areNotEqual(
- '<a href="' + url + '">Title</a>',
- module._get_objectlink_html(link));
- }
-
-}));
-
-/**
- * Test for safely_render_description() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test variable substitution in subscription descriptions.',
-
- _should: {
- error: {
- test_non_link: new Error('Not a proper ObjectLink.')
- }
- },
-
- test_no_variables: function() {
- // For a string with no variables, no substitution is performed.
- var sub = {
- reason: 'test string with no vars',
- vars: { no: 'vars' }
- };
-
- Y.Assert.areEqual(
- sub.reason,
- module._safely_render_description(sub));
- },
-
- test_missing_variable: function() {
- // If a variable is missing, it is not substituted.
- var sub = {
- reason: 'test string with {foo}',
- vars: {}
- };
-
- Y.Assert.areEqual(
- 'test string with {foo}',
- module._safely_render_description(sub));
- },
-
- test_string_variable: function() {
- // Plain string variables are directly substituted.
- var sub = {
- reason: 'test string with {foo}',
- vars: { foo: 'nothing' }
- };
-
- Y.Assert.areEqual(
- 'test string with nothing',
- module._safely_render_description(sub));
- },
-
- _constructObjectLink: function(title, url) {
- // Constructs a mock ObjectLink.
- return { title: title, url: url };
- },
-
- test_objectlink_variable: function() {
- // ObjectLink variables get turned into actual HTML links.
- var sub = {
- reason: 'test string with {foo}',
- vars: { foo: this._constructObjectLink('Title', 'http://link/') }
- };
-
- Y.Assert.areEqual(
- 'test string with <a href="http://link/";>Title</a>',
- module._safely_render_description(sub));
- },
-
- test_multiple_variables: function() {
- // For multiple variables, they all get replaced.
- var sub = {
- reason: '{simple} string with {foo} {simple}',
- vars: {
- foo: this._constructObjectLink('Link', 'http://link/'),
- simple: "test"
- }
- };
-
- Y.Assert.areEqual(
- 'test string with <a href="http://link/";>Link</a> test',
- module._safely_render_description(sub));
- },
-
- test_extra_variable: function() {
- // Passing in extra variables causes them to be replaced as well.
- var sub = {
- reason: 'test string with {extra}',
- vars: {}
- };
- var extra_vars = {
- extra: 'something extra'
- };
-
- Y.Assert.areEqual(
- 'test string with something extra',
- module._safely_render_description(sub, extra_vars));
- },
-
- test_extra_objectlink_variable: function() {
- // Passing in extra ObjectLink variable gets properly substituted.
- var sub = {
- reason: 'test string with {extra}',
- vars: {}
- };
- var extra_vars = {
- extra: this._constructObjectLink('extras', 'http://link/')
- };
-
- Y.Assert.areEqual(
- 'test string with <a href="http://link/";>extras</a>',
- module._safely_render_description(sub, extra_vars));
- }
-
-}));
-
-reduction_ids = [module._action_ids.mute,
- module._action_ids.subscribe_only_metadata,
- module._action_ids.subscribe_only_closed,
- module._action_ids.unsubscribe];
-
-increasing_ids = [module._action_ids.unmute,
- module._action_ids.subscribe_all,
- module._action_ids.subscribe_metadata,
- module._action_ids.subscribe_closed];
-
-/**
- * Test for get_direct_description_node() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test direct node construction.',
-
- setUp: function () {
- window.LP = { links: {},
- cache: {},
- subscription_info: {
- direct: _constructCategory(),
- bug_id: 1,
- count: 0
- }
- };
- },
-
- tearDown: function () {
- delete window.LP;
- },
-
- test_basic_structure: function() {
- // The node has the three main components we expect.
- var node = module._get_direct_description_node();
- Y.Assert.areEqual('direct-subscription', node.get('id'));
- Y.Assert.isTrue(Y.Lang.isValue(node.one('.reason')));
- Y.Assert.isTrue(Y.Lang.isValue(node.one('.reductions')));
- Y.Assert.isTrue(Y.Lang.isValue(node.one('.increases')));
- Y.Assert.isTrue(
- Y.Lang.isValue(
- node.one('#'+module._action_ids.unsubscribe_with_warning)));
- },
-
- test_reductions_structure: function() {
- var node = module._get_direct_description_node().one('.reductions');
- var i;
- for (i = 0; i < reduction_ids; i++) {
- Y.Assert.isTrue(
- Y.Lang.isValue(node.one('#'+reduction_ids[i])));
- }
- },
-
- test_increases_structure: function() {
- var node = module._get_direct_description_node().one('.increases');
- var i;
- for (i = 0; i < increasing_ids; i++) {
- Y.Assert.isTrue(
- Y.Lang.isValue(node.one('#'+increasing_ids[i])));
- }
- }
-
-}));
-
-/**
- * Test for reveal_direct_description_actions() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test direct node modification with appropriate description.',
-
- setUp: function () {
- window.LP = { links: {},
- cache: {},
- subscription_info: {
- direct: _constructCategory(),
- bug_id: 1,
- count: 0
- }
- };
- },
-
- tearDown: function () {
- delete window.LP;
- },
-
- test_reason_displayed: function() {
- // A description is added in.
- var node = module._get_direct_description_node();
- var expected_text = 'Kumquat rutebega papaya';
- var info = {
- reason: expected_text,
- increases: [],
- reductions: []
- };
- module._reveal_direct_description_actions(node, info);
- Y.Assert.isTrue(node.get('text').indexOf(expected_text) !== -1);
- },
-
- test_reductions_displayed: function() {
- // Reductions are revealed, increases are not.
- var node = module._get_direct_description_node();
- var expected_text = 'Kumquat rutebega papaya';
- var info = {
- reason: expected_text,
- increases: [],
- reductions: reduction_ids
- };
- module._reveal_direct_description_actions(node, info);
- var i;
- for (i = 0; i < reduction_ids.length; i++) {
- Y.Assert.isFalse(
- node.one('#'+reduction_ids[i]).hasClass('hidden'));
- }
- for (i = 0; i < increasing_ids.length; i++) {
- Y.Assert.isTrue(
- node.one('#'+increasing_ids[i]).hasClass('hidden'));
- }
- },
-
- test_increases_displayed: function() {
- // Increases are revealed, reductions are not.
- var node = module._get_direct_description_node();
- var expected_text = 'Kumquat rutebega papaya';
- var info = {
- reason: expected_text,
- increases: increasing_ids,
- reductions: []
- };
- module._reveal_direct_description_actions(node, info);
- var i;
- for (i = 0; i < reduction_ids.length; i++) {
- Y.Assert.isTrue(
- node.one('#'+reduction_ids[i]).hasClass('hidden'));
- }
- for (i = 0; i < increasing_ids.length; i++) {
- Y.Assert.isFalse(
- node.one('#'+increasing_ids[i]).hasClass('hidden'));
- }
- },
-
- test_unsubscribe_with_warning_displayed: function() {
- // Unsubscribe with warning is special because it is not
- // one of the reductions that gets displayed the reduction box.
- var node = module._get_direct_description_node();
- var expected_text = 'Kumquat rutebega papaya';
- // Get a copy of the reduction ids.
- var reductions = reduction_ids.slice(0);
- // Remove unsubscribe.
- reductions.splice(
- reductions.indexOf(module._action_ids.unsubscribe), 1);
- // Add unsubscribe_with_warning.
- reductions.push(module._action_ids.unsubscribe_with_warning);
- var info = {
- reason: expected_text,
- increases: [],
- reductions: reductions
- };
- module._reveal_direct_description_actions(node, info);
- var i;
- for (i = 0; i < reductions.length; i++) {
- Y.Assert.isFalse(
- node.one('#'+reductions[i]).hasClass('hidden'));
- }
- Y.Assert.isTrue(
- node.one('#'+module._action_ids.unsubscribe).hasClass('hidden'));
- for (i = 0; i < increasing_ids.length; i++) {
- Y.Assert.isTrue(
- node.one('#'+increasing_ids[i]).hasClass('hidden'));
- }
- },
-
- test_redisplay: function() {
- // If the function is called twice with different values, the
- // redisplay is correct.
- var node = module._get_direct_description_node();
- var expected_text = 'Kumquat rutebega papaya';
- var info = {
- reason: expected_text,
- increases: increasing_ids,
- reductions: []
- };
- module._reveal_direct_description_actions(node, info);
- info = {
- reason: expected_text,
- increases: [],
- reductions: reduction_ids
- };
- module._reveal_direct_description_actions(node, info);
- var i;
- for (i = 0; i < reduction_ids.length; i++) {
- Y.Assert.isFalse(
- node.one('#'+reduction_ids[i]).hasClass('hidden'));
- }
- for (i = 0; i < increasing_ids.length; i++) {
- Y.Assert.isTrue(
- node.one('#'+increasing_ids[i]).hasClass('hidden'));
- }
- }
-
-}));
-
-/**
- * Tests for the *_action functions used for direct personal subscriptions.
- */
-suite.add(new Y.Test.Case({
- name: 'Test the direct personal subscription action node functions.',
-
- assert_action_matches_expectations: function (
- node_function, expected_id, expected_text, expected_class,
- expected_method, expected_args, begin_with_subscription,
- send_subscription, private_bug, click_ok) {
- // We begin with set up.
- module._lp_client = new LPClient();
- var sub = {
- bug: {
- 'private': private_bug,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- if (send_subscription) {
- module._lp_client.named_post.args = [
- {getAttrs: function () {
- return sub.subscription;
- }}];
- } else {
- module._lp_client.named_post.args = [];
- }
- var initial_subscriptions = [];
- if (begin_with_subscription) {
- initial_subscriptions.push(sub);
- }
- var bug_link = 'http://example.net/firefox/bug/1';
- window.LP = { links: {me: '~tweedledee'},
- cache: {context: {bug_link: bug_link},
- bug_is_private: private_bug,
- bug_subscription_info: {
- direct: _constructCategory(
- initial_subscriptions),
- bug_id: 1,
- count: initial_subscriptions.length
- }
- }
- };
- var display = module._get_direct_description_node();
- Y.one('body').appendChild(display);
- // Now we are actually ready to begin the tests. First we verify
- // the id, text, and link class are all as we expect.
- var node = node_function();
- Y.Assert.areEqual(expected_id, node.get('id'));
- Y.Assert.areEqual(expected_text, node.get('text'));
- Y.Assert.isTrue(node.one('a').hasClass(expected_class));
- // Now we verify that the link has been set up with the expected
- // method name and arguments. For this, we use the version of the
- // node that was actually inserted into the display. It shares the
- // same id.
- node = display.one('#'+expected_id);
- node.one('a').simulate('click');
- var co = Y.one('.yui3-overlay.yui3-lp-app-confirmationoverlay');
- if (!private_bug) {
- // If this a public bug, check the confirmation overlay is
- // nowhere to be found
- Y.Assert.isNull(co);
- } else {
- // Otherwise (private bug), click true
- var div = co.one('.yui3-lazr-formoverlay-actions');
- if (click_ok) {
- var ok = div.one('.ok-btn');
- ok.simulate('click');
- } else {
- var cancel = div.one('.cancel-btn');
- cancel.simulate('click');
- Y.Assert.areEqual(
- initial_subscriptions.length,
- window.LP.cache.bug_subscription_info.count);
- return;
- }
- }
- // We should have had a named_post to the bug_link, calling the
- // expected_method with the expected_args.
- Y.Assert.areEqual(1, module._lp_client.received.length);
- Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(bug_link, args[0]);
- Y.Assert.areEqual(expected_method, args[1]);
- Y.ObjectAssert.areEqual(expected_args, args[2].parameters);
- },
-
- tearDown: function() {
- var display = Y.one('#direct-subscription');
- if (Y.Lang.isValue(display)) {
- display.remove();
- }
- delete window.LP;
- delete module._lp_client;
- },
-
- test_mute_action: function() {
- this.assert_action_matches_expectations(
- module._mute_action, module._action_ids.mute,
- 'mute all emails from this bug',
- 'mute', 'mute', {}, true, true, false, true);
- },
-
- test_unmute_action: function() {
- this.assert_action_matches_expectations(
- module._unmute_action, module._action_ids.unmute,
- 'unmute emails from this bug',
- 'unmute', 'unmute', {}, true, false, false, true);
- },
-
- test_subscribe_all_action: function() {
- this.assert_action_matches_expectations(
- module._subscribe_all_action, module._action_ids.subscribe_all,
- 'receive all emails about this bug',
- 'edit', 'subscribe',
- {person: '~tweedledee', level: 'Discussion'},
- false, true, false, true);
- },
-
- test_subscribe_metadata_action: function() {
- this.assert_action_matches_expectations(
- module._subscribe_metadata_action,
- module._action_ids.subscribe_metadata,
- 'receive all emails about this bug except comments',
- 'edit', 'subscribe',
- {person: '~tweedledee', level: 'Details'},
- false, true, false, true);
- },
-
- test_subscribe_closed_action: function() {
- this.assert_action_matches_expectations(
- module._subscribe_closed_action,
- module._action_ids.subscribe_closed,
- 'only receive email when this bug is closed',
- 'edit', 'subscribe',
- {person: '~tweedledee', level: 'Lifecycle'},
- false, true, false, true);
- },
-
- test_subscribe_only_metadata_action: function() {
- this.assert_action_matches_expectations(
- module._subscribe_only_metadata_action,
- module._action_ids.subscribe_only_metadata,
- 'stop receiving comments from this bug',
- 'edit', 'subscribe',
- {person: '~tweedledee', level: 'Details'},
- false, true, false, true);
- },
-
- test_subscribe_only_closed_action: function() {
- this.assert_action_matches_expectations(
- module._subscribe_only_closed_action,
- module._action_ids.subscribe_only_closed,
- 'only receive email when this bug is closed',
- 'edit', 'subscribe',
- {person: '~tweedledee', level: 'Lifecycle'},
- false, true, false, true);
- },
-
- test_unsubscribe_action: function() {
- this.assert_action_matches_expectations(
- module._unsubscribe_action, module._action_ids.unsubscribe,
- 'unsubscribe from this bug',
- 'remove', 'unsubscribe', {}, true, false, false, true);
- },
-
- test_unsubscribe_with_warning_action: function() {
- this.assert_action_matches_expectations(
- module._unsubscribe_with_warning_action,
- module._action_ids.unsubscribe_with_warning,
- 'You can also unsubscribe from this bug. However, you also '+
- 'have other subscriptions to this bug that may send you email '+
- 'once you have unsubscribed.',
- 'remove', 'unsubscribe', {}, true, false, false, true);
- },
-
- test_unsubscribe_action_private_bug_cancel: function() {
- this.assert_action_matches_expectations(
- module._unsubscribe_action, module._action_ids.unsubscribe,
- 'unsubscribe from this bug',
- 'remove', null, {}, true, false, true, false);
- },
-
- test_unsubscribe_action_private_bug: function() {
- this.assert_action_matches_expectations(
- module._unsubscribe_action, module._action_ids.unsubscribe,
- 'unsubscribe from this bug',
- 'remove', 'unsubscribe', {}, true, false, true, true);
- }
-
-}));
-
-/**
- * Test for get_single_description_node() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test single subscription description node construction.',
-
- test_simple_text: function() {
- // A simple subscription with 'Text' as the reason and no variables.
- var sub = { reason: 'Text', vars: {}, action: function() {} };
- var node = module._get_single_description_node(sub);
-
- // The node has appropriate CSS class set.
- Y.Assert.isTrue(node.hasClass('subscription-description'));
-
- // There is also a sub-node containing the actual description.
- var subnode = node.one('.description-text');
- Y.Assert.areEqual('Text', subnode.get('text'));
- },
-
- test_variable_substitution: function() {
- // A subscription with variables and extra variables
- // has them replaced.
- var sub = { reason: 'Test {var1} {var2}',
- vars: { var1: 'my text'},
- action: function() {} };
- var extra_data = { var2: 'globally' };
- var node = module._get_single_description_node(sub, extra_data);
-
- // The node has appropriate CSS class set.
- Y.Assert.isTrue(node.hasClass('subscription-description'));
-
- // There is also a sub-node containing the actual description.
- var subnode = node.one('.description-text');
- Y.Assert.areEqual('Test my text globally', subnode.get('text'));
- }
-
-}));
-
-/**
- * Test for get_other_descriptions_node() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test creation of node describing all non-direct subscriptions.',
-
- setUp: function() {
- // Monkey patch effects duration to make effects instant.
- // This keeps wait times to a minimum.
- this.original_defaults = Y.lazr.effects.slide_effect_defaults;
- Y.lazr.effects.slide_effect_defaults.duration = 0;
- },
-
- tearDown: function() {
- Y.lazr.effects.slide_effect_defaults = this.original_defaults;
- },
-
-
- test_no_subscriptions: function() {
- // With just a personal subscription, undefined is returned.
- var info = {
- direct: _constructCategory([{ bug: {} }]),
- from_duplicate: _constructCategory(),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- count: 1
- };
- window.LP = { cache: {} };
- Y.Assert.areSame(
- undefined,
- module._get_other_descriptions_node(info));
- delete window.LP;
- },
-
- test_one_subscription: function() {
- // There is a subscription on the duplicate bug.
- var info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory([{ bug: {id: 1} }]),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- count: 1
- };
- window.LP = { links: { me: '~' } };
-
- // A node is returned with ID of 'other-subscriptions'.
- var node = module._get_other_descriptions_node(info);
- Y.Assert.areEqual(
- 'other-subscriptions', node.get('id'));
- // And it contains single '.subscription-description' node.
- Y.Assert.areEqual(
- 1, node.all('.subscription-description').size());
- delete window.LP;
- },
-
- test_multiple_subscription: function() {
- // There is a subscription on the duplicate bug 1,
- // and another as assignee on bug 2.
- var info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory([{ bug: {id: 1} }]),
- as_assignee: _constructCategory([{ bug: {id: 2} }]),
- as_owner: _constructCategory(),
- count: 1
- };
- window.LP = { cache: { context: { web_link: '/' } },
- links: { me: '~' } };
-
- // A node is returned containing two
- // '.subscription-description' nodes.
- var node = module._get_other_descriptions_node(info);
- Y.Assert.areEqual(
- 2, node.all('.subscription-description').size());
- delete window.LP;
- },
-
- test_no_direct_has_structural_subscriptions: function() {
- // With no non-personal subscriptions, and a structural
- // subscription, the node is still constructed because
- // structural subscriptions go there as well.
- var info = {
- direct: _constructCategory([{ bug: {} }]),
- from_duplicate: _constructCategory(),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- count: 1
- };
- window.LP = { cache: { subscription_info: ['1'] } };
- Y.Assert.isNotUndefined(
- module._get_other_descriptions_node(info));
- delete window.LP;
- },
-
- test_header: function() {
- // There is a subscription on the duplicate bug.
- var info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory([{ bug: {id: 1} }]),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- count: 1
- };
-
- window.LP = { links: { me: '~' } };
-
- // A returned node contains the 'other-subscriptions-header'
- // div with the link.
- var node = module._get_other_descriptions_node(info);
- var header = node.one('#other-subscriptions-header');
- Y.Assert.isNotUndefined(header);
- var link = header.one('a');
- Y.Assert.areEqual('Other subscriptions', link.get('text'));
-
- delete window.LP;
- },
-
- test_header_slideout: function() {
- // Clicking on the header slides-out the box, and
- // clicking it again slides it back in.
- var info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory([{ bug: {id: 1} }]),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- count: 1
- };
-
- window.LP = { links: { me: '~' } };
-
- // A returned node contains the 'other-subscriptions-header'
- // div with the link.
- var node = module._get_other_descriptions_node(info);
- var link = node.one('#other-subscriptions-header a');
- var list = node.one('#other-subscriptions-list');
-
- // Initially, the list is hidden.
- Y.Assert.isTrue(link.hasClass('treeCollapsed'));
- Y.Assert.isTrue(list.hasClass('lazr-closed'));
- Y.Assert.areEqual('none', list.getStyle('display'));
-
- // Clicking the link slides out the list of other subscriptions.
- link.simulate('click');
- this.wait(function() {
- Y.Assert.isFalse(link.hasClass('treeCollapsed'));
- Y.Assert.isTrue(link.hasClass('treeExpanded'));
- Y.Assert.isFalse(list.hasClass('lazr-closed'));
- Y.Assert.areNotEqual('none', list.getStyle('display'));
-
- // Clicking it again, slides it back in.
- // It has to be nested inside 'wait' because we need
- // to wait for the first click to "finish".
+ };
+ var cache = {
+ 'subscription-cache-reference-1': 'OK'
+ };
+ module._replace_textual_references(object, cache);
+ Y.Assert.areEqual('OK', object.something);
+ },
+
+ test_multiple: function() {
+ // With multiple references, they all get substituted.0
+ var object = {
+ something: 'subscription-cache-reference-1',
+ other: 'subscription-cache-reference-2'
+ };
+ var cache = {
+ 'subscription-cache-reference-1': 'OK 1',
+ 'subscription-cache-reference-2': 'OK 2'
+ };
+ module._replace_textual_references(object, cache);
+ Y.Assert.areEqual('OK 1', object.something);
+ Y.Assert.areEqual('OK 2', object.other);
+ },
+
+ test_recursive: function() {
+ // Even references in nested objects get replaced.
+ var object = {
+ nested: {
+ something: 'subscription-cache-reference-1'
+ }
+ };
+ var cache = {
+ 'subscription-cache-reference-1': 'OK'
+ };
+ module._replace_textual_references(object, cache);
+ Y.Assert.areEqual('OK', object.nested.something);
+ }
+ }));
+
+
+ /**
+ * Gather subscription records for all assignments.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Gather assignment subscription information',
+
+ test_nothing: function() {
+ // When there are no subscriptions as assignee, returns empty list.
+ var mock_category = {
+ count: 0,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ module._gather_subscriptions_as_assignee(mock_category));
+ },
+
+ test_personal: function() {
+ // When a person is directly the bug assignee, we get that
+ // subscription details returned.
+ var mock_category = {
+ count: 1,
+ personal: [{}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.YOU_ASSIGNED, subs[0].reason);
+ Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
+ },
+
+ test_team_member: function() {
+ // When a person is the bug assignee through team membership,
+ // we get that subscription details returned.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{ principal: 'my team'}],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAM_ASSIGNED, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple: function() {
+ // If a person is a member of multiple teams are assigned to work
+ // on a single bug (eg. on different bug tasks) they get only one
+ // subscription returned.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: 'team1'},
+ { principal: 'team2'}],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAMS_ASSIGNED, subs[0].reason);
+ // And there is a 'teams' variable containing all the team objects.
+ Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
+ subs[0].vars.teams);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple_duplicate: function() {
+ // As with the previous test, but we need to show that each team is
+ // only represented once even if they are responsible for multiple
+ // bug tasks.
+ // We test with full-fledged objects to make sure they work with the
+ // mechanism used to find dupes.
+ var team1 = {display_name: 'team 1',
+ web_link: 'http://launchpad.net/~team1'},
+ team2 = {display_name: 'team 2',
+ web_link: 'http://launchpad.net/~team2'},
+ mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: team1 },
+ { principal: team2 },
+ { principal: team2 }],
+ as_team_admin: []
+ },
+ subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAMS_ASSIGNED, subs[0].reason);
+ // And there is a 'teams' variable containing all the team objects.
+ var teams_found = [];
+ var index;
+ for (index = 0; index < subs[0].vars.teams.length; index++) {
+ teams_found.push(subs[0].vars.teams[index].title);
+ }
+ Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
+ },
+
+ test_team_admin: function() {
+ // When a person is the bug assignee through team membership,
+ // and a team admin at the same time, that subscription is returned.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'my team' }]
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAM_ASSIGNED, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
+ },
+
+ test_team_admin_multiple: function() {
+ // If a person is a member of multiple teams are assigned to work
+ // on a single bug (eg. on different bug tasks) they get only one
+ // subscription returned.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'team1'},
+ { principal: 'team2'}]
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAMS_ASSIGNED, subs[0].reason);
+ // And there is a 'teams' variable containing all the team objects.
+ Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
+ subs[0].vars.teams);
+ Y.Assert.areEqual(module._actions.CHANGE_ASSIGNEES, subs[0].action);
+ },
+
+ test_team_admin_multiple_duplicate: function() {
+ // As with the previous test, but we need to show that each team is
+ // only represented once even if they are responsible for multiple
+ // bug tasks.
+ // We test with full-fledged objects to make sure they work with the
+ // mechanism used to find dupes.
+ var team1 = {display_name: 'team 1',
+ web_link: 'http://launchpad.net/~team1'},
+ team2 = {display_name: 'team 2',
+ web_link: 'http://launchpad.net/~team2'},
+ mock_category = {
+ count: 2,
+ personal: [],
+ as_team_admin: [{ principal: team1 },
+ { principal: team2 },
+ { principal: team2 }],
+ as_team_member: []
+ },
+ subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ // And there is a 'teams' variable containing all the team objects.
+ var teams_found = [];
+ for (index = 0; index < subs[0].vars.teams.length; index++) {
+ teams_found.push(subs[0].vars.teams[index].title);
+ }
+ Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
+ },
+
+ test_combined: function() {
+ // Test that multiple assignments, even if they are in different
+ // categories, work properly.
+ var mock_category = {
+ count: 3,
+ personal: [{}],
+ as_team_member: [{ principal: 'users' }],
+ as_team_admin: [{ principal: 'admins' }]
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(3, subs.length);
+ },
+
+ test_object_links: function() {
+ // Test that team assignments actually provide decent link data.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [
+ { principal: { display_name: 'My team',
+ web_link: 'http://link' } }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual('My team', subs[0].vars.team.title);
+ Y.Assert.areEqual('http://link', subs[0].vars.team.url);
+ }
+ }));
+
+ /**
+ * Gather subscription records for bug supervisor.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Gather bug supervisor subscription information',
+
+ test_nothing: function() {
+ // When there are no subscriptions as bug supervisor,
+ // returns empty list.
+ var mock_category = {
+ count: 0,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ module._gather_subscriptions_as_supervisor(mock_category));
+ },
+
+ test_personal: function() {
+ // Person is the implicit bug supervisor by being the owner
+ // of the project with no bug supervisor.
+ var mock_category = {
+ count: 1,
+ personal: [{pillar: 'project'}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.YOU_OWNER, subs[0].reason);
+ Y.Assert.areEqual('project', subs[0].vars.pillar);
+ Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
+ },
+
+ test_personal_multiple: function() {
+ // Person is the implicit bug supervisor by being the owner
+ // of several projects (eg. multiple bug tasks) with no bug
+ // supervisor.
+ var mock_category = {
+ count: 2,
+ personal: [ {pillar: {title: 'project'} },
+ {pillar: {title:'distro'} }],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_team_member: function() {
+ // Person is a member of the team which is the implicit
+ // bug supervisor.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{ principal: 'my team',
+ pillar: 'project' }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAM_OWNER, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual('project', subs[0].vars.pillar);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple: function() {
+ // Person is a member of several teams which are implicit bug
+ // supervisors on multiple bugtasks, we get subscription
+ // records separately.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: 'team1',
+ pillar: {display_name: 'project'} },
+ { principal: 'team2',
+ pillar: {display_name: 'distro'} }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_team_admin: function() {
+ // Person is an admin of the team which is the implicit
+ // bug supervisor.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'my team',
+ pillar: 'project' }]
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAM_OWNER, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual('project', subs[0].vars.pillar);
+ Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
+ },
+
+ test_team_admin_multiple: function() {
+ // Person is an admin of several teams which are implicit bug
+ // supervisors on multiple bugtasks, we get subscription
+ // records separately.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'team1',
+ pillar: {display_name: 'project'} },
+ { principal: 'team2',
+ pillar: {display_name: 'distro'} }]
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_repeated_pillars: function() {
+ // Different bug tasks might still be on the same pillar,
+ // and we should only get one action.
+ var mock_pillar = { display_name: 'project',
+ web_link: 'http://project/' };
+ var mock_category = {
+ count: 1,
+ personal: [{pillar: mock_pillar},
+ {pillar: mock_pillar}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.YOU_OWNER, subs[0].reason);
+ Y.Assert.areEqual(mock_pillar, subs[0].vars.pillar.self);
+ Y.Assert.areEqual(module._actions.SET_BUG_SUPERVISOR, subs[0].action);
+ },
+
+ test_combined: function() {
+ // Test that multiple implicit bug supervisor roles
+ // are all returned.
+ var mock_category = {
+ count: 3,
+ personal: [{pillar: 'project1'}],
+ as_team_member: [{ principal: 'users', pillar: 'project2' }],
+ as_team_admin: [{ principal: 'admins', pillar: 'distro' }]
+ };
+ var subs = module._gather_subscriptions_as_assignee(mock_category);
+ Y.Assert.areEqual(3, subs.length);
+ },
+
+ test_object_links: function() {
+ // Test that team-as-supervisor actually provide decent link data,
+ // along with pillars as well.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{
+ principal: { display_name: 'My team',
+ web_link: 'http://link' },
+ pillar: { display_name: 'My project',
+ web_link: 'http://project/' }
+ }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_as_supervisor(mock_category);
+ Y.Assert.areEqual('My team', subs[0].vars.team.title);
+ Y.Assert.areEqual('http://link', subs[0].vars.team.url);
+
+ Y.Assert.areEqual('My project', subs[0].vars.pillar.title);
+ Y.Assert.areEqual('http://project/', subs[0].vars.pillar.url);
+ }
+ }));
+
+ /**
+ * Gather subscription records for dupe bug subscriptions.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Gather subscription information for duplicates',
+
+ test_nothing: function() {
+ // When there are no duplicate subscriptions, returns empty list.
+ var mock_category = {
+ count: 0,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ module._gather_subscriptions_from_duplicates(mock_category));
+ },
+
+ test_personal: function() {
+ // A person is subscribed to a duplicate bug.
+ var mock_category = {
+ count: 1,
+ personal: [{bug: 'dupe bug'}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.YOU_SUBSCRIBED_TO_DUPLICATE, subs[0].reason);
+ Y.Assert.areEqual('dupe bug', subs[0].vars.duplicate_bug);
+ Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
+ subs[0].action);
+ },
+
+ test_personal_multiple: function() {
+ // A person is subscribed to multiple duplicate bugs.
+ // They are returned together as one subscription record.
+ var mock_category = {
+ count: 2,
+ personal: [{bug: 'dupe1'}, {bug: 'dupe2'}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.YOU_SUBSCRIBED_TO_DUPLICATES, subs[0].reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
+ Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
+ subs[0].action);
+ },
+
+ test_team_member: function() {
+ // A person is a member of the team subscribed to a duplicate bug.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{ principal: 'my team',
+ bug: 'dupe' }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.TEAM_SUBSCRIBED_TO_DUPLICATE, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ // And a 'duplicate_bug' variable pointing to the dupe.
+ Y.Assert.areEqual('dupe', subs[0].vars.duplicate_bug);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple_bugs: function() {
+ // A person is a member of the team subscribed to multiple
+ // duplicate bugs.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{
+ principal: 'my team',
+ bug: 'dupe1'
+ }, {
+ principal: 'my team',
+ bug: 'dupe2'
+ }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.TEAM_SUBSCRIBED_TO_DUPLICATES, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ // And a 'duplicate_bugs' variable with the list of dupes.
+ Y.ArrayAssert.itemsAreEqual(
+ ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple: function() {
+ // A person is a member of several teams subscribed to
+ // duplicate bugs.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: 'team1',
+ bug: 'dupe1' },
+ { principal: 'team2',
+ bug: 'dupe1' }],
+ as_team_admin: []
+ };
+
+ // Result is two separate subscription records.
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_team_admin: function() {
+ // A person is an admin of the team subscribed to a duplicate bug.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'my team',
+ bug: 'dupe' }]
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAM_SUBSCRIBED_TO_DUPLICATE,
+ subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ // And a 'duplicate_bug' variable pointing to the dupe.
+ Y.Assert.areEqual('dupe', subs[0].vars.duplicate_bug);
+ Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
+ subs[0].action);
+ },
+
+ test_team_admin_multiple_bugs: function() {
+ // A person is an admin of the team subscribed to multiple
+ // duplicate bugs.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{
+ principal: 'my team',
+ bug: 'dupe1'
+ }, {
+ principal: 'my team',
+ bug: 'dupe2'
+ }]
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAM_SUBSCRIBED_TO_DUPLICATES,
+ subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ // And a 'duplicate_bugs' variable with the list of dupes.
+ Y.ArrayAssert.itemsAreEqual(
+ ['dupe1', 'dupe2'], subs[0].vars.duplicate_bugs);
+ Y.Assert.areEqual(module._actions.UNSUBSCRIBE_DUPLICATES,
+ subs[0].action);
+ },
+
+ test_team_admin_multiple: function() {
+ // A person is an admin of several teams subscribed to
+ // duplicate bugs.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'team1',
+ bug: 'dupe1' },
+ { principal: 'team2',
+ bug: 'dupe1' }]
+ };
+
+ // Result is two separate subscription records.
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_object_links: function() {
+ // Test that team dupe subscriptions actually provide decent
+ // link data, including duplicate bugs link data.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{
+ principal: { display_name: 'My team',
+ web_link: 'http://link' },
+ bug: { id: 1,
+ web_link: 'http://launchpad/bug/1' }
+ }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_from_duplicates(
+ mock_category);
+ Y.Assert.areEqual('My team', subs[0].vars.team.title);
+ Y.Assert.areEqual('http://link', subs[0].vars.team.url);
+
+ Y.Assert.areEqual('#1', subs[0].vars.duplicate_bug.title);
+ Y.Assert.areEqual(
+ 'http://launchpad/bug/1', subs[0].vars.duplicate_bug.url);
+ }
+ }));
+
+ /**
+ * Gather subscription records for direct team subscriptions.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Gather team subscription information',
+
+ test_nothing: function() {
+ // When there are no subscriptions through team, returns empty list.
+ var mock_category = {
+ count: 0,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ module._gather_subscriptions_through_team(mock_category));
+ },
+
+ test_personal: function() {
+ // A personal subscription is not considered a team subscription.
+ var mock_category = {
+ count: 1,
+ personal: [{}],
+ as_team_member: [],
+ as_team_admin: []
+ };
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ module._gather_subscriptions_through_team(mock_category));
+ },
+
+ test_team_member: function() {
+ // Person is a member of the team subscribed to the bug.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [{ principal: 'my team'}],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAM_SUBSCRIBED, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple: function() {
+ // Person is a member of several teams subscribed to the bug.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: 'team1'},
+ { principal: 'team2'}],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(module._reasons.TEAMS_SUBSCRIBED, subs[0].reason);
+ // And there is a 'teams' variable containing all the team objects.
+ Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
+ subs[0].vars.teams);
+ Y.Assert.areEqual(module._actions.CONTACT_TEAMS, subs[0].action);
+ },
+
+ test_team_member_multiple_duplicate: function() {
+ // As with the previous test, but we need to show that each team is
+ // only represented once even if they are responsible for multiple
+ // bug tasks.
+ // We test with full-fledged objects to make sure they work with the
+ // mechanism used to find dupes.
+ var team1 = {display_name: 'team 1',
+ web_link: 'http://launchpad.net/~team1'},
+ team2 = {display_name: 'team 2',
+ web_link: 'http://launchpad.net/~team2'},
+ mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [{ principal: team1 },
+ { principal: team2 },
+ { principal: team2 }],
+ as_team_admin: []
+ },
+ subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ // And there is a 'teams' variable containing all the team objects.
+ var teams_found = [];
+ for (index = 0; index < subs[0].vars.teams.length; index++) {
+ teams_found.push(subs[0].vars.teams[index].title);
+ }
+ Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
+ },
+
+ test_team_admin: function() {
+ // Person is an admin of the team subscribed to the bug.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'my team' }]
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAM_SUBSCRIBED, subs[0].reason);
+ // And there is a 'team' variable containing the team object.
+ Y.Assert.areEqual('my team', subs[0].vars.team);
+ Y.Assert.areEqual(module._actions.CHANGE_TEAM_SUBSCRIPTIONS,
+ subs[0].action);
+ },
+
+ test_team_admin_multiple: function() {
+ // Person is an admin of the several teams subscribed to the bug.
+ var mock_category = {
+ count: 2,
+ personal: [],
+ as_team_member: [],
+ as_team_admin: [{ principal: 'team1'},
+ { principal: 'team2'}]
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ Y.Assert.areEqual(
+ module._reasons.ADMIN_TEAMS_SUBSCRIBED, subs[0].reason);
+ // And there is a 'teams' variable containing all the team objects.
+ Y.ArrayAssert.itemsAreEqual(['team1', 'team2'],
+ subs[0].vars.teams);
+ Y.Assert.areEqual(module._actions.CHANGE_TEAM_SUBSCRIPTIONS,
+ subs[0].action);
+ },
+
+ test_team_admin_multiple_duplicate: function() {
+ // As with the previous test, but we need to show that each team is
+ // only represented once even if they are responsible for multiple
+ // bug tasks.
+ // We test with full-fledged objects to make sure they work with the
+ // mechanism used to find dupes.
+ var team1 = {display_name: 'team 1',
+ web_link: 'http://launchpad.net/~team1'},
+ team2 = {display_name: 'team 2',
+ web_link: 'http://launchpad.net/~team2'},
+ mock_category = {
+ count: 2,
+ personal: [],
+ as_team_admin: [{ principal: team1 },
+ { principal: team2 },
+ { principal: team2 }],
+ as_team_member: []
+ },
+ subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(1, subs.length);
+ // And there is a 'teams' variable containing all the team objects.
+ var teams_found = [];
+ for (index = 0; index < subs[0].vars.teams.length; index++) {
+ teams_found.push(subs[0].vars.teams[index].title);
+ }
+ Y.ArrayAssert.itemsAreEqual(['team 1', 'team 2'], teams_found);
+ },
+
+ test_combined: function() {
+ // Test that multiple subscriptions, even if they are in different
+ // categories, work properly, and that personal subscriptions are
+ // still ignored.
+ var mock_category = {
+ count: 3,
+ personal: [{}],
+ as_team_member: [{ principal: 'users' }],
+ as_team_admin: [{ principal: 'admins' }]
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual(2, subs.length);
+ },
+
+ test_object_links: function() {
+ // Test that team subscriptions actually provide decent link data.
+ var mock_category = {
+ count: 1,
+ personal: [],
+ as_team_member: [
+ { principal: { display_name: 'My team',
+ web_link: 'http://link' } }],
+ as_team_admin: []
+ };
+ var subs = module._gather_subscriptions_through_team(mock_category);
+ Y.Assert.areEqual('My team', subs[0].vars.team.title);
+ Y.Assert.areEqual('http://link', subs[0].vars.team.url);
+ }
+ }));
+
+ /**
+ * Get the reason for a direct subscription.
+ * Tests for method get_direct_subscription_information().
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Get reason and actions for a direct subscription',
+
+ _should: {
+ error: {
+ test_multiple_direct_subscriptions:
+ new Error('Programmer error: a person should not have more than '+
+ 'one direct personal subscription.'),
+ test_direct_subscription_at_unknown_level:
+ new Error('Programmer error: unknown bug notification level: '+
+ 'The Larch')
+ }
+ },
+
+ setUp: function() {
+ window.LP = {cache: {subscription_info: []}};
+ },
+
+ tearDown: function() {
+ delete window.LP;
+ },
+
+ test_multiple_direct_subscriptions: function() {
+ // It should not be possible to have multiple direct,
+ // personal subscriptions.
+ // This errors out (see _should.error above).
+ var info = {
+ direct: _constructCategory(['1', '2']),
+ count: 2
+ };
+ module._get_direct_subscription_information(info);
+ },
+
+ test_no_subscriptions_at_all: function() {
+ // There are no subscriptions at all.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicates: _constructCategory()
+ };
+ info.count = info.direct.count + info.from_duplicates.count;
+
+ direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areEqual(
+ module._reasons.NOT_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['select-direct-subscription-discussion',
+ 'select-direct-subscription-metadata',
+ 'select-direct-subscription-lifecycle'],
+ direct_info.increases);
+ },
+
+ test_only_structural_subscriptions: function() {
+ // There are only structural subscriptions.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicates: _constructCategory()
+ };
+ info.count = info.direct.count + info.from_duplicates.count;
+ window.LP.cache.subscription_info.push(true);
+
+ direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.NOT_PERSONALLY_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['select-direct-subscription-discussion'],
+ direct_info.increases);
+ },
+
+ test_no_direct_subscriptions: function() {
+ // There is no direct subscription, but there are
+ // other subscriptions.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicates: _constructCategory(['dupe'])
+ };
+ info.count = info.direct.count + info.from_duplicates.count;
+ direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.NOT_PERSONALLY_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['select-direct-subscription-discussion'],
+ direct_info.increases);
+ },
+
+ test_muted_subscription: function() {
+ // The direct subscription is muted.
+ var info = {
+ direct: _constructCategory(['direct']),
+ muted: true
+ };
+ info.count = info.direct.count;
+ direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.MUTED_SUBSCRIPTION,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['unmute-direct-subscription'],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_at_discussion_level: function() {
+ // The larch^D^D^D^D^D^D simple direct subscription.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Discussion'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_at_metadata_level: function() {
+ // The simple direct subscription at metadata level.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['select-direct-subscription-discussion'],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_at_lifecycle_level: function() {
+ // The simple direct subscription at lifecycle level.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Lifecycle'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ ['select-direct-subscription-discussion',
+ 'select-direct-subscription-metadata'],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_at_unknown_level: function() {
+ // The simple direct subscription at unknown level.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'The Larch'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+ // This should raise an error.
+ module._get_direct_subscription_information(info);
+ },
+
+ test_direct_subscription_as_reporter: function() {
+ // The direct subscription created for bug reporter.
+ var sub = {
+ bug: {},
+ principal_is_reporter: true,
+ subscription: {bug_notification_level: 'Discussion'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_REPORTED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_for_supervisor: function() {
+ // The direct subscription created on private bugs for
+ // the bug supervisor.
+ var sub = {
+ bug: {
+ 'private': true
+ },
+ subscription: {bug_notification_level: 'Discussion'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED_BUG_SUPERVISOR,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_for_security_contact: function() {
+ // The simple direct subscription.
+ var sub = {
+ bug: {
+ security_related: true
+ },
+ subscription: {bug_notification_level: 'Discussion'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ count: 1
+ };
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED_SECURITY_CONTACT,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.increases);
+ },
+
+ test_direct_subscription_and_other_subscriptions: function() {
+ // Other subscriptions are present along with the simple direct
+ // subscription.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Discussion'}
+ };
+ var info = {
+ direct: _constructCategory([sub]),
+ from_duplicates: _constructCategory(['dupe']),
+ count: 2
+ };
+
+ var direct_info = module._get_direct_subscription_information(info);
+ Y.Assert.areSame(
+ module._reasons.YOU_SUBSCRIBED,
+ direct_info.reason);
+ Y.ArrayAssert.itemsAreEqual(
+ ['mute-direct-subscription',
+ 'select-only-direct-subscription-metadata',
+ 'select-only-direct-subscription-lifecycle',
+ 'remove-direct-subscription-with-warning'],
+ direct_info.reductions);
+ Y.ArrayAssert.itemsAreEqual(
+ [],
+ direct_info.increases);
+ }
+
+ }));
+
+ /**
+ * Test for get_objectlink_html() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test conversion of ObjectLink to HTML.',
+
+ _should: {
+ error: {
+ test_non_link: new Error('Not a proper ObjectLink.')
+ }
+ },
+
+ test_string: function() {
+ // When a string is passed in, it is returned unmodified.
+ var link = 'test';
+ Y.Assert.areEqual(
+ link,
+ module._get_objectlink_html(link));
+ },
+
+ test_non_link: function() {
+ // When an object that doesn't have both 'title' and 'url'
+ // passed in, it fails. (see _should.error above)
+ var link = {};
+ module._get_objectlink_html(link);
+ },
+
+ test_simple: function() {
+ // When a string is passed in, it is returned unmodified.
+ var link = {
+ title: 'Title',
+ url: 'http://url/'
+ };
+ Y.Assert.areEqual(
+ '<a href="http://url/";>Title</a>',
+ module._get_objectlink_html(link));
+ },
+
+ test_escaping_title: function() {
+ // Even with title containing HTML characters, they are properly
+ // escaped.
+ var link = {
+ title: 'Title<script>',
+ url: 'http://url/'
+ };
+ Y.Assert.areEqual(
+ '<a href="http://url/";>Title<script></a>',
+ module._get_objectlink_html(link));
+ },
+
+ test_escaping_url: function() {
+ // Even with title containing HTML characters, they are properly
+ // escaped.
+ var url = 'http://url/"; onclick="javascript:alert(\'test\');" a="';
+ var link = {
+ title: 'Title',
+ url: url
+ };
+ // Firefox returns:
+ // '<a href="http://url/%22%20onclick=%22' +
+ // 'javascript:alert%28%27test%27%29;%22%20a=%22">Title</a>'
+ // WebKit returns:
+ // '<a href="http://url/" onclick="'+
+ // 'javascript:alert(\'test\');" a="">Title</a>'
+ Y.Assert.areNotEqual(
+ '<a href="' + url + '">Title</a>',
+ module._get_objectlink_html(link));
+ }
+
+ }));
+
+ /**
+ * Test for safely_render_description() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test variable substitution in subscription descriptions.',
+
+ _should: {
+ error: {
+ test_non_link: new Error('Not a proper ObjectLink.')
+ }
+ },
+
+ test_no_variables: function() {
+ // For a string with no variables, no substitution is performed.
+ var sub = {
+ reason: 'test string with no vars',
+ vars: { no: 'vars' }
+ };
+
+ Y.Assert.areEqual(
+ sub.reason,
+ module._safely_render_description(sub));
+ },
+
+ test_missing_variable: function() {
+ // If a variable is missing, it is not substituted.
+ var sub = {
+ reason: 'test string with {foo}',
+ vars: {}
+ };
+
+ Y.Assert.areEqual(
+ 'test string with {foo}',
+ module._safely_render_description(sub));
+ },
+
+ test_string_variable: function() {
+ // Plain string variables are directly substituted.
+ var sub = {
+ reason: 'test string with {foo}',
+ vars: { foo: 'nothing' }
+ };
+
+ Y.Assert.areEqual(
+ 'test string with nothing',
+ module._safely_render_description(sub));
+ },
+
+ _constructObjectLink: function(title, url) {
+ // Constructs a mock ObjectLink.
+ return { title: title, url: url };
+ },
+
+ test_objectlink_variable: function() {
+ // ObjectLink variables get turned into actual HTML links.
+ var sub = {
+ reason: 'test string with {foo}',
+ vars: { foo: this._constructObjectLink('Title', 'http://link/') }
+ };
+
+ Y.Assert.areEqual(
+ 'test string with <a href="http://link/";>Title</a>',
+ module._safely_render_description(sub));
+ },
+
+ test_multiple_variables: function() {
+ // For multiple variables, they all get replaced.
+ var sub = {
+ reason: '{simple} string with {foo} {simple}',
+ vars: {
+ foo: this._constructObjectLink('Link', 'http://link/'),
+ simple: "test"
+ }
+ };
+
+ Y.Assert.areEqual(
+ 'test string with <a href="http://link/";>Link</a> test',
+ module._safely_render_description(sub));
+ },
+
+ test_extra_variable: function() {
+ // Passing in extra variables causes them to be replaced as well.
+ var sub = {
+ reason: 'test string with {extra}',
+ vars: {}
+ };
+ var extra_vars = {
+ extra: 'something extra'
+ };
+
+ Y.Assert.areEqual(
+ 'test string with something extra',
+ module._safely_render_description(sub, extra_vars));
+ },
+
+ test_extra_objectlink_variable: function() {
+ // Passing in extra ObjectLink variable gets properly substituted.
+ var sub = {
+ reason: 'test string with {extra}',
+ vars: {}
+ };
+ var extra_vars = {
+ extra: this._constructObjectLink('extras', 'http://link/')
+ };
+
+ Y.Assert.areEqual(
+ 'test string with <a href="http://link/";>extras</a>',
+ module._safely_render_description(sub, extra_vars));
+ }
+
+ }));
+
+
+ /**
+ * Test for get_direct_description_node() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test direct node construction.',
+
+ setUp: function () {
+ window.LP = { links: {},
+ cache: {},
+ subscription_info: {
+ direct: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ }
+ };
+ },
+
+ tearDown: function () {
+ delete window.LP;
+ },
+
+ test_basic_structure: function() {
+ // The node has the three main components we expect.
+ var node = module._get_direct_description_node();
+ Y.Assert.areEqual('direct-subscription', node.get('id'));
+ Y.Assert.isTrue(Y.Lang.isValue(node.one('.reason')));
+ Y.Assert.isTrue(Y.Lang.isValue(node.one('.reductions')));
+ Y.Assert.isTrue(Y.Lang.isValue(node.one('.increases')));
+ Y.Assert.isTrue(
+ Y.Lang.isValue(
+ node.one('#'+module._action_ids.unsubscribe_with_warning)));
+ },
+
+ test_reductions_structure: function() {
+ var node = module._get_direct_description_node().one('.reductions');
+ var i;
+ for (i = 0; i < reduction_ids; i++) {
+ Y.Assert.isTrue(
+ Y.Lang.isValue(node.one('#'+reduction_ids[i])));
+ }
+ },
+
+ test_increases_structure: function() {
+ var node = module._get_direct_description_node().one('.increases');
+ var i;
+ for (i = 0; i < increasing_ids; i++) {
+ Y.Assert.isTrue(
+ Y.Lang.isValue(node.one('#'+increasing_ids[i])));
+ }
+ }
+
+ }));
+
+ /**
+ * Test for reveal_direct_description_actions() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test direct node modification with appropriate description.',
+
+ setUp: function () {
+ window.LP = { links: {},
+ cache: {},
+ subscription_info: {
+ direct: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ }
+ };
+ },
+
+ tearDown: function () {
+ delete window.LP;
+ },
+
+ test_reason_displayed: function() {
+ // A description is added in.
+ var node = module._get_direct_description_node();
+ var expected_text = 'Kumquat rutebega papaya';
+ var info = {
+ reason: expected_text,
+ increases: [],
+ reductions: []
+ };
+ module._reveal_direct_description_actions(node, info);
+ Y.Assert.isTrue(node.get('text').indexOf(expected_text) !== -1);
+ },
+
+ test_reductions_displayed: function() {
+ // Reductions are revealed, increases are not.
+ var node = module._get_direct_description_node();
+ var expected_text = 'Kumquat rutebega papaya';
+ var info = {
+ reason: expected_text,
+ increases: [],
+ reductions: reduction_ids
+ };
+ module._reveal_direct_description_actions(node, info);
+ var i;
+ for (i = 0; i < reduction_ids.length; i++) {
+ Y.Assert.isFalse(
+ node.one('#'+reduction_ids[i]).hasClass('hidden'));
+ }
+ for (i = 0; i < increasing_ids.length; i++) {
+ Y.Assert.isTrue(
+ node.one('#'+increasing_ids[i]).hasClass('hidden'));
+ }
+ },
+
+ test_increases_displayed: function() {
+ // Increases are revealed, reductions are not.
+ var node = module._get_direct_description_node();
+ var expected_text = 'Kumquat rutebega papaya';
+ var info = {
+ reason: expected_text,
+ increases: increasing_ids,
+ reductions: []
+ };
+ module._reveal_direct_description_actions(node, info);
+ var i;
+ for (i = 0; i < reduction_ids.length; i++) {
+ Y.Assert.isTrue(
+ node.one('#'+reduction_ids[i]).hasClass('hidden'));
+ }
+ for (i = 0; i < increasing_ids.length; i++) {
+ Y.Assert.isFalse(
+ node.one('#'+increasing_ids[i]).hasClass('hidden'));
+ }
+ },
+
+ test_unsubscribe_with_warning_displayed: function() {
+ // Unsubscribe with warning is special because it is not
+ // one of the reductions that gets displayed the reduction box.
+ var node = module._get_direct_description_node();
+ var expected_text = 'Kumquat rutebega papaya';
+ // Get a copy of the reduction ids.
+ var reductions = reduction_ids.slice(0);
+ // Remove unsubscribe.
+ reductions.splice(
+ reductions.indexOf(module._action_ids.unsubscribe), 1);
+ // Add unsubscribe_with_warning.
+ reductions.push(module._action_ids.unsubscribe_with_warning);
+ var info = {
+ reason: expected_text,
+ increases: [],
+ reductions: reductions
+ };
+ module._reveal_direct_description_actions(node, info);
+ var i;
+ for (i = 0; i < reductions.length; i++) {
+ Y.Assert.isFalse(
+ node.one('#'+reductions[i]).hasClass('hidden'));
+ }
+ Y.Assert.isTrue(
+ node.one('#'+module._action_ids.unsubscribe).hasClass('hidden'));
+ for (i = 0; i < increasing_ids.length; i++) {
+ Y.Assert.isTrue(
+ node.one('#'+increasing_ids[i]).hasClass('hidden'));
+ }
+ },
+
+ test_redisplay: function() {
+ // If the function is called twice with different values, the
+ // redisplay is correct.
+ var node = module._get_direct_description_node();
+ var expected_text = 'Kumquat rutebega papaya';
+ var info = {
+ reason: expected_text,
+ increases: increasing_ids,
+ reductions: []
+ };
+ module._reveal_direct_description_actions(node, info);
+ info = {
+ reason: expected_text,
+ increases: [],
+ reductions: reduction_ids
+ };
+ module._reveal_direct_description_actions(node, info);
+ var i;
+ for (i = 0; i < reduction_ids.length; i++) {
+ Y.Assert.isFalse(
+ node.one('#'+reduction_ids[i]).hasClass('hidden'));
+ }
+ for (i = 0; i < increasing_ids.length; i++) {
+ Y.Assert.isTrue(
+ node.one('#'+increasing_ids[i]).hasClass('hidden'));
+ }
+ }
+
+ }));
+
+ /**
+ * Tests for the *_action functions used for direct personal subscriptions.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test the direct personal subscription action node functions.',
+
+ assert_action_matches_expectations: function (
+ node_function, expected_id, expected_text, expected_class,
+ expected_method, expected_args, begin_with_subscription,
+ send_subscription, private_bug, click_ok) {
+ // We begin with set up.
+ module._lp_client = new Y.lp.testing.helpers.LPClient();
+
+ var sub = {
+ bug: {
+ 'private': private_bug,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ if (send_subscription) {
+ module._lp_client.named_post.args = [
+ {getAttrs: function () {
+ return sub.subscription;
+ }}];
+ } else {
+ module._lp_client.named_post.args = [];
+ }
+ var initial_subscriptions = [];
+ if (begin_with_subscription) {
+ initial_subscriptions.push(sub);
+ }
+ var bug_link = 'http://example.net/firefox/bug/1';
+ window.LP = { links: {me: '~tweedledee'},
+ cache: {context: {bug_link: bug_link},
+ bug_is_private: private_bug,
+ bug_subscription_info: {
+ direct: _constructCategory(
+ initial_subscriptions),
+ bug_id: 1,
+ count: initial_subscriptions.length
+ }
+ }
+ };
+ var display = module._get_direct_description_node();
+ Y.one('body').appendChild(display);
+ // Now we are actually ready to begin the tests. First we verify
+ // the id, text, and link class are all as we expect.
+ var node = node_function();
+ Y.Assert.areEqual(expected_id, node.get('id'));
+ Y.Assert.areEqual(expected_text, node.get('text'));
+ Y.Assert.isTrue(node.one('a').hasClass(expected_class));
+ // Now we verify that the link has been set up with the expected
+ // method name and arguments. For this, we use the version of the
+ // node that was actually inserted into the display. It shares the
+ // same id.
+ node = display.one('#'+expected_id);
+ node.one('a').simulate('click');
+ var co = Y.one('.yui3-overlay.yui3-lp-app-confirmationoverlay');
+ if (!private_bug) {
+ // If this a public bug, check the confirmation overlay is
+ // nowhere to be found
+ Y.Assert.isNull(co);
+ } else {
+ // Otherwise (private bug), click true
+ var div = co.one('.yui3-lazr-formoverlay-actions');
+ if (click_ok) {
+ var ok = div.one('.ok-btn');
+ ok.simulate('click');
+ } else {
+ var cancel = div.one('.cancel-btn');
+ cancel.simulate('click');
+ Y.Assert.areEqual(
+ initial_subscriptions.length,
+ window.LP.cache.bug_subscription_info.count);
+ return;
+ }
+ }
+ // We should have had a named_post to the bug_link, calling the
+ // expected_method with the expected_args.
+ Y.Assert.areEqual(1, module._lp_client.received.length);
+ Y.Assert.areEqual('named_post', module._lp_client.received[0][0]);
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(bug_link, args[0]);
+ Y.Assert.areEqual(expected_method, args[1]);
+ Y.ObjectAssert.areEqual(expected_args, args[2].parameters);
+ },
+
+ tearDown: function() {
+ var display = Y.one('#direct-subscription');
+ if (Y.Lang.isValue(display)) {
+ display.remove();
+ }
+ delete window.LP;
+ delete module._lp_client;
+ },
+
+ test_mute_action: function() {
+ this.assert_action_matches_expectations(
+ module._mute_action, module._action_ids.mute,
+ 'mute all emails from this bug',
+ 'mute', 'mute', {}, true, true, false, true);
+ },
+
+ test_unmute_action: function() {
+ this.assert_action_matches_expectations(
+ module._unmute_action, module._action_ids.unmute,
+ 'unmute emails from this bug',
+ 'unmute', 'unmute', {}, true, false, false, true);
+ },
+
+ test_subscribe_all_action: function() {
+ this.assert_action_matches_expectations(
+ module._subscribe_all_action, module._action_ids.subscribe_all,
+ 'receive all emails about this bug',
+ 'edit', 'subscribe',
+ {person: '~tweedledee', level: 'Discussion'},
+ false, true, false, true);
+ },
+
+ test_subscribe_metadata_action: function() {
+ this.assert_action_matches_expectations(
+ module._subscribe_metadata_action,
+ module._action_ids.subscribe_metadata,
+ 'receive all emails about this bug except comments',
+ 'edit', 'subscribe',
+ {person: '~tweedledee', level: 'Details'},
+ false, true, false, true);
+ },
+
+ test_subscribe_closed_action: function() {
+ this.assert_action_matches_expectations(
+ module._subscribe_closed_action,
+ module._action_ids.subscribe_closed,
+ 'only receive email when this bug is closed',
+ 'edit', 'subscribe',
+ {person: '~tweedledee', level: 'Lifecycle'},
+ false, true, false, true);
+ },
+
+ test_subscribe_only_metadata_action: function() {
+ this.assert_action_matches_expectations(
+ module._subscribe_only_metadata_action,
+ module._action_ids.subscribe_only_metadata,
+ 'stop receiving comments from this bug',
+ 'edit', 'subscribe',
+ {person: '~tweedledee', level: 'Details'},
+ false, true, false, true);
+ },
+
+ test_subscribe_only_closed_action: function() {
+ this.assert_action_matches_expectations(
+ module._subscribe_only_closed_action,
+ module._action_ids.subscribe_only_closed,
+ 'only receive email when this bug is closed',
+ 'edit', 'subscribe',
+ {person: '~tweedledee', level: 'Lifecycle'},
+ false, true, false, true);
+ },
+
+ test_unsubscribe_action: function() {
+ this.assert_action_matches_expectations(
+ module._unsubscribe_action, module._action_ids.unsubscribe,
+ 'unsubscribe from this bug',
+ 'remove', 'unsubscribe', {}, true, false, false, true);
+ },
+
+ test_unsubscribe_with_warning_action: function() {
+ this.assert_action_matches_expectations(
+ module._unsubscribe_with_warning_action,
+ module._action_ids.unsubscribe_with_warning,
+ 'You can also unsubscribe from this bug. However, you also '+
+ 'have other subscriptions to this bug that may send you email '+
+ 'once you have unsubscribed.',
+ 'remove', 'unsubscribe', {}, true, false, false, true);
+ },
+
+ test_unsubscribe_action_private_bug_cancel: function() {
+ this.assert_action_matches_expectations(
+ module._unsubscribe_action, module._action_ids.unsubscribe,
+ 'unsubscribe from this bug',
+ 'remove', null, {}, true, false, true, false);
+ },
+
+ test_unsubscribe_action_private_bug: function() {
+ this.assert_action_matches_expectations(
+ module._unsubscribe_action, module._action_ids.unsubscribe,
+ 'unsubscribe from this bug',
+ 'remove', 'unsubscribe', {}, true, false, true, true);
+ }
+
+ }));
+
+ /**
+ * Test for get_single_description_node() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test single subscription description node construction.',
+
+ test_simple_text: function() {
+ // A simple subscription with 'Text' as the reason and no variables.
+ var sub = { reason: 'Text', vars: {}, action: function() {} };
+ var node = module._get_single_description_node(sub);
+
+ // The node has appropriate CSS class set.
+ Y.Assert.isTrue(node.hasClass('subscription-description'));
+
+ // There is also a sub-node containing the actual description.
+ var subnode = node.one('.description-text');
+ Y.Assert.areEqual('Text', subnode.get('text'));
+ },
+
+ test_variable_substitution: function() {
+ // A subscription with variables and extra variables
+ // has them replaced.
+ var sub = { reason: 'Test {var1} {var2}',
+ vars: { var1: 'my text'},
+ action: function() {} };
+ var extra_data = { var2: 'globally' };
+ var node = module._get_single_description_node(sub, extra_data);
+
+ // The node has appropriate CSS class set.
+ Y.Assert.isTrue(node.hasClass('subscription-description'));
+
+ // There is also a sub-node containing the actual description.
+ var subnode = node.one('.description-text');
+ Y.Assert.areEqual('Test my text globally', subnode.get('text'));
+ }
+
+ }));
+
+ /**
+ * Test for get_other_descriptions_node() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test creation of node describing all non-direct subscriptions.',
+
+ setUp: function() {
+ // Monkey patch effects duration to make effects instant.
+ // This keeps wait times to a minimum.
+ this.original_defaults = Y.lazr.effects.slide_effect_defaults;
+ Y.lazr.effects.slide_effect_defaults.duration = 0;
+ },
+
+ tearDown: function() {
+ Y.lazr.effects.slide_effect_defaults = this.original_defaults;
+ },
+
+ test_no_subscriptions: function() {
+ // With just a personal subscription, undefined is returned.
+ var info = {
+ direct: _constructCategory([{ bug: {} }]),
+ from_duplicate: _constructCategory(),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+ window.LP = { cache: {} };
+ Y.Assert.areSame(
+ undefined,
+ module._get_other_descriptions_node(info));
+ delete window.LP;
+ },
+
+ test_one_subscription: function() {
+ // There is a subscription on the duplicate bug.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory([{ bug: {id: 1} }]),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+ window.LP = { links: { me: '~' } };
+
+ // A node is returned with ID of 'other-subscriptions'.
+ var node = module._get_other_descriptions_node(info);
+ Y.Assert.areEqual(
+ 'other-subscriptions', node.get('id'));
+ // And it contains single '.subscription-description' node.
+ Y.Assert.areEqual(
+ 1, node.all('.subscription-description').size());
+ delete window.LP;
+ },
+
+ test_multiple_subscription: function() {
+ // There is a subscription on the duplicate bug 1,
+ // and another as assignee on bug 2.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory([{ bug: {id: 1} }]),
+ as_assignee: _constructCategory([{ bug: {id: 2} }]),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+ window.LP = { cache: { context: { web_link: '/' } },
+ links: { me: '~' } };
+
+ // A node is returned containing two
+ // '.subscription-description' nodes.
+ var node = module._get_other_descriptions_node(info);
+ Y.Assert.areEqual(
+ 2, node.all('.subscription-description').size());
+ delete window.LP;
+ },
+
+ test_no_direct_has_structural_subscriptions: function() {
+ // With no non-personal subscriptions, and a structural
+ // subscription, the node is still constructed because
+ // structural subscriptions go there as well.
+ var info = {
+ direct: _constructCategory([{ bug: {} }]),
+ from_duplicate: _constructCategory(),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+ window.LP = { cache: { subscription_info: ['1'] } };
+ Y.Assert.isNotUndefined(
+ module._get_other_descriptions_node(info));
+ delete window.LP;
+ },
+
+ test_header: function() {
+ // There is a subscription on the duplicate bug.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory([{ bug: {id: 1} }]),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+
+ window.LP = { links: { me: '~' } };
+
+ // A returned node contains the 'other-subscriptions-header'
+ // div with the link.
+ var node = module._get_other_descriptions_node(info);
+ var header = node.one('#other-subscriptions-header');
+ Y.Assert.isNotUndefined(header);
+ var link = header.one('a');
+ Y.Assert.areEqual('Other subscriptions', link.get('text'));
+
+ delete window.LP;
+ },
+
+ test_header_slideout: function() {
+ // Clicking on the header slides-out the box, and
+ // clicking it again slides it back in.
+ var info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory([{ bug: {id: 1} }]),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ count: 1
+ };
+
+ window.LP = { links: { me: '~' } };
+
+ // A returned node contains the 'other-subscriptions-header'
+ // div with the link.
+ var node = module._get_other_descriptions_node(info);
+ var link = node.one('#other-subscriptions-header a');
+ var list = node.one('#other-subscriptions-list');
+
+ // Initially, the list is hidden.
+ Y.Assert.isTrue(link.hasClass('treeCollapsed'));
+ Y.Assert.isTrue(list.hasClass('lazr-closed'));
+ Y.Assert.areEqual('none', list.getStyle('display'));
+
+ // Clicking the link slides out the list of other subscriptions.
link.simulate('click');
-
- this.wait(function() {
- Y.Assert.isTrue(link.hasClass('treeCollapsed'));
- Y.Assert.isFalse(link.hasClass('treeExpanded'));
- Y.Assert.isTrue(list.hasClass('lazr-closed'));
+ this.wait(function() {
+ Y.Assert.isFalse(link.hasClass('treeCollapsed'));
+ Y.Assert.isTrue(link.hasClass('treeExpanded'));
+ Y.Assert.isFalse(list.hasClass('lazr-closed'));
+ Y.Assert.areNotEqual('none', list.getStyle('display'));
+
+ // Clicking it again, slides it back in.
+ // It has to be nested inside 'wait' because we need
+ // to wait for the first click to "finish".
+ link.simulate('click');
+
+ this.wait(function() {
+ Y.Assert.isTrue(link.hasClass('treeCollapsed'));
+ Y.Assert.isFalse(link.hasClass('treeExpanded'));
+ Y.Assert.isTrue(list.hasClass('lazr-closed'));
+ delete window.LP;
+ }, 50);
+ }, 50);
+ }
+
+ }));
+
+ /**
+ * Test for show_subscription_description() method.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test showing of subscription descriptions.',
+
+ setUp: function() {
+ this.content_node = Y.Node.create('<div></div>')
+ .set('id', 'description-container');
+ this.parent_node = Y.one('#test-root');
+ this.parent_node.appendChild(this.content_node);
+ this.config = {
+ description_box: '#description-container'
+ };
+ },
+
+ tearDown: function() {
+ this.parent_node.empty(true);
+ delete this.config;
+ },
+
+ test_no_subscriptions: function() {
+ // With no subscriptions, a simple description of that state
+ // is added.
+ this.config.subscription_info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory(),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ };
+ window.LP = { links: {},
+ cache: {} };
+ module.show_subscription_description(this.config);
+ this.wait(function() {
+ Y.Assert.areEqual(
+ 1, this.content_node.all('#direct-subscription').size());
+ Y.Assert.areEqual(
+ 0, this.content_node.all('#other-subscriptions').size());
+ }, 10);
+ delete window.LP;
+ },
+
+ test_combined_subscriptions: function() {
+ // With both direct and implicit subscriptions,
+ // we get a simple description and a node with other descriptions.
+ this.config.subscription_info = {
+ direct: _constructCategory([{ bug: {id:1} }]),
+ from_duplicate: _constructCategory([{ bug: {id:2} }]),
+ as_assignee: _constructCategory([{ bug: {id:3} }]),
+ as_owner: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ };
+ window.LP = { cache: { context: { web_link: '/' } },
+ links: { me: '~' } };
+ module.show_subscription_description(this.config);
+ this.wait(function() {
+ Y.Assert.areEqual(
+ 1, this.content_node.all('#direct-subscription').size());
+ Y.Assert.areEqual(
+ 1, this.content_node.all('#other-subscriptions').size());
delete window.LP;
- }, 50);
- }, 50);
- }
-
-}));
-
-/**
- * Test for show_subscription_description() method.
- */
-suite.add(new Y.Test.Case({
- name: 'Test showing of subscription descriptions.',
-
- setUp: function() {
- this.content_node = Y.Node.create('<div></div>')
- .set('id', 'description-container');
- this.parent_node = Y.one('#test-root');
- this.parent_node.appendChild(this.content_node);
- this.config = {
- description_box: '#description-container'
- };
- },
-
- tearDown: function() {
- this.parent_node.empty(true);
- delete this.config;
- },
-
- test_no_subscriptions: function() {
- // With no subscriptions, a simple description of that state
- // is added.
- this.config.subscription_info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory(),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- bug_id: 1,
- count: 0
- };
- window.LP = { links: {},
- cache: {} };
- module.show_subscription_description(this.config);
- this.wait(function() {
- Y.Assert.areEqual(
- 1, this.content_node.all('#direct-subscription').size());
- Y.Assert.areEqual(
- 0, this.content_node.all('#other-subscriptions').size());
- }, 10);
- delete window.LP;
- },
-
- test_combined_subscriptions: function() {
- // With both direct and implicit subscriptions,
- // we get a simple description and a node with other descriptions.
- this.config.subscription_info = {
- direct: _constructCategory([{ bug: {id:1} }]),
- from_duplicate: _constructCategory([{ bug: {id:2} }]),
- as_assignee: _constructCategory([{ bug: {id:3} }]),
- as_owner: _constructCategory(),
- bug_id: 1,
- count: 0
- };
- window.LP = { cache: { context: { web_link: '/' } },
- links: { me: '~' } };
- module.show_subscription_description(this.config);
- this.wait(function() {
- Y.Assert.areEqual(
- 1, this.content_node.all('#direct-subscription').size());
- Y.Assert.areEqual(
- 1, this.content_node.all('#other-subscriptions').size());
- delete window.LP;
- }, 10);
- },
-
- test_reference_substitutions: function() {
- // References of the form `subscription-cache-reference-*` get
- // replaced with LP.cache[...] values.
- this.config.subscription_info = {
- reference: 'subscription-cache-reference-X',
- direct: _constructCategory(),
- from_duplicate: _constructCategory(),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- bug_id: 1,
- count: 0
- };
- window.LP = {
- links: {},
- cache: {
- 'subscription-cache-reference-X': 'value'
- }
- };
- module.show_subscription_description(this.config);
- Y.Assert.areEqual(
- 'value',
- this.config.subscription_info.reference);
- delete window.LP;
- }
-
-}));
-
-/**
- * Test for helper method to construct actions text and subscriptions list
- * for duplicate subscriptions:
- * get_unsubscribe_duplicates_text_and_subscriptions()
- */
-suite.add(new Y.Test.Case({
- name: 'Test duplicate actions text and subscriptions list.',
-
- _should: {
- error: {
- test_multiple_teams_fails:
- new Error('We can only unsubscribe a single team from ' +
- 'multiple duplicate bugs.')
- }
- },
-
- setUp: function() {
- window.LP = { cache: { context: { web_link: 'http://test/' } },
- links: { me: '~' } };
- },
-
- tearDown: function() {
- delete window.LP;
- },
-
- test_yourself_single_bug: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug/' } } ] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- Y.Assert.areEqual('Unsubscribe yourself from the duplicate',
- data.text);
- Y.Assert.areEqual(1, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
- Y.Assert.areEqual('http://bug/', sub.bug);
- },
-
- test_yourself_multiple_bug: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug1/' } },
- { self: { self_link: 'http://bug2/' } }] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- Y.Assert.areEqual('Unsubscribe yourself from all duplicates',
- data.text);
- Y.Assert.areEqual(2, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
- Y.Assert.areEqual('http://bug1/', sub.bug);
-
- sub = data.subscriptions[1];
- Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
- Y.Assert.areEqual('http://bug2/', sub.bug);
- },
-
- test_team_single_bug: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
- teams: [ { self: { self_link: 'http://team/' } } ] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- Y.Assert.areEqual('Unsubscribe this team from the duplicate',
- data.text);
- Y.Assert.areEqual(1, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual('http://team/', sub.subscriber);
- Y.Assert.areEqual('http://bug/', sub.bug);
- },
-
- test_team_multiple_bugs: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug1/' } },
- { self: { self_link: 'http://bug2/' } }],
- teams: [ { self: { self_link: 'http://team/' } } ] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- Y.Assert.areEqual('Unsubscribe this team from all duplicates',
- data.text);
- Y.Assert.areEqual(2, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual('http://team/', sub.subscriber);
- Y.Assert.areEqual('http://bug1/', sub.bug);
-
- sub = data.subscriptions[1];
- Y.Assert.areEqual('http://team/', sub.subscriber);
- Y.Assert.areEqual('http://bug2/', sub.bug);
- },
-
- test_multiple_teams_fails: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
- teams: [ { self: { self_link: 'http://team1/' } },
- { self: { self_link: 'http://team2/' } }] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- }
-
-}));
-
-/**
- * Test for helper method to get modified object links:
- * add_url_element_to_links()
- */
-suite.add(new Y.Test.Case({
- name: 'Test add_url_element_to_links helper.',
-
- compare_object_links: function (first, second) {
- return first.title === second.title &&
- first.url === second.url &&
- first.self === second.self;
- },
-
- test_single_link: function () {
- var self = 'object stand-in',
- original = {
- title: 'Rutebega',
- url: 'http://example.net/kumquat',
- self: self
- },
- modified = module._add_url_element_to_links(
- [original], '/avocado');
- Y.ArrayAssert.itemsAreEquivalent(
- [{title: 'Rutebega',
- url: 'http://example.net/kumquat/avocado',
- self: self}],
- modified,
- this.compare_object_links);
- // The original was not modified.
- Y.Assert.areEqual(original.url, 'http://example.net/kumquat');
- },
-
- test_multiple_link: function () {
- var self1 = 'object stand-in 1',
- original1 = {
- title: 'Rutebega',
- url: 'http://example.net/kumquat',
- self: self1
- },
- self2 = 'object stand-in 2',
- original2 = {
- title: 'Shazam',
- url: 'http://example.net/abracadabra',
- self: self2
- },
- modified = module._add_url_element_to_links(
- [original1, original2], '/avocado');
- Y.ArrayAssert.itemsAreEquivalent(
- [{title: 'Rutebega',
- url: 'http://example.net/kumquat/avocado',
- self: self1},
- {title: 'Shazam',
- url: 'http://example.net/abracadabra/avocado',
- self: self2}],
- modified,
- this.compare_object_links);
- // The originals were not modified.
- Y.Assert.areEqual(original1.url, 'http://example.net/kumquat');
- Y.Assert.areEqual(original2.url, 'http://example.net/abracadabra');
- }
-
-}));
-
-/**
- * Test for helper method to construct action "unsubscribe" node:
- * get_node_for_unsubscribing()
- */
-suite.add(new Y.Test.Case({
- name: 'Test duplicate actions text and subscriptions list.',
-
- setUp: function () {
- module._lp_client = new LPClient();
- this.wrapper_node = Y.Node.create(
- '<div class="subscription-description"></div>');
- Y.one('body').appendChild(this.wrapper_node);
- },
-
- tearDown: function () {
- delete module._lp_client;
- this.wrapper_node.remove();
- var error_overlay = Y.one('.yui3-lazr-formoverlay');
- if (Y.Lang.isValue(error_overlay)) {
- error_overlay.remove();
- }
- },
-
- get_subscriptions: function () {
- // Usually multiple subscriptions will share a subscriber. This
- // function under test does not actually care, so we make it possible
- // to distinguish between the first and the second.
- return [{subscriber: 'http://example.net/~person1',
- bug: 'http://example.net/firefox/bug/1'},
- {subscriber: 'http://example.net/~person2',
- bug: 'http://example.net/firefox/bug/2'}];
- },
-
- test_node_basic: function () {
- var node = module._get_node_for_unsubscribing(
- 'Rutebega', this.get_subscriptions());
- Y.Assert.areEqual(node.get('text'), 'Rutebega');
- Y.Assert.isTrue(node.hasClass('sprite'));
- Y.Assert.isTrue(node.hasClass('modify'));
- Y.Assert.isTrue(node.hasClass('remove'));
- },
-
- test_one_subscription_success: function () {
- var subscriptions = this.get_subscriptions();
- subscriptions.pop();
- Y.Assert.areEqual(subscriptions.length, 1);
- var node = module._get_node_for_unsubscribing(
- 'Rutebega', subscriptions);
- module._lp_client.named_post.args = [];
- module._lp_client.named_post.halt = true;
- Y.one('.subscription-description').appendChild(node);
- node.simulate('click');
- // Now it is as if we are waiting for the server to reply. The
- // spinner spins.
- Y.Assert.isTrue(node.hasClass('spinner'));
- Y.Assert.isFalse(node.hasClass('remove'));
- // Now the server replies back with a success.
- module._lp_client.named_post.resume();
- // We have no spinner.
- Y.Assert.isTrue(node.hasClass('remove'));
- Y.Assert.isFalse(node.hasClass('spinner'));
- // The subscriptions array is empty.
- Y.Assert.areEqual(subscriptions.length, 0);
- // We called unsubscribe on the server once, with the right arguments.
- Y.Assert.areEqual(module._lp_client.received.length, 1);
- Y.Assert.areEqual(module._lp_client.received[0][0], 'named_post');
- var args = module._lp_client.received[0][1];
- Y.Assert.areEqual(args[0], 'http://example.net/firefox/bug/1');
- Y.Assert.areEqual(args[1], 'unsubscribe');
- Y.Assert.areEqual(args[2].parameters.person,
- 'http://example.net/~person1');
- // The parent node is gone, after giving some time to collapse.
- this.wait(
- function () {
- Y.Assert.isNull(Y.one('.subscription-description'));
- },
- 50
- );
- },
-
- test_two_subscriptions_success: function () {
- var subscriptions = this.get_subscriptions();
- Y.Assert.areEqual(subscriptions.length, 2);
- var node = module._get_node_for_unsubscribing(
- 'Rutebega', subscriptions);
- module._lp_client.named_post.args = [];
- Y.one('.subscription-description').appendChild(node);
- node.simulate('click');
- // The subscriptions array is empty.
- Y.Assert.areEqual(subscriptions.length, 0);
- // We called unsubscribe on the server twice, once for each
- // subscription.
- Y.Assert.areEqual(module._lp_client.received.length, 2);
- },
-
- test_failure: function () {
- var subscriptions = this.get_subscriptions();
- var node = module._get_node_for_unsubscribing(
- 'Rutebega', subscriptions);
- module._lp_client.named_post.fail = true;
- module._lp_client.named_post.args = [
- true,
- {status: 400, responseText: 'Rutebegas!'}];
- module._lp_client.named_post.halt = true;
- Y.one('.subscription-description').appendChild(node);
- node.simulate('click');
- // Right now, this is as if we are waiting for the server to
- // reply. The link is spinning.
- Y.Assert.isTrue(node.hasClass('spinner'));
- Y.Assert.isFalse(node.hasClass('remove'));
- // Now the server replies with an error.
- module._lp_client.named_post.resume();
- // We have no spinner.
- Y.Assert.isTrue(node.hasClass('remove'));
- Y.Assert.isFalse(node.hasClass('spinner'));
- // The page has rendered the error overlay.
- var error_box = Y.one('.yui3-lazr-formoverlay-errors');
- // The way the LP error display works now is that it flashes the
- // problem area red for 1 second (the lp.anim default), and
- // *then* shows the overlay.
- this.wait(
- function () {
- Y.Assert.areEqual(
- "The following errors were encountered: Rutebegas!",
- error_box.get('text'));
- },
- 1100
- );
- }
-
-}));
-
-
-/**
- * Test for helper method to construct actions text and subscriptions list
- * for team subscriptions:
- * get_team_unsubscribe_text_and_subscriptions()
- */
-suite.add(new Y.Test.Case({
- name: 'Test duplicate actions text and subscriptions list.',
-
- _should: {
- error: {
- test_multiple_teams_fails:
- new Error('We can only unsubscribe a single team from ' +
- 'multiple duplicate bugs.')
- }
- },
-
- setUp: function() {
- window.LP = { cache: { context: { bug_link: 'http://bug/' } },
- links: { me: '~' } };
- },
-
- tearDown: function() {
- delete window.LP;
- },
-
- test_single_team: function() {
- // There is a single team you admin that is subscribed to the bug.
- var args = { teams: [ { self: { self_link: 'http://team/' } } ] };
- var data = module._get_team_unsubscribe_text_and_subscriptions(args);
- Y.Assert.areEqual('Unsubscribe this team', data.text);
- Y.Assert.areEqual(1, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual('http://team/', sub.subscriber);
- Y.Assert.areEqual('http://bug/', sub.bug);
- },
-
- test_multiple_teams: function() {
- // There are multiple teams you admin that are subscribed to the bug.
- var args = { teams: [ { self: { self_link: 'http://team1/' } },
- { self: { self_link: 'http://team2/' } }] };
- var data = module._get_team_unsubscribe_text_and_subscriptions(args);
- Y.Assert.areEqual('Unsubscribe all of these teams', data.text);
- Y.Assert.areEqual(2, data.subscriptions.length);
- var sub = data.subscriptions[0];
- Y.Assert.areEqual('http://team1/', sub.subscriber);
- Y.Assert.areEqual('http://bug/', sub.bug);
-
- sub = data.subscriptions[1];
- Y.Assert.areEqual('http://team2/', sub.subscriber);
- Y.Assert.areEqual('http://bug/', sub.bug);
- },
-
- test_multiple_teams_fails: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
- teams: [ { self: { self_link: 'http://team1/' } },
- { self: { self_link: 'http://team2/' } }] };
- var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
- args);
- }
-
-}));
-
-/**
- * Test for actions node construction.
- */
-suite.add(new Y.Test.Case({
- name: 'Test node construction for actions.',
-
- setUp: function() {
- window.LP = { cache: { context: { web_link: 'http://test/' } },
- links: { me: '~' } };
- },
-
- tearDown: function() {
- delete window.LP;
- },
-
- test_change_assignees: function() {
- // Change assignees action.
- var link = module._actions.CHANGE_ASSIGNEES();
- Y.Assert.areEqual('Change assignees for this bug', link.get('text'));
- Y.Assert.areEqual('http://test/', link.get('href'));
- },
-
- test_unsubscribe_duplicates: function() {
- // There is a single duplicate bug you are subscribed to.
- var args = { bugs: [ { self: { self_link: 'http://bug/' } } ] };
- var node = module._actions.UNSUBSCRIBE_DUPLICATES(args);
- Y.Assert.areEqual('Unsubscribe yourself from the duplicate',
- node.get('text'));
- Y.Assert.isTrue(node.hasClass('js-action'));
- Y.Assert.isTrue(node.hasClass('remove'));
- },
-
- test_set_bug_supervisor: function() {
- // You are the pillar owner and can set the supervisor.
- var args = { pillar: { title: 'Project',
- web_link: 'http://pillar' } };
- var node = module._actions.SET_BUG_SUPERVISOR(args);
- Y.Assert.areEqual('Set the bug supervisor for Project',
- node.get('text'));
- Y.Assert.areEqual('http://pillar/+bugsupervisor', node.get('href'));
- },
-
- test_contact_teams: function() {
- // You are only a member of the subscribed team,
- // so you need to contact the team admin to unsubscribe.
- var args = { teams: [{ title: 'Team <1>',
- url: 'http://team',
- self: 'self' }] };
- var node = module._actions.CONTACT_TEAMS(args);
- Y.Assert.areEqual(
- 'Contact ' +
- '<a href="http://team/+contactuser";>Team <1></a>' +
- ' to request the administrators make a change',
- node.get('innerHTML'));
- var link = node.one('a');
- Y.Assert.areEqual('http://team/+contactuser', link.get('href'));
- }
-
-}));
-
-
-/**
- * Tests for make_action_link.
- */
-suite.add(new Y.Test.Case({
- name: 'Test make_action_link.',
-
- setUp: function() {
- window.LP = {cache: {},
- links: {}};
- module._lp_client = new LPClient();
- // Set up the required elements.
- this.content_node = Y.Node.create('<div></div>')
- .set('id', 'description-container');
- this.parent_node = Y.one('#test-root');
- this.parent_node.appendChild(this.content_node);
- this.config = {
- description_box: '#description-container'
- };
- this.config.subscription_info = {
- direct: _constructCategory(),
- from_duplicate: _constructCategory(),
- as_assignee: _constructCategory(),
- as_owner: _constructCategory(),
- bug_id: 1,
- count: 0
- };
- },
-
- tearDown: function() {
- delete window.LP;
- delete module._lp_client;
- this.content_node.remove();
- this.parent_node.empty(true);
- delete this.config;
- var error_overlay = Y.one('.yui3-lazr-formoverlay');
- if (Y.Lang.isValue(error_overlay)) {
- error_overlay.remove();
- }
- },
-
- test_link_parts: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'a_method', {});
- Y.Assert.isTrue(node.hasClass('some-sprite'));
- Y.Assert.isTrue(node.hasClass('js-action'));
- Y.Assert.isTrue(node.hasClass('sprite'));
- Y.Assert.isTrue(node.hasClass('modify'));
- Y.Assert.isFalse(node.hasClass('spinner'));
- Y.Assert.areEqual('some text', node.get('text'));
- },
-
- test_on_click_handles_spinner: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'a_method', {});
- window.LP.cache = {context: {bug_link: 'http://bug/'},
- bug_subscription_info: {
- direct: {personal: []}
- }
- };
- module.show_subscription_description(this.config);
- // Setup the LP client to simulate a response.
- //module._lp_client.named_post.halt = true;
- module._lp_client.named_post.args = [];
- module._lp_client.named_post.halt = true;
- node.simulate('click');
- this.wait(function() {
- Y.Assert.isTrue(node.hasClass('spinner'));
- }, 10);
- module._lp_client.named_post.resume();
- Y.Assert.isFalse(node.hasClass('spinner'));
- },
-
- test_on_subscribe_updates_info: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'subscribe', {});
- var bug_link = 'http://example.net/firefox/bug/1';
- window.LP.cache = {context: {bug_link: bug_link},
- bug_subscription_info: {
- direct: {personal: [],
- count:0}
- }
- };
- module.show_subscription_description(this.config);
- // Simulated return values from the named_post call.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- module._lp_client.named_post.args = [
- {getAttrs: function () {
- return sub.subscription;
- }}];
- // Before clicking on the link the direct
- // subscription count is 0.
- Y.Assert.areEqual(
- 0, window.LP.cache.bug_subscription_info.direct.count);
- node.simulate('click');
- this.wait(function() {
- // And afterwards it has been incremented to 1.
- Y.Assert.areEqual(
- 1, window.LP.cache.bug_subscription_info.direct.count);
- }, 10);
- },
-
- test_on_mute_updates_info: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'mute', {});
- var bug_link = 'http://example.net/firefox/bug/1';
- module.show_subscription_description(this.config);
- // Simulated return values from the named_post call.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- module._lp_client.named_post.args = [
- {getAttrs: function () {
- return sub.subscription;
- }}];
- window.LP.cache = {context: {bug_link: bug_link},
- bug_subscription_info: {
- direct: {personal: [sub],
- count:1}
- }
- };
- // Before clicking on the link the direct
- // subscription count is 1.
- Y.Assert.areEqual(
- 1, window.LP.cache.bug_subscription_info.direct.count);
- node.simulate('click');
- this.wait(function() {
- // And afterwards it is still 1.
- Y.Assert.areEqual(
- 1, window.LP.cache.bug_subscription_info.direct.count);
- }, 10);
- },
-
- test_on_unsubscribe_updates_info: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'unsubscribe', {});
- var bug_link = 'http://example.net/firefox/bug/1';
- window.LP.cache = {context: {bug_link: bug_link},
- bug_subscription_info: {
- direct: {personal: ['fakesub'],
- count:3}
- }
- };
- module.show_subscription_description(this.config);
- // Simulated return values from the named_post call.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- module._lp_client.named_post.args = [];
- // Before clicking on the link the direct
- // subscription count is 3.
- Y.Assert.areEqual(
- 3, window.LP.cache.bug_subscription_info.direct.count);
- node.simulate('click');
- this.wait(function() {
- // And afterwards it has been decremented to 2.
- Y.Assert.areEqual(
- 2, window.LP.cache.bug_subscription_info.direct.count);
- }, 10);
- },
-
- test_on_unmute_updates_info: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'unmute', {});
- var bug_link = 'http://example.net/firefox/bug/1';
- window.LP.cache = {context: {bug_link: bug_link},
- bug_subscription_info: {
- direct: {personal: ['fakesub'],
- count:3}
- }
- };
- module.show_subscription_description(this.config);
- // Simulated return values from the named_post call.
- var sub = {
- bug: {
- 'private': false,
- security_related: false
- },
- principal_is_reporter: false,
- subscription: {bug_notification_level: 'Details'}
- };
- module._lp_client.named_post.args = [];
- // Before clicking on the link the direct subscription count
- // is 3.
- Y.Assert.areEqual(
- 3, window.LP.cache.bug_subscription_info.direct.count);
- node.simulate('click');
- this.wait(function() {
- // And afterwards it has been decremented to 2.
- Y.Assert.areEqual(
- 2, window.LP.cache.bug_subscription_info.direct.count);
- }, 10);
- },
-
- test_fail: function() {
- var node = module._make_action_link(
- 'some text', 'some-sprite', 'unmute', {});
- var bug_link = 'http://example.net/firefox/bug/1';
- window.LP.cache = {context: {bug_link: bug_link},
- bug_subscription_info: {
- direct: {personal: ['fakesub'],
- count:3}
- }
- };
- module._lp_client.named_post.fail = true;
- module._lp_client.named_post.args = [
- true,
- {status: 400, responseText: 'Oopsie!'}];
- module.show_subscription_description(this.config);
- node.simulate('click');
- this.wait(
- function () {
- var error_box = Y.one('.yui3-lazr-formoverlay-errors');
- Y.Assert.isNotNull(error_box);
- Y.Assert.areEqual(
- "The following errors were encountered: Oopsie!",
- error_box.get('text'));
- }, 1100);
- }
-
-}));
-
-
-var handle_complete = function(data) {
- window.status = '::::' + JSON.stringify(data);
- };
-Y.Test.Runner.on('complete', handle_complete);
-Y.Test.Runner.add(suite);
-
-var console = new Y.Console({newestOnTop: false});
-console.render('#log');
-
-Y.on('domready', function() {
- Y.Test.Runner.run();
-});
+ }, 10);
+ },
+
+ test_reference_substitutions: function() {
+ // References of the form `subscription-cache-reference-*` get
+ // replaced with LP.cache[...] values.
+ this.config.subscription_info = {
+ reference: 'subscription-cache-reference-X',
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory(),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ };
+ window.LP = {
+ links: {},
+ cache: {
+ 'subscription-cache-reference-X': 'value'
+ }
+ };
+ module.show_subscription_description(this.config);
+ Y.Assert.areEqual(
+ 'value',
+ this.config.subscription_info.reference);
+ delete window.LP;
+ }
+
+ }));
+
+ /**
+ * Test for helper method to construct actions text and subscriptions list
+ * for duplicate subscriptions:
+ * get_unsubscribe_duplicates_text_and_subscriptions()
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test duplicate actions text and subscriptions list.',
+
+ _should: {
+ error: {
+ test_multiple_teams_fails:
+ new Error('We can only unsubscribe a single team from ' +
+ 'multiple duplicate bugs.')
+ }
+ },
+
+ setUp: function() {
+ window.LP = { cache: { context: { web_link: 'http://test/' } },
+ links: { me: '~' } };
+ },
+
+ tearDown: function() {
+ delete window.LP;
+ },
+
+ test_yourself_single_bug: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug/' } } ] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ Y.Assert.areEqual('Unsubscribe yourself from the duplicate',
+ data.text);
+ Y.Assert.areEqual(1, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
+ Y.Assert.areEqual('http://bug/', sub.bug);
+ },
+
+ test_yourself_multiple_bug: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug1/' } },
+ { self: { self_link: 'http://bug2/' } }] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ Y.Assert.areEqual('Unsubscribe yourself from all duplicates',
+ data.text);
+ Y.Assert.areEqual(2, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
+ Y.Assert.areEqual('http://bug1/', sub.bug);
+
+ sub = data.subscriptions[1];
+ Y.Assert.areEqual(window.LP.links.me, sub.subscriber);
+ Y.Assert.areEqual('http://bug2/', sub.bug);
+ },
+
+ test_team_single_bug: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
+ teams: [ { self: { self_link: 'http://team/' } } ] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ Y.Assert.areEqual('Unsubscribe this team from the duplicate',
+ data.text);
+ Y.Assert.areEqual(1, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual('http://team/', sub.subscriber);
+ Y.Assert.areEqual('http://bug/', sub.bug);
+ },
+
+ test_team_multiple_bugs: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug1/' } },
+ { self: { self_link: 'http://bug2/' } }],
+ teams: [ { self: { self_link: 'http://team/' } } ] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ Y.Assert.areEqual('Unsubscribe this team from all duplicates',
+ data.text);
+ Y.Assert.areEqual(2, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual('http://team/', sub.subscriber);
+ Y.Assert.areEqual('http://bug1/', sub.bug);
+
+ sub = data.subscriptions[1];
+ Y.Assert.areEqual('http://team/', sub.subscriber);
+ Y.Assert.areEqual('http://bug2/', sub.bug);
+ },
+
+ test_multiple_teams_fails: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
+ teams: [ { self: { self_link: 'http://team1/' } },
+ { self: { self_link: 'http://team2/' } }] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ }
+
+ }));
+
+ /**
+ * Test for helper method to get modified object links:
+ * add_url_element_to_links()
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test add_url_element_to_links helper.',
+
+ compare_object_links: function (first, second) {
+ return first.title === second.title &&
+ first.url === second.url &&
+ first.self === second.self;
+ },
+
+ test_single_link: function () {
+ var self = 'object stand-in',
+ original = {
+ title: 'Rutebega',
+ url: 'http://example.net/kumquat',
+ self: self
+ },
+ modified = module._add_url_element_to_links(
+ [original], '/avocado');
+ Y.ArrayAssert.itemsAreEquivalent(
+ [{title: 'Rutebega',
+ url: 'http://example.net/kumquat/avocado',
+ self: self}],
+ modified,
+ this.compare_object_links);
+ // The original was not modified.
+ Y.Assert.areEqual(original.url, 'http://example.net/kumquat');
+ },
+
+ test_multiple_link: function () {
+ var self1 = 'object stand-in 1',
+ original1 = {
+ title: 'Rutebega',
+ url: 'http://example.net/kumquat',
+ self: self1
+ },
+ self2 = 'object stand-in 2',
+ original2 = {
+ title: 'Shazam',
+ url: 'http://example.net/abracadabra',
+ self: self2
+ },
+ modified = module._add_url_element_to_links(
+ [original1, original2], '/avocado');
+ Y.ArrayAssert.itemsAreEquivalent(
+ [{title: 'Rutebega',
+ url: 'http://example.net/kumquat/avocado',
+ self: self1},
+ {title: 'Shazam',
+ url: 'http://example.net/abracadabra/avocado',
+ self: self2}],
+ modified,
+ this.compare_object_links);
+ // The originals were not modified.
+ Y.Assert.areEqual(original1.url, 'http://example.net/kumquat');
+ Y.Assert.areEqual(original2.url, 'http://example.net/abracadabra');
+ }
+
+ }));
+
+ /**
+ * Test for helper method to construct action "unsubscribe" node:
+ * get_node_for_unsubscribing()
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test duplicate actions text and subscriptions list.',
+
+ setUp: function () {
+ module._lp_client = new Y.lp.testing.helpers.LPClient();
+ this.wrapper_node = Y.Node.create(
+ '<div class="subscription-description"></div>');
+ Y.one('body').appendChild(this.wrapper_node);
+ },
+
+ tearDown: function () {
+ delete module._lp_client;
+ this.wrapper_node.remove();
+ var error_overlay = Y.one('.yui3-lazr-formoverlay');
+ if (Y.Lang.isValue(error_overlay)) {
+ error_overlay.remove();
+ }
+ },
+
+ get_subscriptions: function () {
+ // Usually multiple subscriptions will share a subscriber. This
+ // function under test does not actually care, so we make it possible
+ // to distinguish between the first and the second.
+ return [{subscriber: 'http://example.net/~person1',
+ bug: 'http://example.net/firefox/bug/1'},
+ {subscriber: 'http://example.net/~person2',
+ bug: 'http://example.net/firefox/bug/2'}];
+ },
+
+ test_node_basic: function () {
+ var node = module._get_node_for_unsubscribing(
+ 'Rutebega', this.get_subscriptions());
+ Y.Assert.areEqual(node.get('text'), 'Rutebega');
+ Y.Assert.isTrue(node.hasClass('sprite'));
+ Y.Assert.isTrue(node.hasClass('modify'));
+ Y.Assert.isTrue(node.hasClass('remove'));
+ },
+
+ test_one_subscription_success: function () {
+ var subscriptions = this.get_subscriptions();
+ subscriptions.pop();
+ Y.Assert.areEqual(subscriptions.length, 1);
+ var node = module._get_node_for_unsubscribing(
+ 'Rutebega', subscriptions);
+ module._lp_client.named_post.args = [];
+ module._lp_client.named_post.halt = true;
+ Y.one('.subscription-description').appendChild(node);
+ node.simulate('click');
+ // Now it is as if we are waiting for the server to reply. The
+ // spinner spins.
+ Y.Assert.isTrue(node.hasClass('spinner'));
+ Y.Assert.isFalse(node.hasClass('remove'));
+ // Now the server replies back with a success.
+ module._lp_client.named_post.resume();
+ // We have no spinner.
+ Y.Assert.isTrue(node.hasClass('remove'));
+ Y.Assert.isFalse(node.hasClass('spinner'));
+ // The subscriptions array is empty.
+ Y.Assert.areEqual(subscriptions.length, 0);
+ // We called unsubscribe on the server once, with the right arguments.
+ Y.Assert.areEqual(module._lp_client.received.length, 1);
+ Y.Assert.areEqual(module._lp_client.received[0][0], 'named_post');
+ var args = module._lp_client.received[0][1];
+ Y.Assert.areEqual(args[0], 'http://example.net/firefox/bug/1');
+ Y.Assert.areEqual(args[1], 'unsubscribe');
+ Y.Assert.areEqual(args[2].parameters.person,
+ 'http://example.net/~person1');
+ // The parent node is gone, after giving some time to collapse.
+ this.wait(
+ function () {
+ Y.Assert.isNull(Y.one('.subscription-description'));
+ },
+ 50
+ );
+ },
+
+ test_two_subscriptions_success: function () {
+ var subscriptions = this.get_subscriptions();
+ Y.Assert.areEqual(subscriptions.length, 2);
+ var node = module._get_node_for_unsubscribing(
+ 'Rutebega', subscriptions);
+ module._lp_client.named_post.args = [];
+ Y.one('.subscription-description').appendChild(node);
+ node.simulate('click');
+ // The subscriptions array is empty.
+ Y.Assert.areEqual(subscriptions.length, 0);
+ // We called unsubscribe on the server twice, once for each
+ // subscription.
+ Y.Assert.areEqual(module._lp_client.received.length, 2);
+ },
+
+ test_failure: function () {
+ var subscriptions = this.get_subscriptions();
+ var node = module._get_node_for_unsubscribing(
+ 'Rutebega', subscriptions);
+ module._lp_client.named_post.fail = true;
+ module._lp_client.named_post.args = [
+ true,
+ {status: 400, responseText: 'Rutebegas!'}];
+ module._lp_client.named_post.halt = true;
+ Y.one('.subscription-description').appendChild(node);
+ node.simulate('click');
+ // Right now, this is as if we are waiting for the server to
+ // reply. The link is spinning.
+ Y.Assert.isTrue(node.hasClass('spinner'));
+ Y.Assert.isFalse(node.hasClass('remove'));
+ // Now the server replies with an error.
+ module._lp_client.named_post.resume();
+ // We have no spinner.
+ Y.Assert.isTrue(node.hasClass('remove'));
+ Y.Assert.isFalse(node.hasClass('spinner'));
+ // The page has rendered the error overlay.
+ var error_box = Y.one('.yui3-lazr-formoverlay-errors');
+ // The way the LP error display works now is that it flashes the
+ // problem area red for 1 second (the lp.anim default), and
+ // *then* shows the overlay.
+ this.wait(
+ function () {
+ Y.Assert.areEqual(
+ "The following errors were encountered: Rutebegas!",
+ error_box.get('text'));
+ },
+ 1100
+ );
+ }
+
+ }));
+
+ /**
+ * Test for helper method to construct actions text and subscriptions list
+ * for team subscriptions:
+ * get_team_unsubscribe_text_and_subscriptions()
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test duplicate actions text and subscriptions list.',
+
+ _should: {
+ error: {
+ test_multiple_teams_fails:
+ new Error('We can only unsubscribe a single team from ' +
+ 'multiple duplicate bugs.')
+ }
+ },
+
+ setUp: function() {
+ window.LP = { cache: { context: { bug_link: 'http://bug/' } },
+ links: { me: '~' } };
+ },
+
+ tearDown: function() {
+ delete window.LP;
+ },
+
+ test_single_team: function() {
+ // There is a single team you admin that is subscribed to the bug.
+ var args = { teams: [ { self: { self_link: 'http://team/' } } ] };
+ var data = module._get_team_unsubscribe_text_and_subscriptions(args);
+ Y.Assert.areEqual('Unsubscribe this team', data.text);
+ Y.Assert.areEqual(1, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual('http://team/', sub.subscriber);
+ Y.Assert.areEqual('http://bug/', sub.bug);
+ },
+
+ test_multiple_teams: function() {
+ // There are multiple teams you admin that are subscribed to the bug.
+ var args = { teams: [ { self: { self_link: 'http://team1/' } },
+ { self: { self_link: 'http://team2/' } }] };
+ var data = module._get_team_unsubscribe_text_and_subscriptions(args);
+ Y.Assert.areEqual('Unsubscribe all of these teams', data.text);
+ Y.Assert.areEqual(2, data.subscriptions.length);
+ var sub = data.subscriptions[0];
+ Y.Assert.areEqual('http://team1/', sub.subscriber);
+ Y.Assert.areEqual('http://bug/', sub.bug);
+
+ sub = data.subscriptions[1];
+ Y.Assert.areEqual('http://team2/', sub.subscriber);
+ Y.Assert.areEqual('http://bug/', sub.bug);
+ },
+
+ test_multiple_teams_fails: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug/' } } ],
+ teams: [ { self: { self_link: 'http://team1/' } },
+ { self: { self_link: 'http://team2/' } }] };
+ var data = module._get_unsubscribe_duplicates_text_and_subscriptions(
+ args);
+ }
+
+ }));
+
+ /**
+ * Test for actions node construction.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test node construction for actions.',
+
+ setUp: function() {
+ window.LP = { cache: { context: { web_link: 'http://test/' } },
+ links: { me: '~' } };
+ },
+
+ tearDown: function() {
+ delete window.LP;
+ },
+
+ test_change_assignees: function() {
+ // Change assignees action.
+ var link = module._actions.CHANGE_ASSIGNEES();
+ Y.Assert.areEqual('Change assignees for this bug', link.get('text'));
+ Y.Assert.areEqual('http://test/', link.get('href'));
+ },
+
+ test_unsubscribe_duplicates: function() {
+ // There is a single duplicate bug you are subscribed to.
+ var args = { bugs: [ { self: { self_link: 'http://bug/' } } ] };
+ var node = module._actions.UNSUBSCRIBE_DUPLICATES(args);
+ Y.Assert.areEqual('Unsubscribe yourself from the duplicate',
+ node.get('text'));
+ Y.Assert.isTrue(node.hasClass('js-action'));
+ Y.Assert.isTrue(node.hasClass('remove'));
+ },
+
+ test_set_bug_supervisor: function() {
+ // You are the pillar owner and can set the supervisor.
+ var args = { pillar: { title: 'Project',
+ web_link: 'http://pillar' } };
+ var node = module._actions.SET_BUG_SUPERVISOR(args);
+ Y.Assert.areEqual('Set the bug supervisor for Project',
+ node.get('text'));
+ Y.Assert.areEqual('http://pillar/+bugsupervisor', node.get('href'));
+ },
+
+ test_contact_teams: function() {
+ // You are only a member of the subscribed team,
+ // so you need to contact the team admin to unsubscribe.
+ var args = { teams: [{ title: 'Team <1>',
+ url: 'http://team',
+ self: 'self' }] };
+ var node = module._actions.CONTACT_TEAMS(args);
+ Y.Assert.areEqual(
+ 'Contact ' +
+ '<a href="http://team/+contactuser";>Team <1></a>' +
+ ' to request the administrators make a change',
+ node.get('innerHTML'));
+ var link = node.one('a');
+ Y.Assert.areEqual('http://team/+contactuser', link.get('href'));
+ }
+
+ }));
+
+
+ /**
+ * Tests for make_action_link.
+ */
+ tests.suite.add(new Y.Test.Case({
+ name: 'Test make_action_link.',
+
+ setUp: function() {
+ window.LP = {cache: {},
+ links: {}};
+ module._lp_client = new Y.lp.testing.helpers.LPClient();
+ // Set up the required elements.
+ this.content_node = Y.Node.create('<div></div>')
+ .set('id', 'description-container');
+ this.parent_node = Y.one('#test-root');
+ this.parent_node.appendChild(this.content_node);
+ this.config = {
+ description_box: '#description-container'
+ };
+ this.config.subscription_info = {
+ direct: _constructCategory(),
+ from_duplicate: _constructCategory(),
+ as_assignee: _constructCategory(),
+ as_owner: _constructCategory(),
+ bug_id: 1,
+ count: 0
+ };
+ },
+
+ tearDown: function() {
+ delete window.LP;
+ delete module._lp_client;
+ this.content_node.remove();
+ this.parent_node.empty(true);
+ delete this.config;
+ var error_overlay = Y.one('.yui3-lazr-formoverlay');
+ if (Y.Lang.isValue(error_overlay)) {
+ error_overlay.remove();
+ }
+ },
+
+ test_link_parts: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'a_method', {});
+ Y.Assert.isTrue(node.hasClass('some-sprite'));
+ Y.Assert.isTrue(node.hasClass('js-action'));
+ Y.Assert.isTrue(node.hasClass('sprite'));
+ Y.Assert.isTrue(node.hasClass('modify'));
+ Y.Assert.isFalse(node.hasClass('spinner'));
+ Y.Assert.areEqual('some text', node.get('text'));
+ },
+
+ test_on_click_handles_spinner: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'a_method', {});
+ window.LP.cache = {context: {bug_link: 'http://bug/'},
+ bug_subscription_info: {
+ direct: {personal: []}
+ }
+ };
+ module.show_subscription_description(this.config);
+ // Setup the LP client to simulate a response.
+ //module._lp_client.named_post.halt = true;
+ module._lp_client.named_post.args = [];
+ module._lp_client.named_post.halt = true;
+ node.simulate('click');
+ this.wait(function() {
+ Y.Assert.isTrue(node.hasClass('spinner'));
+ }, 10);
+ module._lp_client.named_post.resume();
+ Y.Assert.isFalse(node.hasClass('spinner'));
+ },
+
+ test_on_subscribe_updates_info: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'subscribe', {});
+ var bug_link = 'http://example.net/firefox/bug/1';
+ window.LP.cache = {context: {bug_link: bug_link},
+ bug_subscription_info: {
+ direct: {personal: [],
+ count:0}
+ }
+ };
+ module.show_subscription_description(this.config);
+ // Simulated return values from the named_post call.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ module._lp_client.named_post.args = [
+ {getAttrs: function () {
+ return sub.subscription;
+ }}];
+ // Before clicking on the link the direct
+ // subscription count is 0.
+ Y.Assert.areEqual(
+ 0, window.LP.cache.bug_subscription_info.direct.count);
+ node.simulate('click');
+ this.wait(function() {
+ // And afterwards it has been incremented to 1.
+ Y.Assert.areEqual(
+ 1, window.LP.cache.bug_subscription_info.direct.count);
+ }, 10);
+ },
+
+ test_on_mute_updates_info: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'mute', {});
+ var bug_link = 'http://example.net/firefox/bug/1';
+ module.show_subscription_description(this.config);
+ // Simulated return values from the named_post call.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ module._lp_client.named_post.args = [
+ {getAttrs: function () {
+ return sub.subscription;
+ }}];
+ window.LP.cache = {context: {bug_link: bug_link},
+ bug_subscription_info: {
+ direct: {personal: [sub],
+ count:1}
+ }
+ };
+ // Before clicking on the link the direct
+ // subscription count is 1.
+ Y.Assert.areEqual(
+ 1, window.LP.cache.bug_subscription_info.direct.count);
+ node.simulate('click');
+ this.wait(function() {
+ // And afterwards it is still 1.
+ Y.Assert.areEqual(
+ 1, window.LP.cache.bug_subscription_info.direct.count);
+ }, 10);
+ },
+
+ test_on_unsubscribe_updates_info: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'unsubscribe', {});
+ var bug_link = 'http://example.net/firefox/bug/1';
+ window.LP.cache = {context: {bug_link: bug_link},
+ bug_subscription_info: {
+ direct: {personal: ['fakesub'],
+ count:3}
+ }
+ };
+ module.show_subscription_description(this.config);
+ // Simulated return values from the named_post call.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ module._lp_client.named_post.args = [];
+ // Before clicking on the link the direct
+ // subscription count is 3.
+ Y.Assert.areEqual(
+ 3, window.LP.cache.bug_subscription_info.direct.count);
+ node.simulate('click');
+ this.wait(function() {
+ // And afterwards it has been decremented to 2.
+ Y.Assert.areEqual(
+ 2, window.LP.cache.bug_subscription_info.direct.count);
+ }, 10);
+ },
+
+ test_on_unmute_updates_info: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'unmute', {});
+ var bug_link = 'http://example.net/firefox/bug/1';
+ window.LP.cache = {context: {bug_link: bug_link},
+ bug_subscription_info: {
+ direct: {personal: ['fakesub'],
+ count:3}
+ }
+ };
+ module.show_subscription_description(this.config);
+ // Simulated return values from the named_post call.
+ var sub = {
+ bug: {
+ 'private': false,
+ security_related: false
+ },
+ principal_is_reporter: false,
+ subscription: {bug_notification_level: 'Details'}
+ };
+ module._lp_client.named_post.args = [];
+ // Before clicking on the link the direct subscription count
+ // is 3.
+ Y.Assert.areEqual(
+ 3, window.LP.cache.bug_subscription_info.direct.count);
+ node.simulate('click');
+ this.wait(function() {
+ // And afterwards it has been decremented to 2.
+ Y.Assert.areEqual(
+ 2, window.LP.cache.bug_subscription_info.direct.count);
+ }, 10);
+ },
+
+ test_fail: function() {
+ var node = module._make_action_link(
+ 'some text', 'some-sprite', 'unmute', {});
+ var bug_link = 'http://example.net/firefox/bug/1';
+ window.LP.cache = {context: {bug_link: bug_link},
+ bug_subscription_info: {
+ direct: {personal: ['fakesub'],
+ count:3}
+ }
+ };
+ module._lp_client.named_post.fail = true;
+ module._lp_client.named_post.args = [
+ true,
+ {status: 400, responseText: 'Oopsie!'}];
+ module.show_subscription_description(this.config);
+ node.simulate('click');
+ this.wait(
+ function () {
+ var error_box = Y.one('.yui3-lazr-formoverlay-errors');
+ Y.Assert.isNotNull(error_box);
+ Y.Assert.areEqual(
+ "The following errors were encountered: Oopsie!",
+ error_box.get('text'));
+ }, 1100);
+ }
+
+ }));
+
+}, '0.1', {
+ requires: ['test', 'lp.testing.helpers', 'console',
+ 'lp.bugs.subscription', 'node-event-simulate', 'lazr.effects']
});
=== modified file 'lib/lp/registry/javascript/tests/test_structural_subscription.html'
--- lib/lp/registry/javascript/tests/test_structural_subscription.html 2012-03-14 04:41:36 +0000
+++ lib/lp/registry/javascript/tests/test_structural_subscription.html 2012-06-28 15:32:36 +0000
@@ -21,6 +21,8 @@
<script type="text/javascript"
src="../../../../../build/js/lp/app/testing/testrunner.js"></script>
+ <script type="text/javascript"
+ src="../../../../../build/js/lp/app/testing/helpers.js"></script>
<link rel="stylesheet" href="../../../app/javascript/testing/test.css" />
=== modified file 'lib/lp/registry/javascript/tests/test_structural_subscription.js'
--- lib/lp/registry/javascript/tests/test_structural_subscription.js 2012-06-15 15:55:32 +0000
+++ lib/lp/registry/javascript/tests/test_structural_subscription.js 2012-06-28 15:32:36 +0000
@@ -1,8 +1,8 @@
/* Copyright (c) 2011, Canonical Ltd. All rights reserved. */
-YUI().use('lp.testing.runner', 'test', 'console', 'node',
- 'node-event-simulate', 'lazr.effects',
- 'lp.client', 'lp.registry.structural_subscription', function(Y) {
+YUI().use('lp.testing.runner', 'lp.testing.helpers', 'test', 'console',
+ 'node', 'node-event-simulate', 'lazr.effects', 'lp.client',
+ 'lp.registry.structural_subscription', function(Y) {
var suite = new Y.Test.Suite("Structural subscription overlay tests");
@@ -90,64 +90,13 @@
return original_lp;
}
- function LPClient(){
- if (!(this instanceof LPClient)) {
- throw new Error("Constructor called as a function");
- }
- this.received = [];
- // We create new functions every time because we allow them to be
- // configured.
- this.named_post = function(url, func, config) {
- this._call('named_post', config, arguments);
- };
- this.patch = function(bug_filter, data, config) {
- this._call('patch', config, arguments);
- };
- }
-
- LPClient.prototype._call = function(name, config, args) {
- this.received.push(
- [name, Array.prototype.slice.call(args)]);
- if (!Y.Lang.isValue(args.callee.args)) {
- throw new Error("Set call_args on "+name);
- }
- var do_action = function () {
- if (Y.Lang.isValue(args.callee.fail) && args.callee.fail) {
- config.on.failure.apply(undefined, args.callee.args);
- } else {
- config.on.success.apply(undefined, args.callee.args);
- }
- };
- if (Y.Lang.isValue(args.callee.halt) && args.callee.halt) {
- args.callee.resume = do_action;
- } else {
- do_action();
- }
- };
// DELETE uses Y.io directly as of this writing, so we cannot stub it
// here.
function make_lp_client_stub() {
- return new LPClient();
+ return new Y.lp.testing.helpers.LPClient();
}
- suite.add(new Y.Test.Case({
- name: 'Tests for the fake LP client (LPClient) used in these tests.',
-
- _should: {
- error: {
- test_error_when_used_as_a_function: new Error(
- 'Constructor called as a function')
- }
- },
-
- test_error_when_used_as_a_function: function() {
- // LPClient has a built-in safety to ensure that it's instantiated
- // correctly.
- LPClient();
- }
- }));
-
test_case = new Y.Test.Case({
name: 'structural_subscription_overlay',
=== modified file 'standard_test_template.js'
--- standard_test_template.js 2012-06-27 17:36:32 +0000
+++ standard_test_template.js 2012-06-28 15:32:36 +0000
@@ -1,4 +1,4 @@
-/* Copyright (c) 2012, Canonical Ltd. All rights reserved. */
+/* Copyright (c) 2012 Canonical Ltd. All rights reserved. */
YUI.add('lp.${LIBRARY}.test', function (Y) {
Follow ups
