nesta 0.11.1 → 0.12.0

data/test/integration/commands/theme/create_test.rb

@@ -0,0 +1,35 @@
+require 'test_helper'
+require_relative '../../../../lib/nesta/commands'
+
+describe 'nesta theme:create' do
+  include TemporaryFiles
+
+  def theme_path(path = '')
+    File.join('themes', 'theme-name', path)
+  end
+
+  before do
+    FileUtils.mkdir_p(project_root)
+  end
+
+  after do
+    remove_temp_directory
+  end
+
+  it 'creates default files in the theme directory' do
+    Dir.chdir(project_root) do
+      Nesta::Commands::Theme::Create.new('theme-name').execute
+    end
+    assert_exists_in_project theme_path('README.md')
+    assert_exists_in_project theme_path('app.rb')
+  end
+
+  it 'copies default view templates into views directory' do
+    Dir.chdir(project_root) do
+      Nesta::Commands::Theme::Create.new('theme-name').execute
+    end
+    %w(layout.haml page.haml master.sass).each do |template|
+      assert_exists_in_project theme_path("views/#{template}")
+    end
+  end
+end

data/test/integration/commands/theme/enable_test.rb

@@ -0,0 +1,22 @@
+require 'test_helper'
+require_relative '../../../../lib/nesta/commands'
+
+describe 'nesta theme:enable' do
+  include TemporaryFiles
+
+  before do
+    FileUtils.mkdir_p(File.join(project_root, 'config'))
+    File.open(File.join(project_root, 'config', 'config.yml'), 'w').close
+  end
+
+  after do
+    remove_temp_directory
+  end
+
+  it 'enables the theme' do
+    Dir.chdir(project_root) do
+      Nesta::Commands::Theme::Enable.new('theme-name').execute
+      assert_match /^theme: theme-name/, File.read('config/config.yml')
+    end
+  end
+end

data/test/integration/commands/theme/install_test.rb

@@ -0,0 +1,62 @@
+require 'test_helper'
+require_relative '../../../support/silence_commands_during_tests'
+require_relative '../../../../lib/nesta/commands'
+
+Nesta::Commands::Theme::Install.send(:include, SilenceCommandsDuringTests)
+
+describe 'nesta theme:install' do
+  include TemporaryFiles
+
+  def theme_name
+    'test'
+  end
+
+  def repo_url
+    "../../fixtures/nesta-theme-#{theme_name}.git"
+  end
+
+  def theme_dir
+    project_path("themes/#{theme_name}")
+  end
+
+  before do
+    FileUtils.mkdir_p(project_root)
+  end
+
+  after do
+    remove_temp_directory
+  end
+
+  it 'clones the repository' do
+    in_temporary_project do
+      Nesta::Commands::Theme::Install.new(repo_url).execute
+      assert File.directory?(theme_dir), 'theme not cloned'
+    end
+  end
+
+  it "removes the theme's .git directory" do
+    in_temporary_project do
+      Nesta::Commands::Theme::Install.new(repo_url).execute
+      refute File.exist?("#{theme_dir}/.git"), '.git folder found'
+    end
+  end
+
+  it 'enables the freshly installed theme' do
+    in_temporary_project do
+      Nesta::Commands::Theme::Install.new(repo_url).execute
+      assert_match /theme: #{theme_name}/, File.read('config/config.yml')
+    end
+  end
+
+  it 'determines name of theme from name of repository' do
+    url = 'git://foobar.com/path/to/nesta-theme-the-name.git'
+    command = Nesta::Commands::Theme::Install.new(url)
+    assert_equal 'the-name', command.theme_name
+  end
+
+  it "falls back to name of repo when theme name doesn't match correct format" do
+    url = 'git://foobar.com/path/to/mytheme.git'
+    command = Nesta::Commands::Theme::Install.new(url)
+    assert_equal 'mytheme', command.theme_name
+  end
+end

data/test/integration/default_theme_test.rb

@@ -0,0 +1,220 @@
+require 'integration_test_helper'
+
+describe 'Default theme' do
+  include Nesta::IntegrationTest
+
+  def in_categories(*categories)
+    { 'categories' => categories.map(&:path).join(', ') }
+  end
+
+  it 'includes description meta tag' do
+    with_temp_content_directory do
+      model = create(:page, metadata: { 'Description' => 'A good page' })
+      visit model.path
+      xpath = "//meta[@name='description'][@content='A good page']"
+      assert_has_xpath xpath, visible: false
+    end
+  end
+
+  it 'includes keywords meta tag' do
+    with_temp_content_directory do
+      model = create(:page, metadata: { 'Keywords' => 'good, content' })
+      visit model.path
+      xpath = "//meta[@name='keywords'][@content='good, content']"
+      assert_has_xpath xpath, visible: false
+    end
+  end
+
+  it "doesn't include Google Analytics JavaScript snippet by default" do
+    with_temp_content_directory do
+      visit '/'
+      assert_has_no_css 'script', text: 'google-analytics.com'
+    end
+  end
+
+  it 'includes Google Analytics JavaScript when configured' do
+    analytics_code = { 'google_analytics_code' => 'UA-1234' }
+    stub_config(temp_content.merge('test' => analytics_code)) do
+      visit '/'
+      assert_nil all('script').find { |s| s[:src].match /analytics\.js/ }
+    end
+  end
+
+  it 'displays site title' do
+    site_config = {
+      'title' => 'My blog',
+      'subtitle' => 'about stuff',
+    }
+    stub_config(temp_content.merge(site_config)) do
+      visit '/'
+      assert_has_css 'h1', text: 'My blog'
+      assert_has_css 'h2', text: 'about stuff'
+    end
+  end
+
+  describe 'menus' do
+    def create_pages_in_menu
+      pages
+    end
+
+    it "doesn't include menu markup if menu not configured" do
+      with_temp_content_directory do
+        visit '/'
+        assert_has_no_css 'ul.menu'
+      end
+    end
+
+    it 'only displays first two levels of menu items' do
+      with_temp_content_directory do
+        level1, level2, level3 = (0..2).map { create(:page) }
+        text = "%s\n  %s\n    %s\n" % [level1, level2, level3].map(&:abspath)
+        create_menu(text)
+        visit '/'
+        assert_has_css "ul.menu li a:contains('#{level1.link_text}')"
+        assert_has_css "ul.menu li ul li a:contains('#{level2.link_text}')"
+        assert_has_no_css "ul.menu a:contains('#{level3.link_text}')"
+      end
+    end
+
+    def create_page_and_menu
+      model = create(:page)
+      create_menu(model.path)
+      model
+    end
+
+    it "highlights current page's menu item" do
+      with_temp_content_directory do
+        model = create_page_and_menu
+        visit model.path
+        assert_has_css "ul.menu li[class='current']:contains('#{model.link_text}')"
+      end
+    end
+
+    it "highlights current page's menu item when app mounted at /prefix" do
+      Capybara.app = Rack::Builder.new do
+        map '/prefix' do
+          run Nesta::App.new
+        end
+      end
+
+      with_temp_content_directory do
+        model = create_page_and_menu
+        visit "/prefix/#{model.path}"
+        assert_has_css "ul.menu li[class='current']:contains('#{model.link_text}')"
+      end
+    end
+  end
+
+  it 'only displays read more link for summarised pages' do
+    with_temp_content_directory do
+      category = create(:category)
+      metadata = in_categories(category)
+      summarised = create(:page, metadata: metadata.merge('summary' => 'Summary'))
+      not_summarised = create(:page, metadata: metadata)
+      visit category.path
+      assert_has_css 'li:nth-child(1) p', text: summarised.read_more
+      assert_has_no_css 'li:nth-child(2) p', text: not_summarised.read_more
+    end
+  end
+
+  it 'displays page summaries or full content of unsummarised pages' do
+    with_temp_content_directory do
+      category = create(:category)
+      metadata = in_categories(category)
+      summarised = create(:page,
+                          content: 'Summarised content',
+                          metadata: metadata.merge(summary: 'Summary'))
+      not_summarised = create(:page,
+                              content: 'Unsummarised content',
+                              metadata: metadata)
+      visit category.path
+
+      # Page with a summary
+      assert_has_css 'li:nth-child(1) p', text: 'Summary'
+      assert_has_no_css 'li:nth-child(1) p', text: 'content'
+
+      # Page without a summary
+      assert_has_css 'li:nth-child(2) p', text: 'Unsummarised content'
+      assert_has_no_css 'li:nth-child(2) p', text: 'Summary'
+    end
+  end
+
+  it 'displays contents of page' do
+    with_temp_content_directory do
+      model = create(:page, content: 'Body of page')
+      visit model.path
+      assert_has_css 'p', text: 'Body of page'
+    end
+  end
+
+  describe 'category' do
+    it 'displays its "articles heading" above the articles' do
+      with_temp_content_directory do
+        category = create(:category, metadata: {
+          'articles heading' => 'Articles on this topic'
+        })
+        create(:article, metadata: in_categories(category))
+        visit category.path
+        assert_has_css 'h1', text: 'Articles on this topic'
+      end
+    end
+
+    it 'links to articles in category using article title' do
+      with_temp_content_directory do
+        category = create(:category)
+        article = create(:article, metadata: in_categories(category))
+        visit category.path
+        link_text, href = article.link_text, article.abspath
+        assert_has_css "ol h1 a[href$='#{href}']", text: link_text
+      end
+    end
+  end
+
+  describe 'article' do
+    it 'displays the date' do
+      with_temp_content_directory do
+        article = create(:article)
+        visit article.path
+        assert_has_css "time[datetime='#{article.date}']"
+      end
+    end
+
+    it 'links to parent page in breadcrumb' do
+      with_temp_content_directory do
+        parent = create(:category)
+        article = create(:article, path: "#{parent.path}/child")
+        visit article.path
+        href, link_text = parent.abspath, parent.link_text
+        assert_has_css "nav.breadcrumb a[href='#{href}']", text: link_text
+      end
+    end
+
+    it 'links to its categories at end of article' do
+      with_temp_content_directory do
+        categories = [create(:category), create(:category)]
+        article = create(:article, metadata: in_categories(*categories))
+        visit article.path
+        categories.each do |category|
+          href, link_text = category.abspath, category.link_text
+          assert_has_css "p.meta a[href$='#{href}']", text: link_text
+        end
+      end
+    end
+
+    it 'displays comments from Disqus' do
+      stub_config(temp_content.merge('disqus_short_name' => 'mysite')) do
+        article = create(:article)
+        visit article.path
+        assert_has_css '#disqus_thread'
+        assert_has_css 'script[src*="mysite.disqus.com"]', visible: false
+      end
+    end
+
+    it "doesn't use Disqus if it's not configured" do
+      with_temp_content_directory do
+        visit create(:article).path
+        assert_has_no_css '#disqus_thread'
+      end
+    end
+  end
+end

data/test/integration/overrides_test.rb

@@ -0,0 +1,118 @@
+require 'integration_test_helper'
+
+describe 'Overriding files in gem and themes' do
+  include Nesta::IntegrationTest
+
+  def in_nesta_project(config = {}, &block)
+    app_root = temp_path('root')
+    content_config = { 'content' => File.join(app_root, 'content') }
+    stub_config(content_config.merge(config)) do
+      with_app_root(app_root) do
+        yield
+      end
+    end
+  ensure
+    remove_temp_directory
+  end
+
+  def theme_name
+    'my-theme'
+  end
+
+  def create_fixture(type, name, content)
+    base_path = {
+      local: Nesta::Path.local,
+      theme: Nesta::Path.themes(theme_name)
+    }[type]
+    path = File.join(base_path, name)
+    FileUtils.mkdir_p(File.dirname(path))
+    open(path, 'w') { |file| file.write(content) }
+  end
+
+  def create_app_file(type)
+    create_fixture(type, 'app.rb', "FROM_#{type.to_s.upcase} = true")
+  end
+
+  describe 'app.rb' do
+    it 'loads app.rb from configured theme' do
+      in_nesta_project('theme' => theme_name) do
+        create_app_file(:theme)
+        Nesta::Overrides.load_theme_app
+        assert Object.const_get(:FROM_THEME), 'should load app.rb in theme'
+      end
+    end
+
+    it 'loads both local and theme app.rb files' do
+      in_nesta_project('theme' => theme_name) do
+        create_app_file(:local)
+        Nesta::Overrides.load_local_app
+        create_app_file(:theme)
+        Nesta::Overrides.load_theme_app
+        assert Object.const_get(:FROM_THEME), 'should load app.rb in theme'
+        assert Object.const_get(:FROM_LOCAL), 'should load local app.rb'
+      end
+    end
+  end
+
+  def create_view(type, name, content)
+    create_fixture(type, File.join('views', name), content)
+  end
+
+  describe 'rendering stylesheets' do
+    it 'renders Sass stylesheets' do
+      in_nesta_project do
+        create_view(:local, 'master.sass', "body\n  width: 10px * 2")
+        visit '/css/master.css'
+        assert_match /width: 20px;/, body, 'should match /width: 20px;/'
+      end
+    end
+
+    it 'renders SCSS stylesheets' do
+      in_nesta_project do
+        create_view(:local, 'master.scss', "body {\n  width: 10px * 2;\n}")
+        visit '/css/master.css'
+        assert_match /width: 20px;/, body, 'should match /width: 20px;/'
+      end
+    end
+
+    it 'renders stylesheet in the the gem if no others found' do
+      in_nesta_project do
+        visit '/css/master.css'
+        assert_equal 200, page.status_code
+      end
+    end
+  end
+
+  def create_haml(type, name, content)
+    create_view(type, 'layout.haml', '= yield')
+    create_view(type, name, content)
+  end
+
+  describe 'rendering Haml' do
+    it 'uses local template in place of default' do
+      in_nesta_project do
+        create_haml(:local, 'page.haml', '%p Local template')
+        visit create(:category).abspath
+        assert_has_xpath '//p', text: 'Local template'
+      end
+    end
+
+    it 'uses theme template in place of default' do
+      in_nesta_project('theme' => theme_name) do
+        create_haml(:theme, 'page.haml', '%p Theme template')
+        visit create(:category).abspath
+        assert_has_xpath '//p', text: 'Theme template'
+      end
+    end
+
+    it 'prioritise local templates over theme templates' do
+      in_nesta_project('theme' => theme_name) do
+        create_haml(:local, 'page.haml', '%p Local template')
+        create_haml(:theme, 'page.haml', '%p Theme template')
+        visit create(:category).abspath
+        assert_has_xpath '//p', text: 'Local template'
+        assert_has_no_xpath '//p', text: 'Theme template'
+      end
+    end
+  end
+end