Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Commit cbe42e7

Browse files
Revert "DEV: Add test to cover proofreading in rich editor (#1259)"
This reverts commit e81108a.
1 parent e81108a commit cbe42e7

File tree

1 file changed

+3
-37
lines changed

1 file changed

+3
-37
lines changed

spec/system/ai_helper/ai_proofreading_spec.rb

Lines changed: 3 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,8 @@
1212
end
1313

1414
let(:composer) { PageObjects::Components::Composer.new }
15-
let(:rich) { composer.rich_editor }
1615
let(:toasts) { PageObjects::Components::Toasts.new }
1716
let(:diff_modal) { PageObjects::Modals::DiffModal.new }
18-
let(:keyboard_shortcut) { [PLATFORM_KEY_MODIFIER, :alt, "p"] }
1917

2018
context "when triggering via keyboard shortcut" do
2119
it "proofreads selected text using" do
@@ -26,7 +24,7 @@
2624
composer.select_range(6, 12)
2725

2826
DiscourseAi::Completions::Llm.with_prepared_responses(["world"]) do
29-
composer.composer_input.send_keys(keyboard_shortcut)
27+
composer.composer_input.send_keys([PLATFORM_KEY_MODIFIER, :alt, "p"])
3028
diff_modal.confirm_changes
3129
expect(composer.composer_input.value).to eq("hello world !")
3230
end
@@ -39,7 +37,7 @@
3937

4038
# Simulate AI response
4139
DiscourseAi::Completions::Llm.with_prepared_responses(["hello world"]) do
42-
composer.composer_input.send_keys(keyboard_shortcut)
40+
composer.composer_input.send_keys([PLATFORM_KEY_MODIFIER, :alt, "p"])
4341
diff_modal.confirm_changes
4442
expect(composer.composer_input.value).to eq("hello world")
4543
end
@@ -50,41 +48,9 @@
5048

5149
# Simulate AI response
5250
DiscourseAi::Completions::Llm.with_prepared_responses(["hello world"]) do
53-
composer.composer_input.send_keys(keyboard_shortcut)
51+
composer.composer_input.send_keys([PLATFORM_KEY_MODIFIER, :alt, "p"])
5452
expect(toasts).to have_error(I18n.t("js.discourse_ai.ai_helper.no_content_error"))
5553
end
5654
end
57-
58-
context "when using rich text editor" do
59-
before { SiteSetting.rich_editor = true }
60-
61-
it "proofreads selected text and replaces it" do
62-
visit "/new-topic"
63-
expect(composer).to be_opened
64-
composer.toggle_rich_editor
65-
composer.type_content("hello worldd !")
66-
67-
# NOTE: The rich text editor cannot use select_range on the page object since it is
68-
# a contenteditable element. It would be hard to make this generic enough to put in
69-
# the page object, maybe at some point in the future we can refactor this.
70-
execute_script(<<~JS, text)
71-
const composer = document.querySelector("#reply-control .d-editor-input");
72-
const startNode = composer.firstChild.firstChild;
73-
composer.focus();
74-
const range = document.createRange();
75-
range.setStart(startNode, 6);
76-
range.setEnd(startNode, 12);
77-
const selection = window.getSelection();
78-
selection.removeAllRanges();
79-
selection.addRange(range);
80-
JS
81-
82-
DiscourseAi::Completions::Llm.with_prepared_responses(["world"]) do
83-
composer.composer_input.send_keys(keyboard_shortcut)
84-
diff_modal.confirm_changes
85-
expect(rich).to have_css("p", text: "hello world !")
86-
end
87-
end
88-
end
8955
end
9056
end

0 commit comments

Comments
 (0)