Reverts r22507, r22508 and r22510 (#36320, #39803).

git-svn-id: https://svn.redmine.org/redmine/trunk@22515 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/config/application.rb b/config/application.rb
index f5062dfe7fa2b8a245b25454b5140e8bb9cf3fc2..48e855c9b1f2064d6b3db155b83ddda149c9aeb2 100644 (file)
--- a/config/application.rb
+++ b/config/application.rb
@@ -85,9 +85,6 @@ module RedmineApp
     # for more options (same options as config.cache_store).
     config.redmine_search_cache_store = :memory_store
 
-    # Sets default plugin directory
-    config.redmine_plugin_directory = 'plugins'
-
     # Configure log level here so that additional environment file
     # can change it (environments/ENV.rb would take precedence over it)
     config.log_level = Rails.env.production? ? :info : :debug

diff --git a/config/environments/test.rb b/config/environments/test.rb
index e67cea76c3486092a7bb38451da38932fec27218..199f1bb2736c04004d7a6056a4f867ac7596bea5 100644 (file)
--- a/config/environments/test.rb
+++ b/config/environments/test.rb
@@ -18,9 +18,6 @@ Rails.application.configure do
   # preloads Rails for running tests, you may have to set it to true.
   config.eager_load = false
 
-  # Change the plugin directory when testing to avoid clashes with real plugins.
-  config.redmine_plugin_directory = 'test/fixtures/plugins'
-
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
   config.public_file_server.headers = {

diff --git a/lib/redmine/plugin_loader.rb b/lib/redmine/plugin_loader.rb
index 04d7e4af0dce5cc9177aa71d855ec613caf7475f..135df09ba816ccc5ad2700d3e95671e2666352d0 100644 (file)
--- a/lib/redmine/plugin_loader.rb
+++ b/lib/redmine/plugin_loader.rb
@@ -84,7 +84,7 @@ module Redmine
   class PluginLoader
     # Absolute path to the directory where plugins are located
     cattr_accessor :directory
-    self.directory = Rails.root.join Rails.application.config.redmine_plugin_directory
+    self.directory = Rails.root.join('plugins')
 
     # Absolute path to the public directory where plugins assets are copied
     cattr_accessor :public_directory

diff --git a/test/fixtures/plugins/foo_plugin/app/models/foo.rb b/test/fixtures/plugins/foo_plugin/app/models/foo.rb
deleted file mode 100644 (file)
index 0152b6e..0000000
--- a/test/fixtures/plugins/foo_plugin/app/models/foo.rb
+++ /dev/null
@@ -1,3 +0,0 @@
-# frozen_string_literal: true
-class Foo < ActiveRecord::Base
-end

diff --git a/test/unit/lib/redmine/plugin_loader_test.rb b/test/unit/lib/redmine/plugin_loader_test.rb
index a0bec218e7bc1154290f4a449aad1dee4b84fdf7..ed59433468e307a829ac4cde6066f39fd3936b88 100644 (file)
--- a/test/unit/lib/redmine/plugin_loader_test.rb
+++ b/test/unit/lib/redmine/plugin_loader_test.rb
@@ -23,9 +23,8 @@ class Redmine::PluginLoaderTest < ActiveSupport::TestCase
   def setup
     clear_public
 
-    # Change plugin directory for testing to default in config/environments/tesr.rb.
-    # plugins/foo => test/fixtures/plugins/foo
     @klass = Redmine::PluginLoader
+    @klass.directory = Rails.root.join('test/fixtures/plugins')
     @klass.public_directory = Rails.root.join('tmp/public/plugin_assets')
     @klass.load
   end
@@ -56,10 +55,6 @@ class Redmine::PluginLoaderTest < ActiveSupport::TestCase
     assert File.exist?("#{@klass.public_directory}/foo_plugin/stylesheets/foo.css")
   end
 
-  def test_autoload
-    assert_equal true, Object.const_defined?(:Foo)
-  end
-
   def clear_public
     FileUtils.rm_rf 'tmp/public'
   end

diff --git a/test/unit/lib/redmine/plugin_test.rb b/test/unit/lib/redmine/plugin_test.rb
index d2766824795154e14ca50116bc6bb453a1189af0..02df2323684800818514934af5e649942bf37889 100644 (file)
--- a/test/unit/lib/redmine/plugin_test.rb
+++ b/test/unit/lib/redmine/plugin_test.rb
@@ -22,11 +22,14 @@ require_relative '../../../test_helper'
 class Redmine::PluginTest < ActiveSupport::TestCase
   def setup
     @klass = Redmine::Plugin
-    # Change plugin directory for testing to default in config/environments/test.rb.
+    # Change plugin directory for testing to default
     # plugins/foo => test/fixtures/plugins/foo
+    @klass.directory = Rails.root.join('test/fixtures/plugins')
     # In case some real plugins are installed
     @klass.clear
 
+    # Change plugin loader's directory for testing
+    Redmine::PluginLoader.directory = @klass.directory
     Redmine::PluginLoader.setup
   end