FIX: disallow invalid top_menu and post_menu and share_links

In the past any text could be entered there causing big potential issues
This commit is contained in:
Sam
2018-04-26 17:00:56 +10:00
parent 6676bbd38b
commit 88f5251415
4 changed files with 39 additions and 3 deletions

View File

@ -52,7 +52,19 @@ describe SiteSetting do
end
describe "top_menu" do
before { SiteSetting.top_menu = 'one,-nope|two|three,-not|four,ignored|category/xyz|latest' }
describe "validations" do
it "always demands latest" do
expect do
SiteSetting.top_menu = 'categories'
end.to raise_error(Discourse::InvalidParameters)
end
it "does not allow random text" do
expect do
SiteSetting.top_menu = 'latest|random'
end.to raise_error(Discourse::InvalidParameters)
end
end
describe "items" do
let(:items) { SiteSetting.top_menu_items }
@ -64,7 +76,8 @@ describe SiteSetting do
describe "homepage" do
it "has homepage" do
expect(SiteSetting.homepage).to eq('one')
SiteSetting.top_menu = "bookmarks|latest"
expect(SiteSetting.homepage).to eq('bookmarks')
end
end
end