Adds model validations and test units

app/models/record.rb

@@ -1,4 +1,8 @@
 class Record < ApplicationRecord
   belongs_to :user
   belongs_to :task
+
+  validates :duration, presence: :true
+  validates :task, presence: true
+  validates :user, presence: true
 end

db/migrate/20210511212634_create_records.rb

@@ -2,10 +2,10 @@ class CreateRecords < ActiveRecord::Migration[6.1]
   def change
     create_table :records do |t|
       t.integer :duration, null: false
-      t.boolean :is_handled, default: false
-      t.references :handled_by, default: :null
       t.references :user, null: false
       t.references :task, null: false
+      t.boolean :is_handled, default: false
+      t.references :handled_by, default: nil
 
       t.timestamps
     end

db/schema.rb

@@ -34,10 +34,10 @@ ActiveRecord::Schema.define(version: 2021_05_11_212634) do
 
   create_table "records", force: :cascade do |t|
     t.integer "duration", null: false
-    t.boolean "is_handled", default: false
-    t.integer "handled_by_id"
     t.integer "user_id", null: false
     t.integer "task_id", null: false
+    t.boolean "is_handled", default: false
+    t.integer "handled_by_id"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
     t.index ["handled_by_id"], name: "index_records_on_handled_by_id"

test/fixtures/records.yml

@@ -1,13 +1,7 @@
 # Read about fixtures at https://api.rubyonrails.org/classes/ActiveRecord/FixtureSet.html
 
 one:
-  duration: 2021-05-11 23:26:34
-  handled: false
+  duration: 666
   user: one
   task: one
-
-two:
-  duration: 2021-05-11 23:26:34
-  handled: false
-  user: two
-  task: two
+  is_handled: false

test/models/record_test.rb

@@ -1,7 +1,31 @@
 require "test_helper"
 
 class RecordTest < ActiveSupport::TestCase
-  # test "the truth" do
-  #   assert true
+  setup do
+    @user = users(:one)
+    @task = tasks(:one)
+    @record = records(:one)
+  end
+
+  test "record should have valid duration, user and task ids" do
+    record = Record.new(duration: 100, user_id: @user.id, task_id: @task.id)
+    assert record.valid?
+  end
+
+  test "record without a duration should not be valid" do
+    record = Record.new(user_id: @user.id, task_id: @task.id)
+    assert_not record.valid?
+  end
+
+  test "record without user and task ids should not be valid" do
+    record = Record.new(duration: 100)
+    assert_not record.valid?
+  end
+
+  # NOT SURE THE FOLLOWING TEST IS TESTING ANYTHING USEFULL
+  # test "handled records should have a valid handled_by value" do
+  #   @record.update(is_handled: true)
+  #   assert @record.valid?
   # end
+
 end