From 72c5ab1f553f376f3d562360fe83346f01368a74 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               |  9 +++++++++
 spec/jobs/user_stat_collection_job_spec.rb | 21 +++++++++++++++++----
 2 files changed, 26 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..aea0bdb8b
--- /dev/null
+++ b/spec/factories/user_stats.rb
@@ -0,0 +1,9 @@
+FactoryBot.define do
+  factory :user_stat do
+    user_id { create(:user).id }
+    date { Date.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..09a288750 100644
--- a/spec/jobs/user_stat_collection_job_spec.rb
+++ b/spec/jobs/user_stat_collection_job_spec.rb
@@ -13,10 +13,23 @@
 
   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
+
+      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