From 7cf7eb4a48e58cbd2c49dae36aef053b0106fce0 Mon Sep 17 00:00:00 2001 From: Veronica Berglyd Olsen <1619840+vkbo@users.noreply.github.com> Date: Sun, 12 Nov 2023 20:21:35 +0100 Subject: [PATCH] Fix test coverage --- tests/test_gui/test_gui_doceditor.py | 50 ++++------------------------ 1 file changed, 7 insertions(+), 43 deletions(-) diff --git a/tests/test_gui/test_gui_doceditor.py b/tests/test_gui/test_gui_doceditor.py index 064fcf17b..008ad0535 100644 --- a/tests/test_gui/test_gui_doceditor.py +++ b/tests/test_gui/test_gui_doceditor.py @@ -672,7 +672,7 @@ def testGuiEditor_Insert(qtbot, monkeypatch, nwGUI, projPath, ipsumText, mockRnd @pytest.mark.gui -def testGuiEditor_TextManipulation(qtbot, monkeypatch, nwGUI, projPath, ipsumText, mockRnd): +def testGuiEditor_TextManipulation(qtbot, nwGUI, projPath, ipsumText, mockRnd): """Test the text manipulation functions.""" buildTestProject(nwGUI, projPath) assert nwGUI.openDocument(C.hSceneDoc) is True @@ -680,37 +680,6 @@ def testGuiEditor_TextManipulation(qtbot, monkeypatch, nwGUI, projPath, ipsumTex text = "### A Scene\n\n%s" % "\n\n".join(ipsumText) nwGUI.docEditor.replaceText(text) - # Clear Surrounding - # ================= - - # No Selection - text = "### A Scene\n\n%s" % ipsumText[0] - nwGUI.docEditor.replaceText(text) - nwGUI.docEditor.setCursorPosition(45) - - cursor = nwGUI.docEditor.textCursor() - assert nwGUI.docEditor._clearSurrounding(cursor, 1) is False - - # Clear Characters, 1 Layer - repText = text.replace("consectetur", "=consectetur=") - nwGUI.docEditor.replaceText(repText) - nwGUI.docEditor.setCursorPosition(45) - - cursor = nwGUI.docEditor.textCursor() - cursor.select(QTextCursor.WordUnderCursor) - assert nwGUI.docEditor._clearSurrounding(cursor, 1) is True - assert nwGUI.docEditor.getText() == text - - # Clear Characters, 2 Layers - repText = text.replace("consectetur", "==consectetur==") - nwGUI.docEditor.replaceText(repText) - nwGUI.docEditor.setCursorPosition(45) - - cursor = nwGUI.docEditor.textCursor() - cursor.select(QTextCursor.WordUnderCursor) - assert nwGUI.docEditor._clearSurrounding(cursor, 2) is True - assert nwGUI.docEditor.getText() == text - # Wrap Selection # ============== @@ -718,11 +687,6 @@ def testGuiEditor_TextManipulation(qtbot, monkeypatch, nwGUI, projPath, ipsumTex nwGUI.docEditor.replaceText(text) nwGUI.docEditor.setCursorPosition(45) - # No Selection - with monkeypatch.context() as mp: - mp.setattr(nwGUI.docEditor, "_autoSelect", lambda: QTextCursor()) - assert nwGUI.docEditor._wrapSelection("=", "=") is False - # Wrap Equal nwGUI.docEditor.replaceText(text) nwGUI.docEditor.setCursorPosition(45) @@ -752,13 +716,13 @@ def testGuiEditor_TextManipulation(qtbot, monkeypatch, nwGUI, projPath, ipsumTex # ============= text = "### A Scene\n\n%s" % "\n\n".join(ipsumText[0:2]) - nwGUI.docEditor.replaceText(text) - nwGUI.docEditor.setCursorPosition(45) - # No Selection - with monkeypatch.context() as mp: - mp.setattr(nwGUI.docEditor, "_autoSelect", lambda: QTextCursor()) - assert nwGUI.docEditor._toggleFormat(2, "=") is False + # Block format repetition + nwGUI.docEditor.replaceText(text) + nwGUI.docEditor.setCursorPosition(39) + assert nwGUI.docEditor._toggleFormat(1, "=") is True + assert nwGUI.docEditor.getText() == text.replace("amet", "=amet=", 1) + assert nwGUI.docEditor._toggleFormat(1, "=") is False # Wrap Single Equal nwGUI.docEditor.replaceText(text)