Merge lp:~jtatum/mago/gcalctool-lucid into lp:~mago-contributors/mago/mago-1.0

Proposed by James Tatum
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jtatum/mago/gcalctool-lucid
Merge into: lp:~mago-contributors/mago/mago-1.0
Diff against target: 133 lines (+30/-30)
3 files modified
gcalctool/README (+1/-1)
gcalctool/gcalctool_views.xml (+9/-8)
mago/application/gnome.py (+20/-21)
To merge this branch: bzr merge lp:~jtatum/mago/gcalctool-lucid
Reviewer Review Type Date Requested Status
Ara Pulido Approve
Review via email: mp+16925@code.launchpad.net
To post a comment you must log in.
Revision history for this message
James Tatum (jtatum) wrote :

Updating gcalctool tests for Lucid. The gconf key changed as did a majority of the button identifiers.

Resubmitting merge proposal - I had to redo my branch thanks to some operational differences in bzr 2. This should not be merged until Karmic is branched in Mago since these changes will definitely break the tests in Karmic.

Revision history for this message
Ara Pulido (ara) wrote :

Thanks for your changes, James.

They look OK and they work successfully in Lucid. Accepted.
(I have already branched Karmic tests)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'gcalctool/README'
2--- gcalctool/README 2009-09-11 02:51:23 +0000
3+++ gcalctool/README 2010-01-06 19:11:14 +0000
4@@ -13,7 +13,7 @@
5
6 Version
7 -------
8-Tests were updated for Ubuntu Karmic
9+Tests were updated for Ubuntu Lucid
10
11 Available Testsuites
12 --------------------
13
14=== modified file 'gcalctool/gcalctool_views.xml'
15--- gcalctool/gcalctool_views.xml 2009-09-19 03:28:17 +0000
16+++ gcalctool/gcalctool_views.xml 2010-01-06 19:11:14 +0000
17@@ -1,4 +1,5 @@
18-<?xml version="1.0"?>
19+<?xml version="1.0" encoding="UTF-8"?>
20+
21 <suite name="gcalctool views">
22 <class>gcalctool_views.GCalctoolViews</class>
23 <description>
24@@ -6,12 +7,12 @@
25 </description>
26 <case name="Advanced">
27 <method>changeViews</method>
28- <description>Change to the 'advanced' view and verify the square button
29+ <description>Change to the 'advanced' view and verify the mod button
30 is showing</description>
31 <args>
32 <view>ADVANCED</view>
33 <state>showing</state>
34- <button>Square</button>
35+ <button>mod</button>
36 </args>
37 </case>
38 <case name="Financial">
39@@ -21,7 +22,7 @@
40 <args>
41 <view>FINANCIAL</view>
42 <state>showing</state>
43- <button>Future value</button>
44+ <button>Fv</button>
45 </args>
46 </case>
47 <case name="Scientific">
48@@ -36,22 +37,22 @@
49 </case>
50 <case name="Programming">
51 <method>changeViews</method>
52- <description>Change to the 'programming' view and verify the 1's button
53+ <description>Change to the 'programming' view and verify the ones button
54 is showing</description>
55 <args>
56 <view>PROGRAMMING</view>
57 <state>showing</state>
58- <button>1's</button>
59+ <button>ones</button>
60 </args>
61 </case>
62 <case name="Basic">
63 <method>changeViews</method>
64- <description>Change to the 'basic' view and verify the reciprocal button
65+ <description>Change to the 'basic' view and verify the pi button
66 is hidden</description>
67 <args>
68 <view>BASIC</view>
69 <state>hidden</state>
70- <button>Reciprocal</button>
71+ <button>π</button>
72 </args>
73 </case>
74 </suite>
75
76=== modified file 'mago/application/gnome.py'
77--- mago/application/gnome.py 2009-09-11 04:58:37 +0000
78+++ mago/application/gnome.py 2010-01-06 19:11:14 +0000
79@@ -1,3 +1,4 @@
80+#-*- coding:utf-8 -*-
81 """
82 This is the "gnome" module.
83
84@@ -34,30 +35,28 @@
85 "MENU" : MNU_SCIENTIFIC},
86 "PROGRAMMING" : {"WINDOW" : WINDOW_PROGRAMMING,
87 "MENU" : MNU_PROGRAMMING}}
88- BUTTONS = {"0" : "Numeric 0",
89- "1" : "Numeric 1",
90- "2" : "Numeric 2",
91- "3" : "Numeric 3",
92- "4" : "Numeric 4",
93- "5" : "Numeric 5",
94- "6" : "Numeric 6",
95- "7" : "Numeric 7",
96- "8" : "Numeric 8",
97- "9" : "Numeric 9",
98- "=" : "Calculate result",
99- "+" : "Add",
100- "-" : "Subtract",
101- "*" : "Multiply",
102- "/" : "Divide",
103- "c" : "Clear entry",
104- "C" : "Clear",
105- "." : "Numeric point",
106- "@" : "Square",
107- "s" : "Square root"}
108+ BUTTONS = {"0" : "0",
109+ "1" : "1",
110+ "2" : "2",
111+ "3" : "3",
112+ "4" : "4",
113+ "5" : "5",
114+ "6" : "6",
115+ "7" : "7",
116+ "8" : "8",
117+ "9" : "9",
118+ "=" : "=",
119+ "+" : "+",
120+ "-" : "−",
121+ "*" : "×",
122+ "/" : "÷",
123+ "C" : "Clr",
124+ "." : ".",
125+ "s" : "√"}
126 EDITBAR_ROLE = "edit bar"
127 EDITBAR_INDEX = 0
128 SLEEP_DELAY = 2
129- GCONF_MODE_KEY = "/apps/gcalctool/mode"
130+ GCONF_MODE_KEY = "/apps/gcalctool/button_layout"
131 GCONF_MODE_VAL = "BASIC"
132
133 def __init__(self):

Subscribers

People subscribed via source and target branches

to status/vote changes: