1 files changed, 29 insertions, 26 deletions
diff --git a/test/unit/lib/redmine/plugin_test.rb b/test/unit/lib/redmine/plugin_test.rb
index efb28ffcd..29facdd3a 100644
--- a/test/unit/lib/redmine/plugin_test.rb
+++ b/test/unit/lib/redmine/plugin_test.rb
@@ -22,6 +22,9 @@ require File.expand_path('../../../../test_helper', __FILE__)
 class Redmine::PluginTest < ActiveSupport::TestCase
   def setup
     @klass = Redmine::Plugin
+    # 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
   end
@@ -31,7 +34,7 @@ class Redmine::PluginTest < ActiveSupport::TestCase
   end
 
   def test_register
-    @klass.register :foo do
+    @klass.register :foo_plugin do
       name 'Foo plugin'
       url 'http://example.net/plugins/foo'
       author 'John Smith'
@@ -43,9 +46,9 @@ class Redmine::PluginTest < ActiveSupport::TestCase
 
     assert_equal 1, @klass.all.size
 
-    plugin = @klass.find('foo')
+    plugin = @klass.find('foo_plugin')
     assert plugin.is_a?(Redmine::Plugin)
-    assert_equal :foo, plugin.id
+    assert_equal :foo_plugin, plugin.id
     assert_equal 'Foo plugin', plugin.name
     assert_equal 'http://example.net/plugins/foo', plugin.url
     assert_equal 'John Smith', plugin.author
@@ -55,14 +58,14 @@ class Redmine::PluginTest < ActiveSupport::TestCase
   end
 
   def test_installed
-    @klass.register(:foo) {}
-    assert_equal true, @klass.installed?(:foo)
+    @klass.register(:foo_plugin) {}
+    assert_equal true, @klass.installed?(:foo_plugin)
     assert_equal false, @klass.installed?(:bar)
   end
 
   def test_menu
     assert_difference 'Redmine::MenuManager.items(:project_menu).size' do
-      @klass.register :foo do
+      @klass.register :foo_plugin do
         menu :project_menu, :foo_menu_item, '/foo', :caption => 'Foo'
       end
     end
@@ -77,7 +80,7 @@ class Redmine::PluginTest < ActiveSupport::TestCase
   def test_delete_menu_item
     Redmine::MenuManager.map(:project_menu).push(:foo_menu_item, '/foo', :caption => 'Foo')
     assert_difference 'Redmine::MenuManager.items(:project_menu).size', -1 do
-      @klass.register :foo do
+      @klass.register :foo_plugin do
         delete_menu_item :project_menu, :foo_menu_item
       end
     end
@@ -88,18 +91,18 @@ class Redmine::PluginTest < ActiveSupport::TestCase
 
   def test_directory_with_override
     @klass.register(:foo) do
-      directory '/path/to/foo'
+      directory 'test/fixtures/plugins/foo_plugin'
     end
-    assert_equal '/path/to/foo', @klass.find('foo').directory
+    assert_equal 'test/fixtures/plugins/foo_plugin', @klass.find('foo').directory
   end
 
   def test_directory_without_override
-    @klass.register(:foo) {}
-    assert_equal File.join(@klass.directory, 'foo'), @klass.find('foo').directory
+    @klass.register(:other_plugin) {}
+    assert_equal File.join(@klass.directory, 'other_plugin'), @klass.find('other_plugin').directory
   end
 
   def test_requires_redmine
-    plugin = Redmine::Plugin.register(:foo) {}
+    plugin = Redmine::Plugin.register(:foo_plugin) {}
     Redmine::VERSION.stubs(:to_a).returns([2, 1, 3, "stable", 10817])
     # Specific version without hash
     assert plugin.requires_redmine('2.1.3')
@@ -148,24 +151,24 @@ class Redmine::PluginTest < ActiveSupport::TestCase
   def test_requires_redmine_plugin
     test = self
     other_version = '0.5.0'
-    @klass.register :other do
+    @klass.register :other_plugin do
       name 'Other'
       version other_version
     end
-    @klass.register :foo do
-      test.assert requires_redmine_plugin(:other, :version_or_higher => '0.1.0')
-      test.assert requires_redmine_plugin(:other, :version_or_higher => other_version)
-      test.assert requires_redmine_plugin(:other, other_version)
+    @klass.register :foo_plugin do
+      test.assert requires_redmine_plugin(:other_plugin, :version_or_higher => '0.1.0')
+      test.assert requires_redmine_plugin(:other_plugin, :version_or_higher => other_version)
+      test.assert requires_redmine_plugin(:other_plugin, other_version)
       test.assert_raise Redmine::PluginRequirementError do
-        requires_redmine_plugin(:other, :version_or_higher => '99.0.0')
+        requires_redmine_plugin(:other_plugin, :version_or_higher => '99.0.0')
       end
-      test.assert requires_redmine_plugin(:other, :version => other_version)
-      test.assert requires_redmine_plugin(:other, :version => [other_version, '99.0.0'])
+      test.assert requires_redmine_plugin(:other_plugin, :version => other_version)
+      test.assert requires_redmine_plugin(:other_plugin, :version => [other_version, '99.0.0'])
       test.assert_raise Redmine::PluginRequirementError do
-        requires_redmine_plugin(:other, :version => '99.0.0')
+        requires_redmine_plugin(:other_plugin, :version => '99.0.0')
       end
       test.assert_raise Redmine::PluginRequirementError do
-        requires_redmine_plugin(:other, :version => ['98.0.0', '99.0.0'])
+        requires_redmine_plugin(:other_plugin, :version => ['98.0.0', '99.0.0'])
       end
       # Missing plugin
       test.assert_raise Redmine::PluginRequirementError do
@@ -181,17 +184,17 @@ class Redmine::PluginTest < ActiveSupport::TestCase
   end
 
   def test_settings_warns_about_possible_partial_collision
-    @klass.register(:foo) { settings :partial => 'foo/settings' }
+    @klass.register(:foo_plugin) { settings :partial => 'foo/settings' }
     Rails.logger.expects(:warn)
-    @klass.register(:bar) { settings :partial => 'foo/settings' }
+    @klass.register(:other_plugin) { settings :partial => 'foo/settings' }
   end
 
   def test_migrate_redmine_plugin
-    @klass.register :foo do
+    @klass.register :foo_plugin do
       name 'Foo plugin'
       version '0.0.1'
     end
 
-    assert Redmine::Plugin.migrate('foo')
+    assert Redmine::Plugin.migrate('foo_plugin')
   end
 end