protected_record 0.0.0 → 0.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 5936b33d95c7b170f1c1f0a90dbb94ba3d5cddc5
+  data.tar.gz: 4d6bbe694afa07f22a4b04e93fc6dc13fc9819a6
+SHA512:
+  metadata.gz: fd08a85e169ce3ebc9e47f0217fd031d90ef01b1330640113e6b39c1d57a071cb0de107863b5269e2b3904716976131e19b4c974a64773eede60efa33b6d30e4
+  data.tar.gz: f5829512aba3077dc008b4fa47145ecda4c3f54713b50f679ef4f2d7f42da09c3043df93f1db0e1c3864e5a1b7fa6ba44444b05a40d2a771dd7c15147dde9f64

data/.ruby-gemset

@@ -0,0 +1 @@
+protected_record

data/.ruby-version

@@ -0,0 +1 @@
+ruby-2.0.0-p0

data/README.md

@@ -1,7 +1,23 @@
-Just include in your tracked models:
+# protected_record
+## Description
+protected_record will prevent changes to attributes you specify as protected. Any attempted change will be logged as a ProtectedRecord::ChangeRequest::Record. 
+If any changes are allowed through the filter, protected_record will create a ProtectedRecord::ChangeLog::Record to log who changed what, and for which record.
+ProtectedRecord is opt-in only. In order to update with protection, use the following: 
+
+    result = ProtectedRecord::UseCase::Update.new({
+      params: visit_params,
+      protected_record: @patient_visit,
+      user: current_user,
+      protected_keys: %w{ visit_date do_not_resuscitate }
+    }).execute!
+
+
+## Usage
+
+To get started, just include in your tracked models:
 
     include ProtectedRecord::ChangeRequest::Changeling
-    include ProtectedRecord::ChangeLog::Changeling
+    include ProtectedRecord::ChangeLog::Changeling 
 
 and include in your user model:
 

data/Rakefile

@@ -0,0 +1,9 @@
+require "bundler/gem_tasks"
+require "rake/testtask"
+
+Rake::TestTask.new do |t|
+  t.pattern = 'test/**/*_test.rb'
+  t.libs.push 'test'
+end
+
+task :default => :test

data/lib/protected_record/change_log.rb

@@ -1,7 +1,8 @@
+require 'active_record'
 module ProtectedRecord
   module ChangeLog
     class Record < ActiveRecord::Base
-      self.table_name = "change_log_records"
+      self.table_name = "protected_record_change_log_records"
       attr_accessible :recordable, :user
       belongs_to :recordable, polymorphic: true
       belongs_to :user, class_name: "User", foreign_key: :user_id

data/lib/protected_record/change_request.rb

@@ -1,7 +1,8 @@
+require 'active_record'
 module ProtectedRecord
   module ChangeRequest
     class Record < ActiveRecord::Base
-      self.table_name = 'change_request_records'
+      self.table_name = "protected_record_change_request_records"
       attr_accessible :recordable, :user
 
       belongs_to :recordable, polymorphic: true

data/lib/protected_record/use_case/change_filter/create.rb

@@ -0,0 +1,34 @@
+module ProtectedRecord
+  module UseCase
+    module ChangeFilter
+      class Create < PayDirt::Base
+        def initialize(options)
+          load_options(:protected_keys, :protected_record, options) and validate_state
+        end
+
+        def execute!
+          revert_protected_attrs
+
+          return PayDirt::Result.new(data: { change_request_record: @protected_record })
+        end
+
+        private
+        def revert_protected_attrs
+          @protected_keys.each do |key|
+            if @protected_record.send("#{key.to_s}_changed?")
+              @protected_record.send("#{key.to_s}=", @protected_record.send("#{key.to_s}_was"))
+            end
+          end
+        end
+
+        protected
+        def validate_state
+          # If there are no changes, there's no need to do any filtering
+          if !@protected_record.changes.present?
+            raise ActiveRecord::ActiveRecordError.new(':protected_record not dirty')
+          end
+        end
+      end
+    end
+  end
+end

data/lib/protected_record/use_case/change_log/create.rb

@@ -0,0 +1,41 @@
+module ProtectedRecord
+  module UseCase
+    module ChangeLog
+      class Create < PayDirt::Base
+        def initialize(options)
+          options = {
+            record_class: ::ProtectedRecord::ChangeLog::Record
+          }.merge!(options) if !options.has_key?(:record_class)
+
+          load_options(:record_class, :user, :changed_object, options) and validate_state
+        end
+
+        def execute!
+          initialize_change_log_record
+
+          if @record.save
+            return PayDirt::Result.new(data: { change_log_record: @record })
+          else
+            return PayDirt::Result.new(data: { change_log_record: @record }, success: false)
+          end
+        end
+
+        private
+        def initialize_change_log_record
+          @record                   = @record_class.new
+          @record.user              = @user
+          @record.recordable        = @changed_object
+          @record.observed_changes  = ActiveSupport::JSON.encode(@changed_object.previous_changes)
+        end
+
+        protected
+        def validate_state
+          # What are we doing here, if not logging a change that has already happened?
+          if !@changed_object.previous_changes.present?
+            raise ActiveRecord::ActiveRecordError.new(':changed_object has no previous_changes')
+          end
+        end
+      end
+    end
+  end
+end

data/lib/protected_record/use_case/change_request/create.rb

@@ -0,0 +1,60 @@
+module ProtectedRecord
+  module UseCase
+    module ChangeRequest
+      class Create < PayDirt::Base
+        def initialize(options)
+          options = {
+            record_class: ::ProtectedRecord::ChangeRequest::Record
+          }.merge!(options) if !options.has_key?(:record_class)
+
+          load_options(:protected_keys, :record_class, :user, :protected_record, options) and validate_state
+        end
+
+        def execute!
+          return PayDirt::Result.new(data: {}, success: true) if !requested_changes.present?
+
+          initialize_change_request_record
+
+          if @record.save
+            return PayDirt::Result.new(data: { change_request_record: @record })
+          else
+            return PayDirt::Result.new(data: { change_request_record: @record }, success: false)
+          end
+        end
+
+        private
+        def initialize_change_request_record
+          @record                   = @record_class.new
+          @record.user              = @user
+          @record.recordable        = @protected_record
+          @record.requested_changes = ActiveSupport::JSON.encode(requested_changes)
+        end
+
+        def requested_changes
+          @dirty_object.changes.select do |key|
+            @protected_keys.map(&:to_s).include? key.to_s
+          end
+        end
+
+        def revert_protected_attrs
+          @protected_keys.each do |key|
+            if @protected_record.send("#{key.to_s}_changed?")
+              @protected_record.send("#{key.to_s}=", @protected_record.send("#{key.to_s}_was"))
+            end
+          end
+
+          raise if @protected_keys.any? { |key| @protected_record.send("#{key}_changed?") }
+        end
+
+        protected
+        def validate_state
+          # If there are no changes, there's no need to make a request
+          if !@dirty_object.changes.present?
+            raise ActiveRecord::ActiveRecordError.new(':dirty_object not dirty')
+          end
+        end
+
+      end
+    end
+  end
+end

data/lib/protected_record/use_case/update.rb

@@ -0,0 +1,89 @@
+module ProtectedRecord
+  module UseCase
+    class Update < PayDirt::Base
+      def initialize(options)
+        # Defaults
+        options = {
+          change_request: UseCase::ChangeRequest::Create,
+          change_filter:  UseCase::ChangeFilter::Create,
+          change_log:     UseCase::ChangeLog::Create
+        }.merge!(options)
+
+        load_options(:params, :protected_record, :change_request, :change_log, :change_filter, :protected_keys, :user, options)
+        validate_state
+      end
+
+      def execute!
+        form_change_request
+
+        # We are successful if all changes have been applied
+        if !@protected_record.changes.present?
+          return UseCase::Result.new(data: { updated: @protected_record })
+        else
+          return UseCase::Result.new({
+            data: { remaining_changes: @protected_record.changes },
+            success: false
+          })
+        end
+      end
+
+      private
+      def form_change_request
+        @protected_record.attributes = @params
+
+        # We allow any changes on creation
+        unless @protected_record.id_was.nil?
+          request_result = @change_request.new({
+            protected_keys: @protected_keys,
+            protected_record: @protected_record,
+            user: @user
+          }).execute!
+
+          if request_result.successful?
+            revert_protected_attributes ||
+            save_protected_record
+          end
+        end
+      end
+
+      def save_protected_record
+        @protected_record.save
+        log_changes
+      end
+
+      def revert_protected_attributes
+        revert_result = @change_filter.new({
+          protected_keys: @protected_keys,
+          protected_record: @protected_record
+        }).execute!
+
+        revert_result.successful? ? return : raise
+      end
+
+      def log_changes
+        log_result = @change_log.new(user: @user, changed_object: @protected_record).execute!
+
+        result.successful? ? return : raise
+      end
+
+      protected
+      def validate_state
+        # We expect some keys
+        if !@protected_keys.kind_of?(Array)
+          raise TypeError.new(':protected_keys not kind of Array')
+        end
+
+        # The keys should respond to to_s
+        if @protected_keys.any? {|el| !el.respond_to?("to_s") }
+          raise TypeError.new('All :protected_keys should respond to #to_s')
+        end
+
+        # The dirty_object should respond to all keys (as methods)
+        @protected_keys.each do |key|
+          error = ActiveRecord::ActiveRecordError.new(":dirty_object must respond to #{key.to_s}")
+          raise error unless @protected_record.respond_to?(key.to_s)
+        end
+      end
+    end
+  end
+end

data/lib/protected_record/use_case.rb

@@ -0,0 +1,5 @@
+module ProtectedRecord
+  module UseCase
+    require "pay_dirt"
+  end
+end

data/lib/protected_record/version.rb

@@ -1,3 +1,3 @@
 module ProtectedRecord
-  VERSION = "0.0.0"
+  VERSION = "0.0.1"
 end

data/lib/protected_record.rb

@@ -1,5 +1,3 @@
-require "protected_record/use_case/base"
-
 Dir.glob(File.join(File.dirname(__FILE__), '/**/*.rb')) do |c|
   require(c)
 end

data/protected_record.gemspec

@@ -19,8 +19,9 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
+  s.add_runtime_dependency     "pay_dirt"
+  s.add_runtime_dependency "activerecord"
   s.add_development_dependency "minitest"
-
   s.add_development_dependency "rake"
   s.add_development_dependency "pry"
 end

data/test/protected_record/change_filter/create_test.rb

@@ -0,0 +1,134 @@
+require "minitest_helper"
+require "active_record"
+require "pry"
+
+describe ProtectedRecord::UseCase::ChangeFilter::Create do
+  describe "new" do
+    before do
+      class TestCase
+        include ActiveModel::Dirty
+
+        define_attribute_methods [:knowledge, :power]
+
+        def knowledge
+          @knowledge
+        end
+
+        def power
+          @power
+        end
+
+        def knowledge=(val)
+          knowledge_will_change! unless val == @knowledge
+          @knowledge = val
+        end
+
+        def power=(val)
+          power_will_change! unless val == @power
+          @power = val
+        end
+
+        def save
+          @previously_changed = changes
+          @changed_attributes.clear
+        end
+
+        def initialize(attributes = {})
+          attributes.each do |name, value|
+            send("#{name}=", value)
+          end
+        end
+        #####
+      end
+
+      @subject = ProtectedRecord::UseCase::ChangeFilter::Create
+
+      @protected_record = TestCase.new(knowledge: "power", power: "money")
+      @protected_record.save
+    end
+
+    it "initializes with required options" do
+      @protected_record.knowledge="growth"
+      @protected_record.power="knowledge"
+
+      @protected_keys = %w{ power }
+      @subject.new(protected_record: @protected_record, protected_keys: @protected_keys)
+    end
+
+    it "fails to initialize without :protected_record" do
+      begin
+        @subject.new(protected_keys: @protected_keys)
+      rescue => e
+        e.must_be_kind_of RuntimeError
+      end
+    end
+
+    it "fails to initialize without :protected_keys" do
+      begin
+        @subject.new(protected_record: @protected_record)
+      rescue => e
+        e.must_be_kind_of RuntimeError
+      end
+    end
+
+    it "returns a result" do
+      @protected_record.knowledge="growth"
+      @protected_record.power="knowledge"
+
+      @protected_keys = %w{ power }
+      @subject.new({
+        protected_record: @protected_record,
+        protected_keys: @protected_keys
+      }).execute!.must_be_kind_of(PayDirt::Result)
+    end
+
+    it "allows unprotected changes" do
+      @protected_keys = %w{}
+
+      @protected_record.knowledge="growth"
+      @protected_record.power="knowledge"
+
+      result = @subject.new({
+        protected_record: @protected_record,
+        protected_keys: @protected_keys
+      }).execute!
+
+      ret_obj = result.data[:change_request_record]
+
+      ret_obj.knowledge.must_equal "growth"
+      ret_obj.power.must_equal "knowledge"
+    end
+
+    it "filters all protected_keys" do
+      @protected_record.knowledge="growth"
+      @protected_record.power="knowledge"
+
+      @protected_keys = %w{ power knowledge }
+      result = @subject.new({
+        protected_record: @protected_record,
+        protected_keys: @protected_keys
+      }).execute!
+
+      ret_obj = result.data[:change_request_record]
+
+      ret_obj.knowledge.must_equal "power"
+      ret_obj.power.must_equal "money"
+    end
+
+    it "will allow unprotected changes while reverting protected changes" do
+      @protected_record.knowledge="growth"
+      @protected_record.power="knowledge"
+
+      @protected_keys = %w{ power }
+      result = @subject.new({
+        protected_record: @protected_record,
+        protected_keys: @protected_keys
+      }).execute!
+
+      ret_obj = result.data[:change_request_record]
+
+      ret_obj.knowledge.must_equal "growth"
+      ret_obj.power.must_equal "money"
+    end
+  end
+end

data/test/protected_record/change_log/create_test.rb

@@ -0,0 +1,2 @@
+require "minitest_helper"
+

data/test/protected_record/change_request/create_test.rb

@@ -0,0 +1,2 @@
+require "minitest_helper"
+

data/test/protected_record/update_test.rb

@@ -0,0 +1 @@
+require "minitest_helper"

metadata

@@ -1,73 +1,88 @@
 --- !ruby/object:Gem::Specification
 name: protected_record
 version: !ruby/object:Gem::Version
-  prerelease:
-  version: 0.0.0
+  version: 0.0.1
 platform: ruby
 authors:
 - Tad Hosford
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-19 00:00:00.000000000 Z
+date: 2013-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: pay_dirt
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
+        version: '0'
+  type: :runtime
   prerelease: false
-  type: :development
-- !ruby/object:Gem::Dependency
-  name: rake
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
-  prerelease: false
+        version: '0'
   type: :development
-- !ruby/object:Gem::Dependency
-  name: pry
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - '>='
       - !ruby/object:Gem::Version
-        version: !binary |-
-          MA==
-    none: false
+        version: '0'
+  type: :development
   prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-description: ! "\n                      Filters changes & logs changes to protected\
-  \ records.\n                      Creates change requests when changes are attempted\
-  \ on protected attrs\n                    "
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: "\n                      Filters changes & logs changes to protected
+  records.\n                      Creates change requests when changes are attempted
+  on protected attrs\n                    "
 email:
 - tad.hosford@gmail.com
 executables: []
@@ -75,49 +90,47 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .ruby-gemset
+- .ruby-version
 - Gemfile
 - README.md
+- Rakefile
 - lib/protected_record.rb
 - lib/protected_record/change_log.rb
 - lib/protected_record/change_request.rb
-- lib/protected_record/use_case/base.rb
-- lib/protected_record/use_case/change/change_log/create.rb
-- lib/protected_record/use_case/change/change_request/create.rb
-- lib/protected_record/use_case/change/update.rb
-- lib/protected_record/use_case/result.rb
+- lib/protected_record/use_case.rb
+- lib/protected_record/use_case/change_filter/create.rb
+- lib/protected_record/use_case/change_log/create.rb
+- lib/protected_record/use_case/change_request/create.rb
+- lib/protected_record/use_case/update.rb
 - lib/protected_record/version.rb
 - protected_record.gemspec
 - test/minitest_helper.rb
+- test/protected_record/change_filter/create_test.rb
+- test/protected_record/change_log/create_test.rb
+- test/protected_record/change_request/create_test.rb
+- test/protected_record/update_test.rb
 homepage: http://github.com/rthbound/protected_record
 licenses: []
-post_install_message:
+metadata: {}
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
-      segments:
-      - 0
-      hash: 2
-      version: !binary |-
-        MA==
-  none: false
+      version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - '>='
     - !ruby/object:Gem::Version
-      segments:
-      - 0
-      hash: 2
-      version: !binary |-
-        MA==
-  none: false
+      version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 1.8.24
-signing_key:
-specification_version: 3
+rubyforge_project: 
+rubygems_version: 2.0.0.rc.2
+signing_key: 
+specification_version: 4
 summary: ''
 test_files: []

data/lib/protected_record/use_case/base.rb

@@ -1,21 +0,0 @@
-module ProtectedRecord
-  module UseCase
-    class Base
-      def load_option(option, options)
-        instance_variable_set("@#{option}", options.fetch(option.to_sym) { raise "Missing required option: #{option}" } )
-     end
-
-      def load_options(*option_names, options)
-        option_names.each{|o| load_option(o, options) }
-        @errors ||= []
-      end
-
-      def errors
-        @errors
-      end
-    end
-  end
-end
-
-
-

data/lib/protected_record/use_case/change/change_log/create.rb

@@ -1,43 +0,0 @@
-module ProtectedRecord
-  module UseCase
-    module Change
-      module ChangeLog
-        class Create < ::ProtectedRecord::UseCase::Base
-          def initialize(options)
-            options = {
-              record_class: ::ProtectedRecord::ChangeLog::Record
-            }.merge!(options) if !options.has_key?(:record_class)
-
-            load_options(:record_class, :user, :changed_object, options) and validate_state
-          end
-
-          def execute!
-            create_change_log_record if @errors.empty?
-
-            if @errors.empty? && @record.save
-              return UseCase::Result.new(data: { change_log_record: @record })
-            else
-              return UseCase::Result.new(data: { change_log_record: @record }, errors: @errors)
-            end
-          end
-
-          private
-          def create_change_log_record
-            @record                   = @record_class.new
-            @record.user              = @user
-            @record.recordable        = @changed_object
-            @record.observed_changes  = ActiveSupport::JSON.encode(@changed_object.previous_changes)
-          end
-
-          protected
-          def validate_state
-            # What are we doing here, if not logging a change that has already happened?
-            if !@changed_object.previous_changes.present?
-              @errors << ActiveRecord::ActiveRecordError.new(':changed_object has no previous_changes')
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/lib/protected_record/use_case/change/change_request/create.rb

@@ -1,51 +0,0 @@
-module ProtectedRecord
-  module UseCase
-    module Change
-      module ChangeRequest
-        class Create < ::ProtectedRecord::UseCase::Base
-          def initialize(options)
-            options = {
-              record_class: ::ProtectedRecord::ChangeRequest::Record
-            }.merge!(options) if !options.has_key?(:record_class)
-
-            load_options(:protected_keys, :record_class, :user, :dirty_object, options) and validate_state
-          end
-
-          def execute!
-            return UseCase::Result.new(data: nil, success: true) if !requested_changes.present?
-
-            create_change_request_record if @errors.empty?
-
-            if @errors.empty? && @record.save
-              return UseCase::Result.new(data: { change_request_record: @record })
-            else
-              return UseCase::Result.new(data: { change_request_record: @record }, errors: @errors)
-            end
-          end
-
-          private
-          def create_change_request_record
-            @record                   = @record_class.new
-            @record.user              = @user
-            @record.recordable        = @dirty_object
-            @record.requested_changes = ActiveSupport::JSON.encode(requested_changes)
-          end
-
-          def requested_changes
-            @dirty_object.changes.select do |key|
-              @protected_keys.map(&:to_s).include? key.to_s
-            end
-          end
-
-          protected
-          def validate_state
-            # If there are no changes, there's no need to make a request
-            if !@dirty_object.changes.present?
-              @errors << ActiveRecord::ActiveRecordError.new(':dirty_object not dirty')
-            end
-          end
-        end
-      end
-    end
-  end
-end

data/lib/protected_record/use_case/change/update.rb

@@ -1,80 +0,0 @@
-module ProtectedRecord
-  module UseCase
-    module Change
-      class Update < ::ProtectedRecord::UseCase::Base
-        def initialize(options)
-          # Defaults
-          options = {
-            change_request_use_case: UseCase::Change::ChangeRequest::Create,
-            change_log_use_case:     UseCase::Change::ChangeLog::Create
-          }.merge!(options) if !options.has_key?(:record_class)
-
-          load_options(:params, :protected_record, :change_request_use_case, :change_log_use_case, :protected_keys, :user, options)
-          validate_state
-        end
-
-        def execute!
-          form_change_request
-
-          if @errors.empty? && !@protected_record.changes.present?
-            return UseCase::Result.new(data: { params: @protected_record })
-          else
-            return UseCase::Result.new(data: { params: @protected_record }, errors: @errors)
-          end
-        end
-
-        private
-        def form_change_request
-          @protected_record.attributes = @params
-
-          unless @protected_record.id_was.nil?
-            request_result = @change_request_use_case.new(protected_keys: @protected_keys, dirty_object: @protected_record, user: @user).execute!
-            if request_result.successful?
-              revert_protected_attrs
-              save_protected_record
-            end
-          end
-        end
-
-        def revert_protected_attrs
-          @protected_keys.each do |key|
-            if @protected_record.send("#{key.to_s}_changed?")
-              @protected_record.send("#{key.to_s}=", @protected_record.send("#{key.to_s}_was"))
-            end
-          end
-
-          raise if @protected_keys.any? { |key| @protected_record.send("#{key}_changed?") }
-        end
-
-        def save_protected_record
-          @protected_record.save
-          log_changes
-        end
-
-        def log_changes
-          log_result = @change_log_use_case.new(user: @user, changed_object: @protected_record).execute!
-          return true if log_result.successful?
-        end
-
-        protected
-        def validate_state
-          # We expect some keys
-          if !@protected_keys.kind_of?(Array)
-            @errors << TypeError.new(':protected_keys not kind of Array')
-          end
-
-          # The keys should respond to to_s
-          if @protected_keys.any? {|el| !el.respond_to?("to_s") }
-            @errors << TypeError.new('All :protected_keys should respond to #to_s')
-          end
-
-          # The dirty_object should respond to all keys (as methods)
-          @protected_keys.each do |key|
-            error = ActiveRecord::ActiveRecordError.new(":dirty_object must respond to #{key.to_s}")
-            @errors << error unless @protected_record.respond_to?(key.to_s)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/protected_record/use_case/result.rb

@@ -1,40 +0,0 @@
-module ProtectedRecord
-  module UseCase
-    class Result
-      # The response from a use case execution
-      #
-      # Every use case should return a Result after it runs.
-      #
-      # @param [options] options_hash
-      # A hash specifying the appropriate options
-      #
-      # @return [UseCase::Result]
-      # the Result instance
-      #
-      # @example
-      # UseCase::Result.new(success: true, data: {})
-      # # => <UseCase::Result>
-      #
-      # @api public
-      def initialize(options)
-        @success = options.fetch(:success) { !options[:errors].present? }
-        @errors = options[:errors]
-        @data = options[:data]
-      end
-
-      # @api public
-      def successful?
-        !!@success
-      end
-
-      def errors
-        @errors
-      end
-
-      # @api public
-      def data
-        @data
-      end
-    end
-  end
-end