Merge branch 'zoom-article-image' into 'stable'

Shows a zoom link on article images by default https://noosfero.org/Development/ActionItem3086

Merge branch 'zoom-article-image' into 'stable'
Shows a zoom link on article images by default https://noosfero.org/Development/ActionItem3086
Rodrigo Souto
2 parents 8decbb48 7b1826d2
Showing 3 changed files with 32 additions and 11 deletions Show diff stats
app/models/environment.rb
db/migrate/20140408172149_enable_show_zoom_button_on_article_images.rb
test/unit/environment_test.rb
@@ -337,19 +337,22 @@ class Environment &lt; ActiveRecord::Base
     features.delete_if{ |k, v| !self.enabled?(k) }
   end
+  DEFAULT_FEATURES = %w(
+    disable_asset_products
+    disable_gender_icon
+    products_for_enterprises
+    disable_select_city_for_contact
+    enterprise_registration
+    media_panel
+    organizations_are_moderated_by_default
+    show_balloon_with_profile_links_when_clicked
+    show_zoom_button_on_article_images
+    use_portal_community
+  )
+
   before_create :enable_default_features
   def enable_default_features
-    %w(
-      disable_asset_products
-      disable_gender_icon
-      products_for_enterprises
-      disable_select_city_for_contact
-      enterprise_registration
-      media_panel
-      organizations_are_moderated_by_default
-      show_balloon_with_profile_links_when_clicked
-      use_portal_community
-    ).each do |feature|
+    DEFAULT_FEATURES.each do |feature|
       enable(feature, false)
     end
   end
@@ -0,0 +1,11 @@
+class EnableShowZoomButtonOnArticleImages < ActiveRecord::Migration
+  def self.up
+    Environment.find_each do |environment|
+      environment.enable(:show_zoom_button_on_article_images) 
+    end
+  end
+
+  def self.down
+    say("This migration is irreversible.")
+  end
+end
@@ -53,6 +53,13 @@ class EnvironmentTest &lt; ActiveSupport::TestCase
     assert !v.enabled?('feature1') && !v.enabled?('feature2') && !v.enabled?('feature3')
   end
+  def test_default_enabled_features_are_enabled
+    environment = Environment.create(:name => 'Testing')
+    Environment::DEFAULT_FEATURES.each do |features|
+      assert environment.enabled?(features)
+    end
+  end
+
   def test_name_is_mandatory
     v = Environment.new
     v.valid?
@@ -0,0 +1,11 @@		@@ -0,0 +1,11 @@
	1	+class EnableShowZoomButtonOnArticleImages < ActiveRecord::Migration
	2	+ def self.up
	3	+ Environment.find_each do \|environment\|
	4	+ environment.enable(:show_zoom_button_on_article_images)
	5	+ end
	6	+ end
	7	+
	8	+ def self.down
	9	+ say("This migration is irreversible.")
	10	+ end
	11	+end
	@@ -53,6 +53,13 @@ class EnvironmentTest < ActiveSupport::TestCase		@@ -53,6 +53,13 @@ class EnvironmentTest < ActiveSupport::TestCase
53	assert !v.enabled?('feature1') && !v.enabled?('feature2') && !v.enabled?('feature3')	53	assert !v.enabled?('feature1') && !v.enabled?('feature2') && !v.enabled?('feature3')
54	end	54	end
55		55
		56	+ def test_default_enabled_features_are_enabled
		57	+ environment = Environment.create(:name => 'Testing')
		58	+ Environment::DEFAULT_FEATURES.each do \|features\|
		59	+ assert environment.enabled?(features)
		60	+ end
		61	+ end
		62	+
56	def test_name_is_mandatory	63	def test_name_is_mandatory
57	v = Environment.new	64	v = Environment.new
58	v.valid?	65	v.valid?