mirror of
https://github.com/discourse/discourse.git
synced 2024-12-03 01:53:40 +08:00
b1b53da71d
Previously, calling `sign_in` would cause the browser to be redirected to `/`, and would cause the Ember app to boot. We would then call `visit()`, causing the app to boot for a second time. This commit adds a `redirect=false` option to the `/session/username/become` route. This avoids the unnecessary boot of the app, and leads to significantly faster system spec run times. In local testing, this takes the full system-spec suite for chat from ~6min to ~4min.
37 lines
1.1 KiB
Ruby
37 lines
1.1 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
RSpec.describe "Unfollow dm channel", type: :system, js: true do
|
|
fab!(:current_user) { Fabricate(:user) }
|
|
fab!(:other_user) { Fabricate(:user) }
|
|
fab!(:dm_channel_1) { Fabricate(:direct_message_channel, users: [current_user, other_user]) }
|
|
|
|
let!(:chat_page) { PageObjects::Pages::Chat.new }
|
|
let!(:chat_channel_page) { PageObjects::Pages::ChatChannel.new }
|
|
|
|
before do
|
|
SiteSetting.navigation_menu = "sidebar"
|
|
chat_system_bootstrap
|
|
sign_in(current_user)
|
|
end
|
|
|
|
context "when receiving a message after unfollowing" do
|
|
it "correctly shows the channel" do
|
|
visit("/")
|
|
find(".channel-#{dm_channel_1.id}").hover
|
|
find(".channel-#{dm_channel_1.id} .sidebar-section-link-hover").click
|
|
|
|
expect(page).to have_no_css(".channel-#{dm_channel_1.id}")
|
|
|
|
using_session(:user_1) do
|
|
text = "this is fine"
|
|
sign_in(other_user)
|
|
chat_page.visit_channel(dm_channel_1)
|
|
chat_channel_page.send_message(text)
|
|
expect(chat_channel_page).to have_message(text: text)
|
|
end
|
|
|
|
expect(page).to have_css(".channel-#{dm_channel_1.id} .urgent")
|
|
end
|
|
end
|
|
end
|