rails_ops 1.1.22 → 1.1.26

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 31751dfbdb75a4da21a438d5ebb92ef9bbd66dd6e459b273c41e5e4f9d6a737c
-  data.tar.gz: 3caca2b5c09ac40c8715e2e89554e0aaaf9c8b1c1aa9c57b2c73a9e23bafcb18
+  metadata.gz: f526737c309842ce86558e77bc8ac5292d0d5e4faa82a848926cb03e725e51cf
+  data.tar.gz: fba2d918a81609ce6c778eecfe772d0aa252932403774234718309764b2ba348
 SHA512:
-  metadata.gz: 38ed4b42bbb20810c0e01fc82087ff5b9729f77bb820f26b5c7b34c033c49afe486b9b921141f59a438118678d6aa7a360dc781fa3329c55253901574d3099be
-  data.tar.gz: a61c2739ea1daff6e313e943873389d38f2506e647cb29b0edfa12cb054b94befb8e8a67721decba8d753a3e3c291527a0e313413e5722910907c27425e22e15
+  metadata.gz: a3abbc42c29d64386772eceb6095edeecf667ac65987f83acaebe9e93f9a1c6d030536130f9d5430073a6beb8f91e9162f86d0d4e624e214e2a2377ed006ef41
+  data.tar.gz: 5c396aae16103eeec31349daf26708c6eb24b66c9cf39b29a40a1f70e7686d1b6c6b21c8e49af6c5bd000b3ad24ef18c0beb8cf088094192dbaf70a6a45fe512

data/.github/workflows/rubocop.yml

@@ -0,0 +1,17 @@
+name: Rubocop check
+on: push
+
+jobs:
+  verify:
+    name: Test
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v1
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.0.1
+          bundler-cache: true
+      - name: Run rubocop
+        run: bundle exec rubocop

data/.github/workflows/ruby.yml

@@ -1,4 +1,4 @@
-name: Build
+name: Unit tests
 
 on:
   push:
@@ -12,16 +12,46 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby-version: ['2.3.0', '2.5.1', '2.6.2', '2.7.1', '3.0.1']
-
+        include:
+          - rails-version: '5.1'
+            ruby-version: '2.3.0'
+          - rails-version: '5.1'
+            ruby-version: '2.5.1'
+          - rails-version: '5.2'
+            ruby-version: '2.3.0'
+          - rails-version: '5.2'
+            ruby-version: '2.5.1'
+          - rails-version: '5.2'
+            ruby-version: '2.6.2'
+          - rails-version: '6.0'
+            ruby-version: '2.5.1'
+          - rails-version: '6.0'
+            ruby-version: '2.6.2'
+          - rails-version: '6.0'
+            ruby-version: '2.7.1'
+          - rails-version: '6.1'
+            ruby-version: '2.5.1'
+          - rails-version: '6.1'
+            ruby-version: '2.6.2'
+          - rails-version: '6.1'
+            ruby-version: '2.7.1'
+          - rails-version: '6.1'
+            ruby-version: '3.0.1'
+          - rails-version: '7.0'
+            ruby-version: '2.7.1'
+          - rails-version: '7.0'
+            ruby-version: '3.0.1'
+          - rails-version: '7.0'
+            ruby-version: '3.1.0'
+    name: Test against Ruby ${{ matrix.ruby-version }} / Rails ${{ matrix.rails-version }}
+    env:
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/rails_${{ matrix.rails-version }}.gemfile
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Ruby
+    - name: Set up Ruby ${{ matrix.ruby-version }}
       uses: ruby/setup-ruby@v1
       with:
         ruby-version: ${{ matrix.ruby-version }}
         bundler-cache: true
     - name: Run rake tests
-      run: bundle exec rake test
-    - name: Run rubocop
-      run: bundle exec rubocop
+      run: bundle exec rake

data/.gitignore

@@ -12,3 +12,5 @@
 /vendor/bundle
 /test/dummy/tmp/
 /test/dummy/log/
+/test/tmp/
+/gemfiles/*.lock

data/.rubocop.yml

@@ -6,6 +6,7 @@ AllCops:
     - 'tmp/**/*'
     - 'log/**/*'
     - '*.gemspec'
+    - 'test/tmp/**/*'
 
   DisplayCopNames: true
 

data/Appraisals

@@ -0,0 +1,19 @@
+appraise "rails-7.0" do
+  gem "rails", "~> 7.0.1"
+end
+
+appraise "rails-6.1" do
+  gem "rails", "~> 6.1.4"
+end
+
+appraise "rails-6.0" do
+  gem "rails", "~> 6.0.4"
+end
+
+appraise "rails-5.2" do
+  gem "rails", "~> 5.2.6"
+end
+
+appraise "rails-5.1" do
+  gem "rails", "~> 5.1.7"
+end

data/CHANGELOG.md

@@ -1,6 +1,25 @@
 # Changelog
 
-## 1.1.21 (2021-11-01)
+## 1.1.26 (2022-01-18)
+
+* #25: Add test matrix for unit tests
+
+* Add compatibility for Rails 7 and Ruby 3.1.0
+
+## 1.1.25 (2022-01-17)
+
+* #24: Add generator `operation` that generates a controller, operations and empty
+  view files
+
+## 1.1.24 (2021-11-24)
+
+* Add support for STI in model operations
+
+## 1.1.23 (2021-11-01)
+
+* No changes to previous release
+
+## 1.1.22 (2021-11-01)
 
 * Add support for lazy model authorization
 

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright © 2017-2021 Sitrox
+Copyright © 2017-2022 Sitrox
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,4 +1,5 @@
-[![Build](https://github.com/sitrox/rails_ops/actions/workflows/ruby.yml/badge.svg)](https://github.com/sitrox/rails_ops/actions/workflows/ruby.yml)
+[![Unit tests](https://github.com/sitrox/rails_ops/actions/workflows/ruby.yml/badge.svg)](https://github.com/sitrox/rails_ops/actions/workflows/ruby.yml)
+[![Rubocop check](https://github.com/sitrox/rails_ops/actions/workflows/rubocop.yml/badge.svg)](https://github.com/sitrox/rails_ops/actions/workflows/rubocop.yml)
 [![Gem Version](https://badge.fury.io/rb/rails_ops.svg)](https://badge.fury.io/rb/rails_ops)
 
 rails_ops
@@ -22,9 +23,21 @@ Requirements & Installation
 
 ### Requirements
 
-- RailsOps only works with Rails applications and has been tested with Rails >=
-  3.2.0.
-- Prior Rails versions may be supported but this has not been verified.
+- RailsOps only works with Rails applications, with the following Rails versions being tested in the CI:
+  * Rails 5.1.x
+  * Rails 5.2.x
+  * Rails 6.0.x
+  * Rails 6.1.x
+  * Rails 7.0.x
+- Additionally, the following Ruby versions are covered by our unit tests:
+  * 2.3.0
+  * 2.5.1
+  * 2.6.2
+  * 2.7.1
+  * 3.0.1
+  * 3.1.0
+- Please see the [unit test workflow](https://github.com/sitrox/rails_ops/actions/workflows/ruby.yml) for the combinations of the Rails & Ruby versions, as only compatible versions are tested with each other.
+- Prior Rails and Ruby versions may be supported but they are not tested in the CI.
 - Rails Ops' model operations require ActiveRecord but are database / adapter
   agnostic
 
@@ -1194,8 +1207,8 @@ sensible default. See the respective class' source code for details.
 In case of operations inheriting from `RailsOps::Operation::Model::Update`, you
 can specify the `model_authorization_action` to be `lazy`, meaning that it will
 only be checked when *performing* the operation, but not on initialization. This
-can be useful for displaying readonly forms to users which do not have
-read-permissions only:
+can be useful for displaying readonly forms to users which have read-permissions
+only:
 
 ```ruby
 class Operations::User::Update < RailsOps::Operation::Model::Update
@@ -1204,7 +1217,7 @@ class Operations::User::Update < RailsOps::Operation::Model::Update
   # This automatically calls `authorize_model! :read`. Because it is set to be
   # `lazy`, the authorization will only run when the operation is actually
   # *performed*, and not already at instantiation.
-  model_authorization_action :read, lazy: true
+  model_authorization_action :update, lazy: true
 end
 ```
 
@@ -1270,6 +1283,34 @@ This block receives the params hash as it would be passed to the sub operation
 and allows to modify it. The block's return value is then passed to the
 sub-operation. Do not change the params inplace but instead return a new hash.
 
+### Single-table inheritance
+
+Model operations also support STI models (Single Table Inheritance). However,
+there is the caviat that if you do extend your model in the operation (e.g.
+`model Animal do { ... }`), RailsOps automatically creates an anonymous subclass
+of the given class (e.g. `Animal`). Operations will always load / create models
+that are instances of this anonymous class.
+
+Consider the following operation:
+
+```ruby
+class Animal < ApplicationRecord; end
+class Bird < Animal; end
+class Mouse < Animal; end
+
+class LoadAnimal < RailsOps::Operation::Model::Load
+  model Animal do
+    # Something
+  end
+end
+
+bird = Bird.create
+op_bird = LoadAnimal.new(id: bird.id)
+
+bird.class    # => Class "Bird", extending "Animal"
+op_bird.class # => Anonymous class, extending "Animal", not "Bird"
+```
+
 Record extension and virtual records
 ------------------------------------
 
@@ -1460,6 +1501,55 @@ sub-operations, see section *Calling sub-operations* for more information.
 Operation Inheritance
 ---------------------
 
+Generators
+----------
+
+RailsOps features a generator to easily create a structure for common CRUD-style
+constructs. The generator creates the CRUD operations, some empty view files, a
+controller and adds an entry in the routing file.
+
+This is e.g. useful when adding a new model to an application, as the basic structure
+is usually rather similar.
+
+### Usage
+
+Run the generator using the `operation` generator, specifying the name of the
+operation class:
+
+```ruby
+rails g operation User
+```
+
+This will generate the following operations:
+
+* `app/operations/user/load.rb`
+* `app/operations/user/create.rb`
+* `app/operations/user/update.rb`
+* `app/operations/user/destroy.rb`
+
+as well as the controller `app/controllers/users_controller.rb` and the following
+empty view files:
+
+* `app/views/users/index.html.haml`
+* `app/views/users/show.html.haml`
+* `app/views/users/new.html.haml`
+* `app/views/users/edit.html.haml`
+
+It will also add the entry `resources :users` to the `config/routes.rb` file.
+
+If you want to skip the controller, the views or the routes, you can do so using the
+flags:
+
+* `--skip-controller`
+* `--skip-routes`
+* `--skip-views`
+
+Or if you want to skip them all: `--only-operations`.
+
+Of course, at this point, the operations will need some adaptions, especially the
+[parameter schemas](#validating-params), and the controllers need the logic for the
+success and failure cases, as this depends on your application.
+
 Caveats
 -------
 
@@ -1481,4 +1571,4 @@ Rails architecture.
 
 ## Copyright
 
-Copyright © 2017 - 2021 Sitrox. See `LICENSE` for further details.
+Copyright © 2017 - 2022 Sitrox. See `LICENSE` for further details.

data/Rakefile

@@ -15,11 +15,15 @@ task :gemspec do
     spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
     spec.require_paths = ['lib']
 
+    spec.add_development_dependency 'appraisal'
     spec.add_development_dependency 'bundler'
     spec.add_development_dependency 'rake'
     spec.add_development_dependency 'sqlite3'
     spec.add_development_dependency 'cancancan'
+    spec.add_development_dependency 'pry'
+    spec.add_development_dependency 'colorize'
     spec.add_development_dependency 'rubocop', '0.47.1'
+    spec.add_development_dependency 'sprockets-rails'
     spec.add_dependency 'active_type', '>= 1.3.0'
     spec.add_dependency 'minitest'
     spec.add_dependency 'rails'

data/VERSION

@@ -1 +1 @@
-1.1.22
+1.1.26

data/gemfiles/rails_5.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 5.1.7"
+
+gemspec path: "../"

data/gemfiles/rails_5.2.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 5.2.6"
+
+gemspec path: "../"

data/gemfiles/rails_6.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 6.0.4"
+
+gemspec path: "../"

data/gemfiles/rails_6.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 6.1.4"
+
+gemspec path: "../"

data/gemfiles/rails_7.0.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rails", "~> 7.0.1"
+
+gemspec path: "../"

data/lib/generators/operation/USAGE

@@ -0,0 +1,21 @@
+Description:
+    Generates CRUD operations, along with a controller, views and an entry in the route file
+
+Example:
+    bin/rails generate operation User
+
+    This will create:
+        app/operations/user/load.rb
+        app/operations/user/create.rb
+        app/operations/user/update.rb
+        app/operations/user/destroy.rb
+
+        app/controllers/users_controller.rb
+
+        app/views/users/index.html.haml
+        app/views/users/show.html.haml
+        app/views/users/new.html.haml
+        app/views/users/edit.html.haml
+
+    And add a routing entry:
+        resources :users

data/lib/generators/operation/operation_generator.rb

@@ -0,0 +1,46 @@
+class OperationGenerator < Rails::Generators::NamedBase
+  source_root File.expand_path('templates', __dir__)
+
+  class_option :skip_controller, type: :boolean, desc: "Don't add a controller."
+  class_option :skip_views, type: :boolean, desc: "Don't add the views."
+  class_option :skip_routes, type: :boolean, desc: "Don't add routes to config/routes.rb."
+  class_option :only_operations, type: :boolean, desc: 'Only add the operations. This is equal to specifying --skip-controller --skip-routes --skip-views'
+
+  def add_operations
+    @class_name = name.classify
+    @underscored_name = name.underscore
+    @underscored_pluralized_name = name.underscore.pluralize
+
+    operations_path = "app/operations/#{@underscored_name}"
+
+    template 'load.erb', "#{operations_path}/load.rb"
+    template 'create.erb', "#{operations_path}/create.rb"
+    template 'update.erb', "#{operations_path}/update.rb"
+    template 'destroy.erb', "#{operations_path}/destroy.rb"
+  end
+
+  def add_controller
+    return if options[:skip_controller] || options[:only_operations]
+
+    controller_file_path = "app/controllers/#{@underscored_pluralized_name}_controller.rb"
+    @controller_name = "#{@class_name.pluralize}Controller"
+
+    template 'controller.erb', controller_file_path
+  end
+
+  def add_views
+    return if options[:skip_views] || options[:only_operations]
+
+    views_folder = "app/views/#{@underscored_pluralized_name}"
+
+    %w(index show new edit).each do |view|
+      template 'view.erb', "#{views_folder}/#{view}.html.haml"
+    end
+  end
+
+  def add_routes
+    return if options[:skip_routes] || options[:only_operations]
+
+    route "resources :#{@underscored_pluralized_name}"
+  end
+end

data/lib/generators/operation/templates/controller.erb

@@ -0,0 +1,39 @@
+class <%= @controller_name %> < ApplicationController
+  def index; end
+
+  def show
+    op Operations::<%= @class_name %>::Load
+  end
+
+  def new
+    op Operations::<%= @class_name %>::Create
+  end
+
+  def create
+    if run Operations::<%= @class_name %>::Create
+      # handle successful case
+    else
+      # handle error case
+    end
+  end
+
+  def edit
+    op Operations::<%= @class_name %>::Update
+  end
+
+  def update
+    if run Operations::<%= @class_name %>::Update
+      # handle successful case
+    else
+      # handle error case
+    end
+  end
+
+  def destroy
+    if run Operations::<%= @class_name %>::Destroy
+      # handle successful case
+    else
+      # handle error case
+    end
+  end
+end

data/lib/generators/operation/templates/create.erb

@@ -0,0 +1,11 @@
+module Operations::<%= @class_name %>
+  class Create < RailsOps::Operation::Model::Create
+    schema3 do
+      hsh? :<%= @underscored_name %> do
+
+      end
+    end
+
+    model ::<%= @class_name %>
+  end
+end

data/lib/generators/operation/templates/destroy.erb

@@ -0,0 +1,9 @@
+module Operations::<%= @class_name %>
+  class Destroy < RailsOps::Operation::Model::Destroy
+    schema3 do
+      str! :id
+    end
+
+    model ::<%= @class_name %>
+  end
+end

data/lib/generators/operation/templates/load.erb

@@ -0,0 +1,5 @@
+module Operations::<%= @class_name %>
+  class Load < RailsOps::Operation::Model::Load
+    model ::<%= @class_name %>
+  end
+end

data/lib/generators/operation/templates/update.erb

@@ -0,0 +1,12 @@
+module Operations::<%= @class_name %>
+  class Update < RailsOps::Operation::Model::Update
+    schema3 do
+      str! :id
+      hsh? :<%= @underscored_name %> do
+
+      end
+    end
+
+    model ::<%= @class_name %>
+  end
+end

data/lib/rails_ops/model_mixins/sti_fixes.rb

@@ -0,0 +1,17 @@
+module RailsOps
+  module ModelMixins
+    module StiFixes
+      extend ActiveSupport::Concern
+
+      class_methods do
+        def finder_needs_type_condition?
+          base_class.finder_needs_type_condition?
+        end
+
+        def name
+          base_class.name
+        end
+      end
+    end
+  end
+end

data/lib/rails_ops/model_mixins/virtual_attributes/virtual_column_wrapper.rb

@@ -6,4 +6,8 @@ class RailsOps::ModelMixins::VirtualAttributes::VirtualColumnWrapper
   def type
     @virtual_column.instance_variable_get(:@type_caster).instance_variable_get(:@type)
   end
+
+  def default_function
+    nil
+  end
 end

data/lib/rails_ops/model_mixins.rb

@@ -8,5 +8,6 @@ module RailsOps::ModelMixins
     include VirtualAttributes      # Provides virtual attributes functionality.
     include VirtualHasOne          # Provides virtual_has_one.
     include VirtualModelName       # Provides virtual_model_name.
+    include StiFixes               # Fixes using RailsOps models with STI.
   end
 end

data/lib/rails_ops.rb

@@ -88,6 +88,7 @@ require 'rails_ops/model_casting.rb'
 require 'rails_ops/model_mixins.rb'
 require 'rails_ops/model_mixins/ar_extension.rb'
 require 'rails_ops/model_mixins/parent_op.rb'
+require 'rails_ops/model_mixins/sti_fixes.rb'
 require 'rails_ops/model_mixins/virtual_attributes.rb'
 require 'rails_ops/model_mixins/virtual_attributes/virtual_column_wrapper.rb'
 require 'rails_ops/model_mixins/virtual_has_one.rb'