localtower 0.5.0 → 2.0.0

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

@@ -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_pending)).to eq(expected_file_content)
+  end
+end

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

@@ -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_pending)).to eq(expected_file_content)
   end
 end

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

@@ -0,0 +1,101 @@
+require 'spec_helper'
+
+describe ::Localtower::Generators::ServiceObjects::InsertForeignKeys do
+  let(:service) { described_class.new(attributes) }
+
+  before do
+    File.open("#{Rails.root}/db/migrate/0_migration_name_.rb", 'w') { |f| f.write(base_file_content) }
+  end
+
+  context "when foreign_key is true" do
+    let(:attributes) {
+      [
+        { 'user'  =>
+          {
+            'foreign_key' => true,
+          }
+        },
+      ]
+    }
+
+    let(:base_file_content) {
+      <<-MIGRATION.strip_heredoc
+      class CreatePosts < ActiveRecord::Migration[7.2]
+        def change
+          create_table :posts do |t|
+            t.references :user, null: false
+
+            t.timestamps
+          end
+        end
+      end
+      MIGRATION
+    }
+
+    let(:expected_file_content) {
+      <<-MIGRATION.strip_heredoc
+      class CreatePosts < ActiveRecord::Migration[7.2]
+        def change
+          create_table :posts do |t|
+            t.references :user, null: false, foreign_key: true
+
+            t.timestamps
+          end
+        end
+      end
+      MIGRATION
+    }
+
+    it 'works' do
+      service.call
+
+      expect(File.read(last_migration_pending)).to eq(expected_file_content)
+    end
+  end
+
+  context "when foreign_key is false" do
+    let(:attributes) {
+      [
+        { 'user'  =>
+          {
+            'foreign_key' => false,
+          }
+        },
+      ]
+    }
+
+    let(:base_file_content) {
+      <<-MIGRATION.strip_heredoc
+      class CreatePosts < ActiveRecord::Migration[7.2]
+        def change
+          create_table :posts do |t|
+            t.references :user, null: false, foreign_key: true
+
+            t.timestamps
+          end
+        end
+      end
+      MIGRATION
+    }
+
+    let(:expected_file_content) {
+      <<-MIGRATION.strip_heredoc
+      class CreatePosts < ActiveRecord::Migration[7.2]
+        def change
+          create_table :posts do |t|
+            t.references :user, null: false
+
+            t.timestamps
+          end
+        end
+      end
+      MIGRATION
+    }
+
+    it 'works' do
+      service.call
+
+      expect(File.read(last_migration_pending)).to eq(expected_file_content)
+    end
+  end
+end

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_pending)).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, default: 'foo', null: false
+            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_pending)).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,17 +16,23 @@ 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
   Dir["#{Rails.root}/db/migrate/*"]
 end
 
-def last_migration
+def last_migration_pending
   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'
 
@@ -43,15 +49,13 @@ RSpec.configure do |config|
   config.mock_with :rspec
   config.use_transactional_fixtures = false
   config.infer_base_class_for_anonymous_controllers = true
-
-  config.order = 123
-  # config.order = 'random'
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
 
   config.before(:suite) do
     FactoryBot.find_definitions
   end
 
-  config.before(:all) do
+  config.before(:each) do
     clean_files
   end