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
=== modified file 'methanal/js/Methanal/Tests/TestUtil.js'
--- methanal/js/Methanal/Tests/TestUtil.js 2010-01-05 09:52:20 +0000
+++ methanal/js/Methanal/Tests/TestUtil.js 2010-02-23 10:46:13 +0000
@@ -599,11 +599,12 @@
599Divmod.UnitTest.TestCase.subclass(599Divmod.UnitTest.TestCase.subclass(
600 Methanal.Tests.TestUtil, 'TestThrobber').methods(600 Methanal.Tests.TestUtil, 'TestThrobber').methods(
601 function _createThrobber(self, toggleDisplay) {601 function _createThrobber(self, toggleDisplay) {
602 var idNodes = {602 var widget = Nevow.Athena.Widget(
603 'throbber': Methanal.Tests.Util.MockNode('foo')};603 Nevow.Test.WidgetUtil.makeWidgetNode());
604 var widget = Methanal.Tests.Util.MockWidget(idNodes);604 var throbberNode = Methanal.Tests.Util.makeWidgetChildNode(
605 var throbber = Methanal.Util.Throbber(widget, toggleDisplay);605 widget, 'img', 'throbber');
606 return throbber;606 document.body.appendChild(widget.node);
607 return Methanal.Util.Throbber(widget, toggleDisplay);
607 },608 },
608609
609610
@@ -612,7 +613,7 @@
612 */613 */
613 function test_create(self) {614 function test_create(self) {
614 var throbber = self._createThrobber();615 var throbber = self._createThrobber();
615 self.assertIdentical(throbber.node.name, 'foo');616 self.assertIdentical(throbber.node.tagName, 'IMG');
616 },617 },
617618
618619
619620
=== modified file 'methanal/js/Methanal/Tests/Util.js'
--- methanal/js/Methanal/Tests/Util.js 2010-01-09 13:30:59 +0000
+++ methanal/js/Methanal/Tests/Util.js 2010-02-23 10:46:13 +0000
@@ -1,5 +1,6 @@
1// import Divmod.Runtime1// import Divmod.Runtime
2// import Divmod.UnitTest2// import Divmod.UnitTest
3// import Nevow.Test.WidgetUtil
34
45
56
@@ -22,40 +23,6 @@
2223
2324
24/**25/**
25 * A mock of a DOM node.
26 */
27Divmod.Class.subclass(Methanal.Tests.Util, 'MockNode').methods(
28 function __init__(self, name) {
29 self.name = name;
30 self.style = {};
31 });
32
33
34
35/**
36 * A mock of L{Nevow.Athena.Widget}.
37 */
38Divmod.Class.subclass(Methanal.Tests.Util, 'MockWidget').methods(
39 function __init__(self, idNodes) {
40 self.node = Methanal.Tests.Util.MockNode('widget');
41 if (!idNodes) {
42 idNodes = {};
43 }
44 self._idNodes = idNodes;
45 },
46
47
48 function nodeById(self, id) {
49 var node = self._idNodes[id];
50 if (!node) {
51 throw Divmod.Runtime.NodeNotFound('Node not found');
52 }
53 return node;
54 });
55
56
57
58/**
59 * Create a new child DOM element for a widget.26 * Create a new child DOM element for a widget.
60 *27 *
61 * @type widget: L{Nevow.Athena.Widget}28 * @type widget: L{Nevow.Athena.Widget}

Subscribers

People subscribed via source and target branches