Remove grouped updates feature flags

updater/lib/dependabot/dependency_snapshot.rb

@@ -61,16 +61,13 @@ def job_dependencies
     # Returns just the group that is specifically requested to be updated by
     # the job definition
     def job_group
-      return nil unless Dependabot::Experiments.enabled?(:grouped_updates_prototype)
       return nil unless job.dependency_group_to_refresh
       return @job_group if defined?(@job_group)
 
       @job_group = @dependency_group_engine.find_group(name: job.dependency_group_to_refresh)
     end
 
     def groups
-      return [] unless Dependabot::Experiments.enabled?(:grouped_updates_prototype)
-
       @dependency_group_engine.dependency_groups
     end
 
@@ -92,8 +89,6 @@ def initialize(job:, base_commit_sha:, dependency_files:)
 
       @dependencies = parse_files!
 
-      return unless Dependabot::Experiments.enabled?(:grouped_updates_prototype)
-
       @dependency_group_engine = DependencyGroupEngine.from_job_config(job: job)
       @dependency_group_engine.assign_to_groups!(dependencies: allowed_dependencies)
     end

updater/lib/dependabot/updater/operations/group_update_all_versions.rb

@@ -23,7 +23,7 @@ def self.applies_to?(job:)
           return false if job.updating_a_pull_request?
           return false if job.dependencies&.any?
 
-          job.dependency_groups&.any? && Dependabot::Experiments.enabled?(:grouped_updates_prototype)
+          job.dependency_groups&.any?
         end
 
         def self.tag_name

updater/lib/dependabot/updater/operations/refresh_group_update_pull_request.rb

@@ -32,7 +32,7 @@ def self.applies_to?(job:)
           return false unless job.dependencies&.any?
           return false unless job.dependency_group_to_refresh
 
-          job.updating_a_pull_request? && Dependabot::Experiments.enabled?(:grouped_updates_prototype)
+          job.updating_a_pull_request?
         end
 
         def self.tag_name

updater/spec/dependabot/dependency_snapshot_spec.rb

@@ -106,8 +106,6 @@
       end
 
       it "correctly instantiates any configured dependency groups" do
-        Dependabot::Experiments.register("grouped_updates_prototype", true)
-
         snapshot = create_dependency_snapshot
 
         expect(snapshot.groups.length).to eql(1)
@@ -125,12 +123,6 @@
 
         Dependabot::Experiments.reset!
       end
-
-      it "ignores any configured dependency groups when the experiment is disabled" do
-        snapshot = create_dependency_snapshot
-
-        expect(snapshot.groups.length).to eql(0)
-      end
     end
 
     context "when there is a parser error" do

updater/spec/dependabot/updater/operations_spec.rb

@@ -37,38 +37,32 @@
       expect(described_class.class_for(job: job)).to be(Dependabot::Updater::Operations::UpdateAllVersions)
     end
 
-    context "the grouped update experiment is enabled" do
-      it "returns the GroupUpdateAllVersions class when the Job is for a fresh, version update with no dependencies" do
-        Dependabot::Experiments.register("grouped_updates_prototype", true)
-
-        job = instance_double(Dependabot::Job,
-                              security_updates_only?: false,
-                              updating_a_pull_request?: false,
-                              dependencies: [],
-                              dependency_groups: [anything],
-                              is_a?: true)
-
-        expect(described_class.class_for(job: job)).to be(Dependabot::Updater::Operations::GroupUpdateAllVersions)
-
-        Dependabot::Experiments.reset!
-      end
-
-      it "returns the RefreshGroupUpdatePullRequest class when the Job is for an existing group update" do
-        Dependabot::Experiments.register("grouped_updates_prototype", true)
-
-        job = instance_double(Dependabot::Job,
-                              security_updates_only?: false,
-                              updating_a_pull_request?: true,
-                              dependencies: [anything],
-                              dependency_group_to_refresh: anything,
-                              dependency_groups: [anything],
-                              is_a?: true)
-
-        expect(described_class.class_for(job: job))
-          .to be(Dependabot::Updater::Operations::RefreshGroupUpdatePullRequest)
-
-        Dependabot::Experiments.reset!
-      end
+    it "returns the GroupUpdateAllVersions class when the Job is for a fresh, version update with no dependencies" do
+      job = instance_double(Dependabot::Job,
+                            security_updates_only?: false,
+                            updating_a_pull_request?: false,
+                            dependencies: [],
+                            dependency_groups: [anything],
+                            is_a?: true)
+
+      expect(described_class.class_for(job: job)).to be(Dependabot::Updater::Operations::GroupUpdateAllVersions)
+
+      Dependabot::Experiments.reset!
+    end
+
+    it "returns the RefreshGroupUpdatePullRequest class when the Job is for an existing group update" do
+      job = instance_double(Dependabot::Job,
+                            security_updates_only?: false,
+                            updating_a_pull_request?: true,
+                            dependencies: [anything],
+                            dependency_group_to_refresh: anything,
+                            dependency_groups: [anything],
+                            is_a?: true)
+
+      expect(described_class.class_for(job: job))
+        .to be(Dependabot::Updater::Operations::RefreshGroupUpdatePullRequest)
+
+      Dependabot::Experiments.reset!
     end
 
     it "returns the RefreshVersionUpdatePullRequest class when the Job is for an existing dependency version update" do