Refs: #3478@3h; Policy Cleanup and Providing Policy and permissions for all models and actions

 - ApplicationPolicy nondestructive permission depend on model existance
 - ApplicationPolicy destructive permission default to `false`
 - Tied Policy permissions at ApplicationPolicy Level: edit? → update?, new? → create?, index? → show?
 - ApplicationPolicy convenience methods `delete?` & `authorizes_action?(action)`
 - Refactoring of `spec/helpers/table_builder_helper_spec.rb` accordingly

 - Stubbing scope in specs (cannot switch to referential with a `build_stubbed` instance)

1 files changed, 3 insertions, 6 deletions

diff --git a/app/policies/journey_pattern_policy.rb b/app/policies/journey_pattern_policy.rb
index 7ce2da561..99e39eeff 100644
--- a/app/policies/journey_pattern_policy.rb
+++ b/app/policies/journey_pattern_policy.rb
@@ -11,15 +11,12 @@ class JourneyPatternPolicy < ApplicationPolicy
     user.has_permission?('journey_patterns.create')
   end
 
-  def edit?
-    organisation_match? && user.has_permission?('journey_patterns.edit')
-  end
-
   def destroy?
     organisation_match? && user.has_permission?('journey_patterns.destroy')
   end
 
-  def update?  ; edit? end
-  def new?     ; create? end
+  def update?
+    organisation_match? && user.has_permission?('journey_patterns.edit')
+  end
 end