Skip to content

Commit cadcdc0

Browse files
committed
fix python tests
1 parent 34ca898 commit cadcdc0

3 files changed

Lines changed: 13 additions & 8 deletions

File tree

src/cone/app/model.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,9 +158,9 @@ def properties(self):
158158
@instance_property
159159
def metadata(self):
160160
metadata = Metadata()
161-
nodeinfo = self.nodeinfo
162-
if nodeinfo.title:
163-
metadata.title = nodeinfo.title
161+
info = get_node_info(self.node_info_name)
162+
if info and info.title:
163+
metadata.title = info.title
164164
elif self.name:
165165
metadata.title = self.name
166166
else:

src/cone/app/tests/test_browser_ajax.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,8 @@ def test_AjaxMessage(self):
163163
'type': 'message',
164164
'payload': payload,
165165
'flavor': flavor,
166-
'selector': selector
166+
'selector': selector,
167+
'title': None
167168
})
168169

169170
def test_AjaxOverlay(self):
@@ -388,7 +389,8 @@ def test_AjaxFormContinue(self):
388389
'payload': 'Some info message',
389390
'flavor': 'info',
390391
'selector': 'None',
391-
'css': ''
392+
'css': '',
393+
'title': None
392394
}, {
393395
'type': 'overlay',
394396
'action': 'someaction',

src/cone/app/tests/test_browser_sharing.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,8 @@ def test_add_role(self):
212212
'flavor': 'error',
213213
'type': 'message',
214214
'payload': u"Can not add role 'manager' for principal 'viewer'",
215-
'selector': None
215+
'selector': None,
216+
'title': None
216217
}],
217218
'payload': u'',
218219
'mode': 'NONE',
@@ -280,7 +281,8 @@ def test_remove_role(self):
280281
'flavor': 'error',
281282
'type': 'message',
282283
'payload': u"Can not remove role 'inexistent' for principal 'viewer'",
283-
'selector': None
284+
'selector': None,
285+
'title': None
284286
}],
285287
'payload': u'',
286288
'mode': 'NONE',
@@ -302,7 +304,8 @@ def test_remove_role(self):
302304
'flavor': 'error',
303305
'type': 'message',
304306
'payload': u"Can not remove role 'manager' for principal 'foo'",
305-
'selector': None
307+
'selector': None,
308+
'title': None
306309
}],
307310
'payload': u'',
308311
'mode': 'NONE',

0 commit comments

Comments
 (0)