Remove graphql_logging feature flag code [RUN ALL RSPEC] [RUN AS-IF-FOSS]

changelogs/unreleased/35579-remove-graphql_logging-feature-flag.yml

+---
+title: Remove graphql_logging feature flag
+merge_request: 54984
+author:
+type: other

config/feature_flags/development/graphql_logging.yml

----
-name: graphql_logging
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/27885
-rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/35579
-milestone: '12.0'
-type: development
-group: group::project management
-default_enabled: true

lib/gitlab/graphql/query_analyzers/logger_analyzer.rb

@@ -9,10 +9,6 @@ module Gitlab
         FIELD_USAGE_ANALYZER = GraphQL::Analysis::FieldUsage.new { |query, used_fields, used_deprecated_fields| [used_fields, used_deprecated_fields] }
         ALL_ANALYZERS = [COMPLEXITY_ANALYZER, DEPTH_ANALYZER, FIELD_USAGE_ANALYZER].freeze
 
-        def analyze?(query)
-          Feature.enabled?(:graphql_logging, default_enabled: true)
-        end
-
         def initial_value(query)
           variables = process_variables(query.provided_variables)
           default_initial_values(query).merge({

spec/lib/gitlab/graphql/query_analyzers/logger_analyzer_spec.rb

@@ -5,26 +5,29 @@ require 'spec_helper'
 RSpec.describe Gitlab::Graphql::QueryAnalyzers::LoggerAnalyzer do
   subject { described_class.new }
 
-  describe '#analyze?' do
-    context 'feature flag disabled' do
-      before do
-        stub_feature_flags(graphql_logging: false)
-      end
+  describe '#initial_value' do
+    it 'filters out sensitive variables' do
+      doc = GraphQL.parse <<-GRAPHQL
+        mutation createNote($body: String!) {
+          createNote(input: {noteableId: "1", body: $body}) {
+            note {
+              id
+            }
+          }
+        }
+      GRAPHQL
+
+      query = GraphQL::Query.new(GitlabSchema, document: doc, context: {}, variables: { body: "some note" })
 
-      it 'disables the analyzer' do
-        expect(subject.analyze?(anything)).to be_falsey
+      expect(subject.initial_value(query)[:variables]).to eq('{:body=>"[FILTERED]"}')
     end
   end
 
-    context 'feature flag enabled by default' do
+  describe '#final_value' do
     let(:monotonic_time_before) { 42 }
     let(:monotonic_time_after) { 500 }
     let(:monotonic_time_duration) { monotonic_time_after - monotonic_time_before }
 
-      it 'enables the analyzer' do
-        expect(subject.analyze?(anything)).to be_truthy
-      end
-
     it 'returns a duration in seconds' do
       allow(GraphQL::Analysis).to receive(:analyze_query).and_return([4, 2, [[], []]])
       allow(Gitlab::Metrics::System).to receive(:monotonic_time).and_return(monotonic_time_before, monotonic_time_after)
@@ -39,23 +42,4 @@ RSpec.describe Gitlab::Graphql::QueryAnalyzers::LoggerAnalyzer do
       expect(memo[:duration_s]).to eq(expected_duration)
     end
   end
-  end
-
-  describe '#initial_value' do
-    it 'filters out sensitive variables' do
-      doc = GraphQL.parse <<-GRAPHQL
-        mutation createNote($body: String!) {
-          createNote(input: {noteableId: "1", body: $body}) {
-            note {
-              id
-            }
-          }
-        }
-      GRAPHQL
-
-      query = GraphQL::Query.new(GitlabSchema, document: doc, context: {}, variables: { body: "some note" })
-
-      expect(subject.initial_value(query)[:variables]).to eq('{:body=>"[FILTERED]"}')
-    end
-  end
 end