moving simplecov progress to its own service

app/controllers/projects_controller.rb

@@ -1,4 +1,6 @@
 class ProjectsController < ApplicationController
+  helper_method :simplecov_progress
+
   def create
     @project = Project.new(create_params)
 
@@ -26,6 +28,10 @@ def update
 
   private
 
+  memoize def simplecov_progress
+    ::Projects::SimplecovProgress.new(@project).call
+  end
+
   def create_params
     params
       .require(:project)

app/models/project.rb

@@ -19,12 +19,4 @@ def reset_api_auth!
     update!(api_auth_digest: Digest::SHA512.hexdigest(token))
     token
   end
-
-  def reports_coverage_progress
-    reports.map do |report|
-      next if report.general_coverage.blank?
-
-      [ report.created_at, report.general_coverage ]
-    end.compact
-  end
 end

app/services/projects/simplecov_progress.rb

@@ -0,0 +1,19 @@
+module Projects
+  class SimplecovProgress
+    attr_reader :project
+
+    def initialize(project)
+      @project = project
+    end
+
+    def call
+      # TODO: could be improved to only get the last report of each date.
+      # TODO: could be improved to remove blank coverage reports at the SQL level.
+      project.reports.where(branch: "main").order(:created_at).map do |report|
+        next if report.general_coverage.blank?
+
+        [ report.created_at, report.general_coverage ]
+      end.compact
+    end
+  end
+end

app/views/projects/show.html.erb

@@ -4,7 +4,7 @@
 
 <%=
   line_chart(
-    @project.reports_coverage_progress,
+    simplecov_progress,
     library: { scales: { y: { type: 'logarithmic' } } }
   )
 %>