meta-tags 2.0.0 → 2.1.0

data/spec/view_helper/open_search_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'displaying Open Search meta tags' do
+  subject { ActionView::Base.new }
+
+  it 'should display meta tags specified with :open_search' do
+    subject.set_meta_tags(open_search: {
+        title: 'Open Search Title',
+        href:  '/open_search_path.xml'
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('link', with: {
+        href:  '/open_search_path.xml',
+        rel:   'search',
+        title: 'Open Search Title',
+        type:  'application/opensearchdescription+xml',
+      })
+    end
+  end
+
+  it 'should not display meta tags without content' do
+    subject.set_meta_tags(open_search: {
+        title: '',
+        href:  '',
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to_not have_tag('link', with: {
+        rel:   'search',
+        type:  'application/opensearchdescription+xml',
+      })
+    end
+  end
+end

data/spec/view_helper/refresh_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'displaying refresh' do
+  subject { ActionView::Base.new }
+
+  it 'displays refresh when "refresh" is used' do
+    subject.refresh(5)
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: '5', 'http-equiv' => 'refresh' })
+    end
+  end
+
+  it 'displays refresh when "set_meta_tags" used' do
+    subject.set_meta_tags(refresh: 5)
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: '5', 'http-equiv' => 'refresh' })
+    end
+  end
+
+  it 'should use custom refresh if given' do
+    subject.refresh("5;url=http://example.com/")
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: '5;url=http://example.com/', 'http-equiv' => "refresh" })
+    end
+  end
+
+  it 'should display nothing by default' do
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to_not have_tag('meta', with: { 'http-equiv' => "refresh" })
+    end
+  end
+end

data/spec/view_helper/title_spec.rb

@@ -0,0 +1,155 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper do
+  subject { ActionView::Base.new }
+
+  context 'displaying title' do
+    it 'should not display title if blank' do
+      expect(subject.display_meta_tags).to eq('')
+      subject.title('')
+      expect(subject.display_meta_tags).to eq('')
+    end
+
+    it 'should use website name if title is empty' do
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to eq('<title>someSite</title>')
+      end
+    end
+
+    it 'should display title when "title" used' do
+      subject.title('someTitle')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should display title only when "site" is empty' do
+      subject.title('someTitle')
+      expect(subject.display_meta_tags).to eq('<title>someTitle</title>')
+    end
+
+    it 'should display title when "set_meta_tags" used' do
+      subject.set_meta_tags(title: 'someTitle')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should display custom title if given' do
+      subject.title('someTitle')
+      subject.display_meta_tags(site: 'someSite', title: 'defaultTitle').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should use website before page by default' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should only use markup in titles in the view' do
+      expect(subject.title('<b>someTitle</b>')).to eq('<b>someTitle</b>')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should use page before website if :reverse' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle', reverse: true).tap do |meta|
+        expect(meta).to eq('<title>someTitle | someSite</title>')
+      end
+    end
+
+    it 'should be lowercase if :lowercase' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle', lowercase: true).tap do |meta|
+        expect(meta).to eq('<title>someSite | sometitle</title>')
+      end
+    end
+
+    it 'should use custom separator if :separator' do
+      subject.title('someTitle')
+      subject.display_meta_tags(site: 'someSite', separator: '-').tap do |meta|
+        expect(meta).to eq('<title>someSite - someTitle</title>')
+      end
+      subject.display_meta_tags(site: 'someSite', separator: ':').tap do |meta|
+        expect(meta).to eq('<title>someSite : someTitle</title>')
+      end
+      subject.display_meta_tags(site: 'someSite', separator: '&amp;').tap do |meta|
+        expect(meta).to eq('<title>someSite &amp;amp; someTitle</title>')
+      end
+      subject.display_meta_tags(site: 'someSite', separator: '&amp;'.html_safe).tap do |meta|
+        expect(meta).to eq('<title>someSite &amp; someTitle</title>')
+      end
+      subject.display_meta_tags(site: 'someSite: ', separator: false).tap do |meta|
+        expect(meta).to eq('<title>someSite: someTitle</title>')
+      end
+    end
+
+    it 'should use custom prefix and suffix if available' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle', prefix: ' -', suffix: '- ').tap do |meta|
+        expect(meta).to eq('<title>someSite -|- someTitle</title>')
+      end
+    end
+
+    it 'should collapse prefix if false' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle', prefix: false).tap do |meta|
+        expect(meta).to eq('<title>someSite| someTitle</title>')
+      end
+    end
+
+    it 'should collapse suffix if false' do
+      subject.display_meta_tags(site: 'someSite', title: 'someTitle', suffix: false).tap do |meta|
+        expect(meta).to eq('<title>someSite |someTitle</title>')
+      end
+    end
+
+    it 'should use all custom options if available' do
+      subject.display_meta_tags({
+        site:      'someSite',
+        title:     'someTitle',
+        prefix:    ' -',
+        suffix:    '+ ',
+        separator: ':',
+        lowercase: true,
+        reverse:   true,
+      }).tap do |meta|
+        expect(meta).to eq('<title>sometitle -:+ someSite</title>')
+      end
+    end
+
+    it 'should allow Arrays in title' do
+      subject.display_meta_tags(site: 'someSite', title: ['someTitle', 'anotherTitle']).tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle | anotherTitle</title>')
+      end
+    end
+
+    it 'should allow Arrays in title with :lowercase' do
+      subject.display_meta_tags(site: 'someSite', title: ['someTitle', 'anotherTitle'], lowercase: true).tap do |meta|
+        expect(meta).to eq('<title>someSite | sometitle | anothertitle</title>')
+      end
+    end
+
+    it 'should build title in reverse order if :reverse' do
+      subject.display_meta_tags({
+        site:      'someSite',
+        title:     ['someTitle', 'anotherTitle'],
+        prefix:    ' -',
+        suffix:    '+ ',
+        separator: ':',
+        reverse:   true,
+      }).tap do |meta|
+        expect(meta).to eq('<title>anotherTitle -:+ someTitle -:+ someSite</title>')
+      end
+    end
+  end
+
+  context '.display_title' do
+    it 'should display custom title if given' do
+      subject.title('someTitle')
+      subject.display_title(site: 'someSite', title: 'defaultTitle').tap do |meta|
+        expect(meta).to eq('someSite | someTitle')
+      end
+    end
+  end
+end

data/spec/view_helper/twitter_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'displaying Twitter meta tags' do
+  subject { ActionView::Base.new }
+
+  it 'should display meta tags specified with :twitter' do
+    subject.set_meta_tags(twitter: {
+      title: 'Twitter Share Title',
+      card:  'photo',
+      image: {
+        _:      'http://example.com/1.png',
+        width:  123,
+        height: 321,
+      }
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "Twitter Share Title", name: "twitter:title" })
+      expect(meta).to have_tag('meta', with: { content: "photo", name: "twitter:card" })
+      expect(meta).to have_tag('meta', with: { content: "http://example.com/1.png", name: "twitter:image" })
+      expect(meta).to have_tag('meta', with: { content: "123", name: "twitter:image:width" })
+      expect(meta).to have_tag('meta', with: { content: "321", name: "twitter:image:height" })
+    end
+  end
+
+  it "should display mirrored content" do
+    subject.set_meta_tags(title: 'someTitle')
+    subject.display_meta_tags(twitter: { title: :title }).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "someTitle", name: "twitter:title" })
+    end
+  end
+end

data/spec/view_helper_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper do
+  subject { ActionView::Base.new }
+
+  context 'returning values' do
+    it 'should return title' do
+      expect(subject.title('some-title')).to eq('some-title')
+    end
+
+    it 'should return headline if specified' do
+      expect(subject.title('some-title', 'some-headline')).to eq('some-headline')
+    end
+
+    it 'should return title' do
+      expect(subject.title('some-title')).to eq('some-title')
+      expect(subject.title).to eq('some-title')
+    end
+
+    it 'should return description' do
+      expect(subject.description('some-description')).to eq('some-description')
+    end
+
+    it 'should return keywords' do
+      expect(subject.keywords('some-keywords')).to eq('some-keywords')
+    end
+
+    it 'should return noindex' do
+      expect(subject.noindex('some-noindex')).to eq('some-noindex')
+    end
+
+    it 'should return nofollow' do
+      expect(subject.noindex('some-nofollow')).to eq('some-nofollow')
+    end
+  end
+
+  context 'while handling string meta tag names' do
+    it 'should work with common parameters' do
+      subject.display_meta_tags('site' => 'someSite', 'title' => 'someTitle').tap do |meta|
+        expect(meta).to eq('<title>someSite | someTitle</title>')
+      end
+    end
+
+    it 'should work with open graph parameters' do
+      subject.set_meta_tags('og' => {
+        'title'       => 'facebook title',
+        'description' => 'facebook description'
+      })
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "facebook title", property: "og:title" })
+        expect(meta).to have_tag('meta', with: { content: "facebook description", property: "og:description" })
+      end
+    end
+  end
+
+  it_behaves_like '.set_meta_tags'
+end

metadata

@@ -1,83 +1,97 @@
 --- !ruby/object:Gem::Specification
 name: meta-tags
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Dmytro Shteflyuk
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-15 00:00:00.000000000 Z
+date: 2015-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.3.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.3.0
+- !ruby/object:Gem::Dependency
+  name: rspec-html-matchers
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bluecloth
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Search Engine Optimization (SEO) plugin for Ruby on Rails applications.
@@ -89,10 +103,11 @@ extra_rdoc_files:
 - README.md
 - CHANGELOG.md
 files:
-- .gitignore
-- .ruby-gemset
-- .ruby-version
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".ruby-gemset"
+- ".ruby-version"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - MIT-LICENSE
@@ -100,6 +115,7 @@ files:
 - Rakefile
 - lib/meta-tags.rb
 - lib/meta_tags.rb
+- lib/meta_tags/configuration.rb
 - lib/meta_tags/content_tag.rb
 - lib/meta_tags/controller_helper.rb
 - lib/meta_tags/meta_tags_collection.rb
@@ -109,30 +125,46 @@ files:
 - lib/meta_tags/version.rb
 - lib/meta_tags/view_helper.rb
 - meta-tags.gemspec
+- spec/configuration_spec.rb
 - spec/controller_helper_spec.rb
-- spec/meta_tags_spec.rb
 - spec/spec_helper.rb
+- spec/text_normalizer/normalize_title_spec.rb
+- spec/text_normalizer/truncate_array_spec.rb
+- spec/view_helper/charset_spec.rb
+- spec/view_helper/custom_spec.rb
+- spec/view_helper/description_spec.rb
+- spec/view_helper/icon_spec.rb
+- spec/view_helper/keywords_spec.rb
+- spec/view_helper/links_spec.rb
+- spec/view_helper/module_spec.rb
+- spec/view_helper/noindex_spec.rb
+- spec/view_helper/open_graph_spec.rb
+- spec/view_helper/open_search_spec.rb
+- spec/view_helper/refresh_spec.rb
+- spec/view_helper/title_spec.rb
+- spec/view_helper/twitter_spec.rb
+- spec/view_helper_spec.rb
 homepage: http://github.com/kpumuk/meta-tags
 licenses: []
 metadata: {}
 post_install_message: 
 rdoc_options:
-- --charset=UTF-8
+- "--charset=UTF-8"
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Collection of SEO helpers for Ruby on Rails.