From 11c2741dba3ac9559d64153395b75f6881fcd3d0 Mon Sep 17 00:00:00 2001
From: Shana Moore <shana@scientist.com>
Date: Wed, 22 Jan 2025 13:18:36 -0800
Subject: [PATCH] =?UTF-8?q?=E2=9C=85=20Update=20specs=20for=20UserStatColl?=
 =?UTF-8?q?ectionJob?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Added spec contexts to test behavior when UserStat records are present and absent.
- Defined a FactoryBot factory for UserStat to support spec setup.
---
 spec/factories/user_stats.rb               | 10 +++++++++
 spec/jobs/user_stat_collection_job_spec.rb | 25 ++++++++++++++++++----
 2 files changed, 31 insertions(+), 4 deletions(-)
 create mode 100644 spec/factories/user_stats.rb

diff --git a/spec/factories/user_stats.rb b/spec/factories/user_stats.rb
new file mode 100644
index 000000000..a4477022c
--- /dev/null
+++ b/spec/factories/user_stats.rb
@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+FactoryBot.define do
+  factory :user_stat do
+    user_id { create(:user).id }
+    date { Time.zone.today }
+    file_views { rand(1..100) }
+    file_downloads { rand(1..100) }
+    work_views { rand(1..100) }
+  end
+end
diff --git a/spec/jobs/user_stat_collection_job_spec.rb b/spec/jobs/user_stat_collection_job_spec.rb
index 3a15e5dcf..4a5396d13 100644
--- a/spec/jobs/user_stat_collection_job_spec.rb
+++ b/spec/jobs/user_stat_collection_job_spec.rb
@@ -13,10 +13,27 @@
 
   let(:account) { create(:account_with_public_schema) }
 
-  describe '#reenqueue' do
-    it 'Enques an TenantMaintenanceJob after perform' do
-      switch!(account)
-      expect { UserStatCollectionJob.perform_now }.to have_enqueued_job(UserStatCollectionJob)
+  describe '#perform' do
+    context 'when UserStat records exist' do
+      before do
+        FactoryBot.create(:user_stat) # Ensure a UserStat record exists
+      end
+      after do
+        # Ensure we reset to the default tenant after each test
+        Apartment::Tenant.switch!(Apartment.default_tenant)
+      end
+
+      it 'enqueues UserStatCollectionJob after perform' do
+        switch!(account)
+        expect { UserStatCollectionJob.perform_now }.to have_enqueued_job(UserStatCollectionJob)
+      end
+    end
+
+    context 'when no UserStat records exist' do
+      it 'does not enqueue UserStatCollectionJob' do
+        switch!(account)
+        expect { UserStatCollectionJob.perform_now }.not_to have_enqueued_job(UserStatCollectionJob)
+      end
     end
   end
 end