mirror of
https://github.com/discourse/discourse.git
synced 2025-07-13 04:39:50 +08:00

This PR takes the localization features out of "experimental" to prep for the announcement - rename settings and gives them its own area - `experimental_content_localization` to `content_localization_enabled` - `experimental_content_localization_allowed_groups` to `content_localization_allowed_groups` - `experimental_content_localization_supported_locales` to `content_localization_supported_locales` - `experimental_anon_language_switcher` to `content_localization_anon_language_switcher` - migration - related to https://github.com/discourse/discourse-ai/pull/1439 | screenshot 📸 | |---| | <img width="964" alt="Screenshot 2025-06-17 at 5 06 32 PM" src="https://github.com/user-attachments/assets/9a8b2c38-c846-4fc9-8ddd-815c45cc3d0e" /> |
110 lines
3.3 KiB
Ruby
110 lines
3.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
describe TopicLocalizationsController do
|
|
fab!(:user)
|
|
fab!(:group)
|
|
fab!(:topic)
|
|
|
|
let(:locale) { "ja" }
|
|
let(:title) { "これはトピックの翻訳です。" }
|
|
|
|
before do
|
|
SiteSetting.content_localization_enabled = true
|
|
SiteSetting.content_localization_allowed_groups = group.id.to_s
|
|
group.add(user)
|
|
sign_in(user)
|
|
end
|
|
|
|
describe "#create_or_update" do
|
|
context "when localization does not exist" do
|
|
it "creates a new localization" do
|
|
expect {
|
|
post "/topic_localizations/create_or_update.json",
|
|
params: {
|
|
topic_id: topic.id,
|
|
locale:,
|
|
title:,
|
|
}
|
|
}.to change { TopicLocalization.count }.by(1)
|
|
expect(response.status).to eq(201)
|
|
expect(TopicLocalization.last).to have_attributes(
|
|
locale:,
|
|
title:,
|
|
topic_id: topic.id,
|
|
localizer_user_id: user.id,
|
|
)
|
|
end
|
|
end
|
|
|
|
context "when localization already exists" do
|
|
it "updates the existing localization" do
|
|
topic_localization =
|
|
Fabricate(:topic_localization, topic: topic, locale: locale, title: "Old title")
|
|
new_user = Fabricate(:user, groups: [group])
|
|
sign_in(new_user)
|
|
|
|
expect {
|
|
post "/topic_localizations/create_or_update.json",
|
|
params: {
|
|
topic_id: topic.id,
|
|
locale: locale,
|
|
title: title,
|
|
}
|
|
}.not_to change { TopicLocalization.count }
|
|
|
|
expect(response.status).to eq(200)
|
|
topic_localization.reload
|
|
expect(topic_localization).to have_attributes(
|
|
locale: locale,
|
|
title: title,
|
|
localizer_user_id: new_user.id,
|
|
)
|
|
end
|
|
end
|
|
|
|
it "returns forbidden if user not in allowed group" do
|
|
group.remove(user)
|
|
expect {
|
|
post "/topic_localizations/create_or_update.json",
|
|
params: {
|
|
topic_id: topic.id,
|
|
locale:,
|
|
title:,
|
|
}
|
|
}.not_to change { TopicLocalization.count }
|
|
expect(response.status).to eq(403)
|
|
end
|
|
|
|
it "returns not found if topic does not exist" do
|
|
post "/topic_localizations/create_or_update.json", params: { topic_id: -1, locale:, title: }
|
|
expect(response.status).to eq(404)
|
|
end
|
|
end
|
|
|
|
describe "#destroy" do
|
|
fab!(:topic_localization) { Fabricate(:topic_localization, topic:, locale: "ja") }
|
|
|
|
it "destroys the localization" do
|
|
expect {
|
|
delete "/topic_localizations/destroy.json", params: { topic_id: topic.id, locale: "ja" }
|
|
}.to change { TopicLocalization.count }.by(-1)
|
|
expect(response.status).to eq(204)
|
|
end
|
|
|
|
it "returns forbidden if user not allowed" do
|
|
group.remove(user)
|
|
expect {
|
|
delete "/topic_localizations/destroy.json", params: { topic_id: topic.id, locale: "ja" }
|
|
}.not_to change { TopicLocalization.count }
|
|
expect(response.status).to eq(403)
|
|
end
|
|
|
|
it "returns not found if localization is missing" do
|
|
expect {
|
|
delete "/topic_localizations/destroy.json", params: { topic_id: -1, locale: "ja" }
|
|
}.not_to change { TopicLocalization.count }
|
|
expect(response.status).to eq(404)
|
|
end
|
|
end
|
|
end
|