Thinning controllers for easier unit testing and separation of concerns

I wrote this gem, GitHub - NullVoxPopuli/skinny_controllers: A pattern for allowing for easier testing of large projects' business logic
which is influenced by Trailblazer, but less with some different
opinions
here and there.

The main idea that it pulls out of trailblazer is operations and
policies.

Operations are in app/operations
Policies are in app/policies

Operations control what data you are getting, and Policies control if
you
are allowed to get it.

The goal is to have all the logic in ruby objects that don’t require a
controller instance to run (i.e. avoiding put/get/post in your tests).

At the very minimum, you’d need

include SkinnyControllers::Diet
# and then in your action
render json: model

and the rest is implicit.

obviously, not everyone wants the same create / update / destroy
functionality, so that can all be configured:

here is an example of the create operation for the EventsController

module EventOperations
  class Create < SkinnyControllers::Operation::Base
    def run
      return unless allowed?
      # @model is used here, because the method `model` is memoized off
@model
      @model = model_class.new(model_params)
      @model.save
      @model # or just `model`
    end
  end
end

allowed? is a special method that looks up the corresponding policy:

class EventPolicy < SkinnyControllers::Policy::Base
  # for the Create operation for the EventsController is matched to 
create?
in the policy
  # o / object here is the instance of the model trying to be created
  def create?(o = object)
    user.is_admin?
  end
end

Thoughts?

I’m using this in a big side project I’ve been working on for event
registration. (Rails API + Ember)

example of operation specs:
https://github.com/NullVoxPopuli/aeonvera/blob/master/spec/operations/order_operations_spec.rb

example of policy specs:
https://github.com/NullVoxPopuli/aeonvera/blob/master/spec/policies/package_policy_spec.rb