From 4918ffcd7492661f36a763cdf248757993f9bff4 Mon Sep 17 00:00:00 2001
From: Teddy Wing
Date: Tue, 19 Dec 2017 16:35:09 +0100
Subject: Remove `hstore_accessor` usage

The places we were using `hstore_accessor` have now been converted to
Postgres JSON fields. Thus we no longer need `hstore_accessor` in order
to add accessors for the fields.

Instead we can use Rails' built-in support for Postgres JSON fields and
create accessors for our JSON hash keys with `store_accessor`. We get
rid of the data types because `store_accessor` doesn't work like that
and JSON already types our values with primitive data types.

Finally, add some extra validation tests now that the vaildation for our
JSON fields should work.

Refs #5316
---
 spec/support/shared_examples/compliance_control_validation.rb | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'spec/support')

diff --git a/spec/support/shared_examples/compliance_control_validation.rb b/spec/support/shared_examples/compliance_control_validation.rb
index d4ab9f41d..c76712c4c 100644
--- a/spec/support/shared_examples/compliance_control_validation.rb
+++ b/spec/support/shared_examples/compliance_control_validation.rb
@@ -1,6 +1,9 @@
 RSpec.shared_examples_for 'has min_max_values' do
 
   context "is valid" do
+    it { should validate_numericality_of(:minimum) }
+    it { should validate_numericality_of(:maximum) }
+
     it 'if no value is provided' do
       expect_it.to be_valid
     end
@@ -41,3 +44,10 @@ RSpec.shared_examples_for 'has min_max_values' do
     end
   end
 end
+
+
+RSpec.shared_examples_for "has target attribute" do
+  context "is valid" do
+    it { should validate_presence_of(:target) }
+  end
+end
-- 
cgit v1.2.3