mirror of
https://github.com/discourse/discourse.git
synced 2024-12-02 07:23:40 +08:00
c9dab6fd08
It's very easy to forget to add `require 'rails_helper'` at the top of every core/plugin spec file, and omissions can cause some very confusing/sporadic errors. By setting this flag in `.rspec`, we can remove the need for `require 'rails_helper'` entirely.
46 lines
1.5 KiB
Ruby
46 lines
1.5 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require "i18n/duplicate_key_finder"
|
|
|
|
describe "site setting integrity checks" do
|
|
let(:site_setting_file) { File.join(Rails.root, 'config', 'site_settings.yml') }
|
|
let(:yaml) { YAML.load_file(site_setting_file) }
|
|
|
|
%w(hidden client).each do |property|
|
|
it "set #{property} value as true or not set" do
|
|
yaml.each_value do |category|
|
|
category.each_value do |setting|
|
|
next unless setting.is_a?(Hash)
|
|
expect(setting[property] == nil || setting[property] == true).to be_truthy
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
it "has no duplicate keys" do
|
|
duplicates = DuplicateKeyFinder.new.find_duplicates(site_setting_file)
|
|
expect(duplicates).to be_empty
|
|
end
|
|
|
|
it "no locale default has different type than default or invalid key" do
|
|
yaml.each_value do |category|
|
|
category.each do |setting_name, setting|
|
|
next unless setting.is_a?(Hash)
|
|
if setting['locale_default']
|
|
setting['locale_default'].each_pair do |k, v|
|
|
expect(LocaleSiteSetting.valid_value?(k.to_s)).to be_truthy,
|
|
"'#{k}' is not a valid locale_default key for '#{setting_name}' site setting"
|
|
|
|
case setting['default']
|
|
when TrueClass, FalseClass
|
|
expect(v.class == TrueClass || v.class == FalseClass).to be_truthy
|
|
else
|
|
expect(v).to be_a_kind_of(setting['default'].class)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|