Refactor and Finalize Session Tracking (Building on closed unmerged #74)

app/models/marking_session.rb

@@ -15,6 +15,14 @@ def duration_minutes
 
   def update_session_details
     now = DateTime.now
-    update(end_time: now)
+    if start_time.present?
+      duration = ((now.to_f - start_time.to_f) / 60).to_i
+      update(
+        end_time: now,
+        duration_minutes: duration
+      )
+    else
+      update(end_time: now, duration_minutes: 0)
+    end
   end
 end

app/services/session_tracker.rb

@@ -18,10 +18,10 @@ def self.record_assessment_activity(action:, user:, project:, ip_address:, task:
       project_id: project.id,
       task_id: task&.id,
       task_definition_id: task&.task_definition_id,
-      created_at: Time.zone.now
+      created_at: DateTime.now
     )
 
-    session.update_session_details
+    session.update_session_details if action == 'assessing'
 
     activity
   end

db/migrate/20250922103033_create_marking_sessions.rb

@@ -6,6 +6,7 @@ def change
       t.string :ip_address
       t.datetime :start_time
       t.datetime :end_time
+      t.integer :duration_minutes, default: 0
       t.boolean :during_tutorial
 
       t.timestamps

db/schema.rb

@@ -191,6 +191,7 @@
     t.string "ip_address"
     t.datetime "start_time"
     t.datetime "end_time"
+    t.integer "duration_minutes", default: 0
     t.boolean "during_tutorial"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false