mobile_workflow 0.6.3 → 0.6.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa30fd9bb5c6aa9d27333100a4955e32219e8a0241753d03743cb4ae3e769a40
-  data.tar.gz: d23b234015566cd627dc4063efa40b7ef14d0f7a80a5511b5a06739f580f9ee3
+  metadata.gz: 913509d1248a7a63226904e313ba1e56c700df9b95f0e393bb642d8f512c94b5
+  data.tar.gz: f1718f9a7e22d33c90ff546215c3ed6047118fca4148ccff6344ec34bedb375c
 SHA512:
-  metadata.gz: 9861be314639d646a46c7b35d85a91cd4ebf32b4b19320878cc494ac5b0227052e12f7ab0f6f0173a56ab503b3748a40f004b3a3613ae9c511f9e526521ee987
-  data.tar.gz: 95fb08640afa41f41d792b8fdf97d14c4622163600546638733f1d02a29d60c3cf3b4fdf09b1574fcfb3ed9a8094bcc15dcd24552376e686ae1a5a7261de60c8
+  metadata.gz: 2bc79973227c2db3e308c4329076ea441364ed7c65f2b095fabb201e23fedd2d736883c872ed051aa0ee17e2a94e626f4a99c3dbb95a058d8dbf568dc9d25a05
+  data.tar.gz: 5689ffc81f5df062ac81e25fdbe04da2986c10e434ba5bc9ce27d502381a3fbc63da5c8d8ffa698e4d85571b56ea3b74a0e26936c9178874dd66e60b773aa50a

data/lib/generators/mobile_workflow/controller_generator.rb

@@ -33,6 +33,22 @@ module MobileWorkflow
         params = attributes_names.map{ |name| ":#{name}" }
         params.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
 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,36 @@ 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")
       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 +79,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} #{s3_storage? ? '--s3-storage' : ''}".strip
+          route "resources :#{plural_controller_name}, only: [#{actions.map{|a| ":#{a}"}.join(", ")}]"
         end
       end
       
@@ -88,6 +101,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/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 %> = 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

@@ -16,9 +16,21 @@ module MobileWorkflow::Cli
     def rspec_generator
       generate 'rspec:install'
     end
+    
+    def factory_bot
+      inject_into_file 'spec/rails_helper.rb', before: "end\n" do
+        "config.include FactoryBot::Syntax::Methods"
+      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
@@ -33,7 +45,7 @@ module Admin
 end
 CODE
 
-      Bundler.with_unbundled_env { generate 'administrate:routes' }
+      generate 'administrate:routes'
     end
     
     def ability_generator
@@ -51,9 +63,6 @@ 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 s3_backend(region)
@@ -76,6 +85,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

@@ -33,10 +33,11 @@ module MobileWorkflow::Cli
       after_bundle do
         build :procfiles
         build :rspec_generator
+        build :factory_bot
         build :ability_generator
         build :active_storage if options[:s3_storage] 
         build :mobile_workflow_generator, ARGV[1]
-        setup_db
+        build :migrate_db
         build :administrate_generator
         
         generate_dot_env
@@ -57,14 +58,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.3'
+  VERSION = '0.6.9'
   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.3
+  version: 0.6.9
 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,14 @@ 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/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