meta-tags 2.0.0 → 2.1.0

data/spec/view_helper/icon_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper do
+  subject { ActionView::Base.new }
+
+  it 'should not display icon by default' do
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to_not have_tag('link', with: { rel: 'icon' })
+    end
+  end
+
+  it 'should display icon when "set_meta_tags" used' do
+    subject.set_meta_tags(icon: '/favicon.ico')
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('link', with: { href: '/favicon.ico', rel: 'icon', type: 'image/x-icon' })
+    end
+  end
+
+  it 'should display default canonical url' do
+    subject.display_meta_tags(site: 'someSite', icon: '/favicon.ico').tap do |meta|
+      expect(meta).to have_tag('link', with: { href: '/favicon.ico', rel: 'icon', type: 'image/x-icon' })
+    end
+  end
+
+  it 'should allow to specify hash as an icon' do
+    subject.set_meta_tags(icon: { href: '/favicon.png', type: 'image/png' })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('link', with: { href: '/favicon.png', rel: 'icon', type: 'image/png' })
+    end
+  end
+
+  it 'should allow to specify multiple icons' do
+    subject.set_meta_tags(icon: [
+      { href: '/images/icons/icon_96.png', sizes: '32x32 96x96', type: 'image/png' },
+      { href: '/images/icons/icon_itouch_precomp_32.png', rel: 'apple-touch-icon-precomposed', sizes: '32x32', type: 'image/png' },
+    ])
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('link', with: { href: '/images/icons/icon_96.png', rel: 'icon', type: 'image/png', sizes: '32x32 96x96' })
+      expect(meta).to have_tag('link', with: { href: '/images/icons/icon_itouch_precomp_32.png', rel: 'apple-touch-icon-precomposed', type: 'image/png', sizes: '32x32' })
+    end
+  end
+end

data/spec/view_helper/keywords_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'displaying keywords' do
+  subject { ActionView::Base.new }
+
+  it 'should not display keywords if blank' do
+    subject.keywords('')
+    expect(subject.display_meta_tags).to eq('')
+
+    subject.keywords([])
+    expect(subject.display_meta_tags).to eq('')
+  end
+
+  it 'should display keywords when "keywords" used' do
+    subject.keywords('some-keywords')
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "some-keywords", name: "keywords" })
+    end
+  end
+
+  it 'should display keywords when "set_meta_tags" used' do
+    subject.set_meta_tags(keywords: 'some-keywords')
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "some-keywords", name: "keywords" })
+    end
+  end
+
+  it 'should display default keywords' do
+    subject.display_meta_tags(site: 'someSite', keywords: 'some-keywords').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "some-keywords", name: "keywords" })
+    end
+  end
+
+  it 'should use custom keywords if given' do
+    subject.keywords('some-keywords')
+    subject.display_meta_tags(site: 'someSite', keywords: 'default_keywords').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "some-keywords", name: "keywords" })
+    end
+  end
+
+  it 'should lowercase keywords' do
+    subject.display_meta_tags(site: 'someSite', keywords: 'someKeywords').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "somekeywords", name: "keywords" })
+    end
+  end
+
+  it 'should join keywords from Array' do
+    subject.display_meta_tags(site: 'someSite', keywords: %w(keyword1 keyword2)).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "keyword1, keyword2", name: "keywords" })
+    end
+  end
+
+  it 'should join keywords from nested Arrays' do
+    subject.display_meta_tags(site: 'someSite', keywords: [%w(keyword1 keyword2), 'keyword3']).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "keyword1, keyword2, keyword3", name: "keywords" })
+    end
+  end
+end

data/spec/view_helper/links_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper do
+  subject { ActionView::Base.new }
+
+  context 'displaying canonical url' do
+    it 'should not display canonical url by default' do
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('link', with: { href: "http://example.com/base/url", rel: "canonical" })
+      end
+    end
+
+    it 'should display canonical url when "set_meta_tags" used' do
+      subject.set_meta_tags(canonical: 'http://example.com/base/url')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "canonical" })
+      end
+    end
+
+    it 'should display default canonical url' do
+      subject.display_meta_tags(site: 'someSite', canonical: 'http://example.com/base/url').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "canonical" })
+      end
+    end
+  end
+
+  context 'displaying alternate url' do
+    it 'should not display alternate url by default' do
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('link', with: { href: "http://example.fr/base/url", hreflang: "fr", rel: "alternate" })
+      end
+    end
+
+    it 'should display alternate url when "set_meta_tags" used' do
+      subject.set_meta_tags(alternate: { 'fr' => 'http://example.fr/base/url' })
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.fr/base/url", hreflang: "fr", rel: "alternate" })
+      end
+    end
+
+    it 'should display default alternate url' do
+      subject.display_meta_tags(site: 'someSite', alternate: { 'fr' => 'http://example.fr/base/url' }).tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.fr/base/url", hreflang: "fr", rel: "alternate" })
+      end
+    end
+
+    it "should not display alternate without content" do
+      subject.display_meta_tags(site: 'someSite', alternate: {'zh-Hant' => ''}).tap do |meta|
+        expect(meta).to_not have_tag('link', with: { href: "", hreflang: "zh-Hant", rel: "alternate" })
+      end
+    end
+
+    it 'should allow to specify an array of alternate links' do
+      subject.display_meta_tags(site: 'someSite', alternate: [
+        { href: 'http://example.fr/base/url', hreflang: 'fr' },
+        { href: 'http://example.com/feed.rss', type: 'application/rss+xml', title: 'RSS' },
+        { href: 'http://m.example.com/page-1', media: 'only screen and (max-width: 640px)'},
+      ]).tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.fr/base/url", hreflang: "fr", rel: "alternate" })
+        expect(meta).to have_tag('link', with: { href: "http://example.com/feed.rss", type: "application/rss+xml", title: 'RSS', rel: "alternate" })
+        expect(meta).to have_tag('link', with: { href: "http://m.example.com/page-1", media: 'only screen and (max-width: 640px)', rel: "alternate" })
+      end
+    end
+  end
+
+  context 'displaying author link' do
+    it 'should display author link when "set_meta_tags" used' do
+      subject.set_meta_tags(author: 'http://plus.google.com/profile/url')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://plus.google.com/profile/url", rel: "author" })
+      end
+    end
+  end
+
+  context 'displaying publisher link' do
+    it 'should display publisher link when "set_meta_tags" used' do
+      subject.set_meta_tags(publisher: 'http://plus.google.com/myprofile_url')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://plus.google.com/myprofile_url", rel: "publisher" })
+      end
+    end
+  end
+
+  context 'displaying prev url' do
+    it 'should not display prev url by default' do
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('link', with: { href: "http://example.com/base/url", rel: "prev" })
+      end
+    end
+
+    it 'should display prev url when "set_meta_tags" used' do
+      subject.set_meta_tags(prev: 'http://example.com/base/url')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "prev" })
+      end
+    end
+
+    it 'should display default prev url' do
+      subject.display_meta_tags(site: 'someSite', prev: 'http://example.com/base/url').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "prev" })
+      end
+    end
+  end
+
+  context 'displaying next url' do
+    it 'should not display next url by default' do
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('link', with: { href: "http://example.com/base/url", rel: "next" })
+      end
+    end
+
+    it 'should display next url when "set_meta_tags" used' do
+      subject.set_meta_tags(next: 'http://example.com/base/url')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "next" })
+      end
+    end
+
+    it 'should display default next url' do
+      subject.display_meta_tags(site: 'someSite', next: 'http://example.com/base/url').tap do |meta|
+        expect(meta).to have_tag('link', with: { href: "http://example.com/base/url", rel: "next" })
+      end
+    end
+  end
+end

data/spec/view_helper/module_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'module' do
+  subject { ActionView::Base.new }
+
+  it 'should be mixed into ActionView::Base' do
+    expect(ActionView::Base.included_modules).to include(MetaTags::ViewHelper)
+  end
+
+  it 'should respond to "title" helper' do
+    expect(subject).to respond_to(:title)
+  end
+
+  it 'should respond to "description" helper' do
+    expect(subject).to respond_to(:description)
+  end
+
+  it 'should respond to "keywords" helper' do
+    expect(subject).to respond_to(:keywords)
+  end
+
+  it 'should respond to "noindex" helper' do
+    expect(subject).to respond_to(:noindex)
+  end
+
+  it 'should respond to "nofollow" helper' do
+    expect(subject).to respond_to(:nofollow)
+  end
+
+  it 'should respond to "set_meta_tags" helper' do
+    expect(subject).to respond_to(:set_meta_tags)
+  end
+
+  it 'should respond to "display_meta_tags" helper' do
+    expect(subject).to respond_to(:display_meta_tags)
+  end
+
+  it 'should respond to "display_title" helper' do
+    expect(subject).to respond_to(:display_title)
+  end
+end

data/spec/view_helper/noindex_spec.rb

@@ -0,0 +1,107 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper do
+  subject { ActionView::Base.new }
+
+  context 'displaying noindex' do
+    it 'should display noindex when "noindex" used' do
+      subject.noindex(true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex", name: "robots" })
+      end
+    end
+
+    it 'should display noindex when "set_meta_tags" used' do
+      subject.set_meta_tags(noindex: true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex", name: "robots" })
+      end
+    end
+
+    it 'should use custom noindex if given' do
+      subject.noindex('some-noindex')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex", name: "some-noindex" })
+      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: { content: "noindex" })
+      end
+    end
+
+    it "should display nothing if given false" do
+      subject.set_meta_tags(noindex: false)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('meta', with: { content: "robots" })
+      end
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to_not have_tag('meta', with: { content: "noindex" })
+      end
+    end
+  end
+
+  context 'displaying nofollow' do
+    it 'should display nofollow when "nofollow" used' do
+      subject.nofollow(true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "nofollow", name: "robots" })
+      end
+    end
+
+    it 'should display nofollow when "set_meta_tags" used' do
+      subject.set_meta_tags(nofollow: true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "nofollow", name: "robots" })
+      end
+    end
+
+    it 'should use custom nofollow if given' do
+      subject.nofollow('some-nofollow')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "nofollow", name: "some-nofollow" })
+      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: { content: "nofollow" })
+      end
+    end
+  end
+
+  context 'displaying both nofollow and noindex' do
+    it 'should be displayed when set using helpers' do
+      subject.noindex(true)
+      subject.nofollow(true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex, nofollow", name: "robots" })
+      end
+    end
+
+    it 'should be displayed when "set_meta_tags" used' do
+      subject.set_meta_tags(nofollow: true, noindex: true)
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex, nofollow", name: "robots" })
+      end
+    end
+
+    it 'should use custom name if string is used' do
+      subject.noindex('some-name')
+      subject.nofollow('some-name')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex, nofollow", name: "some-name" })
+      end
+    end
+
+    it 'should display two meta tags when different names used' do
+      subject.noindex('some-noindex')
+      subject.nofollow('some-nofollow')
+      subject.display_meta_tags(site: 'someSite').tap do |meta|
+        expect(meta).to have_tag('meta', with: { content: "noindex", name: "some-noindex" })
+        expect(meta).to have_tag('meta', with: { content: "nofollow", name: "some-nofollow" })
+      end
+    end
+  end
+end

data/spec/view_helper/open_graph_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+describe MetaTags::ViewHelper, 'displaying Open Graph meta tags' do
+  subject { ActionView::Base.new }
+
+  it 'should display meta tags specified with :open_graph' do
+    subject.set_meta_tags(open_graph: {
+      title:       'Facebook Share Title',
+      description: 'Facebook Share Description'
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Title", property: "og:title" })
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Description", property: "og:description" })
+    end
+  end
+
+  it 'should display meta tags specified with :og' do
+    subject.set_meta_tags(og: {
+      title:       'Facebook Share Title',
+      description: 'Facebook Share Description'
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Title", property: "og:title" })
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Description", property: "og:description" })
+    end
+  end
+
+  it 'should use deep merge when displaying open graph meta tags' do
+    subject.set_meta_tags(og: { title: 'Facebook Share Title' })
+    subject.display_meta_tags(og: { description: 'Facebook Share Description' }).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Title", property: "og:title" })
+      expect(meta).to have_tag('meta', with: { content: "Facebook Share Description", property: "og:description" })
+    end
+  end
+
+  it "should not display meta tags without content" do
+    subject.set_meta_tags(open_graph: {
+      title:       '',
+      description: ''
+    })
+    subject.display_meta_tags(site: 'someSite').tap do |meta|
+      expect(meta).to_not have_tag('meta', with: { content: "", property: "og:title" })
+      expect(meta).to_not have_tag('meta', with: { content: "", property: "og:description" })
+    end
+  end
+
+  it "should display locale meta tags" do
+    subject.display_meta_tags(open_graph: { locale: { _: 'en_GB', alternate: ['fr_FR', 'es_ES'] } }).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "en_GB", property: "og:locale" })
+      expect(meta).to have_tag('meta', with: { content: "fr_FR", property: "og:locale:alternate" })
+      expect(meta).to have_tag('meta', with: { content: "es_ES", property: "og:locale:alternate" })
+    end
+  end
+
+  it "should display mirrored content" do
+    subject.set_meta_tags(title: 'someTitle')
+    subject.display_meta_tags(open_graph: { title: :title }).tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "someTitle", property: "og:title" })
+    end
+  end
+
+  it "should display open graph meta tags with an array of images" do
+    subject.set_meta_tags(open_graph: {
+      title: 'someTitle',
+      image: [{
+        _:      'http://example.com/1.png',
+        width:  75,
+        height: 75,
+      },
+      {
+        _:      'http://example.com/2.png',
+        width:  50,
+        height: 50,
+      }]
+    })
+    subject.display_meta_tags(site: 'someTitle').tap do |meta|
+      expect(meta).to have_tag('meta', with: { content: "someTitle", property: "og:title" })
+      expect(meta).to have_tag('meta', with: { content: "http://example.com/1.png", property: "og:image" })
+      expect(meta).to have_tag('meta', with: { content: "75", property: "og:image:width" })
+      expect(meta).to have_tag('meta', with: { content: "75", property: "og:image:height" })
+      expect(meta).to have_tag('meta', with: { content: "http://example.com/2.png", property: "og:image" })
+      expect(meta).to have_tag('meta', with: { content: "50", property: "og:image:width" })
+      expect(meta).to have_tag('meta', with: { content: "50", property: "og:image:height" })
+    end
+  end
+end