Adds Rubocop rule for line break after guard clause

Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.

4 files changed, 10 insertions, 0 deletions

diff --git a/lib/declarative_policy/base.rb b/lib/declarative_policy/base.rb
index b028169f500..47542194497 100644
--- a/lib/declarative_policy/base.rb
+++ b/lib/declarative_policy/base.rb
@@ -276,6 +276,7 @@ module DeclarativePolicy
     # boolean `false`
     def cache(key, &b)
       return @cache[key] if cached?(key)
+
       @cache[key] = yield
     end
 
@@ -291,6 +292,7 @@ module DeclarativePolicy
       @_conditions[name] ||=
         begin
           raise "invalid condition #{name}" unless self.class.conditions.key?(name)
+
           ManifestCondition.new(self.class.conditions[name], self)
         end
     end
diff --git a/lib/declarative_policy/cache.rb b/lib/declarative_policy/cache.rb
index 0804edba016..780d8f707bd 100644
--- a/lib/declarative_policy/cache.rb
+++ b/lib/declarative_policy/cache.rb
@@ -3,6 +3,7 @@ module DeclarativePolicy
     class << self
       def user_key(user)
         return '<anonymous>' if user.nil?
+
         id_for(user)
       end
 
@@ -15,6 +16,7 @@ module DeclarativePolicy
       def subject_key(subject)
         return '<nil>' if subject.nil?
         return subject.inspect if subject.is_a?(Symbol)
+
         "#{subject.class.name}:#{id_for(subject)}"
       end
 
diff --git a/lib/declarative_policy/rule.rb b/lib/declarative_policy/rule.rb
index 7cfa82a9a9f..e309244a3b3 100644
--- a/lib/declarative_policy/rule.rb
+++ b/lib/declarative_policy/rule.rb
@@ -83,6 +83,7 @@ module DeclarativePolicy
       def cached_pass?(context)
         condition = context.condition(@name)
         return nil unless condition.cached?
+
         condition.pass?
       end
 
@@ -109,6 +110,7 @@ module DeclarativePolicy
       def delegated_context(context)
         policy = context.delegated_policies[@delegate_name]
         raise MissingDelegate if policy.nil?
+
         policy
       end
 
@@ -121,6 +123,7 @@ module DeclarativePolicy
       def cached_pass?(context)
         condition = delegated_context(context).condition(@name)
         return nil unless condition.cached?
+
         condition.pass?
       rescue MissingDelegate
         false
@@ -157,6 +160,7 @@ module DeclarativePolicy
       def cached_pass?(context)
         runner = context.runner(@ability)
         return nil unless runner.cached?
+
         runner.pass?
       end
 
@@ -258,6 +262,7 @@ module DeclarativePolicy
 
       def score(context)
         return 0 unless cached_pass?(context).nil?
+
         @rules.map { |r| r.score(context) }.inject(0, :+)
       end
 
diff --git a/lib/declarative_policy/runner.rb b/lib/declarative_policy/runner.rb
index 45ff2ef9ced..77c91817382 100644
--- a/lib/declarative_policy/runner.rb
+++ b/lib/declarative_policy/runner.rb
@@ -43,6 +43,7 @@ module DeclarativePolicy
     # used by Rule::Ability. See #steps_by_score
     def score
       return 0 if cached?
+
       steps.map(&:score).inject(0, :+)
     end