Commit 934e0f60 authored by Sean McGivern's avatar Sean McGivern

Use Gitlab::FeatureCategories in specs

This is our single source of truth for feature categories, so we can use
it instead of loading the YAML file each time.
parent 3fc8d136
...@@ -21,7 +21,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Restful do ...@@ -21,7 +21,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Restful do
end end
it 'has a valid feature category for every route', :aggregate_failures do it 'has a valid feature category for every route', :aggregate_failures do
feature_categories = YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).to_set feature_categories = Gitlab::FeatureCategories.default.categories
described_class.ee_routes.each do |route| described_class.ee_routes.each do |route|
expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid" expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid"
......
...@@ -4,7 +4,7 @@ require 'spec_helper' ...@@ -4,7 +4,7 @@ require 'spec_helper'
RSpec.describe "Every controller" do RSpec.describe "Every controller" do
context "feature categories" do context "feature categories" do
let_it_be(:feature_categories) do let_it_be(:feature_categories) do
YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).map(&:to_sym).to_set Gitlab::FeatureCategories.default.categories.map(&:to_sym).to_set
end end
let_it_be(:controller_actions) do let_it_be(:controller_actions) do
......
...@@ -5,7 +5,7 @@ require 'spec_helper' ...@@ -5,7 +5,7 @@ require 'spec_helper'
RSpec.describe 'Every API endpoint' do RSpec.describe 'Every API endpoint' do
context 'feature categories' do context 'feature categories' do
let_it_be(:feature_categories) do let_it_be(:feature_categories) do
YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).map(&:to_sym).to_set Gitlab::FeatureCategories.default.categories.map(&:to_sym).to_set
end end
let_it_be(:api_endpoints) do let_it_be(:api_endpoints) do
......
...@@ -11,7 +11,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Graphql do ...@@ -11,7 +11,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Graphql do
end end
it 'has a valid feature category for every route', :aggregate_failures do it 'has a valid feature category for every route', :aggregate_failures do
feature_categories = YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).to_set feature_categories = Gitlab::FeatureCategories.default.categories
described_class::ROUTES.each do |route| described_class::ROUTES.each do |route|
expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid" expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid"
......
...@@ -107,7 +107,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Restful do ...@@ -107,7 +107,7 @@ RSpec.describe Gitlab::EtagCaching::Router::Restful do
end end
it 'has a valid feature category for every route', :aggregate_failures do it 'has a valid feature category for every route', :aggregate_failures do
feature_categories = YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).to_set feature_categories = Gitlab::FeatureCategories.default.categories
described_class::ROUTES.each do |route| described_class::ROUTES.each do |route|
expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid" expect(feature_categories).to include(route.feature_category), "#{route.name} has a category of #{route.feature_category}, which is not valid"
......
...@@ -349,7 +349,7 @@ RSpec.describe Gitlab::Metrics::RequestsRackMiddleware, :aggregate_failures do ...@@ -349,7 +349,7 @@ RSpec.describe Gitlab::Metrics::RequestsRackMiddleware, :aggregate_failures do
it 'has every label in config/feature_categories.yml' do it 'has every label in config/feature_categories.yml' do
defaults = [::Gitlab::FeatureCategories::FEATURE_CATEGORY_DEFAULT, 'not_owned'] defaults = [::Gitlab::FeatureCategories::FEATURE_CATEGORY_DEFAULT, 'not_owned']
feature_categories = YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).map(&:strip) + defaults feature_categories = Gitlab::FeatureCategories.default.categories + defaults
expect(described_class::FEATURE_CATEGORIES_TO_INITIALIZE).to all(be_in(feature_categories)) expect(described_class::FEATURE_CATEGORIES_TO_INITIALIZE).to all(be_in(feature_categories))
end end
......
...@@ -48,7 +48,7 @@ RSpec.describe 'Every Sidekiq worker' do ...@@ -48,7 +48,7 @@ RSpec.describe 'Every Sidekiq worker' do
describe "feature category declarations" do describe "feature category declarations" do
let(:feature_categories) do let(:feature_categories) do
YAML.load_file(Rails.root.join('config', 'feature_categories.yml')).map(&:to_sym).to_set Gitlab::FeatureCategories.default.categories.map(&:to_sym).to_set
end end
# All Sidekiq worker classes should declare a valid `feature_category` # All Sidekiq worker classes should declare a valid `feature_category`
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment