sc/qa/uitest/range_name/tdf137617.py |   73 ++++++++++++++---------------------
 1 file changed, 31 insertions(+), 42 deletions(-)

New commits:
commit 400e4b1866cbf7ab4b703e31f6c0e9b4040a58e0
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Mon Feb 6 18:00:08 2023 +0100
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Tue Feb 7 09:34:59 2023 +0000

    sc: use try/finally to reset the changes in OptionsTreeDialog
    
    Otherwise it might affect other tests if it hits an assert
    
    Change-Id: I2d2fe1c2dfb931c8c9d95e61d8fdb31236b96c57
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146592
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sc/qa/uitest/range_name/tdf137617.py 
b/sc/qa/uitest/range_name/tdf137617.py
index 234cc99f84ac..c2ac24ebc6e7 100644
--- a/sc/qa/uitest/range_name/tdf137617.py
+++ b/sc/qa/uitest/range_name/tdf137617.py
@@ -15,6 +15,18 @@ from uitest.uihelper.calc import enter_text_to_cell
 
 class tdf137617(UITestCase):
 
+    def change_formula_syntax(self, syntax):
+        with 
self.ui_test.execute_dialog_through_command(".uno:OptionsTreeDialog") as 
xDialogOpt:
+
+            xPages = xDialogOpt.getChild("pages")
+            xCalcEntry = xPages.getChild('3')
+            xCalcEntry.executeAction("EXPAND", tuple())
+            xCalcFormulaEntry = xCalcEntry.getChild('4')
+            xCalcFormulaEntry.executeAction("SELECT", tuple())
+
+            xFormulaSyntax = xDialogOpt.getChild('formulasyntax')
+            select_by_text(xFormulaSyntax, syntax)
+
     def test_tdf137617(self):
 
         with self.ui_test.create_doc_in_start_center("calc"):
@@ -36,7 +48,6 @@ class tdf137617(UITestCase):
                 self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('bottom'))['Selected'])
                 self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('top'))['Selected'])
 
-
             gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "B1"}))
 
             xPosWindow = calcDoc.getChild('pos_window')
@@ -46,56 +57,34 @@ class tdf137617(UITestCase):
 
             self.assertEqual('Result2', get_state_as_dict(xPosWindow)['Text'])
 
-            # Change formula syntax to "Excel R1C1"
-            with 
self.ui_test.execute_dialog_through_command(".uno:OptionsTreeDialog") as 
xDialogOpt:
-
-                xPages = xDialogOpt.getChild("pages")
-                xCalcEntry = xPages.getChild('3')
-                xCalcEntry.executeAction("EXPAND", tuple())
-                xCalcFormulaEntry = xCalcEntry.getChild('4')
-                xCalcFormulaEntry.executeAction("SELECT", tuple())
-
-                xFormulaSyntax = xDialogOpt.getChild('formulasyntax')
-                select_by_text(xFormulaSyntax, "Excel R1C1")
-
-
-            enter_text_to_cell(gridwin, "C1", "Result3")
-            enter_text_to_cell(gridwin, "D1", "Result4")
-
-            gridwin.executeAction("SELECT", mkPropertyValues({"RANGE": 
"C1:D2"}))
-
-            with 
self.ui_test.execute_dialog_through_command(".uno:CreateNames") as xDialog:
-
-
-                # Only top is selected
-                self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('left'))['Selected'])
-                self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('right'))['Selected'])
-                self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('bottom'))['Selected'])
-                self.assertEqual('true', 
get_state_as_dict(xDialog.getChild('top'))['Selected'])
+            try:
+                self.change_formula_syntax("Excel R1C1")
 
+                enter_text_to_cell(gridwin, "C1", "Result3")
+                enter_text_to_cell(gridwin, "D1", "Result4")
 
-            gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "C2"}))
+                gridwin.executeAction("SELECT", mkPropertyValues({"RANGE": 
"C1:D2"}))
 
-            # Without the fix in place, this test would have failed with
-            # AssertionError: 'Result3' != 'R2C3'
-            self.assertEqual('Result3', get_state_as_dict(xPosWindow)['Text'])
+                with 
self.ui_test.execute_dialog_through_command(".uno:CreateNames") as xDialog:
 
-            gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "D2"}))
 
-            self.assertEqual('Result4', get_state_as_dict(xPosWindow)['Text'])
+                    # Only top is selected
+                    self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('left'))['Selected'])
+                    self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('right'))['Selected'])
+                    self.assertEqual('false', 
get_state_as_dict(xDialog.getChild('bottom'))['Selected'])
+                    self.assertEqual('true', 
get_state_as_dict(xDialog.getChild('top'))['Selected'])
 
-            # Change formula syntax back to "Calc A1"
-            with 
self.ui_test.execute_dialog_through_command(".uno:OptionsTreeDialog") as 
xDialogOpt:
+                gridwin.executeAction("SELECT", mkPropertyValues({"CELL": 
"C2"}))
 
-                xPages = xDialogOpt.getChild("pages")
-                xCalcEntry = xPages.getChild('3')
-                xCalcEntry.executeAction("EXPAND", tuple())
-                xCalcFormulaEntry = xCalcEntry.getChild('4')
-                xCalcFormulaEntry.executeAction("SELECT", tuple())
+                # Without the fix in place, this test would have failed with
+                # AssertionError: 'Result3' != 'R2C3'
+                self.assertEqual('Result3', 
get_state_as_dict(xPosWindow)['Text'])
 
-                xFormulaSyntax = xDialogOpt.getChild('formulasyntax')
-                select_by_text(xFormulaSyntax, "Calc A1")
+                gridwin.executeAction("SELECT", mkPropertyValues({"CELL": 
"D2"}))
 
+                self.assertEqual('Result4', 
get_state_as_dict(xPosWindow)['Text'])
 
+            finally:
+                self.change_formula_syntax("Calc A1")
 
 # vim: set shiftwidth=4 softtabstop=4 expandtab:

Reply via email to