From ed83aee59e102e25b5fbe76901847f9d3d53e1e3 Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Wed, 12 Jan 2022 17:35:13 +0000
Subject: [PATCH] Failing spec

---
 spec/models/case_log_spec.rb                  | 30 ---------
 .../validations/property_validations_spec.rb  | 61 +++++++++++++++++++
 2 files changed, 61 insertions(+), 30 deletions(-)
 create mode 100644 spec/models/validations/property_validations_spec.rb

diff --git a/spec/models/case_log_spec.rb b/spec/models/case_log_spec.rb
index 0afdd3a6e..1c77b47d6 100644
--- a/spec/models/case_log_spec.rb
+++ b/spec/models/case_log_spec.rb
@@ -72,36 +72,6 @@ RSpec.describe Form, type: :model do
       }.to raise_error(ActiveRecord::RecordInvalid)
     end
 
-    it "validates number of relets is a number" do
-      expect {
-        CaseLog.create!(
-          offered: "random",
-          owning_organisation: owning_organisation,
-          managing_organisation: managing_organisation,
-        )
-      }.to raise_error(ActiveRecord::RecordInvalid)
-    end
-
-    it "validates number of relets is under 20" do
-      expect {
-        CaseLog.create!(
-          offered: 21,
-          owning_organisation: owning_organisation,
-          managing_organisation: managing_organisation,
-        )
-      }.to raise_error(ActiveRecord::RecordInvalid)
-    end
-
-    it "validates number of relets is over 0" do
-      expect {
-        CaseLog.create!(
-          offered: 0,
-          owning_organisation: owning_organisation,
-          managing_organisation: managing_organisation,
-        )
-      }.to raise_error(ActiveRecord::RecordInvalid)
-    end
-
     context "reasonable preference is yes" do
       it "validates a reason must be selected" do
         expect {
diff --git a/spec/models/validations/property_validations_spec.rb b/spec/models/validations/property_validations_spec.rb
new file mode 100644
index 000000000..17cea86ac
--- /dev/null
+++ b/spec/models/validations/property_validations_spec.rb
@@ -0,0 +1,61 @@
+require "rails_helper"
+require_relative "../../request_helper"
+
+RSpec.describe CaseLog do
+  let(:owning_organisation) { FactoryBot.create(:organisation) }
+  let(:managing_organisation) { owning_organisation }
+
+  before do
+    RequestHelper.stub_http_requests
+  end
+
+  describe "#new" do
+    it "raises an error when offered is present and invalid" do
+      expect {
+        CaseLog.create!(
+          offered: "random",
+          owning_organisation: owning_organisation,
+          managing_organisation: managing_organisation,
+        )
+      }.to raise_error(ActiveRecord::RecordInvalid)
+    end
+  end
+end
+
+RSpec.describe Validations::PropertyValidations do
+  let(:subject) { subject_class.new }
+  let(:subject_class) { Class.new { include Validations::PropertyValidations } }
+  let(:record) { FactoryBot.create(:case_log) }
+  let(:expected_error) { "Number of times property has been offered for relet must be a number between 0 and 20" }
+
+  describe "#validate_property_number_of_times_relet" do
+    it "does not add an error if the record offered is missing" do
+      record.offered = nil
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to be_empty
+    end
+
+    it "does not add an error if offered is valid (number between 0 and 20)" do
+      record.offered = 0
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to be_empty
+      record.offered = 10
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to be_empty
+      record.offered = 20
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to be_empty
+    end
+
+    it "does add an error when offered is invalid" do
+      record.offered = "invalid"
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to_not be_empty
+      expect(record.errors["offered"]).to include(match(expected_error))
+      record.offered = 21
+      subject.validate_property_number_of_times_relet(record)
+      expect(record.errors).to_not be_empty
+      expect(record.errors["offered"]).to include(match(expected_error))
+    end
+  end
+end