From de08be6af68373f6387854960ac967611d69ec9f Mon Sep 17 00:00:00 2001
From: Bouke van der Bijl <boukevanderbijl@gmail.com>
Date: Tue, 1 Nov 2016 14:29:29 -0400
Subject: [PATCH] Don't put db:migrate and db:setup in binfiles if activerecord
 is excluded

---
 .../generators/rails/app/templates/bin/{setup => setup.tt}  | 2 ++
 .../rails/app/templates/bin/{update => update.tt}           | 2 ++
 railties/test/generators/app_generator_test.rb              | 6 ++++++
 3 files changed, 10 insertions(+)
 rename railties/lib/rails/generators/rails/app/templates/bin/{setup => setup.tt} (94%)
 rename railties/lib/rails/generators/rails/app/templates/bin/{update => update.tt} (93%)

diff --git a/railties/lib/rails/generators/rails/app/templates/bin/setup b/railties/lib/rails/generators/rails/app/templates/bin/setup.tt
similarity index 94%
rename from railties/lib/rails/generators/rails/app/templates/bin/setup
rename to railties/lib/rails/generators/rails/app/templates/bin/setup.tt
index acae810c1a..8635e97b76 100644
--- a/railties/lib/rails/generators/rails/app/templates/bin/setup
+++ b/railties/lib/rails/generators/rails/app/templates/bin/setup.tt
@@ -16,6 +16,7 @@ chdir APP_ROOT do
   puts '== Installing dependencies =='
   system! 'gem install bundler --conservative'
   system('bundle check') || system!('bundle install')
+<% unless options.skip_active_record -%>
 
   # puts "\n== Copying sample files =="
   # unless File.exist?('config/database.yml')
@@ -24,6 +25,7 @@ chdir APP_ROOT do
 
   puts "\n== Preparing database =="
   system! 'bin/rails db:setup'
+<% end -%>
 
   puts "\n== Removing old logs and tempfiles =="
   system! 'bin/rails log:clear tmp:clear'
diff --git a/railties/lib/rails/generators/rails/app/templates/bin/update b/railties/lib/rails/generators/rails/app/templates/bin/update.tt
similarity index 93%
rename from railties/lib/rails/generators/rails/app/templates/bin/update
rename to railties/lib/rails/generators/rails/app/templates/bin/update.tt
index 770a605fed..d385b363c6 100644
--- a/railties/lib/rails/generators/rails/app/templates/bin/update
+++ b/railties/lib/rails/generators/rails/app/templates/bin/update.tt
@@ -16,9 +16,11 @@ chdir APP_ROOT do
   puts '== Installing dependencies =='
   system! 'gem install bundler --conservative'
   system('bundle check') || system!('bundle install')
+<% unless options.skip_active_record -%>
 
   puts "\n== Updating database =="
   system! 'bin/rails db:migrate'
+<% end -%>
 
   puts "\n== Removing old logs and tempfiles =="
   system! 'bin/rails log:clear tmp:clear'
diff --git a/railties/test/generators/app_generator_test.rb b/railties/test/generators/app_generator_test.rb
index 9299b9ebea..6c6c5613ab 100644
--- a/railties/test/generators/app_generator_test.rb
+++ b/railties/test/generators/app_generator_test.rb
@@ -363,6 +363,12 @@ def test_generator_if_skip_active_record_is_given
     assert_file "test/test_helper.rb" do |helper_content|
       assert_no_match(/fixtures :all/, helper_content)
     end
+    assert_file "bin/setup" do |setup_content|
+      assert_no_match(/db:setup/, setup_content)
+    end
+    assert_file "bin/update" do |update_content|
+      assert_no_match(/db:migrate/, update_content)
+    end
 
     assert_file "config/initializers/new_framework_defaults.rb" do |initializer_content|
       assert_no_match(/belongs_to_required_by_default/, initializer_content)
-- 
GitLab