RubyGems - localtower - Versions diffs - 0.5.0 → 1.0.0 - Mend

localtower 0.5.0 → 1.0.0

Files changed (61) hide show

checksums.yaml +4 -4
data/README.md +25 -17
data/app/controllers/localtower/pages_controller.rb +27 -12
data/app/views/layouts/localtower/application.html.erb +16 -33
data/app/views/localtower/pages/_alert_no_models.html.erb +3 -0
data/app/views/localtower/pages/migrations.html.erb +52 -83
data/app/views/localtower/pages/models.html.erb +53 -70
data/app/views/localtower/pages/new_migration.html.erb +103 -0
data/app/views/localtower/pages/new_model.html.erb +87 -0
data/config/routes.rb +4 -4
data/lib/localtower/generators/migration.rb +51 -120
data/lib/localtower/generators/model.rb +52 -28
data/lib/localtower/generators/service_objects/insert_array.rb +23 -0
data/lib/localtower/generators/service_objects/insert_defaults.rb +15 -43
data/lib/localtower/generators/service_objects/insert_indexes.rb +80 -0
data/lib/localtower/generators/service_objects/insert_nullable.rb +23 -0
data/lib/localtower/status.rb +1 -1
data/lib/localtower/tools.rb +13 -16
data/lib/localtower/version.rb +1 -1
data/public/css/app.css +0 -49
data/public/js/app.js +215 -70
data/public/screenshots/v1.0.0/migrations.png +0 -0
data/public/screenshots/v1.0.0/models.png +0 -0
data/public/screenshots/v1.0.0/new_migration.png +0 -0
data/public/screenshots/v1.0.0/new_model.png +0 -0
data/spec/dummy/Gemfile +0 -2
data/spec/dummy/Gemfile.lock +1 -10
data/spec/dummy/app/models/post.rb +3 -0
data/spec/dummy/app/models/user.rb +1 -0
data/spec/dummy/config/environments/development.rb +1 -0
data/spec/dummy/config/puma.rb +1 -1
data/spec/dummy/db/migrate/20230119221452_create_users.rb +14 -0
data/spec/dummy/db/migrate/20230119221751_change_users_at1674166670.rb +7 -0
data/spec/dummy/db/migrate/20230119222054_create_posts.rb +11 -0
data/spec/dummy/db/migrate/20230119222106_change_posts_at1674166865.rb +5 -0
data/spec/dummy/db/schema.rb +20 -8
data/spec/dummy/log/development.log +15281 -3345
data/spec/dummy/log/localtower.log +1897 -132
data/spec/dummy/log/test.log +0 -0
data/spec/dummy/test/index.html +38 -0
data/spec/dummy/tmp/pids/server.pid +1 -0
data/spec/factories/migration.rb +25 -41
data/spec/factories/model.rb +39 -25
data/spec/lib/localtower/generators/migration_spec.rb +36 -63
data/spec/lib/localtower/generators/model_spec.rb +43 -34
data/spec/lib/localtower/generators/service_objects/insert_array_spec.rb +47 -0
data/spec/lib/localtower/generators/service_objects/insert_defaults_spec.rb +30 -35
data/spec/lib/localtower/generators/service_objects/insert_indexes_spec.rb +90 -0
data/spec/lib/localtower/generators/service_objects/insert_nullable_spec.rb +61 -0
data/spec/lib/localtower/tools_spec.rb +1 -11
data/spec/spec_helper.rb +8 -3
metadata +34 -18
data/app/views/localtower/pages/_migrations.html.erb +0 -57
data/app/views/localtower/pages/schema.html.erb +0 -67
data/spec/dummy/db/migrate/20221115190039_create_users.rb +0 -13
data/spec/dummy/db/migrate/20221115193020_change_users.rb +0 -8
data/spec/dummy/db/migrate/20221115193532_change_users_at1668540931.rb +0 -5
data/spec/dummy/db/migrate/20221115193605_change_users_at1668540964.rb +0 -5
data/spec/dummy/db/migrate/20221115193637_change_users_at1668540996.rb +0 -5
data/spec/dummy/db/migrate/20221115193642_change_users_at1668541001.rb +0 -5
data/spec/lib/localtower/generators/relation_spec.rb +0 -65

data/spec/dummy/log/test.log ADDED Viewed

File without changes

data/spec/dummy/test/index.html ADDED Viewed

@@ -0,0 +1,38 @@
+<!doctype html>
+<html>
+<head>
+  <meta charset="UTF-8">
+  <meta name="viewport" content="width=device-width, initial-scale=1.0">
+  <script src="https://cdn.tailwindcss.com"></script>
+  <script>
+    tailwind.config = {
+      theme: {
+        extend: {
+          colors: {
+            clifford: '#da373d',
+          }
+        }
+      }
+    }
+  </script>
+    <style type="text/tailwindcss">
+      @layer utilities {
+        .content-auto {
+          content-visibility: auto;
+        }
+        .nude {
+          @apply text-green-500;
+        }
+      }
+    </style>
+</head>
+<body>
+  <div class="container mx-auto">
+    <h1 class="nude text-3xl font-bold underline">
+      Hello world!
+    </h1>
+  </div>
+</body>
+</html>

data/spec/dummy/tmp/pids/server.pid ADDED Viewed

	@@ -0,0 +1 @@
1	+ 67242

data/spec/factories/migration.rb CHANGED Viewed

@@ -1,28 +1,4 @@
 FactoryBot.define do
-  factory :create_table, class: "Hash" do
-    migration_name { "Post" }
-    migrations do
-      [
-        {
-          action: "create_table",
-          table_name: "posts"
-        }
-      ]
-    end
-  end
-  factory :create_table_two, class: "Hash" do
-    migration_name { "User" }
-    migrations do
-      [
-        {
-          action: "create_table",
-          table_name: "users"
-        }
-      ]
-    end
-  end
   factory :add_column, class: "Hash" do
     migration_name { "Post" }
     migrations do
@@ -32,7 +8,6 @@ FactoryBot.define do
           table_name: "posts",
           column: "title",
           column_type: "string",
-          index: true,
           nullable: true,
         },
         {
@@ -40,7 +15,6 @@ FactoryBot.define do
           table_name: "posts",
           column: "content",
           column_type: "text",
-          index: false,
           nullable: true,
         },
         {
@@ -64,20 +38,6 @@ FactoryBot.define do
     end
   end
-  factory :add_column_fail, class: "Hash" do
-    migration_name { "Post" }
-    migrations do
-      [
-        {
-          action: "add_column",
-          table_name: "posts",
-          column: "title",
-          # column_type: "string",
-        }
-      ]
-    end
-  end
   factory :rename_column, class: "Hash" do
     migration_name { "Post" }
     migrations do
@@ -134,10 +94,34 @@ FactoryBot.define do
     migration_name { "Post" }
     migrations do
       [
+        {
+          action: "add_column",
+          table_name: "posts",
+          column: "content",
+          column_type: "text",
+        },
         {
           action: "add_index_to_column",
           table_name: "posts",
-          column: "content_new",
+          column: "content",
+        },
+        {
+          action: "add_index_to_column",
+          table_name: "posts",
+          column: "content_second",
+          index: {
+            using: 'default',
+            unique: true,
+          }
+        },
+        {
+          action: "add_index_to_column",
+          table_name: "posts",
+          column: "content_third",
+          index: {
+            using: 'gin',
+            algorithm: 'concurrently',
+          }
         }
       ]
     end

data/spec/factories/model.rb CHANGED Viewed

@@ -1,40 +1,54 @@
 FactoryBot.define do
-  factory :post_one, class: "Hash" do
-    model_name { "Post" }
+  factory :post_one, class: Hash do
+    model_name { 'Post' }
     attributes do
       [
         {
-          attribute_name: "title",
-          attribute_type: "string",
-          index: true
+          'attribute_name' => 'title',
+          'attribute_type' => 'string',
+          'index' => {
+            'using' => 'default',
+            'unique' => true,
+            'algorithm' => 'default',
+          }
         },
         {
-          attribute_name: "content",
-          attribute_type: "text"
+          'attribute_name' => 'tags',
+          'attribute_type' => 'array',
+          'nullable' => false,
+          'default' => '[]',
+          'index' => {
+            'using' => 'gin',
+            'unique' => true,
+            'algorithm' => 'concurrently',
+          }
         },
-      ]
-    end
-  end
-  factory :user_one, class: "Hash" do
-    model_name { "User" }
-    attributes do
-      [
         {
-          attribute_name: "name",
-          attribute_type: "string",
-          index: true,
+          'attribute_name' => 'content',
+          'attribute_type' => 'text',
+          'nullable' => false,
+          'index' => {
+            'using' => 'gist',
+            'algorithm' => 'default',
+          }
         },
         {
-          attribute_name: "metadata",
-          attribute_type: "jsonb"
+          'attribute_name' => 'likes_count',
+          'attribute_type' => 'integer',
+          'nullable' => true,
+          'default' => '0',
+        },
+        {
+          'attribute_name' => 'score',
+          'attribute_type' => 'float',
+        },
+        {
+          'attribute_name' => 'metadata',
+          'attribute_type' => 'jsonb',
+          'nullable' => false,
+          'default' => '{}',
         },
       ]
     end
   end
-  factory :relation_one, class: "Hash" do
-    model_one_name { "User" }
-    model_two_name { "Post" }
-  end
 end

data/spec/lib/localtower/generators/migration_spec.rb CHANGED Viewed

@@ -3,111 +3,84 @@ require 'spec_helper'
 module Localtower
   module Generators
     describe Migration do
-      before(:all) do
-        clean_files
-      end
-      after(:all) do
-      end
-      it 'create_table' do
-        data = attributes_for(:create_table)
-        data["run_migrate"] = true
-        generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /create_table :posts/)).to eq(true)
-      end
-      it 'create_table_two' do
-        data = attributes_for(:create_table_two)
-        data["run_migrate"] = true
-        generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /create_table :users/)).to eq(true)
-      end
       it 'add_column' do
-        data = attributes_for(:add_column)
-        data["run_migrate"] = true
+        data = attributes_for(:add_column)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :tags, :text, default: \[\], array: true/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_index :posts, :tags, using: :gin/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :views, :integer, default: 0, null: false, index: true/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :content, :text/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :title, :string, index: true/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_column :posts, :tags, :string, default: \[\], array: true/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_column :posts, :views, :integer, default: 0, null: false/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_column :posts, :content, :text/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_column :posts, :title, :string/)).to eq(true)
       end
-      it 'add_column fail' do
-        data = attributes_for(:add_column_fail)
-        data["run_migrate"] = true
+      it 'remove_column' do
+        data = attributes_for(:remove_column)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :tags, :text, default: \[\], array: true/)).to eq(false)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_index :posts, :tags, using: :gin/)).to eq(false)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :views, :integer, default: 0, null: false, index: true/)).to eq(false)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :content, :text/)).to eq(false)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_column :posts, :title, :string, index: true/)).to eq(false)
+        expect(word_in_file?(last_migration, /remove_column :posts, :title/)).to eq(true)
+        expect(word_in_file?(last_migration, /rename_column :posts, :views, :views_by_users/)).to eq(true)
       end
       it 'rename_column' do
-        data = attributes_for(:rename_column)
-        data["run_migrate"] = true
+        data = attributes_for(:rename_column)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /rename_column :posts, :content, :content_new/)).to eq(true)
+        expect(word_in_file?(last_migration, /rename_column :posts, :content, :content_new/)).to eq(true)
       end
-      it 'remove_column' do
-        data = attributes_for(:remove_column)
-        data["run_migrate"] = true
+      it 'change_column_type' do
+        data = attributes_for(:change_column_type)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /remove_column :posts, :title/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /rename_column :posts, :views, :views_by_users/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_index :posts, :views_by_users/)).to eq(true)
+        expect(word_in_file?(last_migration, /change_column :posts, :content_new, :string/)).to eq(true)
       end
-      it 'change_column_type' do
-        data = attributes_for(:change_column_type)
-        data["run_migrate"] = true
+      it 'add_index_to_column' do
+        data = attributes_for(:add_index_to_column)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_index :posts, :views_by_users/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?(last_migration, /change_column :posts, :content_new, :string/)).to eq(true)
+        expect(word_in_file?(last_migration, /disable_ddl_transaction!/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_index :posts, :content/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_index :posts, :content_second, unique: true/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_index :posts, :content_third, using: :gin, algorithm: :concurrently/)).to eq(true)
       end
-      it 'add_index_to_column' do
-        data = attributes_for(:add_index_to_column)
-        data["run_migrate"] = true
+      it 'belongs_to' do
+        data = attributes_for(:belongs_to)[:migrations]
+        model_post = "#{Rails.root}/app/models/post.rb"
+        model_user = "#{Rails.root}/app/models/user.rb"
+        File.open(model_post, 'w') { |f| f.write("class Post < ApplicationRecord\nend")}
+        File.open(model_user, 'w') { |f| f.write("class User < ApplicationRecord\nend")}
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_index :posts, :content_new/)).to eq(true)
+        expect(word_in_file?(model_post, /# belongs_to :user/)).to eq(true)
+        expect(word_in_file?(model_user, /# has_many :posts/)).to eq(true)
+        expect(word_in_file?(last_migration, /add_reference :posts, :user, foreign_key: true, index: true/)).to eq(true)
       end
       it 'remove_index_to_column' do
-        data = attributes_for(:remove_index_to_column)
-        data["run_migrate"] = true
+        data = attributes_for(:remove_index_to_column)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /remove_index :posts, :views_by_users/)).to eq(true)
+        expect(word_in_file?(last_migration, /remove_index :posts, :views_by_users/)).to eq(true)
       end
       it 'drop_table' do
-        data = attributes_for(:drop_table)
-        data["run_migrate"] = true
+        data = attributes_for(:drop_table)[:migrations]
         generator = ::Localtower::Generators::Migration.new(data).run
-        expect(::Localtower::Tools.word_in_file?(last_migration, /drop_table :posts, force: :cascade/)).to eq(true)
+        expect(word_in_file?(last_migration, /drop_table :posts/)).to eq(true)
       end
     end
   end

data/spec/lib/localtower/generators/model_spec.rb CHANGED Viewed

@@ -3,40 +3,49 @@ require 'spec_helper'
 module Localtower
   module Generators
     describe Model do
-      before(:all) do
-        clean_files
-      end
-      after(:all) do
-      end
-      it 'create a post' do
-        data = attributes_for(:post_one)
-        data["run_migrate"] = true
-        ::Localtower::Generators::Model.new(data).run
-        expect(File.exist?("#{Rails.root}/app/models/post.rb")).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/post.rb", /class Post/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /create_table "posts"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.string(.*)"title"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.text(.*)"content"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.index(.*)\["title"\], name: "index_posts_on_title"/)).to eq(true)
-      end
-      it 'create a user' do
-        data = attributes_for(:user_one)
-        data["run_migrate"] = true
-        ::Localtower::Generators::Model.new(data).run
-        expect(File.exist?("#{Rails.root}/app/models/user.rb")).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/user.rb", /class User/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /create_table "users"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.string(.*)"name"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.jsonb(.*)"metadata"/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.index(.*)\["name"\], name: "index_users_on_name"/)).to eq(true)
+      context 'model 1' do
+        let(:expected_migration) do
+          <<-TEXT_MULTILINE.strip_heredoc
+          class CreatePosts < ActiveRecord::Migration[5.2]
+            disable_ddl_transaction!
+            def change
+              create_table :posts do |t|
+                t.string :title
+                t.string :tags, array: true, default: [], null: false
+                t.text :content, null: false
+                t.integer :likes_count, default: 0
+                t.float :score
+                t.jsonb :metadata, default: {}, null: false
+                t.timestamps
+              end
+              add_index :posts, :title, unique: true
+              add_index :posts, :tags, using: :gin, unique: true, algorithm: :concurrently
+              add_index :posts, :content, using: :gist
+              add_index :posts, :likes_count
+              add_index :posts, :score
+              add_index :posts, :metadata
+            end
+          end
+          TEXT_MULTILINE
+        end
+        let(:expected_model) do
+          <<-TEXT_MULTILINE.strip_heredoc
+          class Post < ApplicationRecord
+          end
+          TEXT_MULTILINE
+        end
+        let(:attributes) { attributes_for(:post_one).deep_stringify_keys }
+        it 'create a post' do
+          ::Localtower::Generators::Model.new(attributes).run
+          expect(File.read("#{Rails.root}/app/models/post.rb")).to eq(expected_model)
+          expect(File.read(last_migration)).to eq(expected_migration)
+        end
       end
     end
   end

data/spec/lib/localtower/generators/service_objects/insert_array_spec.rb ADDED Viewed

@@ -0,0 +1,47 @@
+require 'spec_helper'
+describe ::Localtower::Generators::ServiceObjects::InsertArray do
+  let(:service) { described_class.new(attributes) }
+  let(:attributes) {
+    ['tags']
+  }
+  let(:base_file_content) {
+    <<-MIGRATION.strip_heredoc
+      class CreateTests < ActiveRecord::Migration[7.0]
+        def change
+          create_table :tests do |t|
+            t.string :tags
+            t.timestamps
+          end
+        end
+      end
+    MIGRATION
+  }
+  let(:expected_file_content) {
+    <<-MIGRATION.strip_heredoc
+      class CreateTests < ActiveRecord::Migration[7.0]
+        def change
+          create_table :tests do |t|
+            t.string :tags, array: true
+            t.timestamps
+          end
+        end
+      end
+    MIGRATION
+  }
+  before do
+    File.open("#{Rails.root}/db/migrate/0_migration_name_.rb", 'w') { |f| f.write(base_file_content) }
+  end
+  it 'works' do
+    service.call
+    expect(File.read(last_migration)).to eq(expected_file_content)
+  end
+end

data/spec/lib/localtower/generators/service_objects/insert_defaults_spec.rb CHANGED Viewed

@@ -1,32 +1,28 @@
 require 'spec_helper'
 describe ::Localtower::Generators::ServiceObjects::InsertDefaults do
-  let(:latest_migration) do
-    Tempfile.new('foo').tap do |file|
-      file.puts base_file_content
-      file.rewind
-    end
-  end
+  let(:service) { described_class.new(attributes) }
   let(:attributes) {
     [
-      { 'first'  => "true"  },
-      { 'third'  => "nil"   },
+      { 'first'  => "true" },
+      { 'third'  => "nil" },
       { 'fourth' => "false" },
-      { 'fifth'  => "0"     }
+      { 'foo' => "{}" },
+      { 'fifth'  => "0" }
     ]
   }
   let(:base_file_content) {
     <<-MIGRATION.strip_heredoc
-      class CreateTests < ActiveRecord::Migration[5.0]
+      class CreateTests < ActiveRecord::Migration[7.0]
         def change
           create_table :tests do |t|
             t.string  :first
             t.integer :second
             t.string  :third
             t.string  :fourth
+            t.jsonb  :foo
             t.integer :fifth
             t.timestamps
@@ -37,34 +33,33 @@ describe ::Localtower::Generators::ServiceObjects::InsertDefaults do
     MIGRATION
   }
-  let(:expected_file_structure_array) {
-    [
-      "class CreateTests < ActiveRecord::Migration[5.0]\n",
-      "  def change\n",
-      "    create_table :tests do |t|\n",
-      "      t.string  :first, default: true\n",
-      "      t.integer :second\n",
-      "      t.string  :third, default: nil\n",
-      "      t.string  :fourth, default: false\n",
-      "      t.integer :fifth, default: 0\n",
-      "\n",
-      "      t.timestamps\n",
-      "    end\n",
-      "    add_index :tests, :first\n",
-      "  end\n",
-      "end\n"
-    ]
-  }
+  let(:expected_file_content) {
+    <<-MIGRATION.strip_heredoc
+      class CreateTests < ActiveRecord::Migration[7.0]
+        def change
+          create_table :tests do |t|
+            t.string  :first, default: true
+            t.integer :second
+            t.string  :third, default: nil
+            t.string  :fourth, default: false
+            t.jsonb  :foo, default: {}
+            t.integer :fifth, default: 0
-  let(:service) { described_class.new(attributes) }
+            t.timestamps
+          end
+          add_index :tests, :first
+        end
+      end
+    MIGRATION
+  }
   before do
-    allow(service).to receive(:latest_migration).and_return(latest_migration)
-    service.call
+    File.open("#{Rails.root}/db/migrate/0_migration_name_.rb", 'w') { |f| f.write(base_file_content) }
   end
-  it 'inserts default values for attributes' do
-    lines_array = File.readlines(latest_migration)
-    expect(lines_array).to match_array expected_file_structure_array
+  it 'works' do
+    service.call
+    expect(File.read(last_migration)).to eq(expected_file_content)
   end
 end