Merge branch 'prevent_future_licenses_from_being_the_current' into 'master'

Change logic to find the current license

See merge request gitlab-org/gitlab!30296

ee/app/models/license.rb

@@ -267,11 +267,7 @@ class License < ApplicationRecord
     def load_license
       return unless self.table_exists?
 
-      license = self.last
-
-      return unless license && license.valid?
-
-      license
+      self.order(id: :desc).limit(100).find { |license| license.valid? && license.started? }
     end
 
     def global_feature?(feature)

ee/changelogs/unreleased/prevent_future_licenses_from_being_the_current.yml

+---
+title: Change logic to find the current license
+merge_request: 30296
+author:
+type: changed

ee/spec/factories/licenses.rb

@@ -19,8 +19,8 @@ FactoryBot.define do
       plan { License::STARTER_PLAN }
     end
 
-    starts_at { Date.today - 1.month }
-    expires_at { Date.today + 11.months }
+    starts_at { Date.new(1970, 1, 1) }
+    expires_at { Date.current + 11.months }
     block_changes_at { expires_at + 2.weeks }
     notify_users_at  { expires_at }
     notify_admins_at { expires_at }

ee/spec/models/historical_data_spec.rb

@@ -102,7 +102,7 @@ describe HistoricalData do
     end
 
     context 'with data outside of the license period' do
-      let!(:license) { create(:license) }
+      let!(:license) { create(:license, starts_at: Date.current - 1.month) }
 
       context 'with stats before the license period' do
         before do

ee/spec/models/license_spec.rb

@@ -116,7 +116,7 @@ describe License do
         end
 
         context "after the license started" do
-          let(:date) { Date.today }
+          let(:date) { Date.current }
 
           it "is valid" do
             expect(license).to be_valid
@@ -249,7 +249,7 @@ describe License do
     describe 'downgrade' do
       context 'when more users were added in previous period' do
         before do
-          HistoricalData.create!(date: 6.months.ago, active_user_count: 15)
+          HistoricalData.create!(date: described_class.current.starts_at - 6.months, active_user_count: 15)
 
           set_restrictions(restricted_user_count: 5, previous_user_count: 10)
         end
@@ -274,11 +274,8 @@ describe License do
   end
 
   describe "Class methods" do
-    let!(:license) { described_class.last }
-
     before do
       described_class.reset_current
-      allow(described_class).to receive(:last).and_return(license)
     end
 
     describe '.features_for_plan' do
@@ -343,46 +340,48 @@ describe License do
 
     describe ".current" do
       context 'when licenses table does not exist' do
-        before do
+        it 'returns nil' do
           allow(described_class).to receive(:table_exists?).and_return(false)
-        end
 
-        it 'returns nil' do
           expect(described_class.current).to be_nil
         end
       end
 
       context "when there is no license" do
-        let!(:license) { nil }
-
         it "returns nil" do
+          allow(described_class).to receive(:order).and_return(double(limit: []))
+
           expect(described_class.current).to be_nil
         end
       end
 
       context "when the license is invalid" do
-        before do
+        it "returns nil" do
+          allow(described_class).to receive(:order).and_return(double(limit: [license]))
           allow(license).to receive(:valid?).and_return(false)
-        end
 
-        it "returns nil" do
           expect(described_class.current).to be_nil
         end
       end
 
       context "when the license is valid" do
         it "returns the license" do
-          expect(described_class.current).to be_present
+          current_license = create_list(:license, 2).last
+          create(:license, data: create(:gitlab_license, starts_at: Date.current + 1.month).export)
+
+          expect(described_class.current).to eq(current_license)
         end
       end
     end
 
     describe ".block_changes?" do
-      context "when there is no current license" do
       before do
-          allow(described_class).to receive(:current).and_return(nil)
+        allow(License).to receive(:current).and_return(license)
       end
 
+      context "when there is no current license" do
+        let(:license) { nil }
+
         it "returns false" do
           expect(described_class.block_changes?).to be_falsey
         end
@@ -779,12 +778,14 @@ describe License do
   end
 
   def set_restrictions(opts)
+    date = described_class.current.starts_at
+
     gl_license.restrictions = {
       active_user_count: opts[:restricted_user_count],
       previous_user_count: opts[:previous_user_count],
       trueup_quantity: opts[:trueup_quantity],
-      trueup_from: (Date.today - 1.year).to_s,
-      trueup_to: Date.today.to_s
+      trueup_from: (date - 1.year).to_s,
+      trueup_to: date.to_s
     }
   end
 

ee/spec/requests/api/license_spec.rb

@@ -32,7 +32,7 @@ describe API::License, api: true do
       get api('/license', admin)
       expect(response).to have_gitlab_http_status(:ok)
       expect(json_response['user_limit']).to eq 0
-      expect(Date.parse(json_response['starts_at'])).to eq Date.today - 1.month
+      expect(Date.parse(json_response['starts_at'])).to eq Date.new(1970, 1, 1)
       expect(Date.parse(json_response['expires_at'])).to eq Date.today + 11.months
       expect(json_response['active_users']).to eq 1
       expect(json_response['licensee']).not_to be_empty
@@ -51,7 +51,7 @@ describe API::License, api: true do
 
       expect(response).to have_gitlab_http_status(:created)
       expect(json_response['user_limit']).to eq 0
-      expect(Date.parse(json_response['starts_at'])).to eq Date.today - 1.month
+      expect(Date.parse(json_response['starts_at'])).to eq Date.new(1970, 1, 1)
       expect(Date.parse(json_response['expires_at'])).to eq Date.today + 11.months
       expect(json_response['active_users']).to eq 1
       expect(json_response['licensee']).not_to be_empty