Update pyface/tasks/tests/test_dock_pane_toggle_group.py

Improve test comments.
enthought · Nov 16, 2012 · 4dcf242 · 4dcf242
1 parent 5941b70
commit 4dcf242
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/pyface/tasks/tests/test_dock_pane_toggle_group.py b/pyface/tasks/tests/test_dock_pane_toggle_group.py
@@ -20,8 +20,8 @@ def create_central_pane(self):
 
     def create_dock_panes(self):
         self.dock_panes = dock_panes = [
-            DockPane(id='tests.bogus_task.dock_pane_1', name='Dock Pane 2'),
-            DockPane(id='tests.bogus_task.dock_pane_2', name='Dock Pane 1'),
+            DockPane(id='tests.bogus_task.dock_pane_2', name='Dock Pane 2'),
+            DockPane(id='tests.bogus_task.dock_pane_1', name='Dock Pane 1'),
         ]
 
         return dock_panes
@@ -75,6 +75,7 @@ def test_group_content_at_startup(self):
         # Check that there are 2 dock panes in the group at the beginning.
         self.assertEqual(2, len(self.dock_pane_toggle_group.items))
 
+        # Names are sorted by the group.
         names = self.get_dock_pane_toggle_action_names()
         expected_names = ['Dock Pane 1', 'Dock Pane 2']
         self.assertItemsEqual(expected_names, names)
@@ -88,6 +89,7 @@ def test_react_to_dock_pane_added(self):
         # Check that there are 3 dock panes in the group.
         self.assertEqual(3, len(self.dock_pane_toggle_group.items))
 
+        # Names are sorted by the group.
         names = self.get_dock_pane_toggle_action_names()
         expected_names = ['Dock Pane 0', 'Dock Pane 1', 'Dock Pane 2']
         self.assertItemsEqual(expected_names, names)
@@ -96,7 +98,7 @@ def test_react_to_dock_pane_removed(self):
         # Remove a dock pane from the task.
         self.task_state.dock_panes.remove(self.task.dock_panes[0])
 
-        # Check that there are 3 dock panes in the group.
+        # Check that there is only 1 dock pane left in the group.
         self.assertEqual(1, len(self.dock_pane_toggle_group.items))
 
         names = self.get_dock_pane_toggle_action_names()