Merge lp:~methanal-developers/methanal/improve-mock-dom-framework into lp:methanal

Proposed by Jonathan Jacobs
Status: Merged
Approved by: Tristan Seligmann
Approved revision: not available
Merged at revision: not available
Proposed branch: lp:~methanal-developers/methanal/improve-mock-dom-framework
Merge into: lp:methanal
Diff against target: 82 lines (+8/-40)
2 files modified
methanal/js/Methanal/Tests/TestUtil.js (+7/-6)
methanal/js/Methanal/Tests/Util.js (+1/-34)
To merge this branch: bzr merge lp:~methanal-developers/methanal/improve-mock-dom-framework
Reviewer Review Type Date Requested Status
Tristan Seligmann Approve
Review via email: mp+19949@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Tristan Seligmann (mithrandi) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'methanal/js/Methanal/Tests/TestUtil.js'
2--- methanal/js/Methanal/Tests/TestUtil.js 2010-01-05 09:52:20 +0000
3+++ methanal/js/Methanal/Tests/TestUtil.js 2010-02-23 10:46:13 +0000
4@@ -599,11 +599,12 @@
5 Divmod.UnitTest.TestCase.subclass(
6 Methanal.Tests.TestUtil, 'TestThrobber').methods(
7 function _createThrobber(self, toggleDisplay) {
8- var idNodes = {
9- 'throbber': Methanal.Tests.Util.MockNode('foo')};
10- var widget = Methanal.Tests.Util.MockWidget(idNodes);
11- var throbber = Methanal.Util.Throbber(widget, toggleDisplay);
12- return throbber;
13+ var widget = Nevow.Athena.Widget(
14+ Nevow.Test.WidgetUtil.makeWidgetNode());
15+ var throbberNode = Methanal.Tests.Util.makeWidgetChildNode(
16+ widget, 'img', 'throbber');
17+ document.body.appendChild(widget.node);
18+ return Methanal.Util.Throbber(widget, toggleDisplay);
19 },
20
21
22@@ -612,7 +613,7 @@
23 */
24 function test_create(self) {
25 var throbber = self._createThrobber();
26- self.assertIdentical(throbber.node.name, 'foo');
27+ self.assertIdentical(throbber.node.tagName, 'IMG');
28 },
29
30
31
32=== modified file 'methanal/js/Methanal/Tests/Util.js'
33--- methanal/js/Methanal/Tests/Util.js 2010-01-09 13:30:59 +0000
34+++ methanal/js/Methanal/Tests/Util.js 2010-02-23 10:46:13 +0000
35@@ -1,5 +1,6 @@
36 // import Divmod.Runtime
37 // import Divmod.UnitTest
38+// import Nevow.Test.WidgetUtil
39
40
41
42@@ -22,40 +23,6 @@
43
44
45 /**
46- * A mock of a DOM node.
47- */
48-Divmod.Class.subclass(Methanal.Tests.Util, 'MockNode').methods(
49- function __init__(self, name) {
50- self.name = name;
51- self.style = {};
52- });
53-
54-
55-
56-/**
57- * A mock of L{Nevow.Athena.Widget}.
58- */
59-Divmod.Class.subclass(Methanal.Tests.Util, 'MockWidget').methods(
60- function __init__(self, idNodes) {
61- self.node = Methanal.Tests.Util.MockNode('widget');
62- if (!idNodes) {
63- idNodes = {};
64- }
65- self._idNodes = idNodes;
66- },
67-
68-
69- function nodeById(self, id) {
70- var node = self._idNodes[id];
71- if (!node) {
72- throw Divmod.Runtime.NodeNotFound('Node not found');
73- }
74- return node;
75- });
76-
77-
78-
79-/**
80 * Create a new child DOM element for a widget.
81 *
82 * @type widget: L{Nevow.Athena.Widget}

Subscribers

People subscribed via source and target branches