mobile_workflow 0.6.4 → 0.6.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d3b107beb896ad8c143fe376e7532d3f7d3093435f1b0b59a15401dba17d752
-  data.tar.gz: '097097c10f64a32065c90e42779b672fecd4b964a15eb43e887d686718c3acfb'
+  metadata.gz: c1e72adcc8d3a21ec01476098c7f16fe849c2ab8348f0164fe15ffde1872b156
+  data.tar.gz: 5c361e5591040d4a5636aa425dbf77ec533cddbad38e150a6f1f94a1fdca360e
 SHA512:
-  metadata.gz: c8fea51abb663711210ef86a67a8a2b5bcaaedd95d8772fb1297d6deaa10ee4142af6a0a2a6948db56f088da31aa26833b580c6ed7be994c16df6884338394dd
-  data.tar.gz: 41fee4840d3925030a310530cca5f778a1b989d5f37bea0c8fc7440d3947d04fade3ac3abf040b0f6bf6a0e3508b492b840871c7c73c50fb7b9201decf8703b7
+  metadata.gz: 5b66dbaa49b726bc92618c6405af04e3cac7476809ffafeda4d7d444ec3275a993fae2c4a4b6c534ecd5569467c3c8fb4402c97a7b96d1238d1f0c5c34190148
+  data.tar.gz: 67bd8a5575e05e1ec46653b42eadf089a81c05e64ea0c1e6f9702b6e0621e9fd6212e577f13ccbf085a299f53fd1bf288df2db01fd46b2b52d9a748f07aa0518

data/lib/generators/mobile_workflow/controller_generator.rb

@@ -30,8 +30,25 @@ module MobileWorkflow
       end
           
       def permitted_params
-        params = attributes_names.map{ |name| ":#{name}" }
-        params.join(", ")
+        permitted_attributes = attributes_names.dup
+        permitted_attributes.delete('user')
+        permitted_attributes.map{ |name| ":#{name}" }.join(", ")
+      end
+      
+      def index_action?
+        actions.include?("index")
+      end
+      
+      def show_action?
+        actions.include?("show")
+      end
+      
+      def create_action?
+        actions.include?("create")
+      end
+      
+      def actions
+        options[:actions]
       end
     end
   end

data/lib/generators/mobile_workflow/install/install_generator.rb

@@ -1,9 +1,13 @@
 require "rails/generators/base"
+require "rails/generators/active_model"
+require "rails/generators/active_record/migration"
+require "active_record"
 require "mobile_workflow/open_api_spec/parser"
 
 module MobileWorkflow
   module Generators
     class InstallGenerator < Rails::Generators::Base
+      include ActiveRecord::Generators::Migration
       
       source_root File.expand_path("../templates", __FILE__)
 
@@ -24,6 +28,8 @@ module MobileWorkflow
         return unless options[:doorkeeper_oauth]
         say "Generating Doorkeeper OAuth"
         
+        migration_template "create_users.rb", "db/migrate/create_users.rb"
+        
         generate 'doorkeeper:install'
         gsub_file 'config/initializers/doorkeeper.rb', 'raise "Please configure doorkeeper resource_owner_authenticator block located in #{__FILE__}"', 'User.find_by_id(session[:user_id]) || redirect_to(new_session_url(return_to: request.fullpath))'
         generate 'doorkeeper:migration'
@@ -33,29 +39,37 @@ module MobileWorkflow
         route "resources :sessions, only: [:new, :create]"
         
         # View related for login screen
-        copy_file("views/layouts/application.html.erb")
-        copy_file("views/sessions/new.html.erb")
-        copy_file("helpers/application_helper.rb")
+        copy_file("app/views/layouts/application.html.erb")
+        copy_file("app/views/sessions/new.html.erb")
+        copy_file("app/helpers/application_helper.rb")
+        copy_file("spec/factories/users.rb")
       end
       
       def generate_models
         say "Loading OpenAPI Spec: #{open_api_spec_path}"
         say "Generating models"
-        model_name_to_properties.each_pair do |model_name, model_properties|          
+        model_name_to_properties.each_pair do |model_name, model_properties|   
+          
+          if doorkeeper_oauth?
+            model_properties = "#{model_properties} user:references"
+            @model_name_to_properties[model_name] = model_properties
+          end
+                 
           if interactive? && !yes?("Use generated schema #{model_name}(#{model_properties})[yn]?")
             model_properties = ask "Specify schema for #{model_name}: (e.g. text:string image:attachment region:reference)"
+            @model_name_to_properties[model_name] = model_properties
           end
 
           generate_model(model_name, model_properties)
-          @model_name_to_properties[model_name] = model_properties
         end
       end
 
       def generate_controllers_and_routes
         say "Generating controllers"
-        route "root to: 'admin/#{open_api_spec.controller_names.first}#index'"
+        controller_name_to_actions = open_api_spec.controller_name_to_actions
+        route "root to: 'admin/#{controller_name_to_actions.keys.first}#index'"
         
-        open_api_spec.controller_names.each do |plural_controller_name|
+        controller_name_to_actions.each_pair do |plural_controller_name, actions|
           controller_name = plural_controller_name.singularize
           model_properties = model_name_to_properties[controller_name]
           
@@ -66,8 +80,8 @@ module MobileWorkflow
             generate_model(controller_name, model_properties)
           end
           
-          generate "mobile_workflow:controller #{controller_name} --attributes #{model_properties} #{s3_storage? ? '--s3-storage' : ''}".strip
-          route "resources :#{plural_controller_name}, only: [:index, :show, :create]"
+          generate "mobile_workflow:controller #{controller_name} --actions #{actions.join(" ")} --attributes #{model_properties} #{generate_controller_args}".strip
+          route "resources :#{plural_controller_name}, only: [#{actions.map{|a| ":#{a}"}.join(", ")}]"
         end
       end
       
@@ -81,6 +95,13 @@ module MobileWorkflow
         generate("mobile_workflow:model #{model_name} #{model_properties}")
       end
       
+      def generate_controller_args
+        args = ''
+        args += ' --s3-storage' if s3_storage?
+        args += ' --doorkeeper-oauth' if doorkeeper_oauth?
+        args.strip
+      end
+      
       def model_name_to_properties
         @model_name_to_properties ||= open_api_spec.model_name_to_properties
       end
@@ -88,6 +109,10 @@ module MobileWorkflow
       def open_api_spec
         @open_api_spec ||= ::MobileWorkflow::OpenApiSpec::Parser.new(File.read(open_api_spec_path))
       end
+
+      def doorkeeper_oauth?
+        options[:doorkeeper_oauth]
+      end
       
       def s3_storage?
         options[:s3_storage]

data/lib/generators/mobile_workflow/install/templates/Gemfile.erb

@@ -45,6 +45,7 @@ group :development, :test do
   gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
   gem 'dotenv-rails'
   gem 'factory_bot_rails'
+  gem "rufo"
 end
 
 group :production do

data/lib/generators/mobile_workflow/install/templates/spec/factories/users.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :user do
+    email { 'jo@example.com' }
+    password { 'Password123' }
+  end
+end

data/lib/generators/mobile_workflow/templates/controller.rb.erb

@@ -4,16 +4,24 @@ class <%= controller_class_name %>Controller < ApiController
   before_action :rewrite_payload, only: :create
 
   load_and_authorize_resource
-
+  
+<% if index_action? -%>
   def index
     render json: @<%= plural_table_name %>.collect(&:list_item_as_json)
   end
-
+  
+<% end -%>
+<% if show_action? -%>
   def show
     render json: @<%= singular_table_name %>.display_as_json
   end
-
+  
+<% end -%>
+<% if create_action? -%>
   def create
+    <% if doorkeeper_oauth? -%>
+    @<%= singular_table_name %>.user = current_user
+    <% end -%>
     if @<%= singular_table_name %>.save
       <% if s3_storage? -%>
       render json: { binary_urls: binary_urls(@<%= singular_table_name %>), response: @<%= singular_table_name %> }, status: :created
@@ -39,5 +47,6 @@ class <%= controller_class_name %>Controller < ApiController
   def <%= singular_table_name.underscore %>_params
     params.require(:payload).permit(<%= permitted_params %>)
   end
+<% end -%>
 end
 <% end %>

data/lib/generators/mobile_workflow/templates/controller_spec.rb.erb

@@ -10,7 +10,8 @@ RSpec.describe <%= controller_class_name %>Controller do
   let(:user) { create(:user) }
   let(:token) { instance_double('Doorkeeper::AccessToken', :accessible? => true, :acceptable? => true, resource_owner_id: user.id) }
   <% end -%>
-  
+
+<% if index_action? -%>  
   describe 'GET #index' do
     let!(:<%= controller_class_name.singularize.underscore %>) { create(:<%= controller_class_name.singularize.underscore %>) }
     before(:each) do
@@ -25,7 +26,8 @@ RSpec.describe <%= controller_class_name %>Controller do
       it { expect(response.status).to eq 200 }
     end
   end
-  
+<% end -%>
+<% if show_action? -%>
   describe 'GET #show' do
     let(:<%= controller_class_name.singularize.underscore %>) { create(:<%= controller_class_name.singularize.underscore %>) }
     let(:params) { { id: <%= controller_class_name.singularize.underscore %>.id } }
@@ -40,7 +42,9 @@ RSpec.describe <%= controller_class_name %>Controller do
       it { expect(response.status).to eq 200 }
     end
   end
-  
+
+<% end -%>
+<% if create_action? -%>
   describe 'POST #create' do
     let(:payload_params) { {text: 'OK'} }
     let(:params) { { payload: payload_params, binaries: [{identifier: 'record', mimetype: 'video/mp4'}] } }
@@ -59,4 +63,5 @@ RSpec.describe <%= controller_class_name %>Controller do
       <% end %>
     end
   end
+<% end -%>
 end

data/lib/mobile_workflow/cli/app_builder.rb

@@ -18,13 +18,19 @@ module MobileWorkflow::Cli
     end
     
     def factory_bot
-      inject_into_file 'spec/rails_helper.rb', before: "end\n" do
-        "config.include FactoryBot::Syntax::Methods"
+      inject_into_file 'spec/rails_helper.rb', after: "RSpec.configure do |config|\n" do
+        "\tconfig.include FactoryBot::Syntax::Methods\n"
       end
     end
+    
+    def migrate_db 
+      rails_command("db:drop")
+      rails_command("db:create")
+      rails_command("db:migrate")
+    end
 
     def administrate_generator
-      Bundler.with_unbundled_env { generate 'administrate:install' }
+      generate 'administrate:install'
       
       file 'app/assets/config/manifest.js', <<-CODE
 //= link administrate/application.css
@@ -39,7 +45,7 @@ module Admin
 end
 CODE
 
-      Bundler.with_unbundled_env { generate 'administrate:routes' }
+      generate 'administrate:routes'
     end
     
     def ability_generator
@@ -57,9 +63,10 @@ CODE
       gen_opts = ""
       gen_opts += "--doorkeeper_oauth" if options[:doorkeeper_oauth]
       generate "mobile_workflow:install #{gen_opts}"
-      
-      # Copy user migrations if needed
-      rails_command 'mobile_workflow:install:migrations' if options[:doorkeeper_oauth]
+    end
+    
+    def format_source
+      run 'rufo .'
     end
     
     def s3_backend(region)
@@ -82,6 +89,7 @@ CODE
       heroku_backend.create
       heroku_backend.configure_activestorage if options[:s3_storage]
       heroku_backend.deploy
+      heroku_backend.seed_db
       heroku_backend.sync_dotenv
     end
     

data/lib/mobile_workflow/cli/app_server_generator.rb

@@ -37,8 +37,9 @@ module MobileWorkflow::Cli
         build :ability_generator
         build :active_storage if options[:s3_storage] 
         build :mobile_workflow_generator, ARGV[1]
-        setup_db
+        build :migrate_db
         build :administrate_generator
+        build :format_source
         
         generate_dot_env
         initial_git_commit
@@ -58,14 +59,6 @@ module MobileWorkflow::Cli
     # Todo: MBS - move these methods to the builder class
     # Ideally override RailsBuilder methods
     private
-    def setup_db
-      Bundler.with_unbundled_env do 
-        rails_command("db:drop")
-        rails_command("db:create")
-        rails_command("db:migrate")
-      end
-    end
-
     def initial_git_commit
       git add: "."
       git commit: %Q{ -m 'Initial commit' }

data/lib/mobile_workflow/cli/heroku_backend.rb

@@ -21,6 +21,10 @@ module MobileWorkflow::Cli
     def deploy
       heroku_command "git push"
     end
+    
+    def seed_db
+      heroku_command "heroku run rails db:seed"
+    end
   
     def sync_dotenv
       env = File.read(".env").split.join(" ")

data/lib/mobile_workflow/open_api_spec/parser.rb

@@ -21,9 +21,9 @@ module MobileWorkflow
         end
         @model_properties
       end
-  
-      def controller_names
-        @controller_names ||= paths.collect{|url_path| url_path.split('/')[1] }.uniq
+      
+      def controller_name_to_actions
+        @controller_name_to_actions ||= parse_controller_names_to_actions
       end
   
       def paths
@@ -35,6 +35,22 @@ module MobileWorkflow
       end
   
       private
+      def parse_controller_names_to_actions
+        controllers = {}
+        paths.each do |path|
+          path_items = path.split('/')
+          controller_name = path_items[1]
+          controllers[controller_name] = [] unless controllers.key?(controller_name)
+          
+          open_api_spec[:paths][path].keys.each do |method|
+            controllers[controller_name] << 'create' if path_items.count == 2 && method == 'post'
+            controllers[controller_name] << 'index' if path_items.count == 2 && method == 'get'
+            controllers[controller_name] << 'show' if path_items.count == 3 && method == 'get'                        
+          end
+        end
+        controllers
+      end
+      
       def open_api_spec
         @open_api_spec ||= read_openapi_spec
       end

data/lib/mobile_workflow/version.rb

@@ -1,5 +1,5 @@
 module MobileWorkflow
-  VERSION = '0.6.4'
+  VERSION = '0.6.10'
   RUBY_VERSION = '2.7.2'
   RAILS_VERSION = '6.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mobile_workflow
 version: !ruby/object:Gem::Version
-  version: 0.6.4
+  version: 0.6.10
 platform: ruby
 authors:
 - Matt Brooke-Smith
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-27 00:00:00.000000000 Z
+date: 2021-02-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -103,7 +103,6 @@ files:
 - app/views/layouts/mobile_workflow/application.html.erb
 - bin/mwf
 - config/routes.rb
-- db/migrate/20200823174210_create_users.rb
 - lib/generators/mobile_workflow/controller_generator.rb
 - lib/generators/mobile_workflow/install/install_generator.rb
 - lib/generators/mobile_workflow/install/templates/Gemfile.erb
@@ -112,13 +111,15 @@ files:
 - lib/generators/mobile_workflow/install/templates/README.md.erb
 - lib/generators/mobile_workflow/install/templates/ability.rb
 - lib/generators/mobile_workflow/install/templates/api_controller.rb.erb
-- lib/generators/mobile_workflow/install/templates/helpers/application_helper.rb
+- lib/generators/mobile_workflow/install/templates/app/helpers/application_helper.rb
+- lib/generators/mobile_workflow/install/templates/app/views/layouts/application.html.erb
+- lib/generators/mobile_workflow/install/templates/app/views/sessions/new.html.erb
+- lib/generators/mobile_workflow/install/templates/create_users.rb
 - lib/generators/mobile_workflow/install/templates/seeds.rb.erb
 - lib/generators/mobile_workflow/install/templates/sessions_controller.rb.erb
+- lib/generators/mobile_workflow/install/templates/spec/factories/users.rb
 - lib/generators/mobile_workflow/install/templates/storage.s3.yml
 - lib/generators/mobile_workflow/install/templates/user.rb.erb
-- lib/generators/mobile_workflow/install/templates/views/layouts/application.html.erb
-- lib/generators/mobile_workflow/install/templates/views/sessions/new.html.erb
 - lib/generators/mobile_workflow/model_generator.rb
 - lib/generators/mobile_workflow/templates/controller.rb.erb
 - lib/generators/mobile_workflow/templates/controller_spec.rb.erb