Modify CVD Risk for range-score and stale-delete (#5508)

**Story card:**
[sc-14311](https://app.shortcut.com/simpledotorg/story/14311/modify-cvd-risk-for-range-score-and-stale-delete)

## Because

- We would need to give the risk scores as a range sometimes
- Risk scores may need to be deleted when they've gone _stale_

## This addresses

- The blocker building out the CVD sync resource on the Android side

## Test instructions

- suite tests

app/controllers/api/v4/cvd_risks_controller.rb

@@ -32,6 +32,7 @@ def cvd_risk_params
         :id,
         :patient_id,
         :risk_score,
+        :deleted_at,
         :created_at,
         :updated_at
       )

app/schema/api/v4/models.rb

@@ -22,7 +22,8 @@ def cvd_risk
        properties: {
          id: {"$ref" => "#/definitions/uuid"},
          patient_id: {"$ref" => "#/definitions/uuid"},
-         risk_score: {type: :number},
+         risk_score: {"$ref" => "#/definitions/non_empty_string"},
+         deleted_at: {"$ref" => "#/definitions/nullable_timestamp"},
          created_at: {"$ref" => "#/definitions/timestamp"},
          updated_at: {"$ref" => "#/definitions/timestamp"}
        },

app/validators/api/v4/cvd_risk_payload_validator.rb

@@ -1,5 +1,5 @@
 class Api::V4::CvdRiskPayloadValidator < Api::V4::PayloadValidator
-  attr_accessor :id, :patient_id, :risk_score, :created_at, :updated_at
+  attr_accessor :id, :patient_id, :risk_score, :deleted_at, :created_at, :updated_at
 
   validate :validate_schema
 

db/migrate/20241210092449_change_cvd_risk_score_to_string.rb

@@ -0,0 +1,5 @@
+class ChangeCvdRiskScoreToString < ActiveRecord::Migration[6.1]
+  def change
+    change_column(:cvd_risks, :risk_score, :string)
+  end
+end

db/structure.sql

@@ -593,7 +593,7 @@ ALTER SEQUENCE public.cphc_migration_error_logs_id_seq OWNED BY public.cphc_migr
 
 CREATE TABLE public.cvd_risks (
     id uuid DEFAULT gen_random_uuid() NOT NULL,
-    risk_score integer,
+    risk_score character varying,
     patient_id uuid NOT NULL,
     deleted_at timestamp without time zone,
     device_created_at timestamp without time zone,
@@ -7858,6 +7858,7 @@ INSERT INTO "schema_migrations" (version) VALUES
 ('20241125155649'),
 ('20241126111757'),
 ('20241129212725'),
-('20241204155510');
+('20241204155510'),
+('20241210092449');
 
 

spec/factories/cvd_risks.rb

@@ -2,7 +2,7 @@
   factory :cvd_risk do
     id { SecureRandom.uuid }
     patient
-    risk_score { 42 }
+    risk_score { "42" }
     device_created_at { Time.current }
     device_updated_at { Time.current }
 

swagger/v4/swagger.json

@@ -1898,7 +1898,10 @@
           "$ref": "#/definitions/uuid"
         },
         "risk_score": {
-          "type": "number"
+          "$ref": "#/definitions/non_empty_string"
+        },
+        "deleted_at": {
+          "$ref": "#/definitions/nullable_timestamp"
         },
         "created_at": {
           "$ref": "#/definitions/timestamp"