Enable ActiveStorage by default for Solidus' in-memory dummy app

solidusio · Mar 2, 2021 · 46de692 · 46de692
1 parent 3422d20
commit 46de692
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 9 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -98,17 +98,13 @@ jobs:
   postgres:
     executor: postgres
     parallelism: &parallelism 3
-    environment:
-      ENABLE_ACTIVE_STORAGE: true
     steps:
       - setup
       - test
 
   mysql:
     executor: mysql
     parallelism: *parallelism
-    environment:
-      ENABLE_ACTIVE_STORAGE: true
     steps:
       - setup
       - test
@@ -118,6 +114,7 @@ jobs:
     parallelism: *parallelism
     environment:
       RAILS_VERSION: '~> 6.0.0'
+      DISABLE_ACTIVE_STORAGE: true
     steps:
       - setup
       - test
@@ -127,6 +124,7 @@ jobs:
     parallelism: *parallelism
     environment:
       RAILS_VERSION: '~> 5.2.0'
+      DISABLE_ACTIVE_STORAGE: true
     steps:
       - setup
       - test

diff --git a/core/lib/spree/testing_support/dummy_app.rb b/core/lib/spree/testing_support/dummy_app.rb
@@ -68,7 +68,7 @@ class Application < ::Rails::Application
 
     config.storage_path = Rails.root.join('tmp', 'storage')
 
-    if ENV['ENABLE_ACTIVE_STORAGE']
+    unless ENV['DISABLE_ACTIVE_STORAGE']
       initializer 'solidus.active_storage' do
         config.active_storage.service_configurations = {
           test: {
@@ -119,9 +119,9 @@ class Application < ::Rails::Application
 Spree.config do |config|
   config.mails_from = "store@example.com"
 
-  if ENV['ENABLE_ACTIVE_STORAGE']
-    config.image_attachment_module = 'Spree::Image::ActiveStorageAttachment'
-    config.taxon_attachment_module = 'Spree::Taxon::ActiveStorageAttachment'
+  if ENV['DISABLE_ACTIVE_STORAGE']
+    config.image_attachment_module = 'Spree::Image::PaperclipAttachment'
+    config.taxon_attachment_module = 'Spree::Taxon::PaperclipAttachment'
   end
 end
 

diff --git a/core/lib/spree/testing_support/dummy_app/migrations.rb b/core/lib/spree/testing_support/dummy_app/migrations.rb
@@ -27,7 +27,7 @@ def auto_migrate
         ActiveRecord::Base.remove_connection
 
         sh 'rake db:reset VERBOSE=false'
-        if ENV['ENABLE_ACTIVE_STORAGE']
+        unless ENV['DISABLE_ACTIVE_STORAGE']
           sh 'rake active_storage:install db:migrate VERBOSE=false'
         end