From 9a0638cf9e7089528e95831fec2b525e79e0f057 Mon Sep 17 00:00:00 2001
From: Sebastian Riedel <s@kraih.com>
Date: Mon, 14 Oct 2024 15:18:28 +0200
Subject: [PATCH] Also show rejected packages in stats

---
 assets/vue/CavilStatistics.vue | 14 +++++++++++---
 lib/Cavil/Model/Packages.pm    |  1 +
 t/stats.t                      |  2 +-
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/assets/vue/CavilStatistics.vue b/assets/vue/CavilStatistics.vue
index 323925c98..0de7a98b5 100644
--- a/assets/vue/CavilStatistics.vue
+++ b/assets/vue/CavilStatistics.vue
@@ -1,5 +1,11 @@
 <template>
   <div class="row">
+    <div class="col-md-2">
+      <div class="stats">
+        <div class="stats-body">{{ activePackages }}</div>
+        <div class="stats-description">Active Packages</div>
+      </div>
+    </div>
     <div class="col-md-2">
       <div class="stats">
         <div class="stats-body">{{ openReviews }}</div>
@@ -8,8 +14,8 @@
     </div>
     <div class="col-md-2">
       <div class="stats">
-        <div class="stats-body">{{ activePackages }}</div>
-        <div class="stats-description">Active Packages</div>
+        <div class="stats-body">{{ rejectedPackages }}</div>
+        <div class="stats-description">Rejected Packages</div>
       </div>
     </div>
     <div class="col-md-2">
@@ -33,7 +39,8 @@ export default {
       embargoedPackages: 0,
       openReviews: 0,
       refreshDelay: 120000,
-      refreshUrl: '/stats/meta'
+      refreshUrl: '/stats/meta',
+      rejectedPackages: 0
     };
   },
   methods: {
@@ -41,6 +48,7 @@ export default {
       this.activePackages = data.active_packages;
       this.embargoedPackages = data.embargoed_packages;
       this.openReviews = data.open_reviews;
+      this.rejectedPackages = data.rejected_packages;
     }
   }
 };
diff --git a/lib/Cavil/Model/Packages.pm b/lib/Cavil/Model/Packages.pm
index 69bed63f3..39763dce1 100644
--- a/lib/Cavil/Model/Packages.pm
+++ b/lib/Cavil/Model/Packages.pm
@@ -491,6 +491,7 @@ sub stats {
     "SELECT
        (SELECT COUNT(*) FROM bot_packages WHERE obsolete = false) AS active_packages,
        (SELECT COUNT(*) FROM bot_packages WHERE obsolete = false AND embargoed = true) AS embargoed_packages,
+       (SELECT COUNT(*) FROM bot_packages WHERE obsolete = false AND state = 'unacceptable') AS rejected_packages,
        (SELECT COUNT(*) FROM bot_packages WHERE obsolete = false AND state = 'new') AS open_reviews"
   )->hash;
 
diff --git a/t/stats.t b/t/stats.t
index 0ca903305..a274d9b16 100644
--- a/t/stats.t
+++ b/t/stats.t
@@ -33,7 +33,7 @@ subtest 'Statistics' => sub {
 
   $t->get_ok('/stats')->status_is(200)->content_like(qr/id="statistics"/);
   $t->get_ok('/stats/meta')->status_is(200)->json_is('/active_packages', 2)->json_is('/embargoed_packages', 0)
-    ->json_is('/open_reviews' => 2);
+    ->json_is('/open_reviews' => 2)->json_is('/rejected_packages' => 0);
 
   $t->get_ok('/logout')->status_is(302)->header_is(Location => '/');
 };