Launchpad itself

Merge lp:~deryck/launchpad/fix-affects-me-widget-529049 into lp:launchpad

  1. fix-affects-me-widget-529049
  2. Merge into devel
Proposed by Deryck Hodge
Status: Merged
Approved by: Deryck Hodge
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~deryck/launchpad/fix-affects-me-widget-529049
Merge into: lp:launchpad
Diff against target: 130 lines (+18/-50)
5 files modified
lib/canonical/launchpad/javascript/bugs/tests/subscriber.html (+8/-8)
lib/canonical/launchpad/javascript/bugs/tests/test_me_too.html (+7/-7)
lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js (+1/-33)
lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js (+1/-1)
versions.cfg (+1/-1)
To merge this branch: bzr merge lp:~deryck/launchpad/fix-affects-me-widget-529049
Reviewer Review Type Date Requested Status
Francis J. Lacoste (community) release-critical Approve
Abel Deuring (community) Approve
Tim Penhey release-critical Pending
Review via email: mp+20380@code.launchpad.net

Commit message

Update versions.cfg to lazr-js r170 and fix some js unit tests.

To post a comment you must log in.
Revision history for this message
Deryck Hodge (deryck) wrote :

This branch is the second attempt to update to the latest revision in
lazr-js. The first was reverted because the lazr-js change broken the
"affects me too" widget. This has been fixed in lazr-js and now this
branch enables the new version.

I also fixed some tests paths in the js unit tests for bugs. I removed
tests that no longer applied because we removed the flame icon from the
feature.

I've run the Bugs Windmill tests which all pass, but I'll put this
through ec2 before landing, even though it's just UI-related changes.

I would like to have this RC'ed for 10.02 because it allows us to use
our fix for the "changing bug statuses is too easy" bug, which has been
hanging around for awhile causing some pain.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/canonical/launchpad/javascript/bugs/tests/subscriber.html
  versions.cfg
  lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js
  lib/canonical/launchpad/javascript/bugs/tests/test_me_too.html
  lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js

== JSLint notices ==
jslint: No problem found in
'/home/deryck/launchpad/lp-branches/fix-affects-me-widget-529049/lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js'.

jslint: No problem found in
'/home/deryck/launchpad/lp-branches/fix-affects-me-widget-529049/lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js'.

jslint: 2 files to lint.

Revision history for this message
Abel Deuring (adeuring) :
review: Approve
Revision history for this message
Francis J. Lacoste (flacoste) :
review: Approve (release-critical)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/canonical/launchpad/javascript/bugs/tests/subscriber.html'
--- lib/canonical/launchpad/javascript/bugs/tests/subscriber.html 2009-12-08 11:08:43 +0000
+++ lib/canonical/launchpad/javascript/bugs/tests/subscriber.html 2010-03-01 16:50:37 +0000
@@ -4,15 +4,15 @@
44
5 <!-- YUI 3.0 Setup -->5 <!-- YUI 3.0 Setup -->
6 <script type="text/javascript"6 <script type="text/javascript"
7 src="../../../icing/yui/current/build/yui/yui.js"></script>7 src="../../../../../../lazr-js/build/yui/yui/yui.js"></script>
8 <script type="text/javascript"8 <script type="text/javascript"
9 src="../../../icing/lazr/build/lazr.js"></script>9 src="../../../../../../lazr-js/build/lazr.js"></script>
10 <link rel="stylesheet"10 <link rel="stylesheet"
11 href="../../../icing/yui/current/build/cssreset/reset.css"/>11 href="../../../../../../lazr-js/build/yui/cssreset/reset.css"/>
12 <link rel="stylesheet"12 <link rel="stylesheet"
13 href="../../../icing/yui/current/build/cssfonts/fonts.css"/>13 href="../../../../../../lazr-js/build/yui/cssfonts/fonts.css"/>
14 <link rel="stylesheet"14 <link rel="stylesheet"
15 href="../../../icing/yui/current/build/cssbase/base.css"/>15 href="../../../../../../lazr-js/build/yui/cssbase/base.css"/>
16 <link rel="stylesheet"16 <link rel="stylesheet"
17 href="../../test.css" />17 href="../../test.css" />
1818
1919
=== modified file 'lib/canonical/launchpad/javascript/bugs/tests/test_me_too.html'
--- lib/canonical/launchpad/javascript/bugs/tests/test_me_too.html 2009-11-25 19:21:59 +0000
+++ lib/canonical/launchpad/javascript/bugs/tests/test_me_too.html 2010-03-01 16:50:37 +0000
@@ -4,15 +4,15 @@
4 <title>Status Editor</title>4 <title>Status Editor</title>
55
6 <!-- YUI 3.0 Setup -->6 <!-- YUI 3.0 Setup -->
7 <script type="text/javascript" src="../../../icing/yui/current/build/yui/yui.js"></script>7 <script type="text/javascript" src="../../../../../../lazr-js/build/yui/yui/yui.js"></script>
8 <link rel="stylesheet" href="../../../icing/yui/current/build/cssreset/reset.css"/>8 <link rel="stylesheet" href="../../../../../../lazr-js/build/yui/cssreset/reset.css"/>
9 <link rel="stylesheet" href="../../../icing/yui/current/build/cssfonts/fonts.css"/>9 <link rel="stylesheet" href="../../../../../../lazr-js/build/yui/cssfonts/fonts.css"/>
10 <link rel="stylesheet" href="../../../icing/yui/current/build/cssbase/base.css"/>10 <link rel="stylesheet" href="../../../../../../lazr-js/build/yui/cssbase/base.css"/>
1111
12 <!-- Dependency -->12 <!-- Dependency -->
13 <script type="text/javascript" src="../../../icing/lazr/build/lazr.js"></script>13 <script type="text/javascript" src="../../../../../../lazr-js/build/lazr.js"></script>
14 <script type="text/javascript" src="../../../icing/lazr/build/overlay/overlay.js"></script>14 <script type="text/javascript" src="../../../../../../lazr-js/build/overlay/overlay.js"></script>
15 <script type="text/javascript" src="../../../icing/lazr/build/choiceedit/choiceedit.js"></script>15 <script type="text/javascript" src="../../../../../../lazr-js/build/choiceedit/choiceedit.js"></script>
16 <script type="text/javascript" src="../../../javascript/client/client.js"></script>16 <script type="text/javascript" src="../../../javascript/client/client.js"></script>
1717
18 <!-- The module under test -->18 <!-- The module under test -->
1919
=== modified file 'lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js'
--- lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js 2009-12-10 17:33:27 +0000
+++ lib/canonical/launchpad/javascript/bugs/tests/test_me_too.js 2010-03-01 16:50:37 +0000
@@ -1,7 +1,7 @@
1/* Copyright (c) 2008, Canonical Ltd. All rights reserved. */1/* Copyright (c) 2008, Canonical Ltd. All rights reserved. */
22
3YUI({3YUI({
4 base: '../../../icing/yui/current/build/',4 base: '../../../../../../lazr-js/build/yui/',
5 filter: 'raw',5 filter: 'raw',
6 combine: false6 combine: false
7 }).use('event', 'bugs.bugtask_index', 'node', 'test', 'widget-stack', 'console',7 }).use('event', 'bugs.bugtask_index', 'node', 'test', 'widget-stack', 'console',
@@ -116,38 +116,6 @@
116 },116 },
117117
118 /**118 /**
119 * The flame icon should be hidden initially.
120 */
121 test_flame_hidden_initially: function() {
122 var flame_icon = this.choice_edit.get('flame_icon');
123 Assert.isTrue(flame_icon.hasClass('unseen'), "Flame is not hidden.");
124 },
125
126 /**
127 * The flame icon should be hidden when the user has made a
128 * negative choice (i.e. "Does not affect me").
129 */
130 test_flame_hidden_with_negative_choice: function() {
131 simulate(this.choice_edit.get('boundingBox'), '.value', 'click');
132 simulate(this.choice_edit._choice_list.get('boundingBox'),
133 'li a[href$=false]', 'click');
134 var flame_icon = this.choice_edit.get('flame_icon');
135 Assert.isTrue(flame_icon.hasClass('unseen'), "Flame is not hidden.");
136 },
137
138 /**
139 * The flame icon should be shown when the user has made a
140 * positive choice (i.e. "Affects me too").
141 */
142 test_flame_hidden_with_positive_choice: function() {
143 simulate(this.choice_edit.get('boundingBox'), '.value', 'click');
144 simulate(this.choice_edit._choice_list.get('boundingBox'),
145 'li a[href$=true]', 'click');
146 var flame_icon = this.choice_edit.get('flame_icon');
147 Assert.isFalse(flame_icon.hasClass('unseen'), "Flame is hidden.");
148 },
149
150 /**
151 * The UI should be in a waiting state while the save process is119 * The UI should be in a waiting state while the save process is
152 * executing and return to a non-waiting state once it has120 * executing and return to a non-waiting state once it has
153 * finished.121 * finished.
154122
=== modified file 'lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js'
--- lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js 2009-12-08 11:42:52 +0000
+++ lib/canonical/launchpad/javascript/bugs/tests/test_subscriber.js 2010-03-01 16:50:37 +0000
@@ -1,5 +1,5 @@
1YUI({1YUI({
2 base: '../../../icing/yui/current/build/',2 base: '../../../../../../lazr-js/build/yui/',
3 filter: 'raw', combine: false3 filter: 'raw', combine: false
4 }).use('test', 'console', 'lp.subscriber', function(Y) {4 }).use('test', 'console', 'lp.subscriber', function(Y) {
55
66
=== modified file 'versions.cfg'
--- versions.cfg 2010-02-27 15:17:31 +0000
+++ versions.cfg 2010-03-01 16:50:37 +0000
@@ -33,7 +33,7 @@
33lazr.smtptest = 1.133lazr.smtptest = 1.1
34lazr.testing = 0.1.134lazr.testing = 0.1.1
35lazr.uri = 1.0.235lazr.uri = 1.0.2
36lazr-js = 0.9.236lazr-js = 0.9.2DEVr170
37martian = 0.1137martian = 0.11
38mechanize = 0.1.1138mechanize = 0.1.11
39mercurial = 1.3.139mercurial = 1.3.1