hot-glue 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ddd0fff6e983cefb9f4705ecfa218553412a78d6b02e70f246427562b0991e4
-  data.tar.gz: 12971b62efd73dc85db9a48b7d9874f8ba9e81615f1f981af6d00cd2d58a57d7
+  metadata.gz: 59ec85e7253282cf8d83bc7d8137a787ff33cb617aad95f5752efbb6df68acf4
+  data.tar.gz: bdbab52ce54a5c1ed52f03753c01c203cce7109dc58fa73e6b9357deee8c83bc
 SHA512:
-  metadata.gz: 3f436af0a80f4228e2f17f290d9be67aa1ffc7e48379d92a6739889906bf1a148df922db4af8f5989567bc6306336541de7685dbf41a89de4d7c6caa26235d55
-  data.tar.gz: 566393c43f57c2ec3e5e09b2ddd1e3684f643f10fa76468e7343cac20ccd0a015bc6c71f417181cecc60c2e97d10dfecbf36b735c7e2db8dd25556dee5669db0
+  metadata.gz: 995f7a97e1b6f1105b9190a19cc0bfa93ed9490c2080c97332ae745326f1cc86da3e9e0abf401510d926a574686728dc848e155aa7a0f12ac53417147c625185
+  data.tar.gz: b5bda3332c15706e124f577aa77bfaa068d7ad8d45ebe31aacea7cbcf9cb193a20719ff87d8c1b467bfd0162fcbe5ab0e536295849a3f86e30002babe0ad1212

data/.gitignore

@@ -1,16 +1,21 @@
 .bundle/
 log/*.log
 pkg/
-test/dummy/db/*.sqlite3
-test/dummy/db/*.sqlite3-journal
-test/dummy/db/*.sqlite3-*
-test/dummy/log/*.log
-test/dummy/storage/
-test/dummy/tmp/
+
 .byebug_history
 
 .idea/
 *.gem
 .DS_Store
 
-dump.rdb
+dump.rdb
+tmp/
+
+
+db/*.sqlite3
+
+spec/dummy/app/views/
+spec/dummy/app/controllers/
+spec/dummy/specs/
+
+spec/strawman/

data/Gemfile

@@ -1,15 +1,10 @@
 source 'https://rubygems.org'
 git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
-# Declare your gem's dependencies in common_core_js.gemspec.
-# Bundler will treat runtime dependencies like base dependencies, and
-# development dependencies will be added by default to the :development group.
 gemspec
 
-# Declare any dependencies that are still in development here instead of in
-# your gemspec. These might include edge Rails or gems from your path or
-# Git. Remember to move these dependencies to your gemspec before releasing
-# your gem to rubygems.org.
+gem 'sqlite3'
+gem 'byebug'
 
-# To use a debugger
-# gem 'byebug', group: [:development, :test]
+gem 'rails', '6.1'
+gem 'devise', require: true

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    hot-glue (0.0.4)
+    hot-glue (0.0.5)
       ffaker (~> 2.16)
       haml-rails (~> 2.0)
       kaminari (~> 1.2)
@@ -12,68 +12,76 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    actioncable (6.1.3)
-      actionpack (= 6.1.3)
-      activesupport (= 6.1.3)
+    actioncable (6.1.0)
+      actionpack (= 6.1.0)
+      activesupport (= 6.1.0)
       nio4r (~> 2.0)
       websocket-driver (>= 0.6.1)
-    actionmailbox (6.1.3)
-      actionpack (= 6.1.3)
-      activejob (= 6.1.3)
-      activerecord (= 6.1.3)
-      activestorage (= 6.1.3)
-      activesupport (= 6.1.3)
+    actionmailbox (6.1.0)
+      actionpack (= 6.1.0)
+      activejob (= 6.1.0)
+      activerecord (= 6.1.0)
+      activestorage (= 6.1.0)
+      activesupport (= 6.1.0)
       mail (>= 2.7.1)
-    actionmailer (6.1.3)
-      actionpack (= 6.1.3)
-      actionview (= 6.1.3)
-      activejob (= 6.1.3)
-      activesupport (= 6.1.3)
+    actionmailer (6.1.0)
+      actionpack (= 6.1.0)
+      actionview (= 6.1.0)
+      activejob (= 6.1.0)
+      activesupport (= 6.1.0)
       mail (~> 2.5, >= 2.5.4)
       rails-dom-testing (~> 2.0)
-    actionpack (6.1.3)
-      actionview (= 6.1.3)
-      activesupport (= 6.1.3)
+    actionpack (6.1.0)
+      actionview (= 6.1.0)
+      activesupport (= 6.1.0)
       rack (~> 2.0, >= 2.0.9)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actiontext (6.1.3)
-      actionpack (= 6.1.3)
-      activerecord (= 6.1.3)
-      activestorage (= 6.1.3)
-      activesupport (= 6.1.3)
+    actiontext (6.1.0)
+      actionpack (= 6.1.0)
+      activerecord (= 6.1.0)
+      activestorage (= 6.1.0)
+      activesupport (= 6.1.0)
       nokogiri (>= 1.8.5)
-    actionview (6.1.3)
-      activesupport (= 6.1.3)
+    actionview (6.1.0)
+      activesupport (= 6.1.0)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activejob (6.1.3)
-      activesupport (= 6.1.3)
+    activejob (6.1.0)
+      activesupport (= 6.1.0)
       globalid (>= 0.3.6)
-    activemodel (6.1.3)
-      activesupport (= 6.1.3)
-    activerecord (6.1.3)
-      activemodel (= 6.1.3)
-      activesupport (= 6.1.3)
-    activestorage (6.1.3)
-      actionpack (= 6.1.3)
-      activejob (= 6.1.3)
-      activerecord (= 6.1.3)
-      activesupport (= 6.1.3)
+    activemodel (6.1.0)
+      activesupport (= 6.1.0)
+    activerecord (6.1.0)
+      activemodel (= 6.1.0)
+      activesupport (= 6.1.0)
+    activestorage (6.1.0)
+      actionpack (= 6.1.0)
+      activejob (= 6.1.0)
+      activerecord (= 6.1.0)
+      activesupport (= 6.1.0)
       marcel (~> 0.3.1)
       mimemagic (~> 0.3.2)
-    activesupport (6.1.3)
+    activesupport (6.1.0)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
       zeitwerk (~> 2.3)
+    bcrypt (3.1.16)
     builder (3.2.4)
+    byebug (11.1.3)
     concurrent-ruby (1.1.8)
     crass (1.0.6)
+    devise (4.7.3)
+      bcrypt (~> 3.0)
+      orm_adapter (~> 0.1)
+      railties (>= 4.1.0)
+      responders
+      warden (~> 1.2.3)
     erubi (1.10.0)
     erubis (2.7.0)
     ffaker (2.18.0)
@@ -118,41 +126,47 @@ GEM
     method_source (1.0.0)
     mimemagic (0.3.5)
     mini_mime (1.0.2)
+    mini_portile2 (2.5.0)
     minitest (5.14.4)
-    nio4r (2.5.5)
-    nokogiri (1.11.1-x86_64-darwin)
+    nio4r (2.5.7)
+    nokogiri (1.11.1)
+      mini_portile2 (~> 2.5.0)
       racc (~> 1.4)
+    orm_adapter (0.5.0)
     racc (1.5.2)
     rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
-    rails (6.1.3)
-      actioncable (= 6.1.3)
-      actionmailbox (= 6.1.3)
-      actionmailer (= 6.1.3)
-      actionpack (= 6.1.3)
-      actiontext (= 6.1.3)
-      actionview (= 6.1.3)
-      activejob (= 6.1.3)
-      activemodel (= 6.1.3)
-      activerecord (= 6.1.3)
-      activestorage (= 6.1.3)
-      activesupport (= 6.1.3)
+    rails (6.1.0)
+      actioncable (= 6.1.0)
+      actionmailbox (= 6.1.0)
+      actionmailer (= 6.1.0)
+      actionpack (= 6.1.0)
+      actiontext (= 6.1.0)
+      actionview (= 6.1.0)
+      activejob (= 6.1.0)
+      activemodel (= 6.1.0)
+      activerecord (= 6.1.0)
+      activestorage (= 6.1.0)
+      activesupport (= 6.1.0)
       bundler (>= 1.15.0)
-      railties (= 6.1.3)
+      railties (= 6.1.0)
       sprockets-rails (>= 2.0.0)
     rails-dom-testing (2.0.3)
       activesupport (>= 4.2.0)
       nokogiri (>= 1.6)
     rails-html-sanitizer (1.3.0)
       loofah (~> 2.3)
-    railties (6.1.3)
-      actionpack (= 6.1.3)
-      activesupport (= 6.1.3)
+    railties (6.1.0)
+      actionpack (= 6.1.0)
+      activesupport (= 6.1.0)
       method_source
       rake (>= 0.8.7)
       thor (~> 1.0)
     rake (13.0.3)
+    responders (3.0.1)
+      actionpack (>= 5.0)
+      railties (>= 5.0)
     ruby_parser (3.15.1)
       sexp_processor (~> 4.9)
     sass-rails (6.0.0)
@@ -173,6 +187,7 @@ GEM
       actionpack (>= 4.0)
       activesupport (>= 4.0)
       sprockets (>= 3.0.0)
+    sqlite3 (1.4.2)
     temple (0.8.2)
     thor (1.1.0)
     tilt (2.0.10)
@@ -180,6 +195,8 @@ GEM
       rails (>= 6.0.0)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
+    warden (1.2.9)
+      rack (>= 2.0.9)
     websocket-driver (0.7.3)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
@@ -189,7 +206,11 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  byebug
+  devise
   hot-glue!
+  rails (= 6.1)
+  sqlite3
 
 BUNDLED WITH
    2.1.4

data/README.md

@@ -282,7 +282,11 @@ Omits delete action.
 
 # VERSION HISTORY
 
-#### 2021-03-01 - v0.0.4 - Validation magic; refactors the options to the correct Rails::Generators syntax
+#### 2021-03-08 - v0.1   - testing, testing, testing
+
+#### 2021-03-01 - v0.0.5 - Validation magic; refactors the options to the correct Rails::Generators syntax
+
+#### (yanked)   - v0.0.4
 
 #### 2021-02-27 - v0.0.3 - several fixes for namespaces; adds pagination; adds exclude list to fields
 
@@ -300,3 +304,34 @@ Omits delete action.
 I hope one day I will leave this Earth a poor man (like my code) owning only the most simple structure for the simple form of my existence. Thanks for having educated me in this wisdom goes to my former mentor [@trak3r](https://github.com/trak3r)! 
 
 
+
+
+# HOW THIS GEM IS TESTED
+
+The testing of functionality-within-functionality is a little tricky and requires thinking outside the box.
+
+We have two kinds of "sandboxes": a DUMMY sandbox, and also a STRAWMAN sandbox
+
+The dummy sandbox is found at `spec/dummy`
+
+The dummy lives as mostly checked- into the repository, except the folders where the generated code goes (`spec/dummy/app/views/`, `spec/dummy/app/controllers/`, `spec/dummy/specs/` are excluded from Git)
+
+When you run the **internal specs**, which you can do **at the root of this repo** using the command `rspec`, a set of specs will run to assert the generators are erroring when they are supposed to and producing code when they are supposed to.
+
+
+The DUMMY testing DOES NOT test the actual functionality of the output code (it just tests the functionality of the generation process).
+
+For this reason, I've also added something I call STRAWMAN testing, which is a set of steps that does these things:
+
+The Strawman isn't in the repository, but if you build it locally, it will create itself into
+
+`spec/strawman/`
+
+1) Builds you a strawman sandbox app from scratch, using the native `rails new`
+2) Makes a few small modifications to the new app to support this gem
+3) Create two nonsense tables called OmnitableA and OmnitableB. Each omnitable has one of each kind of field (integer, text, string, date, time, etc)
+4) In this way, it is a "Noah's Arc" testing strategy: It will build you a fully functional app using all of the feature sets of the gem.
+5) The built app itself will contain its own specs
+6) You will then cd into `spec/strawman` and run `rspec`. This second test suite now runs to full test all of the *generated* code. 
+
+

data/Rakefile

@@ -14,7 +14,7 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-APP_RAKEFILE = File.expand_path("test/dummy/Rakefile", __dir__)
+APP_RAKEFILE = File.expand_path("spec/dummy/Rakefile", __dir__)
 load 'rails/tasks/engine.rake'
 
 load 'rails/tasks/statistics.rake'

data/bin/rails

@@ -3,8 +3,8 @@
 # installed from the root of your application.
 
 ENGINE_ROOT = File.expand_path('..', __dir__)
-ENGINE_PATH = File.expand_path('../lib/common_core_js/engine', __dir__)
-APP_PATH = File.expand_path('../test/dummy/config/application', __dir__)
+ENGINE_PATH = File.expand_path('../lib/hotglue/engine', __dir__)
+APP_PATH = File.expand_path('../spec/dummy/application', __dir__)
 
 # Set up gems listed in the Gemfile.
 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)

data/config/database.yml

@@ -0,0 +1,11 @@
+development:
+  adapter: sqlite3
+  database: ../spec/dummy/db/development.sqlite3
+  pool: 5
+  timeout: 5000
+
+test:
+  adapter: sqlite3
+  database: ../spec/dummy/db/test.sqlite3
+  pool: 5
+  timeout: 5000

data/db/migrate/20210306212711_create_abcs.rb

@@ -0,0 +1,11 @@
+class CreateAbcs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :abcs do |t|
+      t.integer :xxx
+      t.string :yyy
+      t.integer :def_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223300_create_defs.rb

@@ -0,0 +1,9 @@
+class CreateDefs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :defs do |t|
+      t.integer :user_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223305_create_hgis.rb

@@ -0,0 +1,9 @@
+class CreateHgis < ActiveRecord::Migration[6.1]
+  def change
+    create_table :hgis do |t|
+      t.integer :def_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223309_create_jkls.rb

@@ -0,0 +1,9 @@
+class CreateJkls < ActiveRecord::Migration[6.1]
+  def change
+    create_table :jkls do |t|
+      t.integer :hgi_id
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20210306223701_devise_create_users.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+class DeviseCreateUsers < ActiveRecord::Migration[6.1]
+  def change
+    create_table :users do |t|
+      ## Database authenticatable
+      t.string :email,              null: false, default: ""
+      t.string :encrypted_password, null: false, default: ""
+
+      ## Recoverable
+      t.string   :reset_password_token
+      t.datetime :reset_password_sent_at
+
+      ## Rememberable
+      t.datetime :remember_created_at
+
+      ## Trackable
+      # t.integer  :sign_in_count, default: 0, null: false
+      # t.datetime :current_sign_in_at
+      # t.datetime :last_sign_in_at
+      # t.string   :current_sign_in_ip
+      # t.string   :last_sign_in_ip
+
+      ## Confirmable
+      # t.string   :confirmation_token
+      # t.datetime :confirmed_at
+      # t.datetime :confirmation_sent_at
+      # t.string   :unconfirmed_email # Only if using reconfirmable
+
+      ## Lockable
+      # t.integer  :failed_attempts, default: 0, null: false # Only if lock strategy is :failed_attempts
+      # t.string   :unlock_token # Only if unlock strategy is :email or :both
+      # t.datetime :locked_at
+
+
+      t.timestamps null: false
+    end
+
+    add_index :users, :email,                unique: true
+    add_index :users, :reset_password_token, unique: true
+    # add_index :users, :confirmation_token,   unique: true
+    # add_index :users, :unlock_token,         unique: true
+  end
+end

data/db/migrate/20210306225506_create_xyzs.rb

@@ -0,0 +1,9 @@
+class CreateXyzs < ActiveRecord::Migration[6.1]
+  def change
+    create_table :xyzs do |t|
+      t.integer :nothing_id
+
+      t.timestamps
+    end
+  end
+end

data/db/schema.rb

@@ -0,0 +1,59 @@
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# This file is the source Rails uses to define your schema when running `bin/rails
+# db:schema:load`. When creating a new database, `bin/rails db:schema:load` tends to
+# be faster and is potentially less error prone than running all of your
+# migrations from scratch. Old migrations may fail to apply correctly if those
+# migrations use external dependencies or application code.
+#
+# It's strongly recommended that you check this file into your version control system.
+
+ActiveRecord::Schema.define(version: 2021_03_06_225506) do
+
+  create_table "abcs", force: :cascade do |t|
+    t.integer "xxx"
+    t.string "yyy"
+    t.integer "def_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "defs", force: :cascade do |t|
+    t.integer "user_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "hgis", force: :cascade do |t|
+    t.integer "def_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "jkls", force: :cascade do |t|
+    t.integer "hgi_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+  create_table "users", force: :cascade do |t|
+    t.string "email", default: "", null: false
+    t.string "encrypted_password", default: "", null: false
+    t.string "reset_password_token"
+    t.datetime "reset_password_sent_at"
+    t.datetime "remember_created_at"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+    t.index ["email"], name: "index_users_on_email", unique: true
+    t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
+  end
+
+  create_table "xyzs", force: :cascade do |t|
+    t.integer "nothing_id"
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+  end
+
+end

data/lib/generators/hot_glue/install_generator.rb

@@ -10,9 +10,9 @@ module HotGlue
 
     def initialize(*args) #:nodoc:
       super
-      # copy_file "common_core.js", "app/javascript/common_core.js"
-      # copy_file "common_core.scss", "app/assets/stylesheets/common_core.scss"
-      copy_file "_flash_notices.haml", "app/views/layouts/_flash_notices.haml"
+      # copy_file "hot_glue.js", "#{'spec/dummy/' if Rails.env.test?}app/javascript/hot_glue.js"
+      # copy_file "hot_glue.scss", "#{'spec/dummy/' if Rails.env.test?}app/assets/stylesheets/hot_glue.scss"
+      copy_file "_flash_notices.haml", "#{'spec/dummy/' if Rails.env.test?}app/views/layouts/_flash_notices.haml"
     end
   end
 end

data/lib/generators/hot_glue/scaffold_generator.rb

@@ -3,6 +3,12 @@ require 'ffaker'
 
 
 module HotGlue
+
+
+  class Error < StandardError
+  end
+
+
   module GeneratorHelper
     def text_area_output(col, field_length, col_identifier )
       lines = field_length % 40
@@ -59,32 +65,26 @@ module HotGlue
       begin
         object = eval(class_name)
       rescue StandardError => e
-        puts "*** Oops: It looks like there is no object for #{class_name}. Please define the object + database table first."
-        exit
+        message = "*** Oops: It looks like there is no object for #{class_name}. Please define the object + database table first."
+        raise(HotGlue::Error, message)
+      end
+
+      if @specs_only && @no_specs
+        raise(HotGlue::Error, "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. sorry.")
       end
-      args = meta_args[0]
-      @singular = args.first.tableize.singularize # should be in form hello_world
 
 
+      args = meta_args[0]
+      @singular = args.first.tableize.singularize # should be in form hello_world
       @plural = options['plural'] || @singular + "s" # supply to override; leave blank to use default
       @auth = options['auth'] || "current_user"
       @auth_identifier = options['auth'] || (!@auth.nil? && @auth.gsub("current_", "")) || nil
-
-
       @nest = options['auth'] || nil
       @namespace = options['namespace'] || nil
-
       @singular_class = @singular.titleize.gsub(" ", "")
-
       @exclude_fields = []
-
-
-
       @exclude_fields += options['exclude'].split(",").collect(&:to_sym)
-
       auth_assoc = @auth.gsub("current_","")
-
-
       @god = options['god'] || options['gd'] || false
       @specs_only = options['specs-only'] || false
       @no_specs = options['no-specs'] || false
@@ -92,22 +92,15 @@ module HotGlue
       @no_create = options['no-create'] || false
       @no_paginate = options['no-paginate'] || false
 
-      if @specs_only && @no_specs
-        puts "*** Oops: You seem to have specified both the --specs-only flag and --no-specs flags. this doesn't make any sense, so I am aborting. sorry."
-        exit
-      end
-
       if @god
         @auth = nil
       end
 
-
       # when in self auth, the object is the same as the authenticated object
       if @auth && auth_identifier == @singular
         @self_auth = true
       end
 
-
       @nested_args = []
       if !@nest.nil?
         @nested_args = @nest.split("/")
@@ -134,19 +127,20 @@ module HotGlue
         end
       end
 
-                                 # create the columns
+
       if !@object_owner_sym.empty?
         auth_assoc_field = auth_assoc + "_id"
         assoc = eval("#{singular_class}.reflect_on_association(:#{@object_owner_sym})")
+
         if assoc
           ownership_field = assoc.name.to_s + "_id"
         else
-          if @auth
-            puts "*** Oops: It looks like is no association from current_#{@object_owner_sym} to a class called #{singular_class}. If your user is called something else, pass with flag auth=current_X where X is the model for your users as lowercase. Also, be sure to implement current_X as a method on your controller. (If you really don't want to implement a current_X on your controller and want me to check some other method for your current user, see the section in the docs for auth_identifier.) To make a controller that can read all records, specify with --god."
-          else
-            puts "*** Oops:  god mode could not find the association(?). something is wrong."
-          end
-          exit
+          # if @auth
+            exit_message= "*** Oops: It looks like is no association from current_#{@object_owner_sym} to a class called #{singular_class}. If your user is called something else, pass with flag auth=current_X where X is the model for your users as lowercase. Also, be sure to implement current_X as a method on your controller. (If you really don't want to implement a current_X on your controller and want me to check some other method for your current user, see the section in the docs for auth_identifier.) To make a controller that can read all records, specify with --god."
+          # else
+          #   exit_message= "*** Oops:  god mode could not find the association(?). something is wrong."
+          # end
+          raise(HotGlue::Error, exit_message)
         end
       end
 
@@ -158,18 +152,52 @@ module HotGlue
 
       @exclude_fields.push(auth_assoc_field.to_sym) if !auth_assoc_field.nil?
       @exclude_fields.push(ownership_field.to_sym) if !ownership_field.nil?
+      @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| @exclude_fields.include?(field) }
 
-      begin
-        @columns = object.columns.map(&:name).map(&:to_sym).reject{|field| @exclude_fields.include?(field) }
-      rescue StandardError => e
-        puts "Ooops... #{e} it looks like is no object for #{class_name}. Please create the database table with fields first. "
-        exit
-      end
+      @columns.each do |col|
+        if object.columns_hash[col.to_s].type == :integer
+          if col.to_s.ends_with?("_id")
+            # guess the association name label
+            assoc_name = col.to_s.gsub("_id","")
+            assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
+
+
+            begin
+              eval(assoc.class_name)
+            rescue NameError => e
+              exit_message = "*** Oops: The table #{singular_class} has an association for '#{assoc.name.to_s}', but I can't find an assoicated model for that association. TODO: Please implement a model for #{assoc.name.to_s} that belongs to #{singular_class} "
+              raise(HotGlue::Error, exit_message)
 
+            end
 
 
+            if assoc.nil?
+              exit_message= "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+              raise(HotGlue::Error,exit_message)
+            end
+            assoc_class = eval(assoc.class_name)
+            if assoc_class.column_names.include?("name")
+              # display_column = "name"
+            elsif assoc_class.column_names.include?("to_label")
+              # display_column = "to_label"
+            elsif assoc_class.column_names.include?("full_name")
+              # display_column = "full_name"
+            elsif assoc_class.column_names.include?("display_name")
+              # display_column = "display_name"
+            elsif assoc_class.column_names.include?("email")
+              # display_column = "email"
+            else
+              exit_message= "*** Oops: Can't find any column to use as the display label for the #{assoc.name.to_s} association on the #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, or 5) email directly on your #{assoc.class_name} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
+              raise(HotGlue::Error,exit_message)
+            end
+          end
+        end
+      end
     end
 
+
+
+    #
     def formats
       [format]
     end
@@ -182,17 +210,17 @@ module HotGlue
       @default_colspan = @columns.size
 
       unless @specs_only
-        template "controller.rb", File.join("app/controllers#{namespace_with_dash}", "#{plural}_controller.rb")
+        template "controller.rb", File.join("#{'spec/dummy/' if Rails.env.test?}app/controllers#{namespace_with_dash}", "#{plural}_controller.rb")
         if @namespace &&  defined?(controller_descends_from) == nil
-          template "base_controller.rb", File.join("app/controllers#{namespace_with_dash}", "base_controller.rb")
+          template "base_controller.rb", File.join("#{'spec/dummy/' if Rails.env.test?}app/controllers#{namespace_with_dash}", "base_controller.rb")
         end
       end
 
       unless @no_specs
-        template "controller_spec.rb", File.join("spec/controllers#{namespace_with_dash}", "#{plural}_controller_spec.rb")
+        template "controller_spec.rb", File.join("#{'spec/dummy/' if Rails.env.test?}spec/controllers#{namespace_with_dash}", "#{plural}_controller_spec.rb")
       end
 
-      template "_errors.haml", File.join("app/views#{namespace_with_dash}", "_errors.haml")
+      template "_errors.haml", File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", "_errors.haml")
     end
 
     def list_column_headings
@@ -387,14 +415,14 @@ module HotGlue
       haml_views.each do |view|
         formats.each do |format|
           filename = cc_filename_with_extensions(view, "haml")
-          template filename, File.join("app/views#{namespace_with_dash}", controller_file_path, filename)
+          template filename, File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", controller_file_path, filename)
         end
       end
 
       turbo_stream_views.each do |view|
         formats.each do |format|
           filename = cc_filename_with_extensions(view, 'turbo_stream.haml')
-          template filename, File.join("app/views#{namespace_with_dash}", controller_file_path, filename)
+          template filename, File.join("#{'spec/dummy/' if Rails.env.test?}app/views#{namespace_with_dash}", controller_file_path, filename)
         end
       end
     end
@@ -451,15 +479,15 @@ module HotGlue
         when :integer
           # look for a belongs_to on this object
           if col.to_s.ends_with?("_id")
-            # guess the association name label
-
-
-            assoc_name = col.to_s.gsub("_id","")
-            assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
-            if assoc.nil?
-              puts "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
-              exit
-            end
+            # # guess the association name label
+            #
+            #
+            # assoc_name = col.to_s.gsub("_id","")
+            # assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
+            # if assoc.nil?
+            #   exit_message= "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+            #   exit
+            # end
 
 
             assoc_class = eval(assoc.class_name)
@@ -475,7 +503,8 @@ module HotGlue
             elsif assoc_class.column_names.include?("email")
               display_column = "email"
             else
-              puts "*** Oops: Can't find any column to use as the display label for the #{assoc.name.to_s} association on the #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, or 5) email directly on your #{assoc.class_name} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
+              raise("this should have been caught by the checker in the initializer")
+              # puts "*** Oops: Can't find any column to use as the display label for the #{assoc.name.to_s} association on the #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, or 5) email directly on your #{assoc.class_name} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
             end
 
             "#{col_identifier}{class: \"form-group \#{'alert-danger' if #{singular}.errors.details.keys.include?(:#{assoc_name.to_s})}\"}
@@ -556,8 +585,8 @@ module HotGlue
             assoc = eval("#{singular_class}.reflect_on_association(:#{assoc_name})")
 
             if assoc.nil?
-              puts "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
-              exit
+              exit_message =  "*** Oops. on the #{singular_class} object, there doesn't seem to be an association called '#{assoc_name}'"
+              raise(HotGlue::Error,exit_message)
             end
             
             assoc_class = eval(assoc.class_name)
@@ -664,7 +693,8 @@ module HotGlue
           display_column = "number"
 
         else
-          raise "*** Oops: Can't find any column to use as the display label on #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, 5) email, or 6) number directly on your #{singular_class} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
+          exit_message = "*** Oops: Can't find any column to use as the display label on #{singular_class} model . TODO: Please implement just one of: 1) name, 2) to_label, 3) full_name, 4) display_name, 5) email, or 6) number directly on your #{singular_class} model (either as database field or model methods), then RERUN THIS GENERATOR. (If more than one is implemented, the field to use will be chosen based on the rank here, e.g., if name is present it will be used; if not, I will look for a to_label, etc)"
+          raise(HotGlue::Error, exit_message)
         end
     end
 

data/lib/hotglue/version.rb

@@ -1,3 +1,3 @@
 module HotGlue
-  VERSION = '0.0.5'
+  VERSION = '0.0.6'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hot-glue
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Jason Fleetwood-Boldt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-01 00:00:00.000000000 Z
+date: 2021-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -108,9 +108,18 @@ files:
 - LICENCE
 - README.md
 - Rakefile
+- app/assets/config/manifest.js
 - app/helpers/hot_glue/controller_helper.rb
 - app/helpers/hot_glue_helper.rb
 - bin/rails
+- config/database.yml
+- db/migrate/20210306212711_create_abcs.rb
+- db/migrate/20210306223300_create_defs.rb
+- db/migrate/20210306223305_create_hgis.rb
+- db/migrate/20210306223309_create_jkls.rb
+- db/migrate/20210306223701_devise_create_users.rb
+- db/migrate/20210306225506_create_xyzs.rb
+- db/schema.rb
 - lib/generators/hot_glue/install_generator.rb
 - lib/generators/hot_glue/scaffold_generator.rb
 - lib/generators/hot_glue/templates/_errors.haml
@@ -121,9 +130,9 @@ files:
 - lib/generators/hot_glue/templates/_new_button.haml
 - lib/generators/hot_glue/templates/_new_form.haml
 - lib/generators/hot_glue/templates/_show.haml
-- lib/generators/hot_glue/templates/base_controller.rb
-- lib/generators/hot_glue/templates/controller.rb
-- lib/generators/hot_glue/templates/controller_spec.rb
+- lib/generators/hot_glue/templates/base_controller.rb.erb
+- lib/generators/hot_glue/templates/controller.rb.erb
+- lib/generators/hot_glue/templates/controller_spec.rb.erb
 - lib/generators/hot_glue/templates/create.turbo_stream.haml
 - lib/generators/hot_glue/templates/destroy.turbo_stream.haml
 - lib/generators/hot_glue/templates/edit.haml
@@ -172,7 +181,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.11
+rubygems_version: 3.0.8
 signing_key:
 specification_version: 4
 summary: A gem build scaffolding.