[Libreoffice-commits] core.git: uitest/writer_tests
Mike Kaganski (via logerrit)
logerrit at kemper.freedesktop.org
Fri Dec 27 23:19:51 UTC 2019
uitest/writer_tests/spellDialog.py | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
New commits:
commit e22a3f596ce50b5166063e217d96ef674a54d380
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Sat Dec 28 00:03:09 2019 +0300
Commit: Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Sat Dec 28 00:19:17 2019 +0100
UITest: fix spellDialog.SpellingAndGrammarDialog.test_tdf46852
Two problems were present:
1. If local system default locale is not en_US, then in created document,
text has some other language initially; and instrted text also gets this
language. This may result in no replacement suggestions for the spelling
errors, and "changeall" is not active, so expected sequence of changes
and clicks does not happen. Fixed by setting language at cursor before
paste explicitly.
2. On Windows, the returned text has \r\n paragraph breaks, which didn't
match the regex having \n. Fixed by postprocessing the text before match.
Change-Id: I320fc62c3c27dfe699fb45669c081df30cb66d6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/85877
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
diff --git a/uitest/writer_tests/spellDialog.py b/uitest/writer_tests/spellDialog.py
index 9f69fc2c8fe3..e678afea53e9 100644
--- a/uitest/writer_tests/spellDialog.py
+++ b/uitest/writer_tests/spellDialog.py
@@ -10,6 +10,7 @@ from uitest.framework import UITestCase
from uitest.uihelper.common import get_state_as_dict
from libreoffice.linguistic.linguservice import get_spellchecker
+from com.sun.star.lang import Locale
class SpellingAndGrammarDialog(UITestCase):
@@ -62,6 +63,8 @@ frog, dogg, catt"""
self.ui_test.create_doc_in_start_center("writer")
document = self.ui_test.get_component()
cursor = document.getCurrentController().getViewCursor()
+ # Inserted text must be en_US, so make sure to set language in current location
+ cursor.CharLocale = Locale("en", "US", "")
input_text = self.TDF46852_INPUT.replace('\n', '\r') # \r = para break
document.Text.insertString(cursor, input_text, False)
@@ -94,5 +97,6 @@ frog, dogg, catt"""
)
)
- self.assertTrue(re.match(self.TDF46852_REGEX, document.Text.getString()))
+ output_text = document.Text.getString().replace('\r\n', '\n')
+ self.assertTrue(re.match(self.TDF46852_REGEX, output_text))
More information about the Libreoffice-commits
mailing list