camaleon_cms 2.4.5.5 → 2.4.5.7

data/spec/features/frontend/pages_spec.rb

@@ -1,5 +1,86 @@
-# require 'rails_helper'
-# describe "Post frontend", js: true do
-#   login_success
-#   pages_test
-# end
+require 'rails_helper'
+describe "Post frontend", js: true do
+  before(:each) do
+    @site = create(:site).decorate
+    @post = @site.the_post('sample-post').decorate
+  end
+
+  it 'visit post' do
+    visit @post.the_url(as_path: true)
+    expect(page).to have_text(@post.the_title)
+  end
+  
+  describe 'comments' do
+    
+    describe 'anonymous' do
+      before(:each) do
+        @post.set_meta('has_comments', '1') # enable comments for this post
+        @site.set_option('permit_anonimos_comment', true) # enable anonymous comment
+      end
+      
+      it 'anonymous comment' do
+        visit @post.the_url(as_path: true)
+        expect(page).to have_text('New Comment')
+
+        within("#form-comment") do
+          fill_in 'post_comment_name', :with => 'Owen'
+          fill_in 'post_comment_email', :with => 'owenperedo@gmail.com'
+          fill_in 'post_comment_content', :with => 'Sample comment'
+        end
+        click_button 'Comment'
+        expect(page).to have_text('The comment has been created')
+      end
+
+      it 'anonymous comment valid captcha' do
+        @site.set_option('enable_captcha_for_comments', true) # enable anonymous captcha
+        Capybara.using_session("test session") do
+          visit cama_captcha_path(len: 4, t: Time.current.to_i)
+          captcha = page.get_rack_session['cama_captcha']
+          visit @post.the_url(as_path: true)
+          expect(page).to have_text('New Comment')
+          within("#form-comment") do
+            fill_in 'post_comment_name', :with => 'Owen'
+            fill_in 'post_comment_email', :with => 'owenperedo@gmail.com'
+            fill_in 'post_comment_content', :with => 'Sample comment'
+            fill_in 'captcha', :with => captcha
+          end
+          click_button 'Comment'
+          expect(page).to have_text('The comment has been created')
+        end
+      end
+
+      it 'anonymous comment wrong captcha' do
+        @site.set_option('enable_captcha_for_comments', true) # enable anonymous captcha
+        visit @post.the_url(as_path: true)
+        expect(page).to have_text('New Comment')
+
+        within("#form-comment") do
+          fill_in 'post_comment_name', :with => 'Owen'
+          fill_in 'post_comment_email', :with => 'owenperedo@gmail.com'
+          fill_in 'post_comment_content', :with => 'Sample comment'
+        end
+        click_button 'Comment'
+        expect(page).not_to have_text('The comment has been created')
+      end
+    end
+
+    it 'comment with user session' do
+      @post.set_meta('has_comments', '1') # enable comments for this post
+      admin_sign_in
+      visit @post.the_url(as_path: true)
+      expect(page).to have_text('New Comment')
+      within("#form-comment") do
+        fill_in 'post_comment_content', :with => 'Sample comment'
+      end
+      click_button 'Comment'
+      expect(page).to have_text('The comment has been created')
+    end
+
+    it 'post not enabled for comments' do
+      @post.set_meta('has_comments', '0')
+      admin_sign_in
+      visit @post.the_url(as_path: true)
+      expect(page).not_to have_text('New Comment')
+    end
+  end
+end

data/spec/features/frontend/post_type_spec.rb

@@ -0,0 +1,16 @@
+require 'rails_helper'
+describe "Posttype frontend", js: true do
+  before(:each) do
+    @site = create(:site).decorate
+    @post = @site.the_post('sample-post').decorate
+    @post_type = @post.post_type.decorate
+  end
+
+  it 'public uri with group structure' do
+    expect(@post_type.the_group_url(as_path: true)).to include("/group/#{@post_type.id}")
+  end
+
+  it "public url" do
+    expect(@post_type.the_url).to include("/#{@post_type.slug}")
+  end
+end

data/spec/helpers/email_helper_spec.rb

@@ -1,7 +1,7 @@
 require 'rails_helper'
 
 describe "CamaleonCms::EmailHelper" do
-  get_current_test_site()
+  before(:each){get_current_test_site()}
   describe "#send_email (old way)" do
     it "returns empty" do
       expect(helper.send_email('test@gmail.com', 'Test Email', 'test content', 'owenperedo@gmail.com')).to be_a_kind_of(ActionMailer::DeliveryJob)

data/spec/helpers/uploader_helper_spec.rb

@@ -0,0 +1,56 @@
+require 'rails_helper'
+describe CamaleonCms::UploaderHelper do
+  init_site
+  before(:each) do
+    @path = "#{CAMALEON_CMS_ROOT}/spec/support/fixtures/rails_tmp.png"
+    FileUtils.cp("#{CAMALEON_CMS_ROOT}/spec/support/fixtures/rails.png", @path)
+  end
+  
+  it "upload a local file" do
+    expect(upload_file(File.open(@path)).keys.include?(:error)).not_to eql(true)
+    expect(upload_file(File.open(@path), {thumb_size: '20x20'}).keys.include?(:error)).not_to eql(true)
+    expect(upload_file(File.open(@path), {folder: 'sample'}).keys.include?(:error)).not_to eql(true)
+  end
+  
+  it "upload a local file max size" do
+    expect(upload_file(File.open(@path), {maximum: 1.byte}).keys.include?(:error)).to eql(true)
+  end
+
+  it "upload a local file custom dimension" do
+    expect(upload_file(File.open(@path), {dimension: '50x50'}).keys.include?(:error)).not_to eql(true)
+    expect(upload_file(File.open(@path), {dimension: 'x50'}).keys.include?(:error)).not_to eql(true)
+    expect(upload_file(File.open(@path), {dimension: '50x'}).keys.include?(:error)).not_to eql(true)
+    expect(upload_file(File.open(@path), {dimension: '50x20?'}).keys.include?(:error)).not_to eql(true)
+  end
+
+  it "upload a local file invalid format" do
+    expect(upload_file(File.open(@path), {formats: 'audio'}).keys.include?(:error)).to eql(true)
+  end
+
+  it "upload a local file with versions" do
+    expect(upload_file(File.open(@path), {versions: '300x300,505x350,20x'}).keys.include?(:error)).not_to eql(true)
+  end
+  
+  it "add auto orient for cropping images" do
+    callback = lambda do |params| 
+      params[:img] = params[:img].auto_orient 
+    end
+    PluginRoutes.add_anonymous_hook('before_crop_image', callback, 'my_custom_hook')
+    expect(upload_file(File.open(@path), {versions: '300x300,505x350,20x'}).keys.include?(:error)).not_to eql(true)
+    PluginRoutes.remove_anonymous_hook('before_crop_image','my_custom_hook')
+  end
+  
+  it "add auto orient for resizing" do
+    callback = lambda do |params|
+      params[:img] = params[:img].auto_orient 
+    end
+    PluginRoutes.add_anonymous_hook('before_resize_crop', callback, 'my_custom_hook')
+    expect(upload_file(File.open(@path), {versions: '300x300,505x350,20x'}).keys.include?(:error)).not_to eql(true)
+    PluginRoutes.remove_anonymous_hook('before_resize_crop','my_custom_hook')
+  end
+
+  it "upload a external file" do
+    expect(upload_file('http://camaleon.tuzitio.com/media/132/slider/slide33.jpg').keys.include?(:error)).not_to eql(true)
+  end 
+  
+end

data/spec/mailers/send_mail_spec.rb

@@ -1,8 +1,10 @@
 require 'rails_helper'
 
 describe "CamaleonCms::HtmlMailer" do
+  before(:each) do
+    @site = create(:site).decorate
+  end
   describe 'empty content' do
-    get_current_test_site()
     let(:mail) { CamaleonCms::HtmlMailer.sender('test@gmail.com', "test") }
 
     it 'renders the subject' do
@@ -23,9 +25,7 @@ describe "CamaleonCms::HtmlMailer" do
   end
 
   describe 'custom content' do
-    get_current_test_site()
     let(:mail) { CamaleonCms::HtmlMailer.sender('test@gmail.com', "test", content: 'custom content', cc_to: ['a@gmail.com', 'b@gmail.com']) }
-
     it 'renders the sender email' do
       expect(mail.cc).to eql(['a@gmail.com', 'b@gmail.com'])
     end

data/spec/{decorators/camaleon_cms/post_decorator_spec.rb → models/post_spec.rb}

@@ -1,6 +1,6 @@
 require 'rails_helper'
-
-RSpec.describe CamaleonCms::PostDecorator do
+include CamaleonCms::Frontend::ApplicationHelper
+RSpec.describe 'PostDecorator' do
   it "next and previous post related to post type" do
     post_type = create_test_post_type(slug: 'test-post-type')
     post3 = create_test_post(post_type, slug: 'test3', post_order: 3).decorate
@@ -8,7 +8,7 @@ RSpec.describe CamaleonCms::PostDecorator do
     post2 = create_test_post(post_type, slug: 'test2', post_order: 2).decorate
     post2_1 = create_test_post(post_type, slug: 'test2_1', post_order: 2).decorate
     post4 = create_test_post(post_type, slug: 'test4', post_order: 4).decorate
-    
+
     expect(post3.the_next_post.slug).to eq('test4')
     expect(post3.the_prev_post.slug).to eq('test2_1')
 

data/spec/rails_helper.rb

@@ -1 +1,80 @@
 require 'spec_helper'
+require 'transactional_capybara/rspec'
+
+# spec/support/factory_bot.rb
+RSpec.configure do |config|
+  config.include FactoryBot::Syntax::Methods
+end
+
+# RSpec without Rails
+RSpec.configure do |config|
+  config.include FactoryBot::Syntax::Methods
+
+  config.before(:suite) do
+    FactoryBot.find_definitions
+  end
+end
+
+# rspec cleaner
+RSpec.configure do |config|
+
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  config.around(:each) do |example|
+    DatabaseCleaner.cleaning do
+      example.run
+    end
+  end
+
+end
+
+# capyobara cleaner
+RSpec.configure do |config|
+
+  config.use_transactional_fixtures = false
+
+  config.before(:suite) do
+    if config.use_transactional_fixtures?
+      raise(<<-MSG)
+        Delete line `config.use_transactional_fixtures = true` from rails_helper.rb
+        (or set it to false) to prevent uncommitted transactions being used in
+        JavaScript-dependent specs.
+
+        During testing, the app-under-test that the browser driver connects to
+        uses a different database connection to the database connection used by
+        the spec. The app's database connection would not be able to access
+        uncommitted transaction data setup over the spec's database connection.
+      MSG
+    end
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.strategy = :transaction
+  end
+
+  config.before(:each, type: :feature) do
+    # :rack_test driver's Rack app under test shares database connection
+    # with the specs, so continue to use transaction strategy for speed.
+    driver_shares_db_connection_with_specs = Capybara.current_driver == :rack_test
+
+    unless driver_shares_db_connection_with_specs
+      # Driver is probably for an external browser with an app
+      # under test that does *not* share a database connection with the
+      # specs, so use truncation strategy.
+      DatabaseCleaner.strategy = :truncation
+    end
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.append_after(:each) do
+    DatabaseCleaner.clean
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,6 +1,3 @@
-# Configuration was inspired by:
-# https://www.viget.com/articles/rails-engine-testing-with-rspec-capybara-and-factorygirl
-
 ENV['RAILS_ENV'] ||= 'test'
 
 CAMALEON_CMS_ROOT = Pathname.new(__FILE__).join('../..')
@@ -13,45 +10,27 @@ ActiveRecord::Migrator.migrations_paths << File.expand_path('../../db/migrate',
 require 'rspec/rails'
 require "capybara/rspec"
 require 'database_cleaner'
+require "rack_session_access/capybara"
 
-# This file was generated by the `rails generate rspec:install` command. Conventionally, all
-# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# The generated `.rspec` file contains `--require spec_helper` which will cause
-# this file to always be loaded, without a need to explicitly require it in any
-# files.
-#
-# Given that it is always loaded, you are encouraged to keep this file as
-# light-weight as possible. Requiring heavyweight dependencies from this file
-# will add to the boot time of your test suite on EVERY test run, even for an
-# individual file that may not need all of that loaded. Instead, consider making
-# a separate helper file that requires the additional dependencies and performs
-# the additional setup, and require it from the spec files that actually need
-# it.
-#
-# The `.rspec` file also contains a few flags that are not defaults but that
-# users commonly want.
-#
-# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
 require 'capybara/rspec'
 require 'capybara-screenshot/rspec'
-# Capybara.javascript_driver = :webkit
+Capybara.javascript_driver = :webkit
 
 # define screenshot errors name
 Capybara::Screenshot.register_filename_prefix_formatter(:rspec) do |example|
   "screenshot_#{example.description.gsub(' ', '-').gsub(/^.*\/spec\//,'')}"
 end
 
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 #****** configuration to test using poltergeist *****#
-require 'capybara/poltergeist'
-# Note: you need phantomjs 1.9.x which can be downloaded from here: https://code.google.com/archive/p/phantomjs/downloads
-if File.exist?(File.join(CAMALEON_CMS_ROOT, 'spec', 'bin', 'phantomjs'))
-  Capybara.register_driver :poltergeist do |app|
-    Capybara::Poltergeist::Driver.new(app, :phantomjs => File.join(CAMALEON_CMS_ROOT, 'spec', 'bin', 'phantomjs').to_s)
-  end
-end
-Capybara.javascript_driver = :poltergeist
-
+# require 'capybara/poltergeist'
+# Check here if you have problems with phantomjs: https://github.com/teampoltergeist/poltergeist
+# Don't forget bundle update to use last stable poltergaist gem
+# Capybara.register_driver :poltergeist do |app|
+#   Capybara::Poltergeist::Driver.new(app, :inspector => true, timeout: 1.minute)
+# end
+# Capybara.javascript_driver = :poltergeist
 
 RSpec.configure do |config|
   # rspec-expectations config goes here. You can use an alternate
@@ -170,34 +149,24 @@ RSpec.configure do |config|
   # The different available types are documented in the features, such as in
   # https://relishapp.com/rspec/rspec-rails/docs
   config.infer_spec_type_from_file_location!
-  config.before :type => :decorator do
-    ApplicationController.new.view_context
-  end
+  # config.before :type => :decorator do
+  #   ApplicationController.new.view_context
+  # end
 end
 
-RSpec.configure do |config|
+Capybara::Webkit.configure do |config|
+  # Enable debug mode. Prints a log of everything the driver is doing.
+  config.debug = false
 
-  config.before(:suite) do
-    DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner.clean_with(:truncation)
-  end
+  # Allow pages to make requests to any URL without issuing a warning.
+  config.allow_unknown_urls
 
-  config.around(:each) do |example|
-    DatabaseCleaner.cleaning do
-      example.run
-    end
-  end
+  # Don't raise errors when SSL certificates can't be validated
+  config.ignore_ssl_errors
 
-end
+  # Don't load images
+  config.skip_image_loading
 
-# Windows less than 1000px wide produce broken rendering on some pages:
-# https://github.com/owen2345/camaleon-cms/issues/481
-# Workaround: maximize the window in capybara:
-# https://makandracards.com/makandra/9773-how-to-start-selenium-browser-maximized-or-with-custom-window-size
-# http://stackoverflow.com/questions/18390071/change-default-capybara-browser-window-size
-RSpec.configure do |config|
-  config.before(:each, js: true) do
-    Capybara.page.driver.browser.manage.window.maximize rescue nil
-    page.driver.allow_url("fonts.googleapis.com") rescue nil
-  end
-end
+  # Raise JavaScript errors as exceptions
+  # config.raise_javascript_errors = true
+end

data/spec/support/common.rb

@@ -3,32 +3,17 @@ require 'rails_helper'
 
 # do login for admin panel and also verify if the site was created
 # if site is not created, then create a new site
-def login_success
-  if !CamaleonCms::Site.any?  && !defined?($install_called)
-    $install_called = true
-    it "Verify Install Camaleon", js: true do
-      visit "#{cama_root_relative_path}/admin/installers"
-      within("#new_site") do
-        fill_in 'site_name', :with => 'Test Site'
-        select "Default Theme", from: "theme"
-      end
-      click_button 'Submit'
-      expect(page).to have_content 'successfully'
-
-      admin_sign_in(true)
-    end
-  else
-    it "signs me in" do
-      admin_sign_in
-    end
+def init_site
+  before(:each) do
+    @site = create(:site).decorate
+    @post = @site.the_post('sample-post').decorate
   end
 end
 
 # sign in for admin panel
 # skip: true => close the skip button for intro
-def admin_sign_in(close = false, user = "admin", pass = "admin123")
+def admin_sign_in(user = "admin", pass = "admin123")
   visit "#{cama_root_relative_path}/admin/logout"
-  screenshot_and_save_page
   within("#login_user") do
     fill_in 'user[username]', :with => user
     fill_in 'user[password]', :with => pass
@@ -73,6 +58,7 @@ end
 
 # return the id of the first post
 def get_tag_attr(attr = "id", pos = "first")
+  puts "----------------#{Cama::Site.first.decorate.the_tags.to_a.inspect}"
   res = Cama::Site.first.decorate.the_tags.decorate.send(pos).send(attr)
   fix_db
   res
@@ -80,10 +66,10 @@ end
 
 # fix for: SQLite3::BusyException: database is locked: commit transaction
 def fix_db
-  if ActiveRecord::Base.connection.adapter_name.downcase.include?('sqlite')
-    ActiveRecord::Base.connection.execute("END;")
-    ActiveRecord::Base.connection.execute("BEGIN TRANSACTION;")
-  end
+  # if ActiveRecord::Base.connection.adapter_name.downcase.include?('sqlite')
+  #   ActiveRecord::Base.connection.execute("END;")
+  #   ActiveRecord::Base.connection.execute("BEGIN TRANSACTION;")
+  # end
 end
 
 def pages_test