adminpanel 2.1.0 → 2.1.1

data/config/routes.rb

@@ -34,11 +34,16 @@ Adminpanel::Engine.routes.draw do
       end
 
       # normal resource
+      resources resource, resources_parameters(resource).merge(rest_path_names)
+
+      # if resource is going to be shared on facebook
       resources resource, resources_parameters(resource).merge(rest_path_names) do
         member do
-          get :fb_choose_page, as: 'fb_choose_page', path: 'publicar-a-pagina-en-fb'
-          post :fb_save_token, as: 'fb_save_token', path: 'guardar-token'
-          post 'fb_publish/:configuration_id', to: "#{resource}#fb_publish", as: 'fb_publish', path: 'publicar-a-facebook'
+          if has_fb_share?(resource)
+            get :fb_choose_page, as: 'fb_choose_page', path: 'publicar-a-pagina-en-fb'
+            post :fb_save_token, as: 'fb_save_token', path: 'guardar-token'
+            post 'fb_publish/:configuration_id', to: "#{resource}#fb_publish", as: 'fb_publish', path: 'publicar-a-facebook'
+          end
         end
       end
     end

data/lib/adminpanel/version.rb

@@ -1,3 +1,3 @@
 module Adminpanel
-  VERSION = "2.1.0"
+  VERSION = "2.1.1"
 end

data/lib/generators/adminpanel/gallery/templates/gallery_migration.rb

@@ -1,4 +1,4 @@
-class CreateAdminpanel<%= class_name.pluralize %>Table < ActiveRecord::Migration
+class CreateAdminpanel<%= class_name.pluralize %> < ActiveRecord::Migration
 	def change
 	  create_table :adminpanel_<%= lower_name.pluralize %> do |t|
 

data/lib/generators/adminpanel/resource/resource_generator.rb

@@ -41,13 +41,13 @@ module Adminpanel
 			end
 
 			def generate_gallery
-				if has_gallery?
+				if has_gallery? && is_a_resource?
 					invoke 'adminpanel:gallery', [lower_singularized_name]
 				end
 			end
 
 			def add_resource_to_config
-				if setup_is_found?
+				if setup_is_found? && is_a_resource?
 					inject_into_file 'config/initializers/adminpanel_setup.rb',
 						after: 'config.displayable_resources = [' do
 						indent "\n:#{pluralized_name},", 4

data/{spec → test}/dummy/.gitignore

@@ -13,7 +13,6 @@ capybara-*.html
 rerun.txt
 pickle-email-*.html
 config/initializers/secret_token.rb
-config/secrets.yml
 /public/uploads/
 
 ## Environment normalisation:

data/{spec/dummy/config/carrierwve.rb → test/dummy/config/carrierwave.rb}

@@ -3,4 +3,4 @@ if Rails.env.test? or Rails.env.cucumber?
     config.storage = :file
     config.enable_processing = false
   end
-end
+end

data/{spec → test}/dummy/config/environments/test.rb

@@ -23,8 +23,8 @@ Dummy::Application.configure do
   # Raise exceptions instead of rendering exception templates.
   config.action_dispatch.show_exceptions = false
 
-  # Disable request forgery protection in test environment.
-  config.action_controller.allow_forgery_protection = false
+  # Enable request forgery protection in test environment.
+  config.action_controller.allow_forgery_protection = true
 
   # Tell Action Mailer not to deliver emails to the real world.
   # The :test delivery method accumulates sent emails in the

data/test/dummy/config/secrets.yml

@@ -0,0 +1,2 @@
+test:
+  secret_key_base: e152a9bf618bb8871e25e2f6c4472d9a1e5c93a221cafa4ee560fe6f58022c0f6f7554379519387568c456d71c519afe7e9c643c09b011c01d4b9e531a41f99d

data/test/dummy/test/fixtures/adminpanel/galleries.yml

@@ -0,0 +1,15 @@
+one:
+  file: hipster.jpg
+  position: 1
+  created_at: <%= Date.today %>
+  updated_at: <%= Date.today %>
+two:
+  file: hipster.jpg
+  position: 2
+  created_at: <%= Date.today %>
+  updated_at: <%= Date.today %>
+three:
+  file: hipster.jpg
+  position: 3
+  created_at: <%= Date.today %>
+  updated_at: <%= Date.today %>

data/test/dummy/test/fixtures/adminpanel/groups.yml

@@ -0,0 +1,4 @@
+Admin:
+  name: Admin
+  created_at: <%= Date.today %>
+  updated_at: <%= Date.today %>

data/test/dummy/test/fixtures/adminpanel/users.yml

@@ -0,0 +1,8 @@
+valid:
+  name: Example User
+  email: user@example.com
+  password_digest: <%= BCrypt::Password.create('foobar') %>
+  remember_token: <%= Digest::SHA1.hexdigest("adminSat Feb 25 17:14:00 GMT 20060.236961325863376") %>
+  group_id: <%= ActiveRecord::FixtureSet.identify(:Admin) %>
+  created_at: <%= Date.today %>
+  updated_at: <%= Date.today %>

data/test/features/session/login_test.rb

@@ -0,0 +1,33 @@
+require 'test_helper'
+
+class LoginTest < ViewCase
+  fixtures :all
+
+  setup :visiting_signin
+
+  def test_login_messages
+    assert_content I18n.t('authentication.welcome')
+  end
+
+  def test_logging_in_with_valid_information
+    login
+    assert_content I18n.t('authentication.signin_success')
+    assert_content I18n.t('authentication.logout')
+    signout
+    assert_content I18n.t('authentication.welcome')
+  end
+
+  def test_logging_in_with_invalid_information
+    login('foobaz') #pass is foobar
+    assert_content I18n.t('authentication.signin_error')
+  end
+
+  protected
+  def visiting_signin
+    visit adminpanel.signin_path
+  end
+
+  def signout
+    click_link 'signout-button'
+  end
+end

data/test/features/shared/new_test.rb

@@ -0,0 +1,39 @@
+require 'test_helper'
+
+class NewTest < ViewCase
+
+  setup :visit_new_product
+  # teardown :teardown
+
+  def test_shared_new_page_messages
+    assert_button("#{I18n.t('action.add')} #{Adminpanel::Product.display_name}")
+  end
+
+  def test_submitting_with_invalid_information
+    click_button('Agregar Producto')
+    assert_content('Producto no pudo guardarse debido a 3 errores')
+  end
+
+  def test_submitting_with_valid_information
+    fill_in 'product_name', :with => 'product name'
+    fill_in 'product_price', :with => '855.5'
+    page.execute_script(
+      %Q(
+        $('#product_description').data('wysihtml5').editor.setValue('que pepsi');
+      )
+    ) # to fill the wysiwyg editor
+    click_button('Agregar Producto')
+    assert_content(I18n.t('action.save_success'))
+    saved_product = Adminpanel::Product.last
+    assert_equal 'product name', saved_product.name
+    assert_equal '855.5', saved_product.price
+    assert_equal 'que pepsi', saved_product.description
+  end
+
+  protected
+  def visit_new_product
+    visit adminpanel.signin_path
+    login
+    visit adminpanel.new_product_path
+  end
+end

data/test/models/adminpanel/gallery_test.rb

@@ -0,0 +1,37 @@
+require 'test_helper'
+module Adminpanel
+  class GalleryTest < ActiveSupport::TestCase
+
+    # set_fixture_class 'adminpanel/galleries' => Adminpanel::Gallery
+
+    def test_moving_up_with_the_best_position
+      gallery_1 = adminpanel_galleries(:one)
+      gallery_1.move_to_better_position
+      assert_equal 1, gallery_1.position
+    end
+
+    def test_moving_down_with_the_worst_position
+      gallery_3 = adminpanel_galleries(:three)
+      gallery_3.move_to_worst_position
+      assert_equal 3, gallery_3.position
+    end
+
+    def test_moving_up_with_intermediate_position
+      gallery_2 = adminpanel_galleries(:two)
+      gallery_2.move_to_better_position
+      assert_equal 1, gallery_2.position
+    end
+
+    def test_moving_down_with_intermediate_position
+      gallery_2 = adminpanel_galleries(:two)
+      gallery_2.move_to_worst_position
+      assert_equal 3, gallery_2.position
+    end
+
+    def test_file_presence
+      g = Adminpanel::Gallery.new
+      assert_not g.save
+    end
+
+  end
+end

data/test/models/adminpanel/section_test.rb

@@ -0,0 +1,53 @@
+require 'test_helper'
+
+module Adminpanel
+  class SectionTest < ActiveSupport::TestCase
+
+    def test_telephone_validations
+      set_telephone
+      assert @telephone.valid?
+
+      @telephone.update_attribute(:description, '1' * 9)
+      assert @telephone.invalid?
+
+      @telephone.update_attribute(:description, '1' * 11)
+      assert @telephone.invalid?
+
+      @telephone.update_attribute(:description, '01-2-3-4-5')
+      assert @telephone.invalid?
+
+      @telephone.update_attribute(:description, '')
+      assert @telephone.valid?
+    end
+
+    def test_default_scope
+      assert_equal(
+        Adminpanel::Section.all.to_sql,
+        Adminpanel::Section.reorder('').order('page ASC').to_sql
+      )
+    end
+
+  protected
+    def set_telephone
+      @telephone = Adminpanel::Section.new(
+        name: 'Telefono',
+        key: 'phone',
+        has_description: false,
+        description: '1234567890',
+        page: 'home',
+        has_image: false
+      )
+    end
+
+    def set_description
+      @description = Adminpanel::Section.new(
+        name: 'Inicio',
+        key: 'greeting',
+        has_description: true,
+        description: 'lorem ipsum dolor sit amec',
+        page: 'home',
+        has_image: false
+      )
+    end
+  end
+end

data/test/models/adminpanel/user_test.rb

@@ -0,0 +1,51 @@
+require 'test_helper'
+
+module Adminpanel
+  class UserTest < ActiveSupport::TestCase
+    setup :assign_user
+
+    def test_is_valid_user
+      assert @test_user.save
+    end
+
+    def test_name_validation
+      @test_user.name = ""
+      assert_not @test_user.valid?
+    end
+
+    def test_email_validation
+      @test_user.email = "foo@"
+      assert_not @test_user.valid?
+
+      @test_user.email = "foo@bar"
+      assert_not @test_user.valid?
+
+      @test_user.email = "@bar.baz"
+      assert_not @test_user.valid?
+
+      @test_user.email = "foo.baz"
+      assert_not @test_user.valid?
+    end
+
+    def test_password_validation
+      @test_user.password_confirmation = "foobaz"
+      assert_not @test_user.valid?
+
+      @test_user.password_confirmation = "foo"
+      @test_user.password = "foo"
+      assert_not @test_user.valid?
+    end
+
+  protected
+    def assign_user
+      @test_user = Adminpanel::User.new(
+        name: "Example User",
+        email: "foo@bar.com",
+        password: "foobar",
+        password_confirmation: "foobar",
+        group_id: adminpanel_groups(:Admin).id
+      )
+    end
+
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,71 @@
+ENV['RAILS_ENV'] ||= 'test'
+require 'rails'
+require File.expand_path('../dummy/config/environment', __FILE__)
+require 'rails/test_help'
+require 'minitest/autorun'
+require 'minitest/emoji' #emoji output
+require 'capybara/rails'
+require 'minitest/unit'
+require 'mocha/mini_test'
+require 'capybara/poltergeist'
+# require 'minitest/debugger' if ENV['DEBUG'] # for deubgging
+
+
+Capybara.current_driver = :poltergeist
+
+load Rails.root.join('db', 'schema.rb')
+
+
+
+
+class ActiveRecord::Base
+  mattr_accessor :shared_connection
+  @@shared_connection = nil
+
+  def self.connection
+    @@shared_connection || retrieve_connection
+  end
+end
+
+# Forces all threads to share the same connection. This works on
+# Capybara because it starts the web server in a thread.
+ActiveRecord::Base.shared_connection = ActiveRecord::Base.connection
+
+
+class ActiveSupport::TestCase
+
+  #fixtures live inside the dummy app
+  fixtures :all
+
+  # Add more helper methods to be used by all tests here...
+end
+class ViewCase < ActionView::TestCase
+  include Capybara::DSL
+  include Capybara::Assertions
+  include Rails.application.routes.url_helpers
+
+  def teardown
+    Capybara.reset_session!
+  end
+
+  protected
+  def login(password = 'foobar')
+    fill_in 'inputEmail', with: adminpanel_users(:valid).email
+    fill_in 'inputPassword', with: password #pass is foobar
+    click_button I18n.t('authentication.new-session')
+  end
+end
+
+# class SharedTestCase < ViewCase
+#   # include Capybara::DSL
+#   # include Capybara::Assertions
+#   # include Rails.application.routes.url_helpers
+#   def before_setup
+#     sign_in
+#   end
+#   protected
+#   def sign_in
+#     visit adminpanel.signin_path
+#     login
+#   end
+# end