Skip to content

Commit 6e94688

Browse files
committedFeb 1, 2015
Followup 956c155: fix python errors in PAL labeling tests
1 parent 9ba8b29 commit 6e94688

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed
 

‎tests/src/python/test_qgspallabeling_base.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -155,9 +155,8 @@ def removeMapLayer(cls, layer):
155155
lyr_id = layer.id()
156156
cls._MapRegistry.removeMapLayer(lyr_id)
157157
ms_layers = cls._MapSettings.layers()
158-
""":type: QStringList"""
159-
if ms_layers.contains(lyr_id):
160-
ms_layers.removeAt(ms_layers.indexOf(lyr_id))
158+
if lyr_id in ms_layers:
159+
ms_layers.remove(lyr_id)
161160
cls._MapSettings.setLayers(ms_layers)
162161

163162
@classmethod
@@ -390,15 +389,15 @@ def tearDown(self):
390389

391390
def test_default_pal_disabled(self):
392391
# Verify PAL labeling is disabled for layer by default
393-
palset = self.layer.customProperty('labeling', '').toString()
392+
palset = self.layer.customProperty('labeling', '')
394393
msg = '\nExpected: Empty string\nGot: {0}'.format(palset)
395394
self.assertEqual(palset, '', msg)
396395

397396
def test_settings_enable_pal(self):
398397
# Verify default PAL settings enable PAL labeling for layer
399398
lyr = QgsPalLayerSettings()
400399
lyr.writeToLayer(self.layer)
401-
palset = self.layer.customProperty('labeling', '').toString()
400+
palset = self.layer.customProperty('labeling', '')
402401
msg = '\nExpected: Empty string\nGot: {0}'.format(palset)
403402
self.assertEqual(palset, 'pal', msg)
404403

0 commit comments

Comments
 (0)
Please sign in to comment.