refinerycms-images 2.1.5 → 3.0.0

data/spec/models/refinery/thumbnail_dimensions_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+module Refinery
+  describe ThumbnailDimensions, type: :model do
+    subject(:dimensions) { described_class.new(geometry, 500, 375) }
+
+    describe '#scale' do
+      context 'when no width is supplied' do
+        let(:geometry) { 'x225>' }
+
+        it 'scales by height factor' do
+          expect(dimensions.width).to eq 300
+          expect(dimensions.height).to eq 225
+        end
+      end
+    end
+  end
+end

data/spec/support/shared contexts/admin_images_tab.rb

@@ -0,0 +1,17 @@
+shared_context 'admin images tab' do
+
+  let(:open_upload_dialog) { find('a', text: ::I18n.t('create_new_image', scope: 'refinery.admin.images.actions')).trigger(:click) }
+  let(:select_upload) {}
+  let(:initialize_context) {}
+  let(:index_in_frame) {false}
+  let(:dialog_frame_id) {'dialog_iframe'}
+  let(:initial_path) { refinery.admin_images_path(view: %w(grid list).sample) }
+
+  let(:index_item_selector) {'#records li'}
+  let(:gridview_img_selector) {' > img'}
+  let(:gridview_title_selector) {'[tooltip]'}
+  let(:gridview_alt_selector) {'[alt]'}
+  let(:listview_title_selector) {' > span.title'}
+  let(:listview_filename_selector) {' > span.preview'}
+
+end

data/spec/support/shared contexts/many_images.rb

@@ -0,0 +1,5 @@
+shared_context 'many images' do
+  let!(:image) { FactoryGirl.create(:image) }
+  let!(:alt_image) { FactoryGirl.create(:alternate_image) }
+  let!(:another_image) { FactoryGirl.create(:another_image) }
+end

data/spec/support/shared contexts/no_images.rb

@@ -0,0 +1,3 @@
+shared_context "no existing images" do
+ let(:image) { FactoryGirl.create(:image) }
+end

data/spec/support/shared contexts/one_image.rb

@@ -0,0 +1,3 @@
+shared_context "one image" do
+ let!(:image) { FactoryGirl.create(:image) }
+end

data/spec/support/shared contexts/visual_editor_add_image.rb

@@ -0,0 +1,31 @@
+shared_context 'Visual Editor - add image' do
+
+  let(:page_for_image) {
+    page = Refinery::Page.create :title => "Add Image to me"
+    # we need page parts so that there's a visual editor
+    Refinery::Pages.default_parts.each_with_index do |default_page_part, index|
+      page.parts.create(:title => default_page_part, :slug => default_page_part, :body => nil, :position => index)
+    end
+    page
+  }
+
+  let(:initialize_context) {click_on('Add Image') }
+
+  let(:open_upload_dialog) {}
+  let(:select_upload) {choose(::I18n.t('new_image', :scope => 'refinery.admin.images.insert')) }
+
+  let(:initial_path) { refinery.edit_admin_page_path(page_for_image) }
+
+  # let(:target_image) { find(:xpath, "//img[@title='Beach']")}
+  # let(:url){target_image["data-#{size}"]}
+
+  let(:index_in_frame)  {true}
+  let(:dialog_frame_id) {'dialog_frame'}
+  let(:index_item_selector) {'#existing_image_area_content ul li img'}
+  let(:title_attribute_selector) {'[title]'}
+  let(:alt_attribute_selector) {'[alt]'}
+
+  let(:image_title_selector) {|title| '//img[@alt="' << title << '"]'}
+  let(:image_size_selector)  {|size| '//a[@href="#' << size << '"]'}
+
+end

data/spec/support/shared examples/image_deleter.rb

@@ -0,0 +1,31 @@
+shared_examples_for 'deletes an image' do
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+  end
+
+  let(:image_count) {[Refinery::Image.count, Refinery::Images.pages_per_admin_index].min}
+  let(:deleting_an_image) {
+    -> {
+      first("#records li").click_link(::I18n.t('delete', scope: 'refinery.admin.images'))
+    }
+  }
+
+  it 'has a delete image link for each image' do
+    expect(page).to have_selector('#records.images li a.confirm-delete', count: image_count)
+  end
+
+  it "removes an image" do
+    expect(deleting_an_image).to change(Refinery::Image, :count).by(-1)
+  end
+
+  it 'says the image has been removed' do
+    image_title = page.has_selector?('#image_grid') ? find("#image_grid li:first").first("img")[:title] :
+                                                      find("#image_list li:first").first("span.title").text
+
+    expect(image_title).to be_present
+
+    first(".images_list li:first").click_link(::I18n.t('delete', scope: 'refinery.admin.images'))
+    expect(page).to have_content(::I18n.t('destroyed', scope: 'refinery.crudify', what: "'#{image_title}'"))
+  end
+end

data/spec/support/shared examples/image_editor.rb

@@ -0,0 +1,3 @@
+shared_examples 'edits an image' do
+  pending
+end

data/spec/support/shared examples/image_indexer.rb

@@ -0,0 +1,118 @@
+shared_examples_for 'indexes images' do
+
+  let(:image_count) {[Refinery::Image.count, Refinery::Images.pages_per_admin_index].min}
+
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+    initialize_context
+  end
+
+  it 'shows all the images', js: true do
+    if index_in_frame
+      page.within_frame(dialog_frame_id) do
+        expect(page).to have_selector(index_item_selector, count: image_count)
+      end
+    else
+      expect(page).to have_selector(index_item_selector, count: image_count)
+    end
+  end
+
+end # image index
+
+shared_examples_for 'shows list and grid views' do
+
+  let(:image_count) {[Refinery::Image.count, Refinery::Images.pages_per_admin_index].min}
+
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+  end
+
+  context "when in grid view" do
+
+    before { ensure_on(current_path + "?view=grid") }
+
+    it 'shows the images with thumbnails', js: true do
+      expect(page).to have_selector(index_item_selector << gridview_img_selector, count: image_count)
+    end
+
+    it 'makes the title attribute of each image available', js: true do
+      expect(page).to have_selector(index_item_selector << gridview_img_selector << gridview_title_selector, count: image_count)
+    end
+
+    it 'makes the alt attribute of each image available', js: true do
+      expect(page).to have_selector(index_item_selector << gridview_img_selector << gridview_alt_selector, count: image_count)
+    end
+
+    it 'has an option to switch to list view' do
+      expect(page).to have_content(::I18n.t('switch_to', view_name: 'list', scope: 'refinery.admin.images.index.view'))
+    end
+
+  end  # grid view
+
+  context "when in list view" do
+
+    before  do
+      ensure_on(current_path + "?view=list")
+    end
+
+    it 'makes the title attribute of each image available', js: true do
+      expect(page).to have_selector(index_item_selector <<  listview_title_selector, count: image_count)
+    end
+
+    it 'makes the filename of each image available' do
+      expect(page).to have_selector(index_item_selector << listview_filename_selector, count: image_count)
+    end
+
+    it 'has an option to switch to grid view' do
+      ensure_on(current_path + '?view=list')
+
+      expect(page).to have_content(::I18n.t('switch_to', view_name: 'grid', scope: 'refinery.admin.images.index.view'))
+    end
+
+  end # list view
+end
+
+shared_examples_for 'paginates the list of images' do
+
+  let(:image_count) {[Refinery::Image.count, Refinery::Images.pages_per_admin_index].min}
+
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+  end
+
+  describe 'pagination', unless: Refinery::Image.count <= 2 do
+    before {
+      Refinery::Images.pages_per_admin_index = 2
+      Refinery::Images.pages_per_dialog_that_have_size_options = 2 }
+
+    it 'divides the index into pages' do
+      expect(page).to have_selector("div.pagination em.current")
+    end
+
+    context 'when on the first page' do
+      it 'shows a link for the next page' do
+        ensure_on(current_path + '?from_page=1&page=1')
+        expect(page).to have_selector("a.next_page[rel='next']")
+      end
+
+      it 'has disabled the previous page link' do
+        expect(page).to have_selector('.previous_page.disabled')
+      end
+    end
+
+    context 'when on the last page' do
+      it 'shows a link for the previous page' do
+        ensure_on(current_path + '?from_page=1&page=2')
+        expect(page).to have_selector("a.previous_page[rel='prev start']")
+      end
+
+      it 'has disabled the next link' do
+        expect(page).to have_selector('.next_page.disabled')
+      end
+
+    end
+  end  # pagination
+end # image index

data/spec/support/shared examples/image_inserter.rb

@@ -0,0 +1,53 @@
+
+shared_examples 'inserts images' do
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+  end
+
+  def select_and_insert_image(title, size='medium')
+    page.within_frame(dialog_frame_id) do
+      within '#existing_image_area_content' do
+        find(:xpath, "//img[@alt=\"#{title}\"]").click
+        find(:xpath, "//a[@href=\"##{size}\"]").click
+        find(:xpath, '//input[@id="submit_button"]').click
+      end
+    end
+  end
+
+  describe 'resize the image' do
+    pending
+    # it 'inserts an image of the selected size', js: true do
+      # click_on('Add Image')
+      # select_and_insert_image('Beach','large')
+        # sleep 10
+        # puts page.html
+        # page.within_frame(find(:xpath, '//iframe[contains(@id,"WYMeditor")]')) do
+          # expect(page).to have_selector('img[title="Beach"][data-rel*="450x450"]')
+      # end
+    # end
+  end
+
+  context 'when all images are available' do
+    pending
+    # it 'inserts the image into the page', js: true do
+      # click_on('Add Image')
+      # select_and_insert_image('Beach')
+      # expect(page).to have_image(url)
+    # end
+  end
+
+  context "when images are filtered by a search" do
+    pending
+
+    # before do
+      # fill_in "search", :with => "Beach"
+      # click_button "Search"
+    # end
+    # it 'inserts the image into the page', js: true do
+      # click_on('Add Image')
+      # select_and_insert_image('Beach')
+      # expect(page).to have_image(url)
+    # end
+  end
+end

data/spec/support/shared examples/image_previewer.rb

@@ -0,0 +1,41 @@
+def preview_image
+  preview_window = window_opened_by do
+    image_url
+    find(:linkhref, image_url).click
+  end
+
+  page.within_window preview_window do
+    expect(page).to have_image(image_url)
+  end
+  preview_window.close
+end
+
+shared_examples 'shows an image preview' do
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+  end
+
+  let(:image_url) {first(:xpath, "//a[@class='preview_icon']")[:href]}
+
+  context "when in list view" do
+    before  do
+      ensure_on(current_path + "?view=list")
+    end
+
+    it 'displays the image in a new window', js: true do
+      preview_image()
+    end
+  end
+
+
+    context "when in grid view" do
+    before  do
+      ensure_on(current_path + "?view=grid")
+    end
+
+    it 'displays the image in a new window', js: true do
+      preview_image
+    end
+  end
+end

data/spec/support/shared examples/image_translator.rb

@@ -0,0 +1,31 @@
+shared_examples 'translates an image' do
+  before do
+    allow(Refinery::I18n).to receive(:frontend_locales).and_return([:en, :fr])
+    ensure_on(initial_path)
+  end
+
+  context "when in list view" do
+    before  do
+      ensure_on(current_path + "?view=list")
+    end
+
+    it "can have a second locale added to it" do
+      expect(page).to have_content("Beach")
+      expect(page).to have_selector("a[href='/refinery/images/#{image.id}/edit']")
+
+      click_link "Edit this image"
+
+      within "#switch_locale_picker" do
+        click_link "FR"
+      end
+
+      fill_in "Title", :with => "Titre de la première image"
+      fill_in "Alt", :with => "Texte alternatif de la première image"
+
+      click_button "Save"
+
+      expect(page).to have_content("'Titre de la première image' was successfully updated.")
+      expect(Refinery::Image.translation_class.count).to eq(1)
+    end
+  end
+end

data/spec/support/shared examples/image_uploader.rb

@@ -0,0 +1,37 @@
+shared_examples 'uploads images' do
+  before do
+    raise "please set let(:initial_path)" if initial_path.blank?
+    ensure_on(initial_path)
+    initialize_context
+  end
+
+  let(:uploading_an_image) {
+  ->{
+    open_upload_dialog
+    page.within_frame(dialog_frame_id) do
+      select_upload
+      attach_file 'image_image', image_path
+      fill_in  'image_image_title', with: 'Image With Dashes'
+      fill_in  'image_image_alt', with: "Alt description for image"
+      click_button ::I18n.t('save', scope: 'refinery.admin.form_actions')
+    end
+    }
+  }
+
+  context 'when the image type is acceptable' do
+    let(:image_path) {Refinery.roots('refinery/images').join("spec/fixtures/image-with-dashes.jpg")}
+    it 'the image is uploaded', :js => true do
+      expect(uploading_an_image).to change(Refinery::Image, :count).by(1)
+    end
+  end
+
+  context 'when the image type is not acceptable' do
+    let(:image_path) {Refinery.roots('refinery/images').join("spec/fixtures/cape-town-tide-table.pdf")}
+    it 'the image is rejected', :js => true do
+      expect(uploading_an_image).to_not change(Refinery::Image, :count)
+      page.within_frame(dialog_frame_id) do
+        expect(page).to have_content(::I18n.t('incorrect_format', :scope => 'activerecord.errors.models.refinery/image'))
+      end
+    end
+  end
+end

data/spec/support/spec_helper.rb

@@ -0,0 +1,19 @@
+module Capybara
+  class Session
+    def has_image?(src)
+      has_xpath?("//img[contains(@src,'#{src}')]")
+    end
+  end
+
+  add_selector(:linkhref) do
+    xpath {|href| ".//a[@href='#{href}']"}
+  end
+end
+
+def ensure_on(path)
+  visit(path) unless current_path == path
+end
+
+RSpec.configure do |c|
+  c.alias_it_should_behave_like_to :it_has_behaviour, 'has behaviour:'
+end

data/test.html

@@ -0,0 +1,126 @@
+<!DOCTYPE html>
+<!-- paulirish.com/2008/conditional-stylesheets-vs-css-hacks-answer-neither/ -->
+<!--[if lt IE 7 ]> <html lang="en" class="no-js ie6"> <![endif]-->
+<!--[if IE 7 ]>    <html lang="en" class="no-js ie7"> <![endif]-->
+<!--[if IE 8 ]>    <html lang="en" class="no-js ie8"> <![endif]-->
+<!--[if IE 9 ]>    <html lang="en" class="no-js ie9"> <![endif]-->
+<!--[if (gt IE 9)|!(IE)]><!-->
+<html lang="en" class="no-js">
+  <!--<![endif]-->
+
+  <head>
+    <meta charset='utf-8' />
+    <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
+    <meta refinerycms='3.0.0' />
+    <title>Company Name - Refinery</title>
+
+    <link rel="stylesheet" media="screen" href="/assets/refinery/refinery.css" />
+    <link rel="stylesheet" media="screen" href="/assets/wymeditor.css" />
+    <link rel="stylesheet" media="screen" href="/assets/wymeditor/skins/refinery/skin.css" />
+
+    <script type="text/javascript" >
+      var refinery = window.refinery || {};
+      refinery.current_admin_locale = 'en';
+    </script>
+    <script src="/assets/admin.js"></script><script src="/assets/refinery/refinery.js"></script><script src="/assets/refinery/application.js"></script>
+    <script src="/assets/refi
+    nery/wymeditor.js"></script>
+    <script src="/assets/wymeditor/lang/en.js"></script>
+    <script src="/assets/wymeditor/skins/refinery/skin.js"></script>
+
+  </head>
+
+  <body class="index en">
+    <div id='site_bar'>
+      <div id='site_bar_content' class='clearfix'>
+
+        <div id='editor_switch'>
+          <a data-no-turbolink="true" href="/">Switch to your website</a>
+        </div>
+
+        <a id="site_bar_refinery_cms_logo" target="_blank" href="http://refinerycms.com"> <span>Refinery CMS™</span> </a>
+        <div id='site_bar_branding'>
+          <span id='site_bar_company_name'> Company Name </span>
+
+          <a id="logout" href="/refinery/logout">Log out</a>
+        </div>
+      </div>
+    </div>
+
+    <div id='tooltip_container'></div>
+    <div id="admin_container" class="clearfix">
+      <header id="header">
+        <nav id="menu">
+          <a id="plugin_refinery_users" href="/refinery/users">Users</a>
+          <a class="active" id="plugin_refinery_images" href="/refinery/images">Images</a>
+          <a id="plugin_refinery_files" href="/refinery/resources">Files</a>
+          <a id="plugin_refinery_pages" href="/refinery/pages">Pages</a>
+
+          <a id="menu_reorder" title="Reorder menu" class=" reorder_icon" href=""></a>
+          <a id="menu_reorder_done" title="i18n: Reorder Done" class=" reorder_done_icon hidden" href=""></a>
+
+        </nav>
+
+      </header>
+      <div id="page_container">
+        <div id="page">
+          <div id="content" class="clearfix">
+            <div id="flash_container">
+              <noscript>
+                <div class='flash flash_error'>
+                  For full functionality of this page it is necessary to enable JavaScript.
+                  Here are the
+                  <a target="_blank" href="http://www.enable-javascript.com">instructions for how to enable JavaScript in your web browser</a>
+                </div>
+              </noscript>
+
+            </div>
+            <section id="records" class="images">
+
+              <div class="pagination_container">
+
+                <div class="clearfix pagination_frame frame_center">
+                  <h3>March 04, 2015</h3>
+                  <ul class='images_list'>
+                    <li id="sortable_1" class="clearfix record on">
+                      <span class="title"> Beach </span>
+                      <span class='alt'> <a title="Title: Beach Alt text: Beach" class=" info_icon" href="#"></a> </span>
+
+                      <span class="preview">&nbsp;</span>
+                      <span class="actions"> <a title="View this image <br/><em>Opens in a new window</em>" class=" preview_icon" target="_blank" href="/system/images/W1siZiIsIjIwMTUvMDMvMDQvNmhleWpqcDJvZF9iZWFjaC5q
+                        cGVnIl1d/beach.jpeg?sha=9fd966feb5fd57e4"></a> <a title="Edit this image" class=" edit_icon" href="/refinery/images/1/edit"></a> <a data-confirm="Are you sure you want to remove &#39;Beach&#39;?" title="Remove this image forever" class=" delete_icon cancel confirm-delete" rel="nofollow" data-method
+                        ="delete" href="/refinery/images/1"></a> </span>
+                    </li>
+
+                  </ul>
+                </div>
+
+              </div>
+
+            </section>
+            <section id="actions">
+              <ul>
+                <li>
+                  <form method='GET' action='/refinery/images' class='search_form'>
+                    <input id="search" type="search" name="search" title="Put here a string of at least 3 characters containing what you are looking for." />
+
+                    <input type="submit" value="Search" />
+                  </form>
+
+                </li>
+                <li>
+                  <a title="Add new image" class=" add_icon" href="/refinery/images/new?dialog=true">Add new image</a>
+                </li>
+                <li>
+                  <a title="Switch to grid view" class=" switch_view_grid_icon" href="/refinery/images?view=grid">Switch to grid view</a>
+                </li>
+              </ul>
+
+            </section>
+
+          </div>
+        </div>
+      </div>
+    </div>
+  </body>
+</html>