localtower 0.5.0 → 1.0.0

data/spec/lib/localtower/generators/service_objects/insert_indexes_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+describe ::Localtower::Generators::ServiceObjects::InsertIndexes do
+  let(:service) { described_class.new(attributes) }
+
+  let(:attributes) {
+    [
+      { 'first'  =>
+        {
+          'using' => 'gin',
+          'algorithm' => 'concurrently',
+        }
+      },
+      { 'third'  =>
+        {
+          'using' => 'gist',
+          'unique' => true,
+        }
+      },
+      { 'fourth' =>
+        {
+          'using' => 'default',
+          'algorithm' => 'concurrently',
+        }
+      },
+      { 'fifth'  =>
+        {
+          'using' => 'default',
+          'unique' => true,
+        }
+      }
+    ]
+  }
+
+  let(:base_file_content) {
+    <<-MIGRATION.strip_heredoc
+      class CreateTests < ActiveRecord::Migration[7.0]
+        def change
+          create_table :tests do |t|
+            t.string  :zero
+            t.string  :first
+            t.string  :third
+            t.string  :fourth
+            t.integer :fifth
+
+            t.timestamps
+          end
+          add_index :tests, :first
+          add_index :tests, :third
+          add_index :tests, :fourth
+          add_index :tests, :fifth
+        end
+      end
+    MIGRATION
+  }
+
+  let(:expected_file_content) {
+    <<-MIGRATION.strip_heredoc
+      class CreateTests < ActiveRecord::Migration[7.0]
+        disable_ddl_transaction!
+
+        def change
+          create_table :tests do |t|
+            t.string  :zero
+            t.string  :first
+            t.string  :third
+            t.string  :fourth
+            t.integer :fifth
+
+            t.timestamps
+          end
+          add_index :tests, :first, using: :gin, algorithm: :concurrently
+          add_index :tests, :third, using: :gist, unique: true
+          add_index :tests, :fourth, algorithm: :concurrently
+          add_index :tests, :fifth, unique: true
+        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_nullable_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe ::Localtower::Generators::ServiceObjects::InsertNullable do
+  let(:service) { described_class.new(attributes) }
+
+  let(:attributes) {
+    [
+      'first',
+      'third',
+      'fourth'
+    ]
+  }
+
+  let(:base_file_content) {
+    <<-MIGRATION.strip_heredoc
+      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, default: 'foo'
+            t.integer :fifth
+
+            t.timestamps
+          end
+          add_index :tests, :first
+        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  :first, null: false
+            t.integer :second
+            t.string  :third, null: false
+            t.string  :fourth, null: false, default: 'foo'
+            t.integer :fifth
+
+            t.timestamps
+          end
+          add_index :tests, :first
+        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/tools_spec.rb

@@ -1,14 +1,4 @@
 require 'spec_helper'
 
-describe Localtower::Tools do
-  before(:each) do
-  end
-
-  after(:all) do
-  end
-
-  # TODO:
-  it 'list models' do
-    models = ::Localtower::Tools.models_presented
-  end
+RSpec.describe Localtower::Tools do
 end

data/spec/spec_helper.rb

@@ -16,7 +16,9 @@ end
     f.write(content_schema)
   end
 
-  Dir["#{Rails.root}/app/models/**/*.*"].each { |model_file| File.delete(model_file) }
+  Dir["#{Rails.root}/app/models/**/*.*"]
+  .reject { |file_name| file_name['application_record.rb']}
+  .each { |model_file| File.delete(model_file) }
 end
 
 def migration_files
@@ -27,6 +29,10 @@ def last_migration
   migration_files.sort.last
 end
 
+def word_in_file?(file, word_or_exp)
+  File.readlines(file).grep(word_or_exp).size > 0
+end
+
 #============================
 ENV['RAILS_ENV'] = 'test'
 
@@ -45,13 +51,12 @@ RSpec.configure do |config|
   config.infer_base_class_for_anonymous_controllers = true
 
   config.order = 123
-  # config.order = 'random'
 
   config.before(:suite) do
     FactoryBot.find_definitions
   end
 
-  config.before(:all) do
+  config.before(:each) do
     clean_files
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: localtower
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Damian Le Nouaille Diez
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-16 00:00:00.000000000 Z
+date: 2023-01-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -107,16 +107,20 @@ files:
 - app/controllers/localtower/pages_controller.rb
 - app/helpers/localtower/application_helper.rb
 - app/views/layouts/localtower/application.html.erb
-- app/views/localtower/pages/_migrations.html.erb
+- app/views/localtower/pages/_alert_no_models.html.erb
 - app/views/localtower/pages/migrations.html.erb
 - app/views/localtower/pages/models.html.erb
-- app/views/localtower/pages/schema.html.erb
+- app/views/localtower/pages/new_migration.html.erb
+- app/views/localtower/pages/new_model.html.erb
 - config/routes.rb
 - lib/localtower.rb
 - lib/localtower/engine.rb
 - lib/localtower/generators/migration.rb
 - lib/localtower/generators/model.rb
+- lib/localtower/generators/service_objects/insert_array.rb
 - lib/localtower/generators/service_objects/insert_defaults.rb
+- lib/localtower/generators/service_objects/insert_indexes.rb
+- lib/localtower/generators/service_objects/insert_nullable.rb
 - lib/localtower/status.rb
 - lib/localtower/tools.rb
 - lib/localtower/version.rb
@@ -172,6 +176,10 @@ files:
 - public/screenshots/v0.1.6/3_relations.png
 - public/screenshots/v0.1.6/4_migrations.png
 - public/screenshots/v0.1.6/5_capture.png
+- public/screenshots/v1.0.0/migrations.png
+- public/screenshots/v1.0.0/models.png
+- public/screenshots/v1.0.0/new_migration.png
+- public/screenshots/v1.0.0/new_model.png
 - public/vendor/font-awesome.min.css
 - public/vendor/highlight-js-default.min.css
 - public/vendor/highlight.min.js
@@ -192,6 +200,7 @@ files:
 - spec/dummy/app/jobs/application_job.rb
 - spec/dummy/app/mailers/application_mailer.rb
 - spec/dummy/app/models/application_record.rb
+- spec/dummy/app/models/post.rb
 - spec/dummy/app/models/user.rb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/views/layouts/mailer.html.erb
@@ -227,15 +236,15 @@ files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config/secrets.yml
 - spec/dummy/config/spring.rb
-- spec/dummy/db/migrate/20221115190039_create_users.rb
-- spec/dummy/db/migrate/20221115193020_change_users.rb
-- spec/dummy/db/migrate/20221115193532_change_users_at1668540931.rb
-- spec/dummy/db/migrate/20221115193605_change_users_at1668540964.rb
-- spec/dummy/db/migrate/20221115193637_change_users_at1668540996.rb
-- spec/dummy/db/migrate/20221115193642_change_users_at1668541001.rb
+- spec/dummy/db/migrate/20230119221452_create_users.rb
+- spec/dummy/db/migrate/20230119221751_change_users_at1674166670.rb
+- spec/dummy/db/migrate/20230119222054_create_posts.rb
+- spec/dummy/db/migrate/20230119222106_change_posts_at1674166865.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/log/development.log
 - spec/dummy/log/localtower.log
+- spec/dummy/log/test.log
+- spec/dummy/test/index.html
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/-Y/-YOiiBKqc2UODHFjctm8xc7xFoZaL7zOjWQj6qQ2wyE.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/-d/-dwueM4vmPt8L51S3jeSyg_AjGDcj0GUN6pDpCA1gCg.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/0g/0gaJnJQdtd2ACbihXxn8OnjLWlDjnQ_WxfgOpbiLzhg.cache
@@ -303,13 +312,16 @@ files:
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/yH/yHZ-a1J23ZCf2n5mEHINMz23Iec2cLGTKauW7k4yGTE.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/z7/z7KfUWYoSaGq8tQ5mbHwyfXyT3Pa4FWKJswHcIdJuds.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/zu/zuExWc5WHxeOUPZUKHl9a9ZRmN50g7jMOD28macn6M0.cache
+- spec/dummy/tmp/pids/server.pid
 - spec/dummy/tmp/restart.txt
 - spec/factories/migration.rb
 - spec/factories/model.rb
 - spec/lib/localtower/generators/migration_spec.rb
 - spec/lib/localtower/generators/model_spec.rb
-- spec/lib/localtower/generators/relation_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_array_spec.rb
 - spec/lib/localtower/generators/service_objects/insert_defaults_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_indexes_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_nullable_spec.rb
 - spec/lib/localtower/tools_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/damln/localtower
@@ -338,6 +350,7 @@ test_files:
 - spec/spec_helper.rb
 - spec/dummy/app/mailers/application_mailer.rb
 - spec/dummy/app/models/application_record.rb
+- spec/dummy/app/models/post.rb
 - spec/dummy/app/models/user.rb
 - spec/dummy/app/jobs/application_job.rb
 - spec/dummy/app/controllers/application_controller.rb
@@ -353,6 +366,7 @@ test_files:
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/channels/application_cable/connection.rb
 - spec/dummy/app/channels/application_cable/channel.rb
+- spec/dummy/test/index.html
 - spec/dummy/bin/update
 - spec/dummy/bin/rake
 - spec/dummy/bin/setup
@@ -386,16 +400,16 @@ test_files:
 - spec/dummy/README.md
 - spec/dummy/Rakefile
 - spec/dummy/db/schema.rb
-- spec/dummy/db/migrate/20221115193637_change_users_at1668540996.rb
-- spec/dummy/db/migrate/20221115193605_change_users_at1668540964.rb
-- spec/dummy/db/migrate/20221115193532_change_users_at1668540931.rb
-- spec/dummy/db/migrate/20221115193020_change_users.rb
-- spec/dummy/db/migrate/20221115193642_change_users_at1668541001.rb
-- spec/dummy/db/migrate/20221115190039_create_users.rb
+- spec/dummy/db/migrate/20230119221751_change_users_at1674166670.rb
+- spec/dummy/db/migrate/20230119221452_create_users.rb
+- spec/dummy/db/migrate/20230119222054_create_posts.rb
+- spec/dummy/db/migrate/20230119222106_change_posts_at1674166865.rb
 - spec/dummy/Gemfile
 - spec/dummy/log/localtower.log
+- spec/dummy/log/test.log
 - spec/dummy/log/development.log
 - spec/dummy/Gemfile.lock
+- spec/dummy/tmp/pids/server.pid
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/3W/3WBmqd-2V6q5N-jvbyp-tlcfn3aHYMwBppbOUm7x4qg.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/pv/pv6tV3CDkIAmLXdU8EPLlNEkXAKJPufVD4VP30o4fWo.cache
 - spec/dummy/tmp/cache/assets/sprockets/v4.0.0/sY/sYvTb1uPWWmnHJNXKuW_xSco-aUb-rN2f0J35zSIzuc.cache
@@ -469,5 +483,7 @@ test_files:
 - spec/lib/localtower/tools_spec.rb
 - spec/lib/localtower/generators/migration_spec.rb
 - spec/lib/localtower/generators/service_objects/insert_defaults_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_array_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_nullable_spec.rb
+- spec/lib/localtower/generators/service_objects/insert_indexes_spec.rb
 - spec/lib/localtower/generators/model_spec.rb
-- spec/lib/localtower/generators/relation_spec.rb

data/app/views/localtower/pages/_migrations.html.erb

@@ -1,57 +0,0 @@
-<div class="card">
-  <div class="header">
-    <div class="container-fluid">
-      <div class="row">
-        <div class="col-md-6"><h4 class="title">Pending migrations</h4></div>
-        <div class="col-md-6">
-          <div class="pull-right">
-          <% if migrations.any? %>
-          <%= form_tag actions_path, class: "form" do %>
-            <button name="cmd" type="submit" value="rails db:migrate" class="btn btn-warning" data-selector="submit">
-              Run Pending Migrations (<code>bundle exec rails db:migrate</code>)
-            </button>
-          <% end %>
-          <% end %>
-          </div>
-        </div>
-      </div>
-    </div>
-  </div>
-
-  <div class="content">
-  <% if migrations.any? %>
-    <% migrations.each do |entry| %>
-    <div class="container-fluid">
-      <div class="row">
-        <div class="col-md-8">
-            <h5 style="color: <%= entry['status'] == :todo ? 'red' : 'green' %>;">
-              <% if entry['status'] == :todo %>
-                <button type="button" class="btn btn-danger btn-xs">PENDING</button>
-              <% end %>
-
-              <a href="<%= file_link(entry['file_full_path']) %>"><%= entry['name'] %></a>
-            </h5>
-        </div>
-      </div>
-
-      <div class="row">
-        <div class="col-md-12">
-        <% if entry['status'] == :todo %>
-          <pre><code class="language-ruby"><%= entry['content'] %></code></pre>
-        <% end %>
-        </div>
-      </div>
-    </div>
-    <% end %>
-
-  <% else %>
-    <div class="container-fluid">
-      <div class="row">
-        <div class="col-md-8">
-          No pending migrations.
-        </div>
-      </div>
-    </div>
-  <% end %>
-  </div>
-</div>

data/app/views/localtower/pages/schema.html.erb

@@ -1,67 +0,0 @@
-<% content_for :title do %>Schema<% end %>
-
-<div class="row">
-  <div class="col-md-12">
-    <div class="content">
-      <div class="">
-
-        <div class="grid">
-          <div class="grid-sizer"></div>
-
-          <% ::Localtower::Tools.models_presented.each do |model| %>
-
-          <div class="grid-item text-small card">
-            <div class="text-center">
-              <h5><%= model[:name] %></h5>
-            </div>
-
-            <table class="table table-hover table-striped table-condensed">
-              <thead>
-                <th>Field</th>
-                <th>Type</th>
-                <th>Belongs To</th>
-                <th>Index?</th>
-              </thead>
-
-              <tbody>
-                <% model[:attributes_list].each do |attribute_data| %>
-
-                <tr>
-                  <td>
-                    <code>
-                    <%= attribute_data["name"] %>
-                    </code>
-                  </td>
-
-                  <td>
-                    <%= attribute_data["type_clean"] %>
-                  </td>
-
-                  <td>
-                    <% if attribute_data["belongs_to"] %>
-                    <span class="label label-primary">
-                      <%= attribute_data["belongs_to"] %>
-                    </span>
-                    <% end %>
-                  </td>
-
-                  <td>
-                    <% if attribute_data["index"].any? %>
-                    <span class="label label-success">Yes</span>
-                    <% else %>
-                    <span class="label label-default">No</span>
-                    <% end %>
-                  </td>
-                </tr>
-                <% end %>
-              </tbody>
-            </table>
-          </div>
-          <% end %>
-          <div class="clearfix"></div>
-        </div>
-
-      </div>
-    </div>
-  </div>
-</div>

data/spec/dummy/db/migrate/20221115190039_create_users.rb

@@ -1,13 +0,0 @@
-class CreateUsers < ActiveRecord::Migration[7.0]
-  def change
-    create_table :users do |t|
-      t.string :name
-      t.string :email
-      t.integer :age
-
-      t.timestamps
-    end
-    add_index :users, :name
-    add_index :users, :email
-  end
-end

data/spec/dummy/db/migrate/20221115193020_change_users.rb

@@ -1,8 +0,0 @@
-class ChangeUsers < ActiveRecord::Migration[7.0]
-  def change
-    add_index :users, :age
-    remove_index :users, :name
-    change_column :users, :name, :text
-    rename_column :users, :email, :reference
-  end
-end

data/spec/dummy/db/migrate/20221115193532_change_users_at1668540931.rb

@@ -1,5 +0,0 @@
-class ChangeUsersAt1668540931 < ActiveRecord::Migration[7.0]
-  def change
-    add_index :users, :age
-  end
-end

data/spec/dummy/db/migrate/20221115193605_change_users_at1668540964.rb

@@ -1,5 +0,0 @@
-class ChangeUsersAt1668540964 < ActiveRecord::Migration[7.0]
-  def change
-    remove_index :users, :name
-  end
-end

data/spec/dummy/db/migrate/20221115193637_change_users_at1668540996.rb

@@ -1,5 +0,0 @@
-class ChangeUsersAt1668540996 < ActiveRecord::Migration[7.0]
-  def change
-    add_column :users, :family, :string
-  end
-end

data/spec/dummy/db/migrate/20221115193642_change_users_at1668541001.rb

@@ -1,5 +0,0 @@
-class ChangeUsersAt1668541001 < ActiveRecord::Migration[7.0]
-  def change
-    add_column :users, :lol, :string
-  end
-end

data/spec/lib/localtower/generators/relation_spec.rb

@@ -1,65 +0,0 @@
-require 'spec_helper'
-
-module Localtower
-  module Generators
-    describe Relation do
-      before(:all) do
-        clean_files
-      end
-
-      after(:all) do
-      end
-
-      it 'belongs_to' do
-        data1 = attributes_for(:post_one)
-        data1["run_migrate"] = true
-        ::Localtower::Generators::Model.new(data1).run
-
-        data2 = attributes_for(:user_one)
-        data2["run_migrate"] = true
-        ::Localtower::Generators::Model.new(data2).run
-
-        data = attributes_for(:belongs_to)
-        data["run_migrate"] = true
-
-        generator = ::Localtower::Generators::Migration.new(data).run
-
-        expect(::Localtower::Tools.word_in_file?(last_migration, /add_reference :posts, :user, foreign_key: true, index: true/)).to eq(true)
-        expect(File.exist?("#{Rails.root}/app/models/post.rb")).to eq(true)
-        expect(File.exist?("#{Rails.root}/app/models/user.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}/app/models/post.rb", /belongs_to/)).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}/app/models/user.rb", /has_many/)).to eq(true)
-
-        Dir["#{Rails.root}/app/models/**/*.*"].each { |model_file| File.delete(model_file) }
-      end
-
-      it 'create a relation' do
-        clean_files
-
-        ::Localtower::Generators::Model.new(attributes_for(:post_one)).run
-        ::Localtower::Generators::Model.new(attributes_for(:user_one)).run
-
-        ::Localtower::Generators::Relation.new(attributes_for(:relation_one)).run
-
-        expect(File.exist?("#{Rails.root}/app/models/user_post.rb")).to eq(true)
-
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/user_post.rb", /class UserPost/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /create_table "user_posts"/)).to eq(true)
-
-        user_id = ::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.integer(.*)"user_id"/) || ::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.bigint(.*)"user_id"/)
-        post_id = ::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.integer(.*)"post_id"/) || ::Localtower::Tools.word_in_file?("#{Rails.root}/db/schema.rb", /t.bigint(.*)"post_id"/)
-
-        expect(user_id).to eq(true)
-        expect(post_id).to eq(true)
-
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/user.rb", /has_many :posts, through: :user_posts/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/user.rb", /has_many :posts/)).to eq(true)
-
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/post.rb", /has_many :users/)).to eq(true)
-        expect(::Localtower::Tools.word_in_file?("#{Rails.root}/app/models/post.rb", /has_many :users, through: :user_posts/)).to eq(true)
-      end
-    end
-  end
-end